linux-kernel - Re: [PATCH 7/9] ARM: dts: stm32: add Hardware debug port (HDP) on stm32mp25

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: <55beb3e7-65ac-4145-adae-fb064378c78d@kernel.org>
Date: Wed, 26 Feb 2025 16:08:02 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Alexandre TORGUE <alexandre.torgue@...s.st.com>,
 Clement LE GOFFIC <clement.legoffic@...s.st.com>,
 Linus Walleij <linus.walleij@...aro.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Maxime Coquelin <mcoquelin.stm32@...il.com>,
 Bartosz Golaszewski <brgl@...ev.pl>
Cc: linux-kernel@...r.kernel.org, linux-gpio@...r.kernel.org,
 devicetree@...r.kernel.org, linux-stm32@...md-mailman.stormreply.com,
 linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 7/9] ARM: dts: stm32: add Hardware debug port (HDP) on
 stm32mp25

On 26/02/2025 10:33, Alexandre TORGUE wrote:
>>>>> +		hdp: pinctrl@...90000 {
>>>>> +			compatible = "st,stm32mp-hdp";
>>>>
>>>> So here again - you have stm32mp251 SoC, but use entirely different
>>>> compatible.
>>>
>>> Ok so I will use "st,stm32mp15-hdp"
>>
>>
>> This means this is stm32mp15 SoC. I do not see such SoC on list of your
>> SoCs in bindings. What's more, there are no bindings for other SoC
>> components for stm32mp15!
> 
> Yes stm32mp15 is not a "real SoC". I agree that at the beginning of the 
> STM32 story we didn't have a clear rule/view to correctly naming our 
> compatible. We tried to improve the situation to avoid compatible like 
> "st,stm32", "st,stm32mp" or "st,stm32mp1". So we introduced 
> "st,stm32mp13", "st,stm32mp15" or "st,stm32mp25" for new drivers. So yes 
> it represents a SoC family and not a real SoC. We haven't had much 
> negative feedback it.
> 
> But, if it's not clean to do it in this way, lets define SoC compatible 
> for any new driver.

Compatibles are for hardware.

> For the HDP case it is: "st,stm32mp157" and used for STM32MP13, 
> STM32MP15 end STM32MP25 SoC families (if driver is the same for all 
> those SoCs).

No, it's three compatibles, because you have three SoCs. BTW, writing
bindings (and online resources and previous reviews and my talks) are
saying that, so we do not ask for anything new here, anything different.
At least not new when looking at last 5 years, because 10 years ago many
rules were relaxed...



Best regards,
Krzysztof