<div dir="ltr"><div dir="ltr">Hi Niklas,</div><div dir="ltr"><br></div><div dir="ltr">Thanks for your review and comments.</div><div dir="ltr">I will have the next version soon according to your suggestions.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 20, 2020 at 10:48 PM Niklas Söderlund <<a href="mailto:niklas.soderlund@ragnatech.se" target="_blank">niklas.soderlund@ragnatech.se</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2020-03-20 14:50:38 +0100, Niklas Söderlund wrote:<br>
> Hi Show,<br>
> <br>
> Thanks for your work!<br>
> <br>
> On 2020-03-20 16:50:26 +0800, Show Liu wrote:<br>
> > Hi all,<br>
> >   I would like to share my adventure on qcam. <br>
> > I added an option to enable rendering YUV frame by OpenGL shader for qcam.<br>
> > Honestly, it's still in progress, and only support NV12 format at present. <br>
> > However the improvement is obviously. I logged the messages below when qcam running.<br>
> > <a href="https://pastebin.linaro.org/view/20d91f21" rel="noreferrer" target="_blank">https://pastebin.linaro.org/view/20d91f21</a><br>
> > In 1280x800 mode:<br>
> > format convert by CPU is around ~4 fps, rendering by GPU up to ~30.0 fps<br>
> > In 1920x1080 mode:<br>
> > format convert by CPU is around ~2 fps, rendering by GPU still up to ~30.0 fps<br>
> <br>
> I have tested this and I get similar result on different device using <br>
> Panfrost GPU, really nice work!</blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> One general observation is that I do get a graphic artifact in the qcam <br>
> GUI, covering about half the dropdown box for camera selection. If I <br>
> specify --opengl the artifact is filled with black, without --opengl <br>
> it's filled with white. If I run without the OpenGL patches I get no <br>
> artifact. I'm not sure if this is an effect of the panfrost driver or <br>
> not.<br>
<br>
I get same behavior of the artifact on other systems as well so it's <br>
somehow related the this series I think.<br></blockquote><div>I will try to fix this problem in the next version. </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> <br>
> I first tried applying the patches to latest libcamera master, but they <br>
> did not apply. It was easy to solve but you might want to pull in the <br>
> latest changes to qcam so you don't have to solve conflicts later.<br>
> <br>
> I will review the individual patches but just wanted to let you know how <br>
> happy I am seeing and testing this series ;-)<br>
> <br>
> > <br>
> >   The improvement still in progress, including more YUV formats support ... etc.<br>
> > And some parts I added are not in good shape, I am trying to make it better. <br>
> > So I appreciate any review conments and suggestion.<br>
> > <br>
> > It's also available on below.<br>
> > <a href="https://github.com/showliu/libcamera/commits/qcam_opengl_hack2" rel="noreferrer" target="_blank">https://github.com/showliu/libcamera/commits/qcam_opengl_hack2</a><br>
> > <br>
> > I tested with kernel 5.5rc5 + Panfrost GPU driver on Rock Pi 4B platform.<br>
> > <br>
> > Show Liu (3):<br>
> >   qcam: Added statusbar to show the camera stream configuration<br>
> >   qcam: Added opengl widget class for qcam<br>
> >   qcam: added an option to enable rendering via OpenGL shader<br>
> > <br>
> >  src/qcam/glwidget.cpp    | 213 +++++++++++++++++++++++++++++++++++++++<br>
> >  src/qcam/glwidget.h      |  76 ++++++++++++++<br>
> >  src/qcam/main.cpp        |   2 +<br>
> >  src/qcam/main_window.cpp |  36 ++++++-<br>
> >  src/qcam/main_window.h   |   3 +<br>
> >  src/qcam/meson.build     |   4 +-<br>
> >  6 files changed, 330 insertions(+), 4 deletions(-)<br>
> >  create mode 100644 src/qcam/glwidget.cpp<br>
> >  create mode 100644 src/qcam/glwidget.h<br>
> > <br>
> > -- <br>
> > 2.20.1<br>
> > <br>
> > _______________________________________________<br>
> > libcamera-devel mailing list<br>
> > <a href="mailto:libcamera-devel@lists.libcamera.org" target="_blank">libcamera-devel@lists.libcamera.org</a><br>
> > <a href="https://lists.libcamera.org/listinfo/libcamera-devel" rel="noreferrer" target="_blank">https://lists.libcamera.org/listinfo/libcamera-devel</a><br>
> <br>
> -- <br>
> Regards,<br>
> Niklas Söderlund<br>
<br>
-- <br>
Regards,<br>
Niklas Söderlund<br></blockquote><div><br></div><div>BR,</div><div>Show Liu </div></div></div>