netdev - Re: [PATCH 0/7] Refactor sa8775p/qcs9100 to common names lemans-auto/lemans

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: <pzkceoriu5cgvidt4xekauyc2ovqkbuoi32bbornr2wbxmombh@7visdfuos6ml>
Date: Sat, 26 Jul 2025 11:09:30 -0500
From: Bjorn Andersson <andersson@...nel.org>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: Wasim Nazir <wasim.nazir@....qualcomm.com>, 
	Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, 
	Richard Cochran <richardcochran@...il.com>, linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, netdev@...r.kernel.org, kernel@....qualcomm.com
Subject: Re: [PATCH 0/7] Refactor sa8775p/qcs9100 to common names
 lemans-auto/lemans

On Wed, Jul 23, 2025 at 10:32:43AM +0200, Krzysztof Kozlowski wrote:
> On Tue, Jul 22, 2025 at 08:19:19PM +0530, Wasim Nazir wrote:
> > This patch series refactors the sa8775p and qcs9100 platforms and introduces
> > a unified naming convention for current and future platforms (qcs9075).
> > 
> > The motivation behind this change is to group similar platforms under a
> > consistent naming scheme and to avoid using numeric identifiers.
> > For example, qcs9100 and qcs9075 differ only in safety features provided by
> > the Safety-Island (SAIL) subsystem but safety features are currently
> > unsupported, so both can be categorized as the same chip today.
> >
> 
> I expressed strong disagreement with this patchset in individual
> patches. I expect NO NEW versions of it, but by any chance you send it,
> then please always carry my:
> 

I requested Wasim to prepare this patch set. Something that would have
been useful to include in the cover letter and some of the patches...

I definitely agree with your position when it comes to renaming working
platforms and I also think there has been way too much churn in relation
to this platform.

But the thing we call SA8775P upstream is not SA8775P. The hardware +
firmware that is described by sa8775p.dtsi doesn't exist.

Reactively and without telling us the whole story, we seem to have
gotten qcs9100-ride*.dts to represent what folks has been using to
upstream the platform support.

But the board where I see people actually running upstream (the EVK
introduced in this series) is based on the QCS9075 variant, which is the
same die but with some hardware features disabled.

In other words, this is a mess resulting from lacking communication and
reactive shortsighted attempts to get things merged. I don't want to
maintain it in this form.

> Nacked-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> 

I'm fine carrying this token of disapproval for how we got here. 

Regards,
Bjorn

> Best regards,
> Krzysztof
>