-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
video/vnc: add vnc_fb_register #11383
Conversation
6431102
to
ee7e307
Compare
I successfully compiled sim:vncserver in the docker [ https://github.com/apache/nuttx/pkgs/container/nuttx%2Fapache-nuttx-ci-linux],
|
I guess the error is due to linker doesn't move the unused fb_register from final image. It could be fixed by changing fb_register to macro or inline function. |
2ec34bb
to
bda2d2a
Compare
@jianglianfang before we merge it, could you please confirm that this tutorial still working after your modifications: https://nuttx.apache.org/docs/latest/platforms/sim/sim/index.html#running-vnc-server ? |
1030d76
to
70ed1d2
Compare
a79c11a
to
37698bd
Compare
Could you please verify the compilation error? |
bc79173
to
53cbb2c
Compare
To optimize the initialization of vnc, change it to vnc_fb_register. Signed-off-by: jianglianfang <jianglianfang@xiaomi.com>
Summary
Optimize the initialization of vnc, change it to vnc_fb_register.
Impact
Nuttx vnc
Testing
run nuttx with vnc