[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <0a7a8e83-8724-4bec-8b3d-a58cc0a70395@linaro.org>
Date: Thu, 29 Feb 2024 13:16:41 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Sebastian Fricke <sebastian.fricke@...labora.com>,
Devarsh Thakkar <devarsht@...com>
Cc: mchehab@...nel.org, robh@...nel.org, krzysztof.kozlowski+dt@...aro.org,
conor+dt@...nel.org, hverkuil-cisco@...all.nl, linux-media@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
benjamin.gaignard@...labora.com, laurent.pinchart@...asonboard.com,
praneeth@...com, nm@...com, vigneshr@...com, a-bhatia1@...com,
j-luthra@...com, b-brnich@...com, detheridge@...com, p-mantena@...com,
vijayp@...com, andrzej.p@...labora.com, nicolas@...fresne.ca, afd@...com,
milkfafa@...il.com
Subject: Re: [PATCH v6 1/3] media: dt-bindings: Add Imagination E5010 JPEG
Encoder
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@...com>
>> Signed-off-by: David Huang <d-huang@...com>
>> Signed-off-by: Devarsh Thakkar <devarsht@...com>
>> Reviewed-by: Rob Herring <robh@...nel.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.
Best regards,
Krzysztof
Powered by blists - more mailing lists