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. Best regards, Krzysztof