All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/8] xilinx: Various DT changes and syncups
@ 2020-01-09 14:18 Michal Simek
  2020-01-09 14:18 ` [PATCH 1/8] arm64: zynqmp: Remove incorrect phy from DT for zcu102-revB up Michal Simek
                   ` (8 more replies)
  0 siblings, 9 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

Hi,

I am sending some patches to fix bugs we found in xilinx release cycle.
Also do some syncs to keep kernel/u-boot dts in pseudo sync.

Thanks,
Michal


Jan Kiszka (1):
  arm64: dts: zcu100-revC: Give wifi some time after power-on

Michal Simek (4):
  arm64: zynqmp: Remove incorrect phy from DT for zcu102-revB up
  arm64: dts: zynqmp: Fix node names which contain "_"
  arm64: zynqmp: Sync gpio-controller name location
  arm64: zynqmp: Sync gem clock nodes with mainline Linux

Rob Herring (1):
  arm64: dts: Remove inconsistent use of 'arm,armv8' compatible string

Sudeep Holla (1):
  arm64: dts: zynqmp: replace gpio-key,wakeup with wakeup-source
    property

Zumeng Chen (1):
  ARM: dts: zynq: enablement of coresight topology

 arch/arm/dts/zynq-7000.dtsi         | 135 ++++++++++++++++++++++++++++
 arch/arm/dts/zynqmp-clk-ccf.dtsi    |  22 +++--
 arch/arm/dts/zynqmp-clk.dtsi        |   4 +-
 arch/arm/dts/zynqmp-zcu100-revC.dts |   7 +-
 arch/arm/dts/zynqmp-zcu102-revA.dts |  12 +--
 arch/arm/dts/zynqmp-zcu102-revB.dts |   2 +-
 arch/arm/dts/zynqmp-zcu106-revA.dts |   4 +-
 arch/arm/dts/zynqmp-zcu111-revA.dts |   4 +-
 arch/arm/dts/zynqmp.dtsi            |  14 +--
 9 files changed, 172 insertions(+), 32 deletions(-)

-- 
2.24.0

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 1/8] arm64: zynqmp: Remove incorrect phy from DT for zcu102-revB up
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-09 14:18 ` [PATCH 2/8] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string Michal Simek
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

zcu102-revB/up are not removing phy from revA properly because of incorrect
name.

Fixes: 2975a42b42c5 ("arm64: zynqmp: Use ethernet-phy as node name for ethernet phys")
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp-zcu102-revB.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/zynqmp-zcu102-revB.dts b/arch/arm/dts/zynqmp-zcu102-revB.dts
index 2132024a253d..38ec18816456 100644
--- a/arch/arm/dts/zynqmp-zcu102-revB.dts
+++ b/arch/arm/dts/zynqmp-zcu102-revB.dts
@@ -25,7 +25,7 @@
 		/* reset-gpios = <&tca6416_u97 6 GPIO_ACTIVE_LOW>; */
 	};
 	/* Cleanup from RevA */
-	/delete-node/ phy at 21;
+	/delete-node/ ethernet-phy at 21;
 };
 
 /* Fix collision with u61 */
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/8] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
  2020-01-09 14:18 ` [PATCH 1/8] arm64: zynqmp: Remove incorrect phy from DT for zcu102-revB up Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-10 15:36   ` Masahiro Yamada
  2020-01-09 14:18 ` [PATCH 3/8] arm64: dts: zynqmp: Fix node names which contain "_" Michal Simek
                   ` (6 subsequent siblings)
  8 siblings, 1 reply; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

From: Rob Herring <robh@kernel.org>

The 'arm,armv8' compatible string is only for software models. It adds
little value otherwise and is inconsistently used as a fallback on some
platforms. Remove it from those platforms.

This fixes warnings generated by the DT schema.

Reported-by: Michal Simek <michal.simek@xilinx.com>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Acked-by: Antoine Tenart <antoine.tenart@bootlin.com>
Acked-by: Nishanth Menon <nm@ti.com>
Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Acked-by: Chanho Min <chanho.min@lge.com>
Acked-by: Krzysztof Kozlowski <krzk@kernel.org>
Acked-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Acked-by: Gregory CLEMENT <gregory.clement@bootlin.com>
Acked-by: Thierry Reding <treding@nvidia.com>
Acked-by: Heiko Stuebner <heiko@sntech.de>
Acked-by: Simon Horman <horms+renesas@verge.net.au>
Acked-by: Tero Kristo <t-kristo@ti.com>
Acked-by: Wei Xu <xuwei5@hisilicon.com>
Acked-by: Liviu Dudau <liviu.dudau@arm.com>
Acked-by: Matthias Brugger <matthias.bgg@gmail.com>
Acked-by: Michal Simek <michal.simek@xilinx.com>
Acked-by: Scott Branden <scott.branden@broadcom.com>
Acked-by: Kevin Hilman <khilman@baylibre.com>
Acked-by: Chunyan Zhang <zhang.lyra@gmail.com>
Acked-by: Robert Richter <rrichter@cavium.com>
Acked-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
Acked-by: Dinh Nguyen <dinguyen@kernel.org>
Signed-off-by: Rob Herring <robh@kernel.org>
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp.dtsi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi
index b453941baf05..8646c4c3a340 100644
--- a/arch/arm/dts/zynqmp.dtsi
+++ b/arch/arm/dts/zynqmp.dtsi
@@ -25,7 +25,7 @@
 		#size-cells = <0>;
 
 		cpu0: cpu at 0 {
-			compatible = "arm,cortex-a53", "arm,armv8";
+			compatible = "arm,cortex-a53";
 			device_type = "cpu";
 			enable-method = "psci";
 			operating-points-v2 = <&cpu_opp_table>;
@@ -34,7 +34,7 @@
 		};
 
 		cpu1: cpu at 1 {
-			compatible = "arm,cortex-a53", "arm,armv8";
+			compatible = "arm,cortex-a53";
 			device_type = "cpu";
 			enable-method = "psci";
 			reg = <0x1>;
@@ -43,7 +43,7 @@
 		};
 
 		cpu2: cpu at 2 {
-			compatible = "arm,cortex-a53", "arm,armv8";
+			compatible = "arm,cortex-a53";
 			device_type = "cpu";
 			enable-method = "psci";
 			reg = <0x2>;
@@ -52,7 +52,7 @@
 		};
 
 		cpu3: cpu at 3 {
-			compatible = "arm,cortex-a53", "arm,armv8";
+			compatible = "arm,cortex-a53";
 			device_type = "cpu";
 			enable-method = "psci";
 			reg = <0x3>;
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 3/8] arm64: dts: zynqmp: Fix node names which contain "_"
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
  2020-01-09 14:18 ` [PATCH 1/8] arm64: zynqmp: Remove incorrect phy from DT for zcu102-revB up Michal Simek
  2020-01-09 14:18 ` [PATCH 2/8] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-09 14:18 ` [PATCH 4/8] arm64: dts: zcu100-revC: Give wifi some time after power-on Michal Simek
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

s/_/-/ for node names.

It fixes warnings like this:
... Warning (node_name_chars_strict): /cpu_opp_table:
Character '_' not recommended in node name ...

Issues reported by make dtbs W=12

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp-clk.dtsi        |  4 ++--
 arch/arm/dts/zynqmp-zcu100-revC.dts |  4 ++--
 arch/arm/dts/zynqmp-zcu102-revA.dts | 10 +++++-----
 arch/arm/dts/zynqmp-zcu106-revA.dts |  2 +-
 arch/arm/dts/zynqmp-zcu111-revA.dts |  2 +-
 arch/arm/dts/zynqmp.dtsi            |  4 ++--
 6 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/arch/arm/dts/zynqmp-clk.dtsi b/arch/arm/dts/zynqmp-clk.dtsi
index 9ef55ad0d18e..c9464ec8eb27 100644
--- a/arch/arm/dts/zynqmp-clk.dtsi
+++ b/arch/arm/dts/zynqmp-clk.dtsi
@@ -61,13 +61,13 @@
 		clock-accuracy = <100>;
 	};
 
-	dpdma_clk: dpdma_clk {
+	dpdma_clk: dpdma-clk {
 		compatible = "fixed-clock";
 		#clock-cells = <0x0>;
 		clock-frequency = <533000000>;
 	};
 
-	drm_clock: drm_clock {
+	drm_clock: drm-clock {
 		compatible = "fixed-clock";
 		#clock-cells = <0x0>;
 		clock-frequency = <262750000>;
diff --git a/arch/arm/dts/zynqmp-zcu100-revC.dts b/arch/arm/dts/zynqmp-zcu100-revC.dts
index 106b56b66222..02573a1f3294 100644
--- a/arch/arm/dts/zynqmp-zcu100-revC.dts
+++ b/arch/arm/dts/zynqmp-zcu100-revC.dts
@@ -95,7 +95,7 @@
 			linux,default-trigger = "bluetooth-power";
 		};
 
-		vbus_det { /* U5 USB5744 VBUS detection via MIO25 */
+		vbus-det { /* U5 USB5744 VBUS detection via MIO25 */
 			label = "vbus_det";
 			gpios = <&gpio 25 GPIO_ACTIVE_HIGH>;
 			default-state = "on";
@@ -120,7 +120,7 @@
 		regulator-boot-on;
 	};
 
-	sdio_pwrseq: sdio_pwrseq {
+	sdio_pwrseq: sdio-pwrseq {
 		compatible = "mmc-pwrseq-simple";
 		reset-gpios = <&gpio 7 GPIO_ACTIVE_LOW>; /* WIFI_EN */
 	};
diff --git a/arch/arm/dts/zynqmp-zcu102-revA.dts b/arch/arm/dts/zynqmp-zcu102-revA.dts
index 78110c490e8d..7d2f59b88c6f 100644
--- a/arch/arm/dts/zynqmp-zcu102-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu102-revA.dts
@@ -58,7 +58,7 @@
 
 	leds {
 		compatible = "gpio-leds";
-		heartbeat_led {
+		heartbeat-led {
 			label = "heartbeat";
 			gpios = <&gpio 23 GPIO_ACTIVE_HIGH>;
 			linux,default-trigger = "heartbeat";
@@ -213,25 +213,25 @@
 		gpio-line-names = "PS_GTR_LAN_SEL0", "PS_GTR_LAN_SEL1", "PS_GTR_LAN_SEL2", "PS_GTR_LAN_SEL3",
 				"PCI_CLK_DIR_SEL", "IIC_MUX_RESET_B", "GEM3_EXP_RESET_B",
 				"", "", "", "", "", "", "", "", "";
-		gtr_sel0 {
+		gtr-sel0 {
 			gpio-hog;
 			gpios = <0 0>;
 			output-low; /* PCIE = 0, DP = 1 */
 			line-name = "sel0";
 		};
-		gtr_sel1 {
+		gtr-sel1 {
 			gpio-hog;
 			gpios = <1 0>;
 			output-high; /* PCIE = 0, DP = 1 */
 			line-name = "sel1";
 		};
-		gtr_sel2 {
+		gtr-sel2 {
 			gpio-hog;
 			gpios = <2 0>;
 			output-high; /* PCIE = 0, USB0 = 1 */
 			line-name = "sel2";
 		};
-		gtr_sel3 {
+		gtr-sel3 {
 			gpio-hog;
 			gpios = <3 0>;
 			output-high; /* PCIE = 0, SATA = 1 */
diff --git a/arch/arm/dts/zynqmp-zcu106-revA.dts b/arch/arm/dts/zynqmp-zcu106-revA.dts
index 7c73ff283e7a..6faac956848b 100644
--- a/arch/arm/dts/zynqmp-zcu106-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu106-revA.dts
@@ -58,7 +58,7 @@
 
 	leds {
 		compatible = "gpio-leds";
-		heartbeat_led {
+		heartbeat-led {
 			label = "heartbeat";
 			gpios = <&gpio 23 GPIO_ACTIVE_HIGH>;
 			linux,default-trigger = "heartbeat";
diff --git a/arch/arm/dts/zynqmp-zcu111-revA.dts b/arch/arm/dts/zynqmp-zcu111-revA.dts
index ca95d7e059a8..5b648ceb2df6 100644
--- a/arch/arm/dts/zynqmp-zcu111-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu111-revA.dts
@@ -58,7 +58,7 @@
 
 	leds {
 		compatible = "gpio-leds";
-		heartbeat_led {
+		heartbeat-led {
 			label = "heartbeat";
 			gpios = <&gpio 23 GPIO_ACTIVE_HIGH>;
 			linux,default-trigger = "heartbeat";
diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi
index 8646c4c3a340..232fd0c11cd7 100644
--- a/arch/arm/dts/zynqmp.dtsi
+++ b/arch/arm/dts/zynqmp.dtsi
@@ -74,7 +74,7 @@
 		};
 	};
 
-	cpu_opp_table: cpu_opp_table {
+	cpu_opp_table: cpu-opp-table {
 		compatible = "operating-points-v2";
 		opp-shared;
 		opp00 {
@@ -252,7 +252,7 @@
 		};
 	};
 
-	amba_apu: amba_apu at 0 {
+	amba_apu: amba-apu at 0 {
 		compatible = "simple-bus";
 		#address-cells = <2>;
 		#size-cells = <1>;
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 4/8] arm64: dts: zcu100-revC: Give wifi some time after power-on
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
                   ` (2 preceding siblings ...)
  2020-01-09 14:18 ` [PATCH 3/8] arm64: dts: zynqmp: Fix node names which contain "_" Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-09 14:18 ` [PATCH 5/8] arm64: dts: zynqmp: replace gpio-key, wakeup with wakeup-source property Michal Simek
                   ` (4 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

From: Jan Kiszka <jan.kiszka@siemens.com>

Somewhere along recent changes to power control of the wl1831, power-on
became very unreliable on the Ultra96, failing like this:

wl1271_sdio: probe of mmc2:0001:1 failed with error -16
wl1271_sdio: probe of mmc2:0001:2 failed with error -16

After playing with some dt parameters and comparing to other users of
this chip, it turned out we need some power-on delay to make things
stable again. In contrast to those other users which define 200 ms,
Ultra96 is already happy with 10 ms.

Fixes: 5869ba0653b9 ("arm64: zynqmp: Add support for Xilinx zcu100-revC")
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Acked-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp-zcu100-revC.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/dts/zynqmp-zcu100-revC.dts b/arch/arm/dts/zynqmp-zcu100-revC.dts
index 02573a1f3294..986f50cf323d 100644
--- a/arch/arm/dts/zynqmp-zcu100-revC.dts
+++ b/arch/arm/dts/zynqmp-zcu100-revC.dts
@@ -123,6 +123,7 @@
 	sdio_pwrseq: sdio-pwrseq {
 		compatible = "mmc-pwrseq-simple";
 		reset-gpios = <&gpio 7 GPIO_ACTIVE_LOW>; /* WIFI_EN */
+		post-power-on-delay-ms = <10>;
 	};
 
 	ina226 {
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 5/8] arm64: dts: zynqmp: replace gpio-key, wakeup with wakeup-source property
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
                   ` (3 preceding siblings ...)
  2020-01-09 14:18 ` [PATCH 4/8] arm64: dts: zcu100-revC: Give wifi some time after power-on Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-09 14:18 ` [PATCH 6/8] arm64: zynqmp: Sync gpio-controller name location Michal Simek
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

From: Sudeep Holla <sudeep.holla@arm.com>

Most of the legacy "gpio-key,wakeup" boolean property is already
replaced with "wakeup-source". However few occurrences of old property
has popped up again, probably from the remnants in downstream trees.

This patch replaces the legacy properties with the unified
"wakeup-source" property introduced by:
"Input: gpio_keys - switch to using generic device properties"
(sha1: 700a38b27eefc582099fdf69effacfad0ad738a4)

Cc: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp-zcu100-revC.dts | 2 +-
 arch/arm/dts/zynqmp-zcu102-revA.dts | 2 +-
 arch/arm/dts/zynqmp-zcu106-revA.dts | 2 +-
 arch/arm/dts/zynqmp-zcu111-revA.dts | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/dts/zynqmp-zcu100-revC.dts b/arch/arm/dts/zynqmp-zcu100-revC.dts
index 986f50cf323d..21118c8cc34a 100644
--- a/arch/arm/dts/zynqmp-zcu100-revC.dts
+++ b/arch/arm/dts/zynqmp-zcu100-revC.dts
@@ -53,7 +53,7 @@
 			label = "sw4";
 			gpios = <&gpio 23 GPIO_ACTIVE_LOW>;
 			linux,code = <KEY_POWER>;
-			gpio-key,wakeup;
+			wakeup-source;
 			autorepeat;
 		};
 	};
diff --git a/arch/arm/dts/zynqmp-zcu102-revA.dts b/arch/arm/dts/zynqmp-zcu102-revA.dts
index 7d2f59b88c6f..b580f9263d02 100644
--- a/arch/arm/dts/zynqmp-zcu102-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu102-revA.dts
@@ -51,7 +51,7 @@
 			label = "sw19";
 			gpios = <&gpio 22 GPIO_ACTIVE_HIGH>;
 			linux,code = <KEY_DOWN>;
-			gpio-key,wakeup;
+			wakeup-source;
 			autorepeat;
 		};
 	};
diff --git a/arch/arm/dts/zynqmp-zcu106-revA.dts b/arch/arm/dts/zynqmp-zcu106-revA.dts
index 6faac956848b..d31982fce784 100644
--- a/arch/arm/dts/zynqmp-zcu106-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu106-revA.dts
@@ -51,7 +51,7 @@
 			label = "sw19";
 			gpios = <&gpio 22 GPIO_ACTIVE_HIGH>;
 			linux,code = <KEY_DOWN>;
-			gpio-key,wakeup;
+			wakeup-source;
 			autorepeat;
 		};
 	};
diff --git a/arch/arm/dts/zynqmp-zcu111-revA.dts b/arch/arm/dts/zynqmp-zcu111-revA.dts
index 5b648ceb2df6..bff224f78d1d 100644
--- a/arch/arm/dts/zynqmp-zcu111-revA.dts
+++ b/arch/arm/dts/zynqmp-zcu111-revA.dts
@@ -51,7 +51,7 @@
 			label = "sw19";
 			gpios = <&gpio 22 GPIO_ACTIVE_HIGH>;
 			linux,code = <KEY_DOWN>;
-			gpio-key,wakeup;
+			wakeup-source;
 			autorepeat;
 		};
 	};
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 6/8] arm64: zynqmp: Sync gpio-controller name location
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
                   ` (4 preceding siblings ...)
  2020-01-09 14:18 ` [PATCH 5/8] arm64: dts: zynqmp: replace gpio-key, wakeup with wakeup-source property Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-09 14:18 ` [PATCH 7/8] arm64: zynqmp: Sync gem clock nodes with mainline Linux Michal Simek
                   ` (2 subsequent siblings)
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

Sync location with mainline kernel.
Added by Linux kernel commit 75926f07baae
("arm64: dts: zynqmp: Add missing gpio-controller to ps gpio").

Fixes: 0b33e0b15600 ("arm64: zynqmp: Add missing gpio property to dtsi")
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi
index 232fd0c11cd7..9e7fae83f787 100644
--- a/arch/arm/dts/zynqmp.dtsi
+++ b/arch/arm/dts/zynqmp.dtsi
@@ -626,12 +626,12 @@
 			compatible = "xlnx,zynqmp-gpio-1.0";
 			status = "disabled";
 			#gpio-cells = <0x2>;
+			gpio-controller;
 			interrupt-parent = <&gic>;
 			interrupts = <0 16 4>;
 			interrupt-controller;
 			#interrupt-cells = <2>;
 			reg = <0x0 0xff0a0000 0x0 0x1000>;
-			gpio-controller;
 			power-domains = <&zynqmp_firmware PD_GPIO>;
 		};
 
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 7/8] arm64: zynqmp: Sync gem clock nodes with mainline Linux
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
                   ` (5 preceding siblings ...)
  2020-01-09 14:18 ` [PATCH 6/8] arm64: zynqmp: Sync gpio-controller name location Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-09 14:18 ` [PATCH 8/8] ARM: dts: zynq: enablement of coresight topology Michal Simek
  2020-01-16  7:08 ` [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

Just fixing indentation and update year in Copyright.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynqmp-clk-ccf.dtsi | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/arch/arm/dts/zynqmp-clk-ccf.dtsi b/arch/arm/dts/zynqmp-clk-ccf.dtsi
index 998298cc9bee..8eacd22d7cda 100644
--- a/arch/arm/dts/zynqmp-clk-ccf.dtsi
+++ b/arch/arm/dts/zynqmp-clk-ccf.dtsi
@@ -2,7 +2,7 @@
 /*
  * Clock specification for Xilinx ZynqMP
  *
- * (C) Copyright 2017, Xilinx, Inc.
+ * (C) Copyright 2017 - 2019, Xilinx, Inc.
  *
  * Michal Simek <michal.simek@xilinx.com>
  */
@@ -173,26 +173,30 @@
 };
 
 &gem0 {
-	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM0_REF>, <&zynqmp_clk GEM0_TX>,
-		 <&zynqmp_clk GEM0_RX>, <&zynqmp_clk GEM_TSU>;
+	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM0_REF>,
+		 <&zynqmp_clk GEM0_TX>, <&zynqmp_clk GEM0_RX>,
+		 <&zynqmp_clk GEM_TSU>;
 	clock-names = "pclk", "hclk", "tx_clk", "rx_clk", "tsu_clk";
 };
 
 &gem1 {
-	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM1_REF>, <&zynqmp_clk GEM1_TX>,
-		 <&zynqmp_clk GEM1_RX>, <&zynqmp_clk GEM_TSU>;
+	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM1_REF>,
+		 <&zynqmp_clk GEM1_TX>, <&zynqmp_clk GEM1_RX>,
+		 <&zynqmp_clk GEM_TSU>;
 	clock-names = "pclk", "hclk", "tx_clk", "rx_clk", "tsu_clk";
 };
 
 &gem2 {
-	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM2_REF>, <&zynqmp_clk GEM2_TX>,
-		 <&zynqmp_clk GEM2_RX>, <&zynqmp_clk GEM_TSU>;
+	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM2_REF>,
+		 <&zynqmp_clk GEM2_TX>, <&zynqmp_clk GEM2_RX>,
+		 <&zynqmp_clk GEM_TSU>;
 	clock-names = "pclk", "hclk", "tx_clk", "rx_clk", "tsu_clk";
 };
 
 &gem3 {
-	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM3_REF>, <&zynqmp_clk GEM3_TX>,
-		 <&zynqmp_clk GEM3_RX>, <&zynqmp_clk GEM_TSU>;
+	clocks = <&zynqmp_clk LPD_LSBUS>, <&zynqmp_clk GEM3_REF>,
+		 <&zynqmp_clk GEM3_TX>, <&zynqmp_clk GEM3_RX>,
+		 <&zynqmp_clk GEM_TSU>;
 	clock-names = "pclk", "hclk", "tx_clk", "rx_clk", "tsu_clk";
 };
 
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 8/8] ARM: dts: zynq: enablement of coresight topology
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
                   ` (6 preceding siblings ...)
  2020-01-09 14:18 ` [PATCH 7/8] arm64: zynqmp: Sync gem clock nodes with mainline Linux Michal Simek
@ 2020-01-09 14:18 ` Michal Simek
  2020-01-16  7:08 ` [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-09 14:18 UTC (permalink / raw)
  To: u-boot

From: Zumeng Chen <zumeng.chen@windriver.com>

This patch is to build the coresight topology structure of zynq-7000
series according to the docs of coresight and userguide of zynq-7000.

Signed-off-by: Zumeng Chen <zumeng.chen@windriver.com>
Signed-off-by: Quanyang Wang <quanyang.wang@windriver.com>
Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm/dts/zynq-7000.dtsi | 135 ++++++++++++++++++++++++++++++++++++
 1 file changed, 135 insertions(+)

diff --git a/arch/arm/dts/zynq-7000.dtsi b/arch/arm/dts/zynq-7000.dtsi
index 07dfa0d1074f..c35eb2344fa8 100644
--- a/arch/arm/dts/zynq-7000.dtsi
+++ b/arch/arm/dts/zynq-7000.dtsi
@@ -62,6 +62,39 @@
 		regulator-always-on;
 	};
 
+	replicator {
+		compatible = "arm,coresight-static-replicator";
+		clocks = <&clkc 27>, <&clkc 46>, <&clkc 47>;
+		clock-names = "apb_pclk", "dbg_trc", "dbg_apb";
+
+		out-ports {
+			#address-cells = <1>;
+			#size-cells = <0>;
+
+			/* replicator output ports */
+			port at 0 {
+				reg = <0>;
+				replicator_out_port0: endpoint {
+					remote-endpoint = <&tpiu_in_port>;
+				};
+			};
+			port at 1 {
+				reg = <1>;
+				replicator_out_port1: endpoint {
+					remote-endpoint = <&etb_in_port>;
+				};
+			};
+		};
+		in-ports {
+			/* replicator input port */
+			port {
+				replicator_in_port0: endpoint {
+					remote-endpoint = <&funnel_out_port>;
+				};
+			};
+		};
+	};
+
 	amba: amba {
 		u-boot,dm-pre-reloc;
 		compatible = "simple-bus";
@@ -415,5 +448,107 @@
 			reg = <0xf8005000 0x1000>;
 			timeout-sec = <10>;
 		};
+
+		etb at f8801000 {
+			compatible = "arm,coresight-etb10", "arm,primecell";
+			reg = <0xf8801000 0x1000>;
+			clocks = <&clkc 27>, <&clkc 46>, <&clkc 47>;
+			clock-names = "apb_pclk", "dbg_trc", "dbg_apb";
+			in-ports {
+				port {
+					etb_in_port: endpoint {
+						remote-endpoint = <&replicator_out_port1>;
+					};
+				};
+			};
+		};
+
+		tpiu at f8803000 {
+			compatible = "arm,coresight-tpiu", "arm,primecell";
+			reg = <0xf8803000 0x1000>;
+			clocks = <&clkc 27>, <&clkc 46>, <&clkc 47>;
+			clock-names = "apb_pclk", "dbg_trc", "dbg_apb";
+			in-ports {
+				port {
+					tpiu_in_port: endpoint {
+						remote-endpoint = <&replicator_out_port0>;
+					};
+				};
+			};
+		};
+
+		funnel at f8804000 {
+			compatible = "arm,coresight-static-funnel", "arm,primecell";
+			reg = <0xf8804000 0x1000>;
+			clocks = <&clkc 27>, <&clkc 46>, <&clkc 47>;
+			clock-names = "apb_pclk", "dbg_trc", "dbg_apb";
+
+			/* funnel output ports */
+			out-ports {
+				port {
+					funnel_out_port: endpoint {
+						remote-endpoint =
+							<&replicator_in_port0>;
+					};
+				};
+			};
+
+			in-ports {
+				#address-cells = <1>;
+				#size-cells = <0>;
+
+				/* funnel input ports */
+				port at 0 {
+					reg = <0>;
+					funnel0_in_port0: endpoint {
+						remote-endpoint = <&ptm0_out_port>;
+					};
+				};
+
+				port at 1 {
+					reg = <1>;
+					funnel0_in_port1: endpoint {
+						remote-endpoint = <&ptm1_out_port>;
+					};
+				};
+
+				port at 2 {
+					reg = <2>;
+					funnel0_in_port2: endpoint {
+					};
+				};
+				/* The other input ports are not connect to anything */
+			};
+		};
+
+		ptm at f889c000 {
+			compatible = "arm,coresight-etm3x", "arm,primecell";
+			reg = <0xf889c000 0x1000>;
+			clocks = <&clkc 27>, <&clkc 46>, <&clkc 47>;
+			clock-names = "apb_pclk", "dbg_trc", "dbg_apb";
+			cpu = <&cpu0>;
+			out-ports {
+				port {
+					ptm0_out_port: endpoint {
+						remote-endpoint = <&funnel0_in_port0>;
+					};
+				};
+			};
+		};
+
+		ptm at f889d000 {
+			compatible = "arm,coresight-etm3x", "arm,primecell";
+			reg = <0xf889d000 0x1000>;
+			clocks = <&clkc 27>, <&clkc 46>, <&clkc 47>;
+			clock-names = "apb_pclk", "dbg_trc", "dbg_apb";
+			cpu = <&cpu1>;
+			out-ports {
+				port {
+					ptm1_out_port: endpoint {
+						remote-endpoint = <&funnel0_in_port1>;
+					};
+				};
+			};
+		};
 	};
 };
-- 
2.24.0

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH 2/8] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string
  2020-01-09 14:18 ` [PATCH 2/8] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string Michal Simek
@ 2020-01-10 15:36   ` Masahiro Yamada
  2020-01-13  6:50     ` Michal Simek
  0 siblings, 1 reply; 12+ messages in thread
From: Masahiro Yamada @ 2020-01-10 15:36 UTC (permalink / raw)
  To: u-boot

On Thu, Jan 9, 2020 at 11:19 PM Michal Simek <michal.simek@xilinx.com> wrote:
>
> From: Rob Herring <robh@kernel.org>
>
> The 'arm,armv8' compatible string is only for software models. It adds
> little value otherwise and is inconsistently used as a fallback on some
> platforms. Remove it from those platforms.
>
> This fixes warnings generated by the DT schema.
>
> Reported-by: Michal Simek <michal.simek@xilinx.com>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Cc: Will Deacon <will.deacon@arm.com>
> Acked-by: Antoine Tenart <antoine.tenart@bootlin.com>
> Acked-by: Nishanth Menon <nm@ti.com>
> Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
> Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> Acked-by: Chanho Min <chanho.min@lge.com>
> Acked-by: Krzysztof Kozlowski <krzk@kernel.org>
> Acked-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> Acked-by: Gregory CLEMENT <gregory.clement@bootlin.com>
> Acked-by: Thierry Reding <treding@nvidia.com>
> Acked-by: Heiko Stuebner <heiko@sntech.de>
> Acked-by: Simon Horman <horms+renesas@verge.net.au>
> Acked-by: Tero Kristo <t-kristo@ti.com>
> Acked-by: Wei Xu <xuwei5@hisilicon.com>
> Acked-by: Liviu Dudau <liviu.dudau@arm.com>
> Acked-by: Matthias Brugger <matthias.bgg@gmail.com>
> Acked-by: Michal Simek <michal.simek@xilinx.com>
> Acked-by: Scott Branden <scott.branden@broadcom.com>
> Acked-by: Kevin Hilman <khilman@baylibre.com>
> Acked-by: Chunyan Zhang <zhang.lyra@gmail.com>
> Acked-by: Robert Richter <rrichter@cavium.com>
> Acked-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
> Acked-by: Dinh Nguyen <dinguyen@kernel.org>


I know you are importing this commit from Linux,
but these Ack lines look weird to me...



> Signed-off-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
>  arch/arm/dts/zynqmp.dtsi | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm/dts/zynqmp.dtsi b/arch/arm/dts/zynqmp.dtsi
> index b453941baf05..8646c4c3a340 100644
> --- a/arch/arm/dts/zynqmp.dtsi
> +++ b/arch/arm/dts/zynqmp.dtsi
> @@ -25,7 +25,7 @@
>                 #size-cells = <0>;
>
>                 cpu0: cpu at 0 {
> -                       compatible = "arm,cortex-a53", "arm,armv8";
> +                       compatible = "arm,cortex-a53";
>                         device_type = "cpu";
>                         enable-method = "psci";
>                         operating-points-v2 = <&cpu_opp_table>;
> @@ -34,7 +34,7 @@
>                 };
>
>                 cpu1: cpu at 1 {
> -                       compatible = "arm,cortex-a53", "arm,armv8";
> +                       compatible = "arm,cortex-a53";
>                         device_type = "cpu";
>                         enable-method = "psci";
>                         reg = <0x1>;
> @@ -43,7 +43,7 @@
>                 };
>
>                 cpu2: cpu at 2 {
> -                       compatible = "arm,cortex-a53", "arm,armv8";
> +                       compatible = "arm,cortex-a53";
>                         device_type = "cpu";
>                         enable-method = "psci";
>                         reg = <0x2>;
> @@ -52,7 +52,7 @@
>                 };
>
>                 cpu3: cpu at 3 {
> -                       compatible = "arm,cortex-a53", "arm,armv8";
> +                       compatible = "arm,cortex-a53";
>                         device_type = "cpu";
>                         enable-method = "psci";
>                         reg = <0x3>;
> --
> 2.24.0
>


-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 2/8] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string
  2020-01-10 15:36   ` Masahiro Yamada
@ 2020-01-13  6:50     ` Michal Simek
  0 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-13  6:50 UTC (permalink / raw)
  To: u-boot

On 10. 01. 20 16:36, Masahiro Yamada wrote:
> On Thu, Jan 9, 2020 at 11:19 PM Michal Simek <michal.simek@xilinx.com> wrote:
>>
>> From: Rob Herring <robh@kernel.org>
>>
>> The 'arm,armv8' compatible string is only for software models. It adds
>> little value otherwise and is inconsistently used as a fallback on some
>> platforms. Remove it from those platforms.
>>
>> This fixes warnings generated by the DT schema.
>>
>> Reported-by: Michal Simek <michal.simek@xilinx.com>
>> Cc: Mark Rutland <mark.rutland@arm.com>
>> Cc: Will Deacon <will.deacon@arm.com>
>> Acked-by: Antoine Tenart <antoine.tenart@bootlin.com>
>> Acked-by: Nishanth Menon <nm@ti.com>
>> Acked-by: Maxime Ripard <maxime.ripard@bootlin.com>
>> Acked-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
>> Acked-by: Chanho Min <chanho.min@lge.com>
>> Acked-by: Krzysztof Kozlowski <krzk@kernel.org>
>> Acked-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>> Acked-by: Gregory CLEMENT <gregory.clement@bootlin.com>
>> Acked-by: Thierry Reding <treding@nvidia.com>
>> Acked-by: Heiko Stuebner <heiko@sntech.de>
>> Acked-by: Simon Horman <horms+renesas@verge.net.au>
>> Acked-by: Tero Kristo <t-kristo@ti.com>
>> Acked-by: Wei Xu <xuwei5@hisilicon.com>
>> Acked-by: Liviu Dudau <liviu.dudau@arm.com>
>> Acked-by: Matthias Brugger <matthias.bgg@gmail.com>
>> Acked-by: Michal Simek <michal.simek@xilinx.com>
>> Acked-by: Scott Branden <scott.branden@broadcom.com>
>> Acked-by: Kevin Hilman <khilman@baylibre.com>
>> Acked-by: Chunyan Zhang <zhang.lyra@gmail.com>
>> Acked-by: Robert Richter <rrichter@cavium.com>
>> Acked-by: Jisheng Zhang <Jisheng.Zhang@synaptics.com>
>> Acked-by: Dinh Nguyen <dinguyen@kernel.org>
> 
> 
> I know you are importing this commit from Linux,
> but these Ack lines look weird to me...

I have not a problem to remove them but not sure if this is the right
thing to do.

Thanks,
Michal

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH 0/8] xilinx: Various DT changes and syncups
  2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
                   ` (7 preceding siblings ...)
  2020-01-09 14:18 ` [PATCH 8/8] ARM: dts: zynq: enablement of coresight topology Michal Simek
@ 2020-01-16  7:08 ` Michal Simek
  8 siblings, 0 replies; 12+ messages in thread
From: Michal Simek @ 2020-01-16  7:08 UTC (permalink / raw)
  To: u-boot

On 09. 01. 20 15:18, Michal Simek wrote:
> Hi,
> 
> I am sending some patches to fix bugs we found in xilinx release cycle.
> Also do some syncs to keep kernel/u-boot dts in pseudo sync.
> 
> Thanks,
> Michal
> 
> 
> Jan Kiszka (1):
>   arm64: dts: zcu100-revC: Give wifi some time after power-on
> 
> Michal Simek (4):
>   arm64: zynqmp: Remove incorrect phy from DT for zcu102-revB up
>   arm64: dts: zynqmp: Fix node names which contain "_"
>   arm64: zynqmp: Sync gpio-controller name location
>   arm64: zynqmp: Sync gem clock nodes with mainline Linux
> 
> Rob Herring (1):
>   arm64: dts: Remove inconsistent use of 'arm,armv8' compatible string
> 
> Sudeep Holla (1):
>   arm64: dts: zynqmp: replace gpio-key,wakeup with wakeup-source
>     property
> 
> Zumeng Chen (1):
>   ARM: dts: zynq: enablement of coresight topology
> 
>  arch/arm/dts/zynq-7000.dtsi         | 135 ++++++++++++++++++++++++++++
>  arch/arm/dts/zynqmp-clk-ccf.dtsi    |  22 +++--
>  arch/arm/dts/zynqmp-clk.dtsi        |   4 +-
>  arch/arm/dts/zynqmp-zcu100-revC.dts |   7 +-
>  arch/arm/dts/zynqmp-zcu102-revA.dts |  12 +--
>  arch/arm/dts/zynqmp-zcu102-revB.dts |   2 +-
>  arch/arm/dts/zynqmp-zcu106-revA.dts |   4 +-
>  arch/arm/dts/zynqmp-zcu111-revA.dts |   4 +-
>  arch/arm/dts/zynqmp.dtsi            |  14 +--
>  9 files changed, 172 insertions(+), 32 deletions(-)
> 

Apply all.
M

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: OpenPGP digital signature
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200116/b3bbe113/attachment.sig>

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2020-01-16  7:08 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-01-09 14:18 [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek
2020-01-09 14:18 ` [PATCH 1/8] arm64: zynqmp: Remove incorrect phy from DT for zcu102-revB up Michal Simek
2020-01-09 14:18 ` [PATCH 2/8] arm64: dts: Remove inconsistent use of 'arm, armv8' compatible string Michal Simek
2020-01-10 15:36   ` Masahiro Yamada
2020-01-13  6:50     ` Michal Simek
2020-01-09 14:18 ` [PATCH 3/8] arm64: dts: zynqmp: Fix node names which contain "_" Michal Simek
2020-01-09 14:18 ` [PATCH 4/8] arm64: dts: zcu100-revC: Give wifi some time after power-on Michal Simek
2020-01-09 14:18 ` [PATCH 5/8] arm64: dts: zynqmp: replace gpio-key, wakeup with wakeup-source property Michal Simek
2020-01-09 14:18 ` [PATCH 6/8] arm64: zynqmp: Sync gpio-controller name location Michal Simek
2020-01-09 14:18 ` [PATCH 7/8] arm64: zynqmp: Sync gem clock nodes with mainline Linux Michal Simek
2020-01-09 14:18 ` [PATCH 8/8] ARM: dts: zynq: enablement of coresight topology Michal Simek
2020-01-16  7:08 ` [PATCH 0/8] xilinx: Various DT changes and syncups Michal Simek

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.