linux-kernel - Re: [PATCH 14/38] dt-bindings: media: mediatek,mt8195-jpeg: Allow range number in node address

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <bcad01be-fd00-4789-ae83-b855e495fab9@kernel.org>
Date: Fri, 12 Sep 2025 15:22:54 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
 linux-mediatek@...ts.infradead.org, robh@...nel.org
Cc: herbert@...dor.apana.org.au, davem@...emloft.net, krzk+dt@...nel.org,
 conor+dt@...nel.org, chunkuang.hu@...nel.org, p.zabel@...gutronix.de,
 airlied@...il.com, simona@...ll.ch, maarten.lankhorst@...ux.intel.com,
 mripard@...nel.org, tzimmermann@...e.de, jassisinghbrar@...il.com,
 mchehab@...nel.org, matthias.bgg@...il.com, chunfeng.yun@...iatek.com,
 vkoul@...nel.org, kishon@...nel.org, sean.wang@...nel.org,
 linus.walleij@...aro.org, lgirdwood@...il.com, broonie@...nel.org,
 andersson@...nel.org, mathieu.poirier@...aro.org, daniel.lezcano@...aro.org,
 tglx@...utronix.de, atenart@...nel.org, jitao.shi@...iatek.com,
 ck.hu@...iatek.com, houlong.wei@...iatek.com,
 kyrie.wu@...iatek.corp-partner.google.com, andy.teng@...iatek.com,
 tinghan.shen@...iatek.com, jiaxin.yu@...iatek.com, shane.chien@...iatek.com,
 olivia.wen@...iatek.com, granquet@...libre.com, eugen.hristev@...aro.org,
 arnd@...db.de, sam.shih@...iatek.com, jieyy.yang@...iatek.com,
 frank-w@...lic-files.de, mwalle@...nel.org, fparent@...libre.com,
 linux-crypto@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
 linux-media@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-phy@...ts.infradead.org, linux-gpio@...r.kernel.org,
 linux-remoteproc@...r.kernel.org, linux-sound@...r.kernel.org
Subject: Re: [PATCH 14/38] dt-bindings: media: mediatek,mt8195-jpeg: Allow
 range number in node address

On 12/09/2025 11:00, AngeloGioacchino Del Regno wrote:
> Il 04/08/25 11:02, AngeloGioacchino Del Regno ha scritto:
>> Il 24/07/25 11:14, Krzysztof Kozlowski ha scritto:
>>> On 24/07/2025 10:38, AngeloGioacchino Del Regno wrote:
>>>> The dual and triple core jpeg encoder and decoder (respectively)
>>>> on MT8195 are far apart: the only way to have this to make sense
>>>> is to split those in multiple address ranges in device trees as
>>>> one big range would overlap with other IP in at least the MT8195
>>>> SoC.
>>>>
>>>> Change both the jpegdec and jpegenc bindings to allow specifying
>>>> children nodes such as "jpegdec@0,10000", "jpegdec@1,0" or for
>>>> encoder "jpegenc@0,0", "jpegenc@1,0" to resolve dtbs_check issues.
>>>
>>>
>>> This should not be needed for standard MMIO/simple-bus nodes. I think
>>> DTS is wrong here.
>>>
>>> Which cases really need the ','?
>>>
>>
>> All of the multi-core JPEG enc/decoders on MT8195 (and newer).
>>
>> The DT changes are included in the same series as this commit; check:
>>
>> 20250724083914.61351-35-angelogioacchino.delregno@...labora.com
>>
>> Cheers,
>> Angelo
>>
> 
> Any further comments on this?

Well yeah, that's still wrong. These are simple MMIO, so comma is not
correct. Rob already commented on this at v1 of Ariel's patchset. It was
BTW the same device - mt8195 jpeg!


Best regards,
Krzysztof