Received-SPF: pass (google.com: domain of linux-kernel+bounces-53897-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Date: Mon, 5 Feb 2024 20:09:45 +0000
From: Rob Herring <robh@kernel.org>
To: Devarsh Thakkar <devarsht@ti.com>
Cc: b-brnich@ti.com, conor+dt@kernel.org, nicolas@ndufresne.ca,
	a-bhatia1@ti.com, praneeth@ti.com, linux-kernel@vger.kernel.org,
	p-mantena@ti.com, hverkuil-cisco@xs4all.nl, vijayp@ti.com,
	laurent.pinchart@ideasonboard.com, j-luthra@ti.com,
	krzysztof.kozlowski+dt@linaro.org, nm@ti.com,
	andrzej.p@collabora.com, robh+dt@kernel.org, mchehab@kernel.org,
	linux-media@vger.kernel.org, vigneshr@ti.com,
	devicetree@vger.kernel.org, detheridge@ti.com
Subject: Re: [PATCH v4 1/3] media: dt-bindings: Add Imagination E5010 JPEG
 Encoder
Message-ID: <170716378412.295212.11603162949482063011.robh@kernel.org>
References: <20240205114239.924697-1-devarsht@ti.com>
 <20240205114239.924697-2-devarsht@ti.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20240205114239.924697-2-devarsht@ti.com>


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@ti.com>
> Signed-off-by: David Huang <d-huang@ti.com>
> Signed-off-by: Devarsh Thakkar <devarsht@ti.com>
> ---
> 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@ti.com/
> ---
>  .../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@kernel.org>