[libcamera-devel] [RFC PATCH v2 4/7] libcamera: ipa: Soft IPA: add a Soft IPA implementation

[Pavel Machek]

Hi!

> > >>> Signed-off-by: Andrey Konovalov <andrey.konovalov at linaro.org>
> > >>
> > >> I'm not sure if we should call implementation "Linaro". Company name
> > >> seems misleading in this context. Could we call implementation "anna",
> > >> for example, after https://en.wikipedia.org/wiki/Anna_Atkins and
> > >> because it is first version, thus "A"?
> > > 
> > > I'd agree here ... I can't imagine libcamera want's "5 different
> > > competing softISP implementations.... all in the same code base"
> > > 
> > > This is just 'SoftISP' in my opinion.
> > 
> > Then SwIspLinaro could be merged into abstract SoftwareIsp,
> > the result being called just SoftwareIsp.
> > And SoftwareIspFactory* are not needed.
> > Correct?
> 
> Has a SoftwareISPFactory requirement been discussed somewhere that I
> haven't been involved in ?

One day we may have SwIspGPU or maybe SwIspIntelMMX, so that support
might be worth keeping.

Best regards,
								Pavel
-- 
People of Russia, stop Putin before his war on Ukraine escalates.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://lists.libcamera.org/pipermail/libcamera-devel/attachments/20231212/34d77d9f/attachment.sig>