[libcamera-devel] [PATCH 2/4] ipa: raspberrypi: cam_helper: Make exposure conversion functions virtual

[Naushir Patuck]

On Tue, 22 Mar 2022 at 23:45, Laurent Pinchart <
laurent.pinchart at ideasonboard.com> wrote:

> On Tue, Mar 22, 2022 at 11:36:23PM +0000, Kieran Bingham wrote:
> > Quoting Laurent Pinchart (2022-03-22 23:25:05)
> > > On Thu, Mar 17, 2022 at 11:28:01AM +0000, Kieran Bingham wrote:
> > > > Quoting Laurent Pinchart (2021-12-19 23:27:12)
> > > > > From: Naushir Patuck <naush at raspberrypi.com>
> > > > >
> > > > > Some sensors (namely the Sony IMX296, whose support will be added
> > > > > shortly) require different conversion formulas between exposure
> time and
> > > > > exposure lines. Make the Exposure() and ExposureLines() functions
> > > > > virtual to allow this.
> > > >
> > > > Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> > >
> > > Naush, could I get your SoB line for this patch ?
> >
> > You have it already in
> >  <CAEmqJPpcDLYRtbu0-5aYZxhn0rMgDR15PJj=i+OKr8dss8NtEw at mail.gmail.com>
>
> Oops, sorry for the noise.
>

Just to confirm :)

Signed-off-by: Naushir Patuck <naush at raspberrypi.com>


>
> > > > > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com
> >
> > > > > ---
> > > > >  src/ipa/raspberrypi/cam_helper.hpp | 4 ++--
> > > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > > >
> > > > > diff --git a/src/ipa/raspberrypi/cam_helper.hpp
> b/src/ipa/raspberrypi/cam_helper.hpp
> > > > > index a776153fcb84..300f8f8a2d3b 100644
> > > > > --- a/src/ipa/raspberrypi/cam_helper.hpp
> > > > > +++ b/src/ipa/raspberrypi/cam_helper.hpp
> > > > > @@ -76,8 +76,8 @@ public:
> > > > >         virtual void Prepare(libcamera::Span<const uint8_t> buffer,
> > > > >                              Metadata &metadata);
> > > > >         virtual void Process(StatisticsPtr &stats, Metadata
> &metadata);
> > > > > -       uint32_t ExposureLines(libcamera::utils::Duration
> exposure) const;
> > > > > -       libcamera::utils::Duration Exposure(uint32_t
> exposure_lines) const;
> > > > > +       virtual uint32_t ExposureLines(libcamera::utils::Duration
> exposure) const;
> > > > > +       virtual libcamera::utils::Duration Exposure(uint32_t
> exposure_lines) const;
> > > > >         virtual uint32_t GetVBlanking(libcamera::utils::Duration
> &exposure,
> > > > >                                       libcamera::utils::Duration
> minFrameDuration,
> > > > >                                       libcamera::utils::Duration
> maxFrameDuration) const;
>
> --
> Regards,
>
> Laurent Pinchart
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.libcamera.org/pipermail/libcamera-devel/attachments/20220323/9bb8fc08/attachment-0001.htm>