Re: [PATCH 3/4] arm64: dts: amd/seattle: Fix bus, mmc, and ethernet node names

Re: [PATCH 3/4] arm64: dts: amd/seattle: Fix bus, mmc, and ethernet node names

[kernel test robot]

:::::: 
:::::: Manual check reason: "dtcheck: binding changes may go via different trees"
:::::: 

BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20250306-dt-amd-fixes-v1-3-011c423ba99a@kernel.org>
References: <20250306-dt-amd-fixes-v1-3-011c423ba99a@kernel.org>
TO: "Rob Herring (Arm)" <robh@kernel.org>
TO: soc@kernel.org
TO: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
TO: Tom Lendacky <thomas.lendacky@amd.com>
TO: Krzysztof Kozlowski <krzk@kernel.org>
TO: Conor Dooley <conor+dt@kernel.org>
TO: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
TO: Richard Cochran <richardcochran@gmail.com>
CC: devicetree@vger.kernel.org
CC: linux-kernel@vger.kernel.org

Hi Rob,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 2014c95afecee3e76ca4a56956a936e23283f05b]

url:    https://github.com/intel-lab-lkp/linux/commits/Rob-Herring-Arm/arm64-dts-amd-seattle-Base-Overdrive-B1-on-top-of-B0-version/20250307-090945
base:   2014c95afecee3e76ca4a56956a936e23283f05b
patch link:    https://lore.kernel.org/r/20250306-dt-amd-fixes-v1-3-011c423ba99a%40kernel.org
patch subject: [PATCH 3/4] arm64: dts: amd/seattle: Fix bus, mmc, and ethernet node names
:::::: branch date: 34 hours ago
:::::: commit date: 34 hours ago
config: arm64-randconfig-051-20250308 (https://download.01.org/0day-ci/archive/20250308/202503081901.SuviWScl-lkp@intel.com/config)
compiler: clang version 16.0.6 (https://github.com/llvm/llvm-project 7cbf1a2591520c2491aa35339f227775f4d3adf6)
dtschema version: 2025.3.dev3+gabf9328
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250308/202503081901.SuviWScl-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202503081901.SuviWScl-lkp@intel.com/

dtcheck warnings: (new ones prefixed by >>)
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: /: failed to match any schema with compatible: ['amd,seattle-overdrive', 'amd,seattle']
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: /: failed to match any schema with compatible: ['amd,seattle-overdrive', 'amd,seattle']
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: sata@e0300000: Unevaluated properties are not allowed ('iommus' was unexpected)
   	from schema $id: http://devicetree.org/schemas/ata/snps,dwc-ahci.yaml#
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: spi@e1020000: Unevaluated properties are not allowed ('spi-controller' was unexpected)
   	from schema $id: http://devicetree.org/schemas/spi/spi-pl022.yaml#
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: spi@e1030000: Unevaluated properties are not allowed ('spi-controller' was unexpected)
   	from schema $id: http://devicetree.org/schemas/spi/spi-pl022.yaml#
>> arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: /bus/ccp@e0100000: failed to match any schema with compatible: ['amd,ccp-seattle-v1a']
>> arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: /bus/ethernet@e0700000: failed to match any schema with compatible: ['amd,xgbe-seattle-v1a']
>> arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dtb: /bus/ethernet@e0900000: failed to match any schema with compatible: ['amd,xgbe-seattle-v1a']
--
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: /: failed to match any schema with compatible: ['amd,seattle-overdrive', 'amd,seattle']
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: /: failed to match any schema with compatible: ['amd,seattle-overdrive', 'amd,seattle']
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: sata@e0300000: Unevaluated properties are not allowed ('iommus' was unexpected)
   	from schema $id: http://devicetree.org/schemas/ata/snps,dwc-ahci.yaml#
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: sata@e0d00000: Unevaluated properties are not allowed ('iommus' was unexpected)
   	from schema $id: http://devicetree.org/schemas/ata/snps,dwc-ahci.yaml#
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: spi@e1020000: Unevaluated properties are not allowed ('spi-controller' was unexpected)
   	from schema $id: http://devicetree.org/schemas/spi/spi-pl022.yaml#
   arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: spi@e1030000: Unevaluated properties are not allowed ('spi-controller' was unexpected)
   	from schema $id: http://devicetree.org/schemas/spi/spi-pl022.yaml#
>> arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: /bus/ccp@e0100000: failed to match any schema with compatible: ['amd,ccp-seattle-v1a']
>> arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: /bus/ethernet@e0700000: failed to match any schema with compatible: ['amd,xgbe-seattle-v1a']
>> arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dtb: /bus/ethernet@e0900000: failed to match any schema with compatible: ['amd,xgbe-seattle-v1a']

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

[PATCH 0/4] arm64: dts: AMD Seattle clean-ups

[Rob Herring (Arm)]

This short series fixes most of the schema warnings for AMD Seattle.

Arnd, Not sure if the AMD maintainers are active. If not, please take 
this directly.

Signed-off-by: "Rob Herring (Arm)" <robh@kernel.org>
---
Rob Herring (Arm) (4):
      arm64: dts: amd/seattle: Base Overdrive B1 on top of B0 version
      arm64: dts: amd/seattle: Move and simplify fixed clocks
      arm64: dts: amd/seattle: Fix bus, mmc, and ethernet node names
      arm64: dts: amd/seattle: Drop undocumented "spi-controller" properties

 arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dts |  2 +-
 arch/arm64/boot/dts/amd/amd-overdrive-rev-b1.dts | 61 +-----------------------
 arch/arm64/boot/dts/amd/amd-seattle-clks.dtsi    | 24 +++-------
 arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi     |  8 ++--
 arch/arm64/boot/dts/amd/amd-seattle-xgbe-b.dtsi  | 32 +------------
 5 files changed, 13 insertions(+), 114 deletions(-)
---
base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b
change-id: 20250306-dt-amd-fixes-6a8984d2d279

Best regards,
-- 
Rob Herring (Arm) <robh@kernel.org>

[PATCH 3/4] arm64: dts: amd/seattle: Fix bus, mmc, and ethernet node names

[Rob Herring (Arm)]

Use preferred node names for bus, mmc, and ethernet.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dts | 2 +-
 arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi     | 2 +-
 arch/arm64/boot/dts/amd/amd-seattle-xgbe-b.dtsi  | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dts b/arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dts
index 7c82d90e940d..8862adae44e9 100644
--- a/arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dts
+++ b/arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dts
@@ -58,7 +58,7 @@ &spi0 {
 
 &spi1 {
 	status = "okay";
-	sdcard0: sdcard@0 {
+	sdcard0: mmc@0 {
 		compatible = "mmc-spi-slot";
 		reg = <0>;
 		spi-max-frequency = <20000000>;
diff --git a/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi b/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi
index 6a073d13478d..aa7fe884a458 100644
--- a/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi
+++ b/arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi
@@ -40,7 +40,7 @@ timer {
 			     <1 10 0xff04>;
 	};
 
-	smb0: smb {
+	smb0: bus {
 		compatible = "simple-bus";
 		#address-cells = <2>;
 		#size-cells = <2>;
diff --git a/arch/arm64/boot/dts/amd/amd-seattle-xgbe-b.dtsi b/arch/arm64/boot/dts/amd/amd-seattle-xgbe-b.dtsi
index 056dd3e35042..18b0c2dd1b2d 100644
--- a/arch/arm64/boot/dts/amd/amd-seattle-xgbe-b.dtsi
+++ b/arch/arm64/boot/dts/amd/amd-seattle-xgbe-b.dtsi
@@ -5,7 +5,7 @@
  * Copyright (C) 2015 Advanced Micro Devices, Inc.
  */
 
-	xgmac0: xgmac@e0700000 {
+	xgmac0: ethernet@e0700000 {
 		compatible = "amd,xgbe-seattle-v1a";
 		reg = <0 0xe0700000 0 0x80000>,
 		      <0 0xe0780000 0 0x80000>,
@@ -31,7 +31,7 @@ xgmac0: xgmac@e0700000 {
 		dma-coherent;
 	};
 
-	xgmac1: xgmac@e0900000 {
+	xgmac1: ethernet@e0900000 {
 		compatible = "amd,xgbe-seattle-v1a";
 		reg = <0 0xe0900000 0 0x80000>,
 		      <0 0xe0980000 0 0x80000>,

-- 
2.47.2

Re: [PATCH 3/4] arm64: dts: amd/seattle: Fix bus, mmc, and ethernet node names

[Krzysztof Kozlowski]

On 07/03/2025 02:07, Rob Herring (Arm) wrote:
> Use preferred node names for bus, mmc, and ethernet.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> ---
>  arch/arm64/boot/dts/amd/amd-overdrive-rev-b0.dts | 2 +-
>  arch/arm64/boot/dts/amd/amd-seattle-soc.dtsi     | 2 +-
>  arch/arm64/boot/dts/amd/amd-seattle-xgbe-b.dtsi  | 4 ++--
>  3 files changed, 4 insertions(+), 4 deletions(-)
> 


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

Best regards,
Krzysztof