Re: [PATCH v6 1/3] media: dt-bindings: Add Imagination E5010 JPEG Encoder

[Sebastian Fricke]

On 29.02.2024 13:16, Krzysztof Kozlowski wrote:
> On 29/02/2024 11:26, Sebastian Fricke wrote:
> > Hey Devarsh,
> >
> > On 28.02.2024 19:41, Devarsh Thakkar wrote:
> > > Add dt-bindings for Imagination E5010 JPEG Encoder [1] which is implemented
> > > as stateful V4L2 M2M driver.
> > >
> > > The device supports baseline encoding with two different quantization
> > > tables and compression ratio as demanded.
> > >
> > > Minimum resolution supported is 64x64 and Maximum resolution supported is
> > > 8192x8192.
> > >
> > > [1]:  AM62A TRM (Section 7.6 is for JPEG Encoder)
> > > Link: https://www.ti.com/lit/pdf/spruj16
> > >
> > > Co-developed-by: David Huang <d-huang@xxxxxx>
> > > Signed-off-by: David Huang <d-huang@xxxxxx>
> > > Signed-off-by: Devarsh Thakkar <devarsht@xxxxxx>
> > > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
> >
> > hmmm when did Rob give his reviewed by on this patch? (As this is not a
> > DT binding I find that odd)
>
> This is a DT binding, which is clearly expressed in subject prefix
> (proper one) and the patch contents.

Yup sorry for the noise, I confused the two patches.

> Best regards,
> Krzysztof

Greetings,
Sebastian