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

[Rob Herring]

On Mon, 05 Feb 2024 17:12:37 +0530, 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>
> ---
> V2: No change
> V3:
> - Add vendor specific compatible
> - Update reg names
> - Update clocks to 1
> - Fix dts example with proper naming
> V4:
>  - Use ti-specific compatible ti,am62a-jpeg-enc as secondary one
>  - Update commit message and title
>  - Remove clock-names as only single clock
> 
> Link to previous commit:
> https://lore.kernel.org/all/20230816152210.4080779-2-devarsht@xxxxxx/
> ---
>  .../bindings/media/img,e5010-jpeg-enc.yaml    | 75 +++++++++++++++++++
>  MAINTAINERS                                   |  5 ++
>  2 files changed, 80 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/media/img,e5010-jpeg-enc.yaml
> 

Reviewed-by: Rob Herring <robh@xxxxxxxxxx>