Messages in this thread | | | Date | Thu, 29 Feb 2024 14:31:37 +0100 | From | Sebastian Fricke <> | Subject | Re: [PATCH v6 1/3] media: dt-bindings: Add Imagination E5010 JPEG Encoder |
| |
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@ti.com> >>> Signed-off-by: David Huang <d-huang@ti.com> >>> Signed-off-by: Devarsh Thakkar <devarsht@ti.com> >>> Reviewed-by: Rob Herring <robh@kernel.org> >> >> 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
| |