[PATCH v7 2/4] ipa: libipa: Copy pwl from rpi
Kieran Bingham
kieran.bingham at ideasonboard.com
Tue Jun 11 12:24:46 CEST 2024
Quoting Paul Elder (2024-06-11 10:58:15)
> On Tue, Jun 11, 2024 at 03:00:41AM +0300, Laurent Pinchart wrote:
> > Hi Paul,
> >
> > Thank you for the patch.
> >
> > On Mon, Jun 10, 2024 at 11:19:39PM +0900, Paul Elder wrote:
> > > Copy the piecewise linear function code from Raspberry Pi.
> > >
> > > Signed-off-by: Paul Elder <paul.elder at ideasonboard.com>
> > > Reviewed-by: Stefan Klug <stefan.klug at ideasonboard.com>
> > > Acked-by: David Plowman <david.plowman at raspberrypi.com>
> > > Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>
> >
> > This will break bisection due to missing documentation. Is that an issue
> > ? Should we squash patches 2/4 and 3/4 ?
>
> If we squash then we can't tell what changed, so it depends on if that
> is more valuable or if maintaining bisection is more valuable.
>
> Or we say that the former is only important for review and it's fine to
> squash.
I would say definitely valuable for review, so a valuable split to start
with and I'm fine if it's preferred to squash for bisectability.
--
Kieran
More information about the libcamera-devel
mailing list