* [PATCH 0/5] ARM: zynq: Update DTs based on the latest YAML checking
@ 2020-11-26 13:24 Michal Simek
2020-11-26 13:25 ` [PATCH 1/5] ARM: zynq: Fix compatible string for adi,adxl345 chip Michal Simek
` (4 more replies)
0 siblings, 5 replies; 6+ messages in thread
From: Michal Simek @ 2020-11-26 13:24 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, git
Cc: Rob Herring, devicetree, linux-arm-kernel
Hi,
several issues have been reported from binding check. The series is
addressing most of them.
Patches are based on
https://github.com/Xilinx/linux-xlnx/tree/zynq/dt
Thanks,
Michal
Michal Simek (5):
ARM: zynq: Fix compatible string for adi,adxl345 chip
ARM: zynq: Rename bus to be align with simple-bus yaml
ARM: zynq: Fix leds subnode name for zc702/zybo-z7
ARM: zynq: Fix OCM mapping to be aligned with binding on zc702
ARM: zynq: Convert at25 binding to new description on zc770-xm013
arch/arm/boot/dts/zynq-7000.dtsi | 2 +-
arch/arm/boot/dts/zynq-zc702.dts | 8 +++++++-
arch/arm/boot/dts/zynq-zc770-xm013.dts | 7 +++----
arch/arm/boot/dts/zynq-zturn-common.dtsi | 2 +-
arch/arm/boot/dts/zynq-zybo-z7.dts | 2 +-
5 files changed, 13 insertions(+), 8 deletions(-)
--
2.29.2
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/5] ARM: zynq: Fix compatible string for adi,adxl345 chip
2020-11-26 13:24 [PATCH 0/5] ARM: zynq: Update DTs based on the latest YAML checking Michal Simek
@ 2020-11-26 13:25 ` Michal Simek
2020-11-26 13:25 ` [PATCH 2/5] ARM: zynq: Rename bus to be align with simple-bus yaml Michal Simek
` (3 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-11-26 13:25 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, git
Cc: Rob Herring, devicetree, linux-arm-kernel
The commit e359a29225dd ("dt-bindings: iio: accel: adxl345: switch to YAML
bindings") switched binding to yaml and the following error pop up:
../zynq-zturn-v5.dt.yaml: accelerometer@53: compatible: 'oneOf' conditional
failed, one must be
fixed:
['adi,adxl345', 'adxl345', 'adi,adxl34x', 'adxl34x'] is too long
Additional items are not allowed ('adi,adxl34x', 'adxl34x' were unexpected)
Additional items are not allowed ('adxl345', 'adi,adxl34x', 'adxl34x' were
unexpected)
'adi,adxl346' was expected
'adi,adxl345' was expected
Use only one compatible string to be aligned with the binding.
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
arch/arm/boot/dts/zynq-zturn-common.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/zynq-zturn-common.dtsi b/arch/arm/boot/dts/zynq-zturn-common.dtsi
index 84f3c85c5bab..bf5d1c4568b0 100644
--- a/arch/arm/boot/dts/zynq-zturn-common.dtsi
+++ b/arch/arm/boot/dts/zynq-zturn-common.dtsi
@@ -104,7 +104,7 @@ stlm75@49 {
};
accelerometer@53 {
- compatible = "adi,adxl345", "adxl345", "adi,adxl34x", "adxl34x";
+ compatible = "adi,adxl345";
reg = <0x53>;
interrupt-parent = <&intc>;
interrupts = <0x0 0x1e 0x4>;
--
2.29.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/5] ARM: zynq: Rename bus to be align with simple-bus yaml
2020-11-26 13:24 [PATCH 0/5] ARM: zynq: Update DTs based on the latest YAML checking Michal Simek
2020-11-26 13:25 ` [PATCH 1/5] ARM: zynq: Fix compatible string for adi,adxl345 chip Michal Simek
@ 2020-11-26 13:25 ` Michal Simek
2020-11-26 13:25 ` [PATCH 3/5] ARM: zynq: Fix leds subnode name for zc702/zybo-z7 Michal Simek
` (2 subsequent siblings)
4 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-11-26 13:25 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, git
Cc: Rob Herring, devicetree, linux-arm-kernel
Rename amba to AXI. Based on Xilinx Zynq TRM (Chapter 5) chip is "AXI
point-to-point channels for communicating addresses, data, and response
transactions between master and slave clients. This ARM AMBA 3.0..."
Issues are reported as:
.. amba: $nodename:0: 'amba' does not match
'^([a-z][a-z0-9\\-]+-bus|bus|soc|axi|ahb|apb)(@[0-9a-f]+)?$'
From schema:
../github.com/devicetree-org/dt-schema/dtschema/schemas/simple-bus.yaml
Similar change has been done for Xilinx ZynqMP SoC.
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
arch/arm/boot/dts/zynq-7000.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/zynq-7000.dtsi b/arch/arm/boot/dts/zynq-7000.dtsi
index db3899b07992..df9ad831cf05 100644
--- a/arch/arm/boot/dts/zynq-7000.dtsi
+++ b/arch/arm/boot/dts/zynq-7000.dtsi
@@ -92,7 +92,7 @@ replicator_in_port0: endpoint {
};
};
- amba: amba {
+ amba: axi {
compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <1>;
--
2.29.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/5] ARM: zynq: Fix leds subnode name for zc702/zybo-z7
2020-11-26 13:24 [PATCH 0/5] ARM: zynq: Update DTs based on the latest YAML checking Michal Simek
2020-11-26 13:25 ` [PATCH 1/5] ARM: zynq: Fix compatible string for adi,adxl345 chip Michal Simek
2020-11-26 13:25 ` [PATCH 2/5] ARM: zynq: Rename bus to be align with simple-bus yaml Michal Simek
@ 2020-11-26 13:25 ` Michal Simek
2020-11-26 13:25 ` [PATCH 4/5] ARM: zynq: Fix OCM mapping to be aligned with binding on zc702 Michal Simek
2020-11-26 13:25 ` [PATCH 5/5] ARM: zynq: Convert at25 binding to new description on zc770-xm013 Michal Simek
4 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-11-26 13:25 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, git
Cc: Rob Herring, devicetree, linux-arm-kernel
Fix the leds subnode names to match (^led-[0-9a-f]$|led).
Similar change has been also done by commit 9a19a39ee48b ("arm64: dts:
zynqmp: Fix leds subnode name for zcu100/ultra96 v1").
The patch is fixing these warnings:
.../zynq-zc702.dt.yaml: leds: 'ds23' does not match any of the regexes:
'(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+'
From schema: .../Documentation/devicetree/bindings/leds/leds-gpio.yaml
.../zynq-zybo-z7.dt.yaml: gpio-leds: 'ld4' does not match any of the
regexes: '(^led-[0-9a-f]$|led)', 'pinctrl-[0-9]+'
From schema: .../Documentation/devicetree/bindings/leds/leds-gpio.yaml
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
arch/arm/boot/dts/zynq-zc702.dts | 2 +-
arch/arm/boot/dts/zynq-zybo-z7.dts | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
index 27cd6cb52f1b..10a7d0b8cf8b 100644
--- a/arch/arm/boot/dts/zynq-zc702.dts
+++ b/arch/arm/boot/dts/zynq-zc702.dts
@@ -49,7 +49,7 @@ sw13 {
leds {
compatible = "gpio-leds";
- ds23 {
+ led-ds23 {
label = "ds23";
gpios = <&gpio0 10 0>;
linux,default-trigger = "heartbeat";
diff --git a/arch/arm/boot/dts/zynq-zybo-z7.dts b/arch/arm/boot/dts/zynq-zybo-z7.dts
index 357b78a5c11b..7b87e10d3953 100644
--- a/arch/arm/boot/dts/zynq-zybo-z7.dts
+++ b/arch/arm/boot/dts/zynq-zybo-z7.dts
@@ -25,7 +25,7 @@ chosen {
gpio-leds {
compatible = "gpio-leds";
- ld4 {
+ led-ld4 {
label = "zynq-zybo-z7:green:ld4";
gpios = <&gpio0 7 GPIO_ACTIVE_HIGH>;
};
--
2.29.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 4/5] ARM: zynq: Fix OCM mapping to be aligned with binding on zc702
2020-11-26 13:24 [PATCH 0/5] ARM: zynq: Update DTs based on the latest YAML checking Michal Simek
` (2 preceding siblings ...)
2020-11-26 13:25 ` [PATCH 3/5] ARM: zynq: Fix leds subnode name for zc702/zybo-z7 Michal Simek
@ 2020-11-26 13:25 ` Michal Simek
2020-11-26 13:25 ` [PATCH 5/5] ARM: zynq: Convert at25 binding to new description on zc770-xm013 Michal Simek
4 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-11-26 13:25 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, git
Cc: Rob Herring, devicetree, linux-arm-kernel
The commit f69629919942 ("dt-bindings: sram: Convert SRAM bindings to
json-schema") converted binding to yaml and some missing required
properties started to be reported. Align binding based on it.
The patch is fixing these warnings:
.../zynq-zc702.dt.yaml: sram@fffc0000: '#address-cells' is a required property
.../zynq-zc702.dt.yaml: sram@fffc0000: '#size-cells' is a required property
.../zynq-zc702.dt.yaml: sram@fffc0000: 'ranges' is a required property
From schema: .../Documentation/devicetree/bindings/sram/sram.yaml
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
arch/arm/boot/dts/zynq-zc702.dts | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
index 10a7d0b8cf8b..cf70aff26c66 100644
--- a/arch/arm/boot/dts/zynq-zc702.dts
+++ b/arch/arm/boot/dts/zynq-zc702.dts
@@ -66,6 +66,12 @@ &amba {
ocm: sram@fffc0000 {
compatible = "mmio-sram";
reg = <0xfffc0000 0x10000>;
+ #address-cells = <1>;
+ #size-cells = <1>;
+ ranges = <0 0xfffc0000 0x10000>;
+ ocm-sram@0 {
+ reg = <0x0 0x10000>;
+ };
};
};
--
2.29.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 5/5] ARM: zynq: Convert at25 binding to new description on zc770-xm013
2020-11-26 13:24 [PATCH 0/5] ARM: zynq: Update DTs based on the latest YAML checking Michal Simek
` (3 preceding siblings ...)
2020-11-26 13:25 ` [PATCH 4/5] ARM: zynq: Fix OCM mapping to be aligned with binding on zc702 Michal Simek
@ 2020-11-26 13:25 ` Michal Simek
4 siblings, 0 replies; 6+ messages in thread
From: Michal Simek @ 2020-11-26 13:25 UTC (permalink / raw)
To: linux-kernel, monstr, michal.simek, git
Cc: Rob Herring, devicetree, linux-arm-kernel
The commit f8f79fa6bb25 ("dt-bindings: at25: convert the binding document
to yaml") converted binding to yaml and 3 deprecated properties pop up.
The patch is fixing these warnings:
.../zynq-zc770-xm013.dt.yaml: eeprom@2: 'pagesize' is a required property
.../zynq-zc770-xm013.dt.yaml: eeprom@2: 'size' is a required property
.../zynq-zc770-xm013.dt.yaml: eeprom@2: 'address-width' is a required property
From schema: .../Documentation/devicetree/bindings/eeprom/at25.yaml
by converting them to new binding.
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---
arch/arm/boot/dts/zynq-zc770-xm013.dts | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/zynq-zc770-xm013.dts b/arch/arm/boot/dts/zynq-zc770-xm013.dts
index 4ae2c85df3a0..38d96adc870c 100644
--- a/arch/arm/boot/dts/zynq-zc770-xm013.dts
+++ b/arch/arm/boot/dts/zynq-zc770-xm013.dts
@@ -63,13 +63,12 @@ &spi0 {
num-cs = <4>;
is-decoded-cs = <0>;
eeprom: eeprom@2 {
- at25,byte-len = <8192>;
- at25,addr-mode = <2>;
- at25,page-size = <32>;
-
compatible = "atmel,at25";
reg = <2>;
spi-max-frequency = <1000000>;
+ size = <8192>;
+ address-width = <16>;
+ pagesize = <32>;
};
};
--
2.29.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-11-26 13:25 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-26 13:24 [PATCH 0/5] ARM: zynq: Update DTs based on the latest YAML checking Michal Simek
2020-11-26 13:25 ` [PATCH 1/5] ARM: zynq: Fix compatible string for adi,adxl345 chip Michal Simek
2020-11-26 13:25 ` [PATCH 2/5] ARM: zynq: Rename bus to be align with simple-bus yaml Michal Simek
2020-11-26 13:25 ` [PATCH 3/5] ARM: zynq: Fix leds subnode name for zc702/zybo-z7 Michal Simek
2020-11-26 13:25 ` [PATCH 4/5] ARM: zynq: Fix OCM mapping to be aligned with binding on zc702 Michal Simek
2020-11-26 13:25 ` [PATCH 5/5] ARM: zynq: Convert at25 binding to new description on zc770-xm013 Michal Simek
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).