Message-Id: <175322585194.629714.3675361832955503635.robh@kernel.org>
Date: Tue, 22 Jul 2025 18:15:15 -0500
From: "Rob Herring (Arm)" <robh@...nel.org>
To: Wasim Nazir <wasim.nazir@....qualcomm.com>
Cc: kernel@....qualcomm.com, devicetree@...r.kernel.org, 
 Conor Dooley <conor+dt@...nel.org>, linux-kernel@...r.kernel.org, 
 Konrad Dybcio <konradybcio@...nel.org>, 
 Richard Cochran <richardcochran@...il.com>, netdev@...r.kernel.org, 
 Bjorn Andersson <andersson@...nel.org>, linux-arm-msm@...r.kernel.org, 
 Krzysztof Kozlowski <krzk+dt@...nel.org>
Subject: Re: [PATCH 0/7] Refactor sa8775p/qcs9100 to common names
 lemans-auto/lemans


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@....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@...00000 (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@...40000 (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@...00000 (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@...40000 (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@...00000 (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@...40000 (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#

Powered by blists - more mailing lists