linux-staging.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/2] staging: mt7621-dts: organice syscons inside device tree
@ 2021-05-05 13:21 Sergio Paracuellos
  2021-05-05 13:21 ` [PATCH 1/2] staging: mt7621-dts: remove ethsys node Sergio Paracuellos
  2021-05-05 13:21 ` [PATCH 2/2] staging: mt7621-dts: use standard 'syscon' string Sergio Paracuellos
  0 siblings, 2 replies; 3+ messages in thread
From: Sergio Paracuellos @ 2021-05-05 13:21 UTC (permalink / raw)
  To: linux-staging; +Cc: gregkh, neil, ilya.lipnitskiy

This two patches organice all the syscons inside the device tree.

We currently have 'sysc' and 'memc' syscons that can be used. There is
also a 'ethsys' node that can be replaced in favor of 'sysc'. We
also should use standard 'syscon' strings in all of them.

Sergio Paracuellos (2):
  staging: mt7621-dts: remove ethsys node
  staging: mt7621-dts: use standard 'syscon' string

 drivers/staging/mt7621-dts/mt7621.dtsi | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

-- 
2.25.1


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

* [PATCH 1/2] staging: mt7621-dts: remove ethsys node
  2021-05-05 13:21 [PATCH 0/2] staging: mt7621-dts: organice syscons inside device tree Sergio Paracuellos
@ 2021-05-05 13:21 ` Sergio Paracuellos
  2021-05-05 13:21 ` [PATCH 2/2] staging: mt7621-dts: use standard 'syscon' string Sergio Paracuellos
  1 sibling, 0 replies; 3+ messages in thread
From: Sergio Paracuellos @ 2021-05-05 13:21 UTC (permalink / raw)
  To: linux-staging; +Cc: gregkh, neil, ilya.lipnitskiy

DT 'ethsys' node is being configured as a syscon
to get access to reset and other registers in the
'mediateķ,mt7621-eth' driver. Since the 'sysc' is also
a syscon, provides the clock and also is virtually
mapped from the same physical address 0x1e000000 we
can just use 'sysc' as the phandle for the syscon in
the ethernet node. Compatible string 'mediatek,mt7621-ethsys'
of the node is not being used anywhere inside the kernel
so, this node can be safely removed.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 drivers/staging/mt7621-dts/mt7621.dtsi | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/drivers/staging/mt7621-dts/mt7621.dtsi b/drivers/staging/mt7621-dts/mt7621.dtsi
index 5759ba8742ca..b39c7968239f 100644
--- a/drivers/staging/mt7621-dts/mt7621.dtsi
+++ b/drivers/staging/mt7621-dts/mt7621.dtsi
@@ -372,13 +372,6 @@ nand: nand@1e003000 {
 		clock-names = "nand";
 	};
 
-	ethsys: syscon@1e000000 {
-		compatible = "mediatek,mt7621-ethsys",
-			     "syscon";
-		reg = <0x1e000000 0x1000>;
-		#clock-cells = <1>;
-	};
-
 	ethernet: ethernet@1e100000 {
 		compatible = "mediatek,mt7621-eth";
 		reg = <0x1e100000 0x10000>;
@@ -396,7 +389,7 @@ ethernet: ethernet@1e100000 {
 		interrupt-parent = <&gic>;
 		interrupts = <GIC_SHARED 3 IRQ_TYPE_LEVEL_HIGH>;
 
-		mediatek,ethsys = <&ethsys>;
+		mediatek,ethsys = <&sysc>;
 
 
 		gmac0: mac@0 {
-- 
2.25.1


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

* [PATCH 2/2] staging: mt7621-dts: use standard 'syscon' string
  2021-05-05 13:21 [PATCH 0/2] staging: mt7621-dts: organice syscons inside device tree Sergio Paracuellos
  2021-05-05 13:21 ` [PATCH 1/2] staging: mt7621-dts: remove ethsys node Sergio Paracuellos
@ 2021-05-05 13:21 ` Sergio Paracuellos
  1 sibling, 0 replies; 3+ messages in thread
From: Sergio Paracuellos @ 2021-05-05 13:21 UTC (permalink / raw)
  To: linux-staging; +Cc: gregkh, neil, ilya.lipnitskiy

Both 'memc' and 'sysc' nodes are not using 'syscon'
as a node string which is the standard one to be used.
Update both of them.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 drivers/staging/mt7621-dts/mt7621.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/mt7621-dts/mt7621.dtsi b/drivers/staging/mt7621-dts/mt7621.dtsi
index b39c7968239f..5623d542bcf2 100644
--- a/drivers/staging/mt7621-dts/mt7621.dtsi
+++ b/drivers/staging/mt7621-dts/mt7621.dtsi
@@ -55,7 +55,7 @@ palmbus: palmbus@1E000000 {
 		#address-cells = <1>;
 		#size-cells = <1>;
 
-		sysc: sysc@0 {
+		sysc: syscon@0 {
 			compatible = "mediatek,mt7621-sysc", "syscon";
 			reg = <0x0 0x100>;
 			#clock-cells = <1>;
@@ -122,7 +122,7 @@ i2s: i2s@a00 {
 			status = "disabled";
 		};
 
-		memc: memc@5000 {
+		memc: syscon@5000 {
 			compatible = "mediatek,mt7621-memc", "syscon";
 			reg = <0x5000 0x1000>;
 		};
-- 
2.25.1


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

end of thread, other threads:[~2021-05-05 13:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-05-05 13:21 [PATCH 0/2] staging: mt7621-dts: organice syscons inside device tree Sergio Paracuellos
2021-05-05 13:21 ` [PATCH 1/2] staging: mt7621-dts: remove ethsys node Sergio Paracuellos
2021-05-05 13:21 ` [PATCH 2/2] staging: mt7621-dts: use standard 'syscon' string Sergio Paracuellos

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).