[PATCH v1 14/14] dt-bindings: media: mediatek,jpeg: Fix jpeg encoder/decoder ranges

Ariel D'Alessandro ariel.dalessandro at collabora.com
Wed Sep 10 08:02:22 PDT 2025


Rob,

On 8/20/25 3:55 PM, Rob Herring wrote:
> On Wed, Aug 20, 2025 at 02:13:02PM -0300, Ariel D'Alessandro wrote:
>> Commit 14176e94bb35d ("arm64: dts: mediatek: mt8195: Fix ranges for jpeg
> 
> That commit is not in any upstream tree.

Ugh, indeed. Dropping this patch.

> 
>> enc/decoder nodes") redefined jpeg encoder/decoder children node ranges.
>> Update the related device tree binding yaml definition to match
>> mediatek/mt8195.dtsi, as this is currently the only one using it.
>>
>> Signed-off-by: Ariel D'Alessandro <ariel.dalessandro at collabora.com>
>> ---
>>   .../media/mediatek,mt8195-jpegdec.yaml        | 31 ++++++++++---------
>>   .../media/mediatek,mt8195-jpegenc.yaml        | 15 ++++-----
>>   2 files changed, 24 insertions(+), 22 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/media/mediatek,mt8195-jpegdec.yaml b/Documentation/devicetree/bindings/media/mediatek,mt8195-jpegdec.yaml
>> index e5448c60e3eb5..b1f3df258dc87 100644
>> --- a/Documentation/devicetree/bindings/media/mediatek,mt8195-jpegdec.yaml
>> +++ b/Documentation/devicetree/bindings/media/mediatek,mt8195-jpegdec.yaml
>> @@ -36,7 +36,7 @@ properties:
>>   
>>   # Required child node:
>>   patternProperties:
>> -  "^jpgdec@[0-9a-f]+$":
>> +  "^jpgdec@[0-9],[0-9a-f]+$":
> 
> This is wrong unless 0-9 is a separate, distinct address (like a chip
> select #).

Ack.

Thanks for the feedback.
Regards,

-- 
Ariel D'Alessandro
Software Engineer

Collabora Ltd.
Platinum Building, St John's Innovation Park, Cambridge CB4 0DS, UK 
Registered in England & Wales, no. 5513718




More information about the linux-arm-kernel mailing list