[libcamera-devel] [PATCH v5 07/10] libcamera: ipa: Extend to support IPA interactions

[Laurent Pinchart]

Hi Jacopo,

On Wed, Oct 09, 2019 at 05:04:41PM +0200, Jacopo Mondi wrote:
> On Wed, Oct 09, 2019 at 05:53:55PM +0300, Laurent Pinchart wrote:
> > Hello,
> 
> [snip]
> 
> > > I might be playing in the 'too verbose' field, but I would expand
> > > these with a few words to make them a little nicer to read. In
> > > example:
> > >
> > > > + * \brief The pixel format
> > >
> > >         \brief The requested stream pixel format
> > >
> > >         The pixel format encoding is specific to the pipeline/IPA
> > >         implementation and might differ from the one configured from
> > >         applications on the Camera.
> >
> > Can it ? I thought it should be the same.
> 
> I assumed it might be possible for some pipes to use external
> components to produce, say, JPEG.

We don't support JPEG compression yet though :-) Longer term we will
probably want to describe more of the pipeline configuration, and extend
IPAStream for that purpose. For now I think we can restrict it to
describe the format produced by the stream.

-- 
Regards,

Laurent Pinchart