linux-kernel - Re: [PATCH 1/3] dt-bindings: media: camss: Add qcom,sm6350-camss

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: <b8550097-76c5-4843-9b05-0802f3a4e3e6@kernel.org>
Date: Tue, 28 Oct 2025 11:39:43 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
 Luca Weiss <luca.weiss@...rphone.com>
Cc: Bryan O'Donoghue <bod@...nel.org>, Robert Foss <rfoss@...nel.org>,
 Todor Tomov <todor.too@...il.com>,
 Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
 Mauro Carvalho Chehab <mchehab@...nel.org>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Bjorn Andersson <andersson@...nel.org>,
 Konrad Dybcio <konradybcio@...nel.org>,
 ~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org,
 linux-arm-msm@...r.kernel.org, linux-media@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/3] dt-bindings: media: camss: Add qcom,sm6350-camss

On 28/10/2025 11:24, Bryan O'Donoghue wrote:
> On 28/10/2025 09:46, Krzysztof Kozlowski wrote:
>>>>> +  power-domain-names:
>>>>> +    items:
>>>>> +      - const: ife0
>>>>> +      - const: ife1
>>>>> +      - const: ife2
>>>>> +      - const: top
>>>> Uh, not your fault, but who came with this list in previous generations?
>>>> Instead of simple and obvious "top+ifeX" which allows growing/shrinking,
>>>> someone put "top" at the end which means this cannot follow same order
>>>> as X1E for example... Heh, it follows at least sm8550.
>>> Shall we put top as first power-domain? I don't think it's an issue to
>>> change the order.
>> Well, it matches sm8550, so I am just grumpy complaining. It's fine.
> 
> The provenance here is "top" was required to be added last because the 
> code depended on magic indexing in dtb to know which was the TOP GDSC.


That's silly, because if it was first element would be much easier.

Best regards,
Krzysztof