[PATCH 2/4] ipa: libipa: Copy pwl from rpi

Kieran Bingham kieran.bingham at ideasonboard.com
Fri May 31 12:34:15 CEST 2024


Quoting Paul Elder (2024-04-26 08:39:39)
> Hi Naush,
> 
> On Mon, Apr 08, 2024 at 09:14:41AM +0100, Naushir Patuck wrote:
> > Hi Paul,
> > 
> > On Fri, 5 Apr 2024 at 09:03, Paul Elder <paul.elder at ideasonboard.com> wrote:
> > >
> > > Copy the piecewise linear function code from Raspberry Pi.
> > >
> > 
> > This seems like a good time to mention something I was intending on
> > doing for a while...
> > 
> > Our PWL library is used internally in a few places
> > (libcamera/libpisp/rpicam-apps/some internal tools).  Right now each
> > of these libraries has a copy of pretty much the same pwl.c/pwl.h
> > file.  I was intending to create a stand alone Raspberry Pi PWL
> > library that could be linked with these so we only have one canonical
> > copy of the source code.  This could then be included into
> > libcamera/libpipa as a subproject if needed.  Do you think this would
> > be helpful and suitable for you?  We will need to look at possibly
> 
> Yeah that might be a good idea. (although I'm not in the position to
> make these decisions)

I don't think core libcamera can rely on a 'subproject' (in meson terms)
as packagers/distributions will not accept a build that has to go and
download more code to perform the build.

But we should certainly find a way to reduce having 2, 3, 4 copies of
the same code in different places...

I think the Pwl is small enough (and stable enough) that it's not too
painful for now. I wonder if we do need to break out more like we have
with libcamera-base ... but I don't know where the bar is for that yet.

> In any case we need this right now so for now we'll go with this :)
> 
> 
> Thanks,
> 
> Paul

--
Kieran


More information about the libcamera-devel mailing list