Message ID | 1333350084-5993-1-git-send-email-shawn.guo@linaro.org |
---|---|
State | New |
Headers | show |
On Mon, Apr 02, 2012 at 03:01:24PM +0800, Shawn Guo wrote: > It has been pointed out by device tree maintainer for several times > that the generic names 'serial' and 'ethernet' should be used for > those devices per ePAPR. Renames imx uart and fec device tree nodes > to stop them being bad examples. > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> Acked-by: Sascha Hauer <s.hauer@pengutronix.de> Sascha > --- > Documentation/devicetree/bindings/net/fsl-fec.txt | 2 +- > .../bindings/tty/serial/fsl-imx-uart.txt | 2 +- > arch/arm/boot/dts/imx27-phytec-phycore.dts | 8 ++++---- > arch/arm/boot/dts/imx27.dtsi | 14 +++++++------- > arch/arm/boot/dts/imx51-babbage.dts | 8 ++++---- > arch/arm/boot/dts/imx51.dtsi | 8 ++++---- > arch/arm/boot/dts/imx53-ard.dts | 2 +- > arch/arm/boot/dts/imx53-evk.dts | 4 ++-- > arch/arm/boot/dts/imx53-qsb.dts | 4 ++-- > arch/arm/boot/dts/imx53-smd.dts | 8 ++++---- > arch/arm/boot/dts/imx53.dtsi | 12 ++++++------ > arch/arm/boot/dts/imx6q-arm2.dts | 4 ++-- > arch/arm/boot/dts/imx6q-sabrelite.dts | 4 ++-- > arch/arm/boot/dts/imx6q.dtsi | 12 ++++++------ > arch/arm/mach-imx/clock-imx6q.c | 12 ++++++------ > 15 files changed, 52 insertions(+), 52 deletions(-) > > diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt > index de43951..7ab9e1a 100644 > --- a/Documentation/devicetree/bindings/net/fsl-fec.txt > +++ b/Documentation/devicetree/bindings/net/fsl-fec.txt > @@ -14,7 +14,7 @@ Optional properties: > > Example: > > -fec@83fec000 { > +ethernet@83fec000 { > compatible = "fsl,imx51-fec", "fsl,imx27-fec"; > reg = <0x83fec000 0x4000>; > interrupts = <87>; > diff --git a/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt > index a9c0406..b462d0c 100644 > --- a/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt > +++ b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt > @@ -11,7 +11,7 @@ Optional properties: > > Example: > > -uart@73fbc000 { > +serial@73fbc000 { > compatible = "fsl,imx51-uart", "fsl,imx21-uart"; > reg = <0x73fbc000 0x4000>; > interrupts = <31>; > diff --git a/arch/arm/boot/dts/imx27-phytec-phycore.dts b/arch/arm/boot/dts/imx27-phytec-phycore.dts > index a51a08f..2b0ff60 100644 > --- a/arch/arm/boot/dts/imx27-phytec-phycore.dts > +++ b/arch/arm/boot/dts/imx27-phytec-phycore.dts > @@ -27,22 +27,22 @@ > status = "okay"; > }; > > - uart@1000a000 { > + serial@1000a000 { > fsl,uart-has-rtscts; > status = "okay"; > }; > > - uart@1000b000 { > + serial@1000b000 { > fsl,uart-has-rtscts; > status = "okay"; > }; > > - uart@1000c000 { > + serial@1000c000 { > fsl,uart-has-rtscts; > status = "okay"; > }; > > - fec@1002b000 { > + ethernet@1002b000 { > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi > index bc5e7d5..2b1a166 100644 > --- a/arch/arm/boot/dts/imx27.dtsi > +++ b/arch/arm/boot/dts/imx27.dtsi > @@ -59,28 +59,28 @@ > status = "disabled"; > }; > > - uart1: uart@1000a000 { > + uart1: serial@1000a000 { > compatible = "fsl,imx27-uart", "fsl,imx21-uart"; > reg = <0x1000a000 0x1000>; > interrupts = <20>; > status = "disabled"; > }; > > - uart2: uart@1000b000 { > + uart2: serial@1000b000 { > compatible = "fsl,imx27-uart", "fsl,imx21-uart"; > reg = <0x1000b000 0x1000>; > interrupts = <19>; > status = "disabled"; > }; > > - uart3: uart@1000c000 { > + uart3: serial@1000c000 { > compatible = "fsl,imx27-uart", "fsl,imx21-uart"; > reg = <0x1000c000 0x1000>; > interrupts = <18>; > status = "disabled"; > }; > > - uart4: uart@1000d000 { > + uart4: serial@1000d000 { > compatible = "fsl,imx27-uart", "fsl,imx21-uart"; > reg = <0x1000d000 0x1000>; > interrupts = <17>; > @@ -183,14 +183,14 @@ > status = "disabled"; > }; > > - uart5: uart@1001b000 { > + uart5: serial@1001b000 { > compatible = "fsl,imx27-uart", "fsl,imx21-uart"; > reg = <0x1001b000 0x1000>; > interrupts = <49>; > status = "disabled"; > }; > > - uart6: uart@1001c000 { > + uart6: serial@1001c000 { > compatible = "fsl,imx27-uart", "fsl,imx21-uart"; > reg = <0x1001c000 0x1000>; > interrupts = <48>; > @@ -206,7 +206,7 @@ > status = "disabled"; > }; > > - fec: fec@1002b000 { > + fec: ethernet@1002b000 { > compatible = "fsl,imx27-fec"; > reg = <0x1002b000 0x4000>; > interrupts = <50>; > diff --git a/arch/arm/boot/dts/imx51-babbage.dts b/arch/arm/boot/dts/imx51-babbage.dts > index fd729d4..d42a404 100644 > --- a/arch/arm/boot/dts/imx51-babbage.dts > +++ b/arch/arm/boot/dts/imx51-babbage.dts > @@ -36,7 +36,7 @@ > status = "okay"; > }; > > - uart3: uart@7000c000 { > + uart3: serial@7000c000 { > fsl,uart-has-rtscts; > status = "okay"; > }; > @@ -173,12 +173,12 @@ > reg = <0x73fa8000 0x4000>; > }; > > - uart1: uart@73fbc000 { > + uart1: serial@73fbc000 { > fsl,uart-has-rtscts; > status = "okay"; > }; > > - uart2: uart@73fc0000 { > + uart2: serial@73fc0000 { > status = "okay"; > }; > }; > @@ -197,7 +197,7 @@ > }; > }; > > - fec@83fec000 { > + ethernet@83fec000 { > phy-mode = "mii"; > status = "okay"; > }; > diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi > index 6663986..66f0ebd 100644 > --- a/arch/arm/boot/dts/imx51.dtsi > +++ b/arch/arm/boot/dts/imx51.dtsi > @@ -86,7 +86,7 @@ > status = "disabled"; > }; > > - uart3: uart@7000c000 { > + uart3: serial@7000c000 { > compatible = "fsl,imx51-uart", "fsl,imx21-uart"; > reg = <0x7000c000 0x4000>; > interrupts = <33>; > @@ -171,14 +171,14 @@ > status = "disabled"; > }; > > - uart1: uart@73fbc000 { > + uart1: serial@73fbc000 { > compatible = "fsl,imx51-uart", "fsl,imx21-uart"; > reg = <0x73fbc000 0x4000>; > interrupts = <31>; > status = "disabled"; > }; > > - uart2: uart@73fc0000 { > + uart2: serial@73fc0000 { > compatible = "fsl,imx51-uart", "fsl,imx21-uart"; > reg = <0x73fc0000 0x4000>; > interrupts = <32>; > @@ -235,7 +235,7 @@ > status = "disabled"; > }; > > - fec@83fec000 { > + ethernet@83fec000 { > compatible = "fsl,imx51-fec", "fsl,imx27-fec"; > reg = <0x83fec000 0x4000>; > interrupts = <87>; > diff --git a/arch/arm/boot/dts/imx53-ard.dts b/arch/arm/boot/dts/imx53-ard.dts > index 4c61b7f..5b8eafc 100644 > --- a/arch/arm/boot/dts/imx53-ard.dts > +++ b/arch/arm/boot/dts/imx53-ard.dts > @@ -40,7 +40,7 @@ > reg = <0x53fa8000 0x4000>; > }; > > - uart1: uart@53fbc000 { > + uart1: serial@53fbc000 { > status = "okay"; > }; > }; > diff --git a/arch/arm/boot/dts/imx53-evk.dts b/arch/arm/boot/dts/imx53-evk.dts > index 8ce5fe1..9c79803 100644 > --- a/arch/arm/boot/dts/imx53-evk.dts > +++ b/arch/arm/boot/dts/imx53-evk.dts > @@ -71,7 +71,7 @@ > reg = <0x53fa8000 0x4000>; > }; > > - uart1: uart@53fbc000 { > + uart1: serial@53fbc000 { > status = "okay"; > }; > }; > @@ -95,7 +95,7 @@ > }; > }; > > - fec@63fec000 { > + ethernet@63fec000 { > phy-mode = "rmii"; > phy-reset-gpios = <&gpio7 6 0>; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx53-qsb.dts b/arch/arm/boot/dts/imx53-qsb.dts > index de34a17..b08b587 100644 > --- a/arch/arm/boot/dts/imx53-qsb.dts > +++ b/arch/arm/boot/dts/imx53-qsb.dts > @@ -45,7 +45,7 @@ > reg = <0x53fa8000 0x4000>; > }; > > - uart1: uart@53fbc000 { > + uart1: serial@53fbc000 { > status = "okay"; > }; > }; > @@ -78,7 +78,7 @@ > }; > }; > > - fec@63fec000 { > + ethernet@63fec000 { > phy-mode = "rmii"; > phy-reset-gpios = <&gpio7 6 0>; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx53-smd.dts b/arch/arm/boot/dts/imx53-smd.dts > index a984fa5..a3529af 100644 > --- a/arch/arm/boot/dts/imx53-smd.dts > +++ b/arch/arm/boot/dts/imx53-smd.dts > @@ -35,7 +35,7 @@ > status = "okay"; > }; > > - uart3: uart@5000c000 { > + uart3: serial@5000c000 { > fsl,uart-has-rtscts; > status = "okay"; > }; > @@ -86,11 +86,11 @@ > reg = <0x53fa8000 0x4000>; > }; > > - uart1: uart@53fbc000 { > + uart1: serial@53fbc000 { > status = "okay"; > }; > > - uart2: uart@53fc0000 { > + uart2: serial@53fc0000 { > status = "okay"; > }; > }; > @@ -138,7 +138,7 @@ > }; > }; > > - fec@63fec000 { > + ethernet@63fec000 { > phy-mode = "rmii"; > phy-reset-gpios = <&gpio7 6 0>; > status = "okay"; > diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi > index 5dd91b9..5188615 100644 > --- a/arch/arm/boot/dts/imx53.dtsi > +++ b/arch/arm/boot/dts/imx53.dtsi > @@ -88,7 +88,7 @@ > status = "disabled"; > }; > > - uart3: uart@5000c000 { > + uart3: serial@5000c000 { > compatible = "fsl,imx53-uart", "fsl,imx21-uart"; > reg = <0x5000c000 0x4000>; > interrupts = <33>; > @@ -173,14 +173,14 @@ > status = "disabled"; > }; > > - uart1: uart@53fbc000 { > + uart1: serial@53fbc000 { > compatible = "fsl,imx53-uart", "fsl,imx21-uart"; > reg = <0x53fbc000 0x4000>; > interrupts = <31>; > status = "disabled"; > }; > > - uart2: uart@53fc0000 { > + uart2: serial@53fc0000 { > compatible = "fsl,imx53-uart", "fsl,imx21-uart"; > reg = <0x53fc0000 0x4000>; > interrupts = <32>; > @@ -226,7 +226,7 @@ > status = "disabled"; > }; > > - uart4: uart@53ff0000 { > + uart4: serial@53ff0000 { > compatible = "fsl,imx53-uart", "fsl,imx21-uart"; > reg = <0x53ff0000 0x4000>; > interrupts = <13>; > @@ -241,7 +241,7 @@ > reg = <0x60000000 0x10000000>; > ranges; > > - uart5: uart@63f90000 { > + uart5: serial@63f90000 { > compatible = "fsl,imx53-uart", "fsl,imx21-uart"; > reg = <0x63f90000 0x4000>; > interrupts = <86>; > @@ -290,7 +290,7 @@ > status = "disabled"; > }; > > - fec@63fec000 { > + ethernet@63fec000 { > compatible = "fsl,imx53-fec", "fsl,imx25-fec"; > reg = <0x63fec000 0x4000>; > interrupts = <87>; > diff --git a/arch/arm/boot/dts/imx6q-arm2.dts b/arch/arm/boot/dts/imx6q-arm2.dts > index 1b2b64f..9c468d2 100644 > --- a/arch/arm/boot/dts/imx6q-arm2.dts > +++ b/arch/arm/boot/dts/imx6q-arm2.dts > @@ -23,7 +23,7 @@ > > soc { > aips-bus@02100000 { /* AIPS2 */ > - enet@02188000 { > + ethernet@02188000 { > phy-mode = "rgmii"; > local-mac-address = [00 04 9F 01 1B 61]; > status = "okay"; > @@ -42,7 +42,7 @@ > status = "okay"; > }; > > - uart4: uart@021f0000 { > + uart4: serial@021f0000 { > status = "okay"; > }; > }; > diff --git a/arch/arm/boot/dts/imx6q-sabrelite.dts b/arch/arm/boot/dts/imx6q-sabrelite.dts > index 4663a4e..a93c593 100644 > --- a/arch/arm/boot/dts/imx6q-sabrelite.dts > +++ b/arch/arm/boot/dts/imx6q-sabrelite.dts > @@ -23,7 +23,7 @@ > > soc { > aips-bus@02100000 { /* AIPS2 */ > - enet@02188000 { > + ethernet@02188000 { > phy-mode = "rgmii"; > phy-reset-gpios = <&gpio3 23 0>; > status = "okay"; > @@ -43,7 +43,7 @@ > status = "okay"; > }; > > - uart2: uart@021e8000 { > + uart2: serial@021e8000 { > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/imx6q.dtsi b/arch/arm/boot/dts/imx6q.dtsi > index 68f2e42..56c5304 100644 > --- a/arch/arm/boot/dts/imx6q.dtsi > +++ b/arch/arm/boot/dts/imx6q.dtsi > @@ -165,7 +165,7 @@ > status = "disabled"; > }; > > - uart1: uart@02020000 { > + uart1: serial@02020000 { > compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; > reg = <0x02020000 0x4000>; > interrupts = <0 26 0x04>; > @@ -506,7 +506,7 @@ > reg = <0x0217c000 0x4000>; > }; > > - enet@02188000 { > + ethernet@02188000 { > compatible = "fsl,imx6q-fec"; > reg = <0x02188000 0x4000>; > interrupts = <0 118 0x04 0 119 0x04>; > @@ -627,28 +627,28 @@ > interrupts = <0 18 0x04>; > }; > > - uart2: uart@021e8000 { > + uart2: serial@021e8000 { > compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; > reg = <0x021e8000 0x4000>; > interrupts = <0 27 0x04>; > status = "disabled"; > }; > > - uart3: uart@021ec000 { > + uart3: serial@021ec000 { > compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; > reg = <0x021ec000 0x4000>; > interrupts = <0 28 0x04>; > status = "disabled"; > }; > > - uart4: uart@021f0000 { > + uart4: serial@021f0000 { > compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; > reg = <0x021f0000 0x4000>; > interrupts = <0 29 0x04>; > status = "disabled"; > }; > > - uart5: uart@021f4000 { > + uart5: serial@021f4000 { > compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; > reg = <0x021f4000 0x4000>; > interrupts = <0 30 0x04>; > diff --git a/arch/arm/mach-imx/clock-imx6q.c b/arch/arm/mach-imx/clock-imx6q.c > index 111c328..be049d1 100644 > --- a/arch/arm/mach-imx/clock-imx6q.c > +++ b/arch/arm/mach-imx/clock-imx6q.c > @@ -1952,12 +1952,12 @@ static struct clk sata_clk = { > } > > static struct clk_lookup lookups[] = { > - _REGISTER_CLOCK("2020000.uart", NULL, uart_clk), > - _REGISTER_CLOCK("21e8000.uart", NULL, uart_clk), > - _REGISTER_CLOCK("21ec000.uart", NULL, uart_clk), > - _REGISTER_CLOCK("21f0000.uart", NULL, uart_clk), > - _REGISTER_CLOCK("21f4000.uart", NULL, uart_clk), > - _REGISTER_CLOCK("2188000.enet", NULL, enet_clk), > + _REGISTER_CLOCK("2020000.serial", NULL, uart_clk), > + _REGISTER_CLOCK("21e8000.serial", NULL, uart_clk), > + _REGISTER_CLOCK("21ec000.serial", NULL, uart_clk), > + _REGISTER_CLOCK("21f0000.serial", NULL, uart_clk), > + _REGISTER_CLOCK("21f4000.serial", NULL, uart_clk), > + _REGISTER_CLOCK("2188000.ethernet", NULL, enet_clk), > _REGISTER_CLOCK("2190000.usdhc", NULL, usdhc1_clk), > _REGISTER_CLOCK("2194000.usdhc", NULL, usdhc2_clk), > _REGISTER_CLOCK("2198000.usdhc", NULL, usdhc3_clk), > -- > 1.7.5.4 > >
Hi Shawn, On 02.04.2012 09:01, Shawn Guo wrote: > It has been pointed out by device tree maintainer for several times > that the generic names 'serial' and 'ethernet' should be used for > those devices per ePAPR. Renames imx uart and fec device tree nodes > to stop them being bad examples. > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > --- > Documentation/devicetree/bindings/net/fsl-fec.txt | 2 +- > .../bindings/tty/serial/fsl-imx-uart.txt | 2 +- > arch/arm/boot/dts/imx27-phytec-phycore.dts | 8 ++++---- > arch/arm/boot/dts/imx27.dtsi | 14 +++++++------- > arch/arm/boot/dts/imx51-babbage.dts | 8 ++++---- > arch/arm/boot/dts/imx51.dtsi | 8 ++++---- > arch/arm/boot/dts/imx53-ard.dts | 2 +- > arch/arm/boot/dts/imx53-evk.dts | 4 ++-- > arch/arm/boot/dts/imx53-qsb.dts | 4 ++-- > arch/arm/boot/dts/imx53-smd.dts | 8 ++++---- > arch/arm/boot/dts/imx53.dtsi | 12 ++++++------ > arch/arm/boot/dts/imx6q-arm2.dts | 4 ++-- > arch/arm/boot/dts/imx6q-sabrelite.dts | 4 ++-- > arch/arm/boot/dts/imx6q.dtsi | 12 ++++++------ > arch/arm/mach-imx/clock-imx6q.c | 12 ++++++------ > 15 files changed, 52 insertions(+), 52 deletions(-) ... > diff --git a/arch/arm/boot/dts/imx6q-sabrelite.dts b/arch/arm/boot/dts/imx6q-sabrelite.dts > index 4663a4e..a93c593 100644 > --- a/arch/arm/boot/dts/imx6q-sabrelite.dts > +++ b/arch/arm/boot/dts/imx6q-sabrelite.dts ... > @@ -43,7 +43,7 @@ > status = "okay"; > }; > > - uart2: uart@021e8000 { > + uart2: serial@021e8000 { > status = "okay"; > }; > > diff --git a/arch/arm/boot/dts/imx6q.dtsi b/arch/arm/boot/dts/imx6q.dtsi > index 68f2e42..56c5304 100644 > --- a/arch/arm/boot/dts/imx6q.dtsi > +++ b/arch/arm/boot/dts/imx6q.dtsi ... > @@ -627,28 +627,28 @@ > interrupts = <0 18 0x04>; > }; > > - uart2: uart@021e8000 { > + uart2: serial@021e8000 { > compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; > reg = <0x021e8000 0x4000>; > interrupts = <0 27 0x04>; > status = "disabled"; > }; Having this patch on a 3.4-rc5 based test branch and booting it on the SabreLite I get (with earlyprintk enabled): Serial: IMX driver imx-uart: probe of 21e8000.serial failed with error -2 resulting in no console output. Reverting this patch I get Serial: IMX driver 21e8000.uart: ttymxc1 at MMIO 0x21e8000 (irq = 59) is a IMX and console works fine again. Have I missed anything? Best regards Dirk
On Fri, May 04, 2012 at 11:12:05AM +0200, Dirk Behme wrote: > Having this patch on a 3.4-rc5 based test branch and booting it on > the SabreLite I get (with earlyprintk enabled): > > Serial: IMX driver > imx-uart: probe of 21e8000.serial failed with error -2 > > resulting in no console output. Reverting this patch I get > > Serial: IMX driver > 21e8000.uart: ttymxc1 at MMIO 0x21e8000 (irq = 59) is a IMX > > and console works fine again. > > Have I missed anything? > The patch changes arch/arm/mach-imx/clock-imx6q.c as well. Not sure if you have the changes incorporated in your testing branch. It reminds me that I should rebase my imx/dt branch on Sascha's clk series to save the conflict.
On 04.05.2012 15:33, Shawn Guo wrote: > On Fri, May 04, 2012 at 11:12:05AM +0200, Dirk Behme wrote: >> Having this patch on a 3.4-rc5 based test branch and booting it on >> the SabreLite I get (with earlyprintk enabled): >> >> Serial: IMX driver >> imx-uart: probe of 21e8000.serial failed with error -2 >> >> resulting in no console output. Reverting this patch I get >> >> Serial: IMX driver >> 21e8000.uart: ttymxc1 at MMIO 0x21e8000 (irq = 59) is a IMX >> >> and console works fine again. >> >> Have I missed anything? >> > The patch changes arch/arm/mach-imx/clock-imx6q.c as well. Not sure > if you have the changes incorporated in your testing branch. Yes, I think so https://github.com/dirkbehme/linux-imx6/commit/ebfac55f157425b5e063cdb2d9668b45a0a6515f#diff-14 > It reminds me that I should rebase my imx/dt branch on Sascha's clk > series to save the conflict. My test branch has Sascha's clk series, too. It's mainly based on https://github.com/dirkbehme/linux-imx6/commits/linux-imx6-staging Best regards Dirk
On Fri, May 04, 2012 at 03:35:51PM +0200, Dirk Behme wrote: > Yes, I think so > > https://github.com/dirkbehme/linux-imx6/commit/ebfac55f157425b5e063cdb2d9668b45a0a6515f#diff-14 > > >It reminds me that I should rebase my imx/dt branch on Sascha's clk > >series to save the conflict. > > My test branch has Sascha's clk series, too. It's mainly based on > > https://github.com/dirkbehme/linux-imx6/commits/linux-imx6-staging > If you have Sascha's clk series merged, the changes on clock-imx6q.c made by this patch could probably get lost, since Sascha's clk series will remove clock-imx6q.c.
diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt index de43951..7ab9e1a 100644 --- a/Documentation/devicetree/bindings/net/fsl-fec.txt +++ b/Documentation/devicetree/bindings/net/fsl-fec.txt @@ -14,7 +14,7 @@ Optional properties: Example: -fec@83fec000 { +ethernet@83fec000 { compatible = "fsl,imx51-fec", "fsl,imx27-fec"; reg = <0x83fec000 0x4000>; interrupts = <87>; diff --git a/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt index a9c0406..b462d0c 100644 --- a/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt +++ b/Documentation/devicetree/bindings/tty/serial/fsl-imx-uart.txt @@ -11,7 +11,7 @@ Optional properties: Example: -uart@73fbc000 { +serial@73fbc000 { compatible = "fsl,imx51-uart", "fsl,imx21-uart"; reg = <0x73fbc000 0x4000>; interrupts = <31>; diff --git a/arch/arm/boot/dts/imx27-phytec-phycore.dts b/arch/arm/boot/dts/imx27-phytec-phycore.dts index a51a08f..2b0ff60 100644 --- a/arch/arm/boot/dts/imx27-phytec-phycore.dts +++ b/arch/arm/boot/dts/imx27-phytec-phycore.dts @@ -27,22 +27,22 @@ status = "okay"; }; - uart@1000a000 { + serial@1000a000 { fsl,uart-has-rtscts; status = "okay"; }; - uart@1000b000 { + serial@1000b000 { fsl,uart-has-rtscts; status = "okay"; }; - uart@1000c000 { + serial@1000c000 { fsl,uart-has-rtscts; status = "okay"; }; - fec@1002b000 { + ethernet@1002b000 { status = "okay"; }; diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi index bc5e7d5..2b1a166 100644 --- a/arch/arm/boot/dts/imx27.dtsi +++ b/arch/arm/boot/dts/imx27.dtsi @@ -59,28 +59,28 @@ status = "disabled"; }; - uart1: uart@1000a000 { + uart1: serial@1000a000 { compatible = "fsl,imx27-uart", "fsl,imx21-uart"; reg = <0x1000a000 0x1000>; interrupts = <20>; status = "disabled"; }; - uart2: uart@1000b000 { + uart2: serial@1000b000 { compatible = "fsl,imx27-uart", "fsl,imx21-uart"; reg = <0x1000b000 0x1000>; interrupts = <19>; status = "disabled"; }; - uart3: uart@1000c000 { + uart3: serial@1000c000 { compatible = "fsl,imx27-uart", "fsl,imx21-uart"; reg = <0x1000c000 0x1000>; interrupts = <18>; status = "disabled"; }; - uart4: uart@1000d000 { + uart4: serial@1000d000 { compatible = "fsl,imx27-uart", "fsl,imx21-uart"; reg = <0x1000d000 0x1000>; interrupts = <17>; @@ -183,14 +183,14 @@ status = "disabled"; }; - uart5: uart@1001b000 { + uart5: serial@1001b000 { compatible = "fsl,imx27-uart", "fsl,imx21-uart"; reg = <0x1001b000 0x1000>; interrupts = <49>; status = "disabled"; }; - uart6: uart@1001c000 { + uart6: serial@1001c000 { compatible = "fsl,imx27-uart", "fsl,imx21-uart"; reg = <0x1001c000 0x1000>; interrupts = <48>; @@ -206,7 +206,7 @@ status = "disabled"; }; - fec: fec@1002b000 { + fec: ethernet@1002b000 { compatible = "fsl,imx27-fec"; reg = <0x1002b000 0x4000>; interrupts = <50>; diff --git a/arch/arm/boot/dts/imx51-babbage.dts b/arch/arm/boot/dts/imx51-babbage.dts index fd729d4..d42a404 100644 --- a/arch/arm/boot/dts/imx51-babbage.dts +++ b/arch/arm/boot/dts/imx51-babbage.dts @@ -36,7 +36,7 @@ status = "okay"; }; - uart3: uart@7000c000 { + uart3: serial@7000c000 { fsl,uart-has-rtscts; status = "okay"; }; @@ -173,12 +173,12 @@ reg = <0x73fa8000 0x4000>; }; - uart1: uart@73fbc000 { + uart1: serial@73fbc000 { fsl,uart-has-rtscts; status = "okay"; }; - uart2: uart@73fc0000 { + uart2: serial@73fc0000 { status = "okay"; }; }; @@ -197,7 +197,7 @@ }; }; - fec@83fec000 { + ethernet@83fec000 { phy-mode = "mii"; status = "okay"; }; diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi index 6663986..66f0ebd 100644 --- a/arch/arm/boot/dts/imx51.dtsi +++ b/arch/arm/boot/dts/imx51.dtsi @@ -86,7 +86,7 @@ status = "disabled"; }; - uart3: uart@7000c000 { + uart3: serial@7000c000 { compatible = "fsl,imx51-uart", "fsl,imx21-uart"; reg = <0x7000c000 0x4000>; interrupts = <33>; @@ -171,14 +171,14 @@ status = "disabled"; }; - uart1: uart@73fbc000 { + uart1: serial@73fbc000 { compatible = "fsl,imx51-uart", "fsl,imx21-uart"; reg = <0x73fbc000 0x4000>; interrupts = <31>; status = "disabled"; }; - uart2: uart@73fc0000 { + uart2: serial@73fc0000 { compatible = "fsl,imx51-uart", "fsl,imx21-uart"; reg = <0x73fc0000 0x4000>; interrupts = <32>; @@ -235,7 +235,7 @@ status = "disabled"; }; - fec@83fec000 { + ethernet@83fec000 { compatible = "fsl,imx51-fec", "fsl,imx27-fec"; reg = <0x83fec000 0x4000>; interrupts = <87>; diff --git a/arch/arm/boot/dts/imx53-ard.dts b/arch/arm/boot/dts/imx53-ard.dts index 4c61b7f..5b8eafc 100644 --- a/arch/arm/boot/dts/imx53-ard.dts +++ b/arch/arm/boot/dts/imx53-ard.dts @@ -40,7 +40,7 @@ reg = <0x53fa8000 0x4000>; }; - uart1: uart@53fbc000 { + uart1: serial@53fbc000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/imx53-evk.dts b/arch/arm/boot/dts/imx53-evk.dts index 8ce5fe1..9c79803 100644 --- a/arch/arm/boot/dts/imx53-evk.dts +++ b/arch/arm/boot/dts/imx53-evk.dts @@ -71,7 +71,7 @@ reg = <0x53fa8000 0x4000>; }; - uart1: uart@53fbc000 { + uart1: serial@53fbc000 { status = "okay"; }; }; @@ -95,7 +95,7 @@ }; }; - fec@63fec000 { + ethernet@63fec000 { phy-mode = "rmii"; phy-reset-gpios = <&gpio7 6 0>; status = "okay"; diff --git a/arch/arm/boot/dts/imx53-qsb.dts b/arch/arm/boot/dts/imx53-qsb.dts index de34a17..b08b587 100644 --- a/arch/arm/boot/dts/imx53-qsb.dts +++ b/arch/arm/boot/dts/imx53-qsb.dts @@ -45,7 +45,7 @@ reg = <0x53fa8000 0x4000>; }; - uart1: uart@53fbc000 { + uart1: serial@53fbc000 { status = "okay"; }; }; @@ -78,7 +78,7 @@ }; }; - fec@63fec000 { + ethernet@63fec000 { phy-mode = "rmii"; phy-reset-gpios = <&gpio7 6 0>; status = "okay"; diff --git a/arch/arm/boot/dts/imx53-smd.dts b/arch/arm/boot/dts/imx53-smd.dts index a984fa5..a3529af 100644 --- a/arch/arm/boot/dts/imx53-smd.dts +++ b/arch/arm/boot/dts/imx53-smd.dts @@ -35,7 +35,7 @@ status = "okay"; }; - uart3: uart@5000c000 { + uart3: serial@5000c000 { fsl,uart-has-rtscts; status = "okay"; }; @@ -86,11 +86,11 @@ reg = <0x53fa8000 0x4000>; }; - uart1: uart@53fbc000 { + uart1: serial@53fbc000 { status = "okay"; }; - uart2: uart@53fc0000 { + uart2: serial@53fc0000 { status = "okay"; }; }; @@ -138,7 +138,7 @@ }; }; - fec@63fec000 { + ethernet@63fec000 { phy-mode = "rmii"; phy-reset-gpios = <&gpio7 6 0>; status = "okay"; diff --git a/arch/arm/boot/dts/imx53.dtsi b/arch/arm/boot/dts/imx53.dtsi index 5dd91b9..5188615 100644 --- a/arch/arm/boot/dts/imx53.dtsi +++ b/arch/arm/boot/dts/imx53.dtsi @@ -88,7 +88,7 @@ status = "disabled"; }; - uart3: uart@5000c000 { + uart3: serial@5000c000 { compatible = "fsl,imx53-uart", "fsl,imx21-uart"; reg = <0x5000c000 0x4000>; interrupts = <33>; @@ -173,14 +173,14 @@ status = "disabled"; }; - uart1: uart@53fbc000 { + uart1: serial@53fbc000 { compatible = "fsl,imx53-uart", "fsl,imx21-uart"; reg = <0x53fbc000 0x4000>; interrupts = <31>; status = "disabled"; }; - uart2: uart@53fc0000 { + uart2: serial@53fc0000 { compatible = "fsl,imx53-uart", "fsl,imx21-uart"; reg = <0x53fc0000 0x4000>; interrupts = <32>; @@ -226,7 +226,7 @@ status = "disabled"; }; - uart4: uart@53ff0000 { + uart4: serial@53ff0000 { compatible = "fsl,imx53-uart", "fsl,imx21-uart"; reg = <0x53ff0000 0x4000>; interrupts = <13>; @@ -241,7 +241,7 @@ reg = <0x60000000 0x10000000>; ranges; - uart5: uart@63f90000 { + uart5: serial@63f90000 { compatible = "fsl,imx53-uart", "fsl,imx21-uart"; reg = <0x63f90000 0x4000>; interrupts = <86>; @@ -290,7 +290,7 @@ status = "disabled"; }; - fec@63fec000 { + ethernet@63fec000 { compatible = "fsl,imx53-fec", "fsl,imx25-fec"; reg = <0x63fec000 0x4000>; interrupts = <87>; diff --git a/arch/arm/boot/dts/imx6q-arm2.dts b/arch/arm/boot/dts/imx6q-arm2.dts index 1b2b64f..9c468d2 100644 --- a/arch/arm/boot/dts/imx6q-arm2.dts +++ b/arch/arm/boot/dts/imx6q-arm2.dts @@ -23,7 +23,7 @@ soc { aips-bus@02100000 { /* AIPS2 */ - enet@02188000 { + ethernet@02188000 { phy-mode = "rgmii"; local-mac-address = [00 04 9F 01 1B 61]; status = "okay"; @@ -42,7 +42,7 @@ status = "okay"; }; - uart4: uart@021f0000 { + uart4: serial@021f0000 { status = "okay"; }; }; diff --git a/arch/arm/boot/dts/imx6q-sabrelite.dts b/arch/arm/boot/dts/imx6q-sabrelite.dts index 4663a4e..a93c593 100644 --- a/arch/arm/boot/dts/imx6q-sabrelite.dts +++ b/arch/arm/boot/dts/imx6q-sabrelite.dts @@ -23,7 +23,7 @@ soc { aips-bus@02100000 { /* AIPS2 */ - enet@02188000 { + ethernet@02188000 { phy-mode = "rgmii"; phy-reset-gpios = <&gpio3 23 0>; status = "okay"; @@ -43,7 +43,7 @@ status = "okay"; }; - uart2: uart@021e8000 { + uart2: serial@021e8000 { status = "okay"; }; diff --git a/arch/arm/boot/dts/imx6q.dtsi b/arch/arm/boot/dts/imx6q.dtsi index 68f2e42..56c5304 100644 --- a/arch/arm/boot/dts/imx6q.dtsi +++ b/arch/arm/boot/dts/imx6q.dtsi @@ -165,7 +165,7 @@ status = "disabled"; }; - uart1: uart@02020000 { + uart1: serial@02020000 { compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; reg = <0x02020000 0x4000>; interrupts = <0 26 0x04>; @@ -506,7 +506,7 @@ reg = <0x0217c000 0x4000>; }; - enet@02188000 { + ethernet@02188000 { compatible = "fsl,imx6q-fec"; reg = <0x02188000 0x4000>; interrupts = <0 118 0x04 0 119 0x04>; @@ -627,28 +627,28 @@ interrupts = <0 18 0x04>; }; - uart2: uart@021e8000 { + uart2: serial@021e8000 { compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; reg = <0x021e8000 0x4000>; interrupts = <0 27 0x04>; status = "disabled"; }; - uart3: uart@021ec000 { + uart3: serial@021ec000 { compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; reg = <0x021ec000 0x4000>; interrupts = <0 28 0x04>; status = "disabled"; }; - uart4: uart@021f0000 { + uart4: serial@021f0000 { compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; reg = <0x021f0000 0x4000>; interrupts = <0 29 0x04>; status = "disabled"; }; - uart5: uart@021f4000 { + uart5: serial@021f4000 { compatible = "fsl,imx6q-uart", "fsl,imx21-uart"; reg = <0x021f4000 0x4000>; interrupts = <0 30 0x04>; diff --git a/arch/arm/mach-imx/clock-imx6q.c b/arch/arm/mach-imx/clock-imx6q.c index 111c328..be049d1 100644 --- a/arch/arm/mach-imx/clock-imx6q.c +++ b/arch/arm/mach-imx/clock-imx6q.c @@ -1952,12 +1952,12 @@ static struct clk sata_clk = { } static struct clk_lookup lookups[] = { - _REGISTER_CLOCK("2020000.uart", NULL, uart_clk), - _REGISTER_CLOCK("21e8000.uart", NULL, uart_clk), - _REGISTER_CLOCK("21ec000.uart", NULL, uart_clk), - _REGISTER_CLOCK("21f0000.uart", NULL, uart_clk), - _REGISTER_CLOCK("21f4000.uart", NULL, uart_clk), - _REGISTER_CLOCK("2188000.enet", NULL, enet_clk), + _REGISTER_CLOCK("2020000.serial", NULL, uart_clk), + _REGISTER_CLOCK("21e8000.serial", NULL, uart_clk), + _REGISTER_CLOCK("21ec000.serial", NULL, uart_clk), + _REGISTER_CLOCK("21f0000.serial", NULL, uart_clk), + _REGISTER_CLOCK("21f4000.serial", NULL, uart_clk), + _REGISTER_CLOCK("2188000.ethernet", NULL, enet_clk), _REGISTER_CLOCK("2190000.usdhc", NULL, usdhc1_clk), _REGISTER_CLOCK("2194000.usdhc", NULL, usdhc2_clk), _REGISTER_CLOCK("2198000.usdhc", NULL, usdhc3_clk),
It has been pointed out by device tree maintainer for several times that the generic names 'serial' and 'ethernet' should be used for those devices per ePAPR. Renames imx uart and fec device tree nodes to stop them being bad examples. Signed-off-by: Shawn Guo <shawn.guo@linaro.org> --- Documentation/devicetree/bindings/net/fsl-fec.txt | 2 +- .../bindings/tty/serial/fsl-imx-uart.txt | 2 +- arch/arm/boot/dts/imx27-phytec-phycore.dts | 8 ++++---- arch/arm/boot/dts/imx27.dtsi | 14 +++++++------- arch/arm/boot/dts/imx51-babbage.dts | 8 ++++---- arch/arm/boot/dts/imx51.dtsi | 8 ++++---- arch/arm/boot/dts/imx53-ard.dts | 2 +- arch/arm/boot/dts/imx53-evk.dts | 4 ++-- arch/arm/boot/dts/imx53-qsb.dts | 4 ++-- arch/arm/boot/dts/imx53-smd.dts | 8 ++++---- arch/arm/boot/dts/imx53.dtsi | 12 ++++++------ arch/arm/boot/dts/imx6q-arm2.dts | 4 ++-- arch/arm/boot/dts/imx6q-sabrelite.dts | 4 ++-- arch/arm/boot/dts/imx6q.dtsi | 12 ++++++------ arch/arm/mach-imx/clock-imx6q.c | 12 ++++++------ 15 files changed, 52 insertions(+), 52 deletions(-)