<div dir="ltr">Hi Han-lin,<br><br>Could you do the final review, and let Kieran or Laurent help to merge the patches?<div>Thanks!</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 24, 2022 at 10:03 PM Kieran Bingham <<a href="mailto:kieran.bingham@ideasonboard.com">kieran.bingham@ideasonboard.com</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">Quoting Cheng-Hao Yang via libcamera-devel (2022-11-01 04:53:10)<br>
> Hello folks,<br>
> <br>
> It's been a while since the last update. If there's no other dispute, may<br>
> we merge the series of patches recently?<br>
> Thanks!<br>
<br>
As this is the android layer - is there anyone at Google/Chromium who<br>
can do a review on this series ?<br>
<br>
--<br>
Kieran<br>
<br>
<br>
> On Fri, Jul 15, 2022 at 11:38 PM Harvey Yang <<a href="mailto:chenghaoyang@chromium.org" target="_blank">chenghaoyang@chromium.org</a>><br>
> wrote:<br>
> <br>
> > Hi all,<br>
> ><br>
> > Fixed thumbnail buffer issue.<br>
> ><br>
> > Please help review and merge them if possible :)<br>
> > Thanks!<br>
> ><br>
> > BR,<br>
> > Harvey<br>
> ><br>
> > Harvey Yang (6):<br>
> >   Allow inheritance of FrameBuffer<br>
> >   Add HALFrameBuffer and replace FrameBuffer in src/android<br>
> >   Add meson.build in src/android/jpeg<br>
> >   Move generateThumbnail from PostProcessorJpeg to Encoder<br>
> >   Pass StreamBuffer to Encoder::encoder<br>
> >   Add JEA implementation<br>
> ><br>
> >  include/libcamera/framebuffer.h               |   3 +-<br>
> >  src/android/camera_device.cpp                 |   3 +-<br>
> >  src/android/cros/camera3_hal.cpp              |   2 +<br>
> >  src/android/cros_mojo_token.h                 |  12 ++<br>
> >  src/android/frame_buffer_allocator.h          |   7 +-<br>
> >  src/android/hal_framebuffer.cpp               |  23 +++<br>
> >  src/android/hal_framebuffer.h                 |  27 ++++<br>
> >  src/android/jpeg/encoder.h                    |  10 +-<br>
> >  src/android/jpeg/encoder_jea.cpp              |  93 ++++++++++++<br>
> >  src/android/jpeg/encoder_jea.h                |  35 +++++<br>
> >  src/android/jpeg/encoder_libjpeg.cpp          | 133 ++++++++++++++----<br>
> >  src/android/jpeg/encoder_libjpeg.h            |  33 ++++-<br>
> >  src/android/jpeg/meson.build                  |  16 +++<br>
> >  src/android/jpeg/post_processor_jpeg.cpp      |  65 ++-------<br>
> >  src/android/jpeg/post_processor_jpeg.h        |  11 +-<br>
> >  src/android/meson.build                       |   6 +-<br>
> >  .../mm/cros_frame_buffer_allocator.cpp        |  14 +-<br>
> >  .../mm/generic_frame_buffer_allocator.cpp     |  12 +-<br>
> >  18 files changed, 391 insertions(+), 114 deletions(-)<br>
> >  create mode 100644 src/android/cros_mojo_token.h<br>
> >  create mode 100644 src/android/hal_framebuffer.cpp<br>
> >  create mode 100644 src/android/hal_framebuffer.h<br>
> >  create mode 100644 src/android/jpeg/encoder_jea.cpp<br>
> >  create mode 100644 src/android/jpeg/encoder_jea.h<br>
> >  create mode 100644 src/android/jpeg/meson.build<br>
> ><br>
> > --<br>
> > 2.37.0.170.g444d1eabd0-goog<br>
> ><br>
> ><br>
</blockquote></div>