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

Wasim Nazir posted 7 patches 2 months, 2 weeks ago
.../devicetree/bindings/arm/qcom.yaml         |  16 +-
arch/arm64/boot/dts/qcom/Makefile             |   8 +-
...8775p-ride.dts => lemans-auto-ride-r3.dts} |  44 +--
...{qcs9100-ride.dts => lemans-auto-ride.dts} |  14 +-
arch/arm64/boot/dts/qcom/lemans-auto.dtsi     | 104 +++++++
arch/arm64/boot/dts/qcom/lemans-evk.dts       | 291 ++++++++++++++++++
.../{sa8775p-pmics.dtsi => lemans-pmics.dtsi} |   0
...775p-ride.dtsi => lemans-ride-common.dtsi} | 168 ----------
.../qcom/lemans-ride-ethernet-88ea1512.dtsi   | 205 ++++++++++++
.../qcom/lemans-ride-ethernet-aqr115c.dtsi    | 205 ++++++++++++
...qcs9100-ride-r3.dts => lemans-ride-r3.dts} |  12 +-
.../dts/qcom/{sa8775p.dtsi => lemans.dtsi}    |  75 +++--
arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts  |  47 ---
13 files changed, 884 insertions(+), 305 deletions(-)
rename arch/arm64/boot/dts/qcom/{sa8775p-ride.dts => lemans-auto-ride-r3.dts} (11%)
rename arch/arm64/boot/dts/qcom/{qcs9100-ride.dts => lemans-auto-ride.dts} (18%)
create mode 100644 arch/arm64/boot/dts/qcom/lemans-auto.dtsi
create mode 100644 arch/arm64/boot/dts/qcom/lemans-evk.dts
rename arch/arm64/boot/dts/qcom/{sa8775p-pmics.dtsi => lemans-pmics.dtsi} (100%)
rename arch/arm64/boot/dts/qcom/{sa8775p-ride.dtsi => lemans-ride-common.dtsi} (87%)
create mode 100644 arch/arm64/boot/dts/qcom/lemans-ride-ethernet-88ea1512.dtsi
create mode 100644 arch/arm64/boot/dts/qcom/lemans-ride-ethernet-aqr115c.dtsi
rename arch/arm64/boot/dts/qcom/{qcs9100-ride-r3.dts => lemans-ride-r3.dts} (36%)
rename arch/arm64/boot/dts/qcom/{sa8775p.dtsi => lemans.dtsi} (99%)
delete mode 100644 arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts
[PATCH 0/7] Refactor sa8775p/qcs9100 to common names lemans-auto/lemans
Posted by Wasim Nazir 2 months, 2 weeks ago
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.

Since, most of our platforms are IoT-based so "lemans" can be served as the
default IoT variant, with "lemans-auto" derived from it. Accordingly:
  - qcs9100/qcs9075 and its associated IoT platforms are renamed to lemans
    which needs different memory-map. So, latest memory-map is updated
    here as per IOT requirements.
  - sa8775p and its associated platforms are renamed to "lemans-auto", which
    is derived from "lemans", that retains the old automotive memory map to
    support legacy use cases.
  - Both lemans & lemans-auto are serving as non-safe chip and if needed
    additional dtsi can be appended in the future to enable safety features.

Additionally:
  - Refactor common daughter cards used in Ride/Ride-R3 platforms into a
    common configuration. Also, introduce new files for different ethernet
    capabilities in Ride/Ride-r3. Since Ethernet functionality in Ride/Ride-r3
    is currently broken upstream, this patch focuses only on refactoring.
  - Include support for qcs9075 EVK[1] platform as lemans-evk. Currently,
    basic features are enabled supporting 'boot to shell'.
  - Remove support for qcs9100-ride, as no platform currently exists for it.

Funtional impact to current boards with refactoring:
  - No functional change on auto boards i.e sa8775p ride/ride-r3 boards
    (renamed as lemans-auto ride/ride-r3), and it is verified by comparing
    decompiled DTB (dtx_diff).
  - qcs9100 ride-r3 (renamed as lemans-ride-r3) is having new memory-map
    and rest other functionalities are still same.

[1] https://lore.kernel.org/all/20250612155437.146925-1-quic_wasimn@quicinc.com/


---
Wasim Nazir (7):
  arm64: dts: qcom: Rename sa8775p SoC to "lemans"
  arm64: dts: qcom: Update memory-map for IoT platforms in lemans
  arm64: dts: qcom: lemans: Separate out ethernet card for ride &
    ride-r3
  arm64: dts: qcom: lemans: Refactor ride/ride-r3 boards based on
    daughter cards
  arm64: dts: qcom: lemans: Rename boards and clean up unsupported
    platforms
  dt-bindings: arm: qcom: Refactor QCS9100 and SA8775P board names to
    reflect Lemans variants
  arm64: dts: qcom: Add lemans evaluation kit (EVK) initial board
    support

 .../devicetree/bindings/arm/qcom.yaml         |  16 +-
 arch/arm64/boot/dts/qcom/Makefile             |   8 +-
 ...8775p-ride.dts => lemans-auto-ride-r3.dts} |  44 +--
 ...{qcs9100-ride.dts => lemans-auto-ride.dts} |  14 +-
 arch/arm64/boot/dts/qcom/lemans-auto.dtsi     | 104 +++++++
 arch/arm64/boot/dts/qcom/lemans-evk.dts       | 291 ++++++++++++++++++
 .../{sa8775p-pmics.dtsi => lemans-pmics.dtsi} |   0
 ...775p-ride.dtsi => lemans-ride-common.dtsi} | 168 ----------
 .../qcom/lemans-ride-ethernet-88ea1512.dtsi   | 205 ++++++++++++
 .../qcom/lemans-ride-ethernet-aqr115c.dtsi    | 205 ++++++++++++
 ...qcs9100-ride-r3.dts => lemans-ride-r3.dts} |  12 +-
 .../dts/qcom/{sa8775p.dtsi => lemans.dtsi}    |  75 +++--
 arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts  |  47 ---
 13 files changed, 884 insertions(+), 305 deletions(-)
 rename arch/arm64/boot/dts/qcom/{sa8775p-ride.dts => lemans-auto-ride-r3.dts} (11%)
 rename arch/arm64/boot/dts/qcom/{qcs9100-ride.dts => lemans-auto-ride.dts} (18%)
 create mode 100644 arch/arm64/boot/dts/qcom/lemans-auto.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/lemans-evk.dts
 rename arch/arm64/boot/dts/qcom/{sa8775p-pmics.dtsi => lemans-pmics.dtsi} (100%)
 rename arch/arm64/boot/dts/qcom/{sa8775p-ride.dtsi => lemans-ride-common.dtsi} (87%)
 create mode 100644 arch/arm64/boot/dts/qcom/lemans-ride-ethernet-88ea1512.dtsi
 create mode 100644 arch/arm64/boot/dts/qcom/lemans-ride-ethernet-aqr115c.dtsi
 rename arch/arm64/boot/dts/qcom/{qcs9100-ride-r3.dts => lemans-ride-r3.dts} (36%)
 rename arch/arm64/boot/dts/qcom/{sa8775p.dtsi => lemans.dtsi} (99%)
 delete mode 100644 arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts


base-commit: 05adbee3ad528100ab0285c15c91100e19e10138
--
2.49.0
Re: [PATCH 0/7] Refactor sa8775p/qcs9100 to common names lemans-auto/lemans
Posted by Rob Herring (Arm) 2 months, 2 weeks ago
On Tue, 22 Jul 2025 20:19:19 +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.
> 
> Since, most of our platforms are IoT-based so "lemans" can be served as the
> default IoT variant, with "lemans-auto" derived from it. Accordingly:
>   - qcs9100/qcs9075 and its associated IoT platforms are renamed to lemans
>     which needs different memory-map. So, latest memory-map is updated
>     here as per IOT requirements.
>   - sa8775p and its associated platforms are renamed to "lemans-auto", which
>     is derived from "lemans", that retains the old automotive memory map to
>     support legacy use cases.
>   - Both lemans & lemans-auto are serving as non-safe chip and if needed
>     additional dtsi can be appended in the future to enable safety features.
> 
> Additionally:
>   - Refactor common daughter cards used in Ride/Ride-R3 platforms into a
>     common configuration. Also, introduce new files for different ethernet
>     capabilities in Ride/Ride-r3. Since Ethernet functionality in Ride/Ride-r3
>     is currently broken upstream, this patch focuses only on refactoring.
>   - Include support for qcs9075 EVK[1] platform as lemans-evk. Currently,
>     basic features are enabled supporting 'boot to shell'.
>   - Remove support for qcs9100-ride, as no platform currently exists for it.
> 
> Funtional impact to current boards with refactoring:
>   - No functional change on auto boards i.e sa8775p ride/ride-r3 boards
>     (renamed as lemans-auto ride/ride-r3), and it is verified by comparing
>     decompiled DTB (dtx_diff).
>   - qcs9100 ride-r3 (renamed as lemans-ride-r3) is having new memory-map
>     and rest other functionalities are still same.
> 
> [1] https://lore.kernel.org/all/20250612155437.146925-1-quic_wasimn@quicinc.com/
> 
> 
> ---
> Wasim Nazir (7):
>   arm64: dts: qcom: Rename sa8775p SoC to "lemans"
>   arm64: dts: qcom: Update memory-map for IoT platforms in lemans
>   arm64: dts: qcom: lemans: Separate out ethernet card for ride &
>     ride-r3
>   arm64: dts: qcom: lemans: Refactor ride/ride-r3 boards based on
>     daughter cards
>   arm64: dts: qcom: lemans: Rename boards and clean up unsupported
>     platforms
>   dt-bindings: arm: qcom: Refactor QCS9100 and SA8775P board names to
>     reflect Lemans variants
>   arm64: dts: qcom: Add lemans evaluation kit (EVK) initial board
>     support
> 
>  .../devicetree/bindings/arm/qcom.yaml         |  16 +-
>  arch/arm64/boot/dts/qcom/Makefile             |   8 +-
>  ...8775p-ride.dts => lemans-auto-ride-r3.dts} |  44 +--
>  ...{qcs9100-ride.dts => lemans-auto-ride.dts} |  14 +-
>  arch/arm64/boot/dts/qcom/lemans-auto.dtsi     | 104 +++++++
>  arch/arm64/boot/dts/qcom/lemans-evk.dts       | 291 ++++++++++++++++++
>  .../{sa8775p-pmics.dtsi => lemans-pmics.dtsi} |   0
>  ...775p-ride.dtsi => lemans-ride-common.dtsi} | 168 ----------
>  .../qcom/lemans-ride-ethernet-88ea1512.dtsi   | 205 ++++++++++++
>  .../qcom/lemans-ride-ethernet-aqr115c.dtsi    | 205 ++++++++++++
>  ...qcs9100-ride-r3.dts => lemans-ride-r3.dts} |  12 +-
>  .../dts/qcom/{sa8775p.dtsi => lemans.dtsi}    |  75 +++--
>  arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts  |  47 ---
>  13 files changed, 884 insertions(+), 305 deletions(-)
>  rename arch/arm64/boot/dts/qcom/{sa8775p-ride.dts => lemans-auto-ride-r3.dts} (11%)
>  rename arch/arm64/boot/dts/qcom/{qcs9100-ride.dts => lemans-auto-ride.dts} (18%)
>  create mode 100644 arch/arm64/boot/dts/qcom/lemans-auto.dtsi
>  create mode 100644 arch/arm64/boot/dts/qcom/lemans-evk.dts
>  rename arch/arm64/boot/dts/qcom/{sa8775p-pmics.dtsi => lemans-pmics.dtsi} (100%)
>  rename arch/arm64/boot/dts/qcom/{sa8775p-ride.dtsi => lemans-ride-common.dtsi} (87%)
>  create mode 100644 arch/arm64/boot/dts/qcom/lemans-ride-ethernet-88ea1512.dtsi
>  create mode 100644 arch/arm64/boot/dts/qcom/lemans-ride-ethernet-aqr115c.dtsi
>  rename arch/arm64/boot/dts/qcom/{qcs9100-ride-r3.dts => lemans-ride-r3.dts} (36%)
>  rename arch/arm64/boot/dts/qcom/{sa8775p.dtsi => lemans.dtsi} (99%)
>  delete mode 100644 arch/arm64/boot/dts/qcom/sa8775p-ride-r3.dts
> 
> 
> base-commit: 05adbee3ad528100ab0285c15c91100e19e10138
> --
> 2.49.0
> 
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


This patch series was applied (using b4) to base:
 Base: using specified base-commit 05adbee3ad528100ab0285c15c91100e19e10138

If this is not the correct base, please add 'base-commit' tag
(or use b4 which does this automatically)

New warnings running 'make CHECK_DTBS=y for arch/arm64/boot/dts/qcom/' for 20250722144926.995064-1-wasim.nazir@oss.qualcomm.com:

arch/arm64/boot/dts/qcom/lemans-auto-ride.dtb: bluetooth (qcom,wcn6855-bt): 'vddwlcx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride.dtb: bluetooth (qcom,wcn6855-bt): 'vddwlmx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride.dtb: bluetooth (qcom,wcn6855-bt): 'vddrfa1p8-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride.dtb: ethernet@23000000 (qcom,sa8775p-ethqos): Unevaluated properties are not allowed ('interconnect-names', 'interconnects' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/qcom,ethqos.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride.dtb: ethernet@23040000 (qcom,sa8775p-ethqos): Unevaluated properties are not allowed ('interconnect-names', 'interconnects' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/qcom,ethqos.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride.dtb: wcn6855-pmu (qcom,wcn6855-pmu): 'vddpmumx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/regulator/qcom,qca6390-pmu.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride.dtb: wcn6855-pmu (qcom,wcn6855-pmu): 'vddpmucx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/regulator/qcom,qca6390-pmu.yaml#
arch/arm64/boot/dts/qcom/lemans-ride-r3.dtb: bluetooth (qcom,wcn6855-bt): 'vddwlcx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-ride-r3.dtb: bluetooth (qcom,wcn6855-bt): 'vddwlmx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-ride-r3.dtb: bluetooth (qcom,wcn6855-bt): 'vddrfa1p8-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride-r3.dtb: bluetooth (qcom,wcn6855-bt): 'vddwlcx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride-r3.dtb: bluetooth (qcom,wcn6855-bt): 'vddwlmx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride-r3.dtb: bluetooth (qcom,wcn6855-bt): 'vddrfa1p8-supply' is a required property
	from schema $id: http://devicetree.org/schemas/net/bluetooth/qualcomm-bluetooth.yaml#
arch/arm64/boot/dts/qcom/lemans-ride-r3.dtb: ethernet@23000000 (qcom,sa8775p-ethqos): Unevaluated properties are not allowed ('interconnect-names', 'interconnects' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/qcom,ethqos.yaml#
arch/arm64/boot/dts/qcom/lemans-ride-r3.dtb: ethernet@23040000 (qcom,sa8775p-ethqos): Unevaluated properties are not allowed ('interconnect-names', 'interconnects' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/qcom,ethqos.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride-r3.dtb: ethernet@23000000 (qcom,sa8775p-ethqos): Unevaluated properties are not allowed ('interconnect-names', 'interconnects' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/qcom,ethqos.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride-r3.dtb: ethernet@23040000 (qcom,sa8775p-ethqos): Unevaluated properties are not allowed ('interconnect-names', 'interconnects' were unexpected)
	from schema $id: http://devicetree.org/schemas/net/qcom,ethqos.yaml#
arch/arm64/boot/dts/qcom/lemans-ride-r3.dtb: wcn6855-pmu (qcom,wcn6855-pmu): 'vddpmumx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/regulator/qcom,qca6390-pmu.yaml#
arch/arm64/boot/dts/qcom/lemans-ride-r3.dtb: wcn6855-pmu (qcom,wcn6855-pmu): 'vddpmucx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/regulator/qcom,qca6390-pmu.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride-r3.dtb: wcn6855-pmu (qcom,wcn6855-pmu): 'vddpmumx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/regulator/qcom,qca6390-pmu.yaml#
arch/arm64/boot/dts/qcom/lemans-auto-ride-r3.dtb: wcn6855-pmu (qcom,wcn6855-pmu): 'vddpmucx-supply' is a required property
	from schema $id: http://devicetree.org/schemas/regulator/qcom,qca6390-pmu.yaml#
Re: [PATCH 0/7] Refactor sa8775p/qcs9100 to common names lemans-auto/lemans
Posted by Krzysztof Kozlowski 2 months, 2 weeks ago
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:

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

Best regards,
Krzysztof
Re: [PATCH 0/7] Refactor sa8775p/qcs9100 to common names lemans-auto/lemans
Posted by Bjorn Andersson 2 months, 1 week ago
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@linaro.org>
> 

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

Regards,
Bjorn

> Best regards,
> Krzysztof
>