[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