linux-kernel - Re: [PATCH v4 2/2] arm64: dts: qcom: qcs8300: Add support for 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 for Android: free password hash cracker in your pocket

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

Message-ID: <8966ddaf-9c10-4626-a4cc-36efd3fc93e2@kernel.org>
Date: Thu, 16 Oct 2025 07:59:37 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
 Vikram Sharma <quic_vikramsa@...cinc.com>, mchehab@...nel.org,
 robh@...nel.org, krzk+dt@...nel.org, conor+dt@...nel.org,
 andersson@...nel.org, konradybcio@...nel.org, hverkuil-cisco@...all.nl,
 cros-qcom-dts-watchers@...omium.org, catalin.marinas@....com, will@...nel.org
Cc: linux-arm-kernel@...ts.infradead.org, quic_svankada@...cinc.com,
 linux-media@...r.kernel.org, linux-arm-msm@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 Konrad Dybcio <konrad.dybcio@....qualcomm.com>, bryan.odonoghue@...aro.org
Subject: Re: [PATCH v4 2/2] arm64: dts: qcom: qcs8300: Add support for camss

On 15/10/2025 20:49, Vladimir Zapolskiy wrote:
> On 10/15/25 16:01, Vikram Sharma wrote:
>> Add changes to support the camera subsystem on the QCS8300.
>>
>> Reviewed-by: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
>> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
>> Signed-off-by: Vikram Sharma <quic_vikramsa@...cinc.com>
>> ---
>>   arch/arm64/boot/dts/qcom/qcs8300.dtsi | 171 ++++++++++++++++++++++++++
>>   1 file changed, 171 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/qcs8300.dtsi b/arch/arm64/boot/dts/qcom/qcs8300.dtsi
>> index 8d78ccac411e..acd475555115 100644
>> --- a/arch/arm64/boot/dts/qcom/qcs8300.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/qcs8300.dtsi
>> @@ -4769,6 +4769,177 @@ videocc: clock-controller@...0000 {
>>   			#power-domain-cells = <1>;
>>   		};
>>   
>> +		camss: isp@...8000 {
>> +			compatible = "qcom,qcs8300-camss";
>> +
>> +			reg = <0x0 0xac78000 0x0 0x1000>,
>> +			      <0x0 0xac7a000 0x0 0xf00>,
>> +			      <0x0 0xac7c000 0x0 0xf00>,
>> +			      <0x0 0xac84000 0x0 0xf00>,
>> +			      <0x0 0xac88000 0x0 0xf00>,
>> +			      <0x0 0xac8c000 0x0 0xf00>,
>> +			      <0x0 0xac90000 0x0 0xf00>,
>> +			      <0x0 0xac94000 0x0 0xf00>,
>> +			      <0x0 0xac9c000 0x0 0x2000>,
>> +			      <0x0 0xac9e000 0x0 0x2000>,
>> +			      <0x0 0xaca0000 0x0 0x2000>,
>> +			      <0x0 0xacac000 0x0 0x400>,
>> +			      <0x0 0xacad000 0x0 0x400>,
>> +			      <0x0 0xacae000 0x0 0x400>,
>> +			      <0x0 0xac4d000 0x0 0xf000>,
>> +			      <0x0 0xac60000 0x0 0xf000>,
>> +			      <0x0 0xac85000 0x0 0xd00>,
>> +			      <0x0 0xac89000 0x0 0xd00>,
>> +			      <0x0 0xac8d000 0x0 0xd00>,
>> +			      <0x0 0xac91000 0x0 0xd00>,
>> +			      <0x0 0xac95000 0x0 0xd00>;
>> +			reg-names = "csid_wrapper",
>> +				    "csid0",
> 
> The list of 'reg-names' is not alphanumerically sorted, this is a newly
> introduced sorting order pattern of CAMSS 'reg' property values.


Please stop inventing ad-hoc or fake rules. There is no such sorting
pattern for this property, which I expressed multiple times. Last time
you claimed there is some sorting by "values", now this.

Best regards,
Krzysztof