mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] MIPS: dts: reuse TP-LINK MR3020 dts file from linux
@ 2016-08-16 21:36 Antony Pavlov
  2016-08-18  6:32 ` Sascha Hauer
  0 siblings, 1 reply; 3+ messages in thread
From: Antony Pavlov @ 2016-08-16 21:36 UTC (permalink / raw)
  To: barebox

At the moment barebox tplink-mr3020.dts file has almost
the same content as linux ar9331_tl_mr3020.dts file
so we can reuse linux dts-file and drop duplicated lines
from barebox dts-file.

Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
---
 arch/mips/dts/tplink-mr3020.dts | 68 +----------------------------------------
 1 file changed, 1 insertion(+), 67 deletions(-)

diff --git a/arch/mips/dts/tplink-mr3020.dts b/arch/mips/dts/tplink-mr3020.dts
index 831fab8..1e843ee 100644
--- a/arch/mips/dts/tplink-mr3020.dts
+++ b/arch/mips/dts/tplink-mr3020.dts
@@ -1,73 +1,7 @@
-/dts-v1/;
-
-#include <dt-bindings/gpio/gpio.h>
-
-#include <mips/qca/ar9331.dtsi>
+#include <mips/qca/ar9331_tl_mr3020.dts>
 
 / {
-	model = "TP-Link TL-MR3020";
-	compatible = "tplink,tl-mr3020";
-
 	aliases {
 		spiflash = &spiflash;
 	};
-
-	memory@0 {
-		device_type = "memory";
-		reg = <0x00000000 0x2000000>;
-	};
-
-	leds {
-		compatible = "gpio-leds";
-
-		wlan {
-			label = "tp-link:green:wlan";
-			gpios = <&gpio 0 GPIO_ACTIVE_HIGH>;
-			default-state = "off";
-		};
-
-		lan {
-			label = "tp-link:green:lan";
-			gpios = <&gpio 17 GPIO_ACTIVE_LOW>;
-			default-state = "off";
-		};
-
-		wps {
-			label = "tp-link:green:wps";
-			gpios = <&gpio 26 GPIO_ACTIVE_LOW>;
-			default-state = "off";
-		};
-
-		led3g {
-			label = "tp-link:green:3g";
-			gpios = <&gpio 27 GPIO_ACTIVE_LOW>;
-			default-state = "off";
-		};
-	};
-};
-
-&ref {
-	clock-frequency = <25000000>;
-};
-
-&uart {
-	status = "okay";
-};
-
-&gpio {
-	status = "okay";
-};
-
-&spi {
-	num-chipselects = <1>;
-	status = "okay";
-
-	/* Spansion S25FL032PIF SPI flash */
-	spiflash: s25sl032p@0 {
-		#address-cells = <1>;
-		#size-cells = <1>;
-		compatible = "spansion,s25sl032p", "jedec,spi-nor";
-		spi-max-frequency = <104000000>;
-		reg = <0>;
-	};
 };
-- 
2.8.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [PATCH] MIPS: dts: reuse TP-LINK MR3020 dts file from linux
  2016-08-16 21:36 [PATCH] MIPS: dts: reuse TP-LINK MR3020 dts file from linux Antony Pavlov
@ 2016-08-18  6:32 ` Sascha Hauer
  2016-08-18  9:43   ` Antony Pavlov
  0 siblings, 1 reply; 3+ messages in thread
From: Sascha Hauer @ 2016-08-18  6:32 UTC (permalink / raw)
  To: Antony Pavlov; +Cc: barebox

On Wed, Aug 17, 2016 at 12:36:09AM +0300, Antony Pavlov wrote:
> At the moment barebox tplink-mr3020.dts file has almost
> the same content as linux ar9331_tl_mr3020.dts file
> so we can reuse linux dts-file and drop duplicated lines
> from barebox dts-file.
> 
> Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
> ---
>  arch/mips/dts/tplink-mr3020.dts | 68 +----------------------------------------
>  1 file changed, 1 insertion(+), 67 deletions(-)

Applied, thanks

There is no difference between the two patches you sent, right?

Sascha


> 
> diff --git a/arch/mips/dts/tplink-mr3020.dts b/arch/mips/dts/tplink-mr3020.dts
> index 831fab8..1e843ee 100644
> --- a/arch/mips/dts/tplink-mr3020.dts
> +++ b/arch/mips/dts/tplink-mr3020.dts
> @@ -1,73 +1,7 @@
> -/dts-v1/;
> -
> -#include <dt-bindings/gpio/gpio.h>
> -
> -#include <mips/qca/ar9331.dtsi>
> +#include <mips/qca/ar9331_tl_mr3020.dts>
>  
>  / {
> -	model = "TP-Link TL-MR3020";
> -	compatible = "tplink,tl-mr3020";
> -
>  	aliases {
>  		spiflash = &spiflash;
>  	};
> -
> -	memory@0 {
> -		device_type = "memory";
> -		reg = <0x00000000 0x2000000>;
> -	};
> -
> -	leds {
> -		compatible = "gpio-leds";
> -
> -		wlan {
> -			label = "tp-link:green:wlan";
> -			gpios = <&gpio 0 GPIO_ACTIVE_HIGH>;
> -			default-state = "off";
> -		};
> -
> -		lan {
> -			label = "tp-link:green:lan";
> -			gpios = <&gpio 17 GPIO_ACTIVE_LOW>;
> -			default-state = "off";
> -		};
> -
> -		wps {
> -			label = "tp-link:green:wps";
> -			gpios = <&gpio 26 GPIO_ACTIVE_LOW>;
> -			default-state = "off";
> -		};
> -
> -		led3g {
> -			label = "tp-link:green:3g";
> -			gpios = <&gpio 27 GPIO_ACTIVE_LOW>;
> -			default-state = "off";
> -		};
> -	};
> -};
> -
> -&ref {
> -	clock-frequency = <25000000>;
> -};
> -
> -&uart {
> -	status = "okay";
> -};
> -
> -&gpio {
> -	status = "okay";
> -};
> -
> -&spi {
> -	num-chipselects = <1>;
> -	status = "okay";
> -
> -	/* Spansion S25FL032PIF SPI flash */
> -	spiflash: s25sl032p@0 {
> -		#address-cells = <1>;
> -		#size-cells = <1>;
> -		compatible = "spansion,s25sl032p", "jedec,spi-nor";
> -		spi-max-frequency = <104000000>;
> -		reg = <0>;
> -	};
>  };
> -- 
> 2.8.1
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [PATCH] MIPS: dts: reuse TP-LINK MR3020 dts file from linux
  2016-08-18  6:32 ` Sascha Hauer
@ 2016-08-18  9:43   ` Antony Pavlov
  0 siblings, 0 replies; 3+ messages in thread
From: Antony Pavlov @ 2016-08-18  9:43 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: barebox

On Thu, 18 Aug 2016 08:32:27 +0200
Sascha Hauer <s.hauer@pengutronix.de> wrote:

> On Wed, Aug 17, 2016 at 12:36:09AM +0300, Antony Pavlov wrote:
> > At the moment barebox tplink-mr3020.dts file has almost
> > the same content as linux ar9331_tl_mr3020.dts file
> > so we can reuse linux dts-file and drop duplicated lines
> > from barebox dts-file.
> > 
> > Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
> > ---
> >  arch/mips/dts/tplink-mr3020.dts | 68 +----------------------------------------
> >  1 file changed, 1 insertion(+), 67 deletions(-)
> 
> Applied, thanks
> 
> There is no difference between the two patches you sent, right?

Yes, there is no difference. My bad --- I have used wrong maillist address in the first patch
(barebox@lists.infradead.orgc instead of barebox@lists.infradead.org).
 
> Sascha
> 
> 
> > 
> > diff --git a/arch/mips/dts/tplink-mr3020.dts b/arch/mips/dts/tplink-mr3020.dts
> > index 831fab8..1e843ee 100644
> > --- a/arch/mips/dts/tplink-mr3020.dts
> > +++ b/arch/mips/dts/tplink-mr3020.dts
> > @@ -1,73 +1,7 @@
> > -/dts-v1/;
> > -
> > -#include <dt-bindings/gpio/gpio.h>
> > -
> > -#include <mips/qca/ar9331.dtsi>
> > +#include <mips/qca/ar9331_tl_mr3020.dts>
> >  
> >  / {
> > -	model = "TP-Link TL-MR3020";
> > -	compatible = "tplink,tl-mr3020";
> > -
> >  	aliases {
> >  		spiflash = &spiflash;
> >  	};
> > -
> > -	memory@0 {
> > -		device_type = "memory";
> > -		reg = <0x00000000 0x2000000>;
> > -	};
> > -
> > -	leds {
> > -		compatible = "gpio-leds";
> > -
> > -		wlan {
> > -			label = "tp-link:green:wlan";
> > -			gpios = <&gpio 0 GPIO_ACTIVE_HIGH>;
> > -			default-state = "off";
> > -		};
> > -
> > -		lan {
> > -			label = "tp-link:green:lan";
> > -			gpios = <&gpio 17 GPIO_ACTIVE_LOW>;
> > -			default-state = "off";
> > -		};
> > -
> > -		wps {
> > -			label = "tp-link:green:wps";
> > -			gpios = <&gpio 26 GPIO_ACTIVE_LOW>;
> > -			default-state = "off";
> > -		};
> > -
> > -		led3g {
> > -			label = "tp-link:green:3g";
> > -			gpios = <&gpio 27 GPIO_ACTIVE_LOW>;
> > -			default-state = "off";
> > -		};
> > -	};
> > -};
> > -
> > -&ref {
> > -	clock-frequency = <25000000>;
> > -};
> > -
> > -&uart {
> > -	status = "okay";
> > -};
> > -
> > -&gpio {
> > -	status = "okay";
> > -};
> > -
> > -&spi {
> > -	num-chipselects = <1>;
> > -	status = "okay";
> > -
> > -	/* Spansion S25FL032PIF SPI flash */
> > -	spiflash: s25sl032p@0 {
> > -		#address-cells = <1>;
> > -		#size-cells = <1>;
> > -		compatible = "spansion,s25sl032p", "jedec,spi-nor";
> > -		spi-max-frequency = <104000000>;
> > -		reg = <0>;
> > -	};
> >  };
> > -- 
> > 2.8.1
> > 
> > 
> > _______________________________________________
> > barebox mailing list
> > barebox@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/barebox
> > 
> 
> -- 
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |


-- 
-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-08-18  9:40 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-16 21:36 [PATCH] MIPS: dts: reuse TP-LINK MR3020 dts file from linux Antony Pavlov
2016-08-18  6:32 ` Sascha Hauer
2016-08-18  9:43   ` Antony Pavlov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox