Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1393654 > unrolled thread

[PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510

Started byDan Haab <dhaab@luxul.com>
First post2016-05-03 19:40 +0200
Last post2016-05-05 23:50 +0200
Articles 5 — 4 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510 Dan Haab <dhaab@luxul.com> - 2016-05-03 19:40 +0200
    Re: [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510 Florian Fainelli <f.fainelli@gmail.com> - 2016-05-04 21:00 +0200
      Re: [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510 Rafał Miłecki <zajec5@gmail.com> - 2016-05-05 00:40 +0200
        Re: [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510 Hauke Mehrtens <hauke@hauke-m.de> - 2016-05-05 23:50 +0200
          Re: [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510 Florian Fainelli <f.fainelli@gmail.com> - 2016-05-05 23:50 +0200

#1393654 — [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510

FromDan Haab <dhaab@luxul.com>
Date2016-05-03 19:40 +0200
Subject[PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510
Message-ID<ruMsH-3Sd-41@gated-at.bofh.it>
Luxul XAP-1510 is an AP device based on BCM4708 SoC. It uses flash
memory connected to the SPI controller.

Signed-off-by: Dan Haab <dhaab@luxul.com>
---
 arch/arm/boot/dts/Makefile                   |    1 +
 arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts |   67 ++++++++++++++++++++++++++
 2 files changed, 68 insertions(+)
 create mode 100644 arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 95c1923..c499d09 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -71,6 +71,7 @@ dtb-$(CONFIG_ARCH_BCM_5301X) += \
 	bcm4708-asus-rt-ac56u.dtb \
 	bcm4708-asus-rt-ac68u.dtb \
 	bcm4708-buffalo-wzr-1750dhp.dtb \
+	bcm4708-luxul-xap-1510.dtb \
 	bcm4708-luxul-xwc-1000.dtb \
 	bcm4708-netgear-r6250.dtb \
 	bcm4708-netgear-r6300-v2.dtb \
diff --git a/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
new file mode 100644
index 0000000..f4460b5
--- /dev/null
+++ b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
@@ -0,0 +1,67 @@
+/*
+ * Broadcom BCM470X / BCM5301X ARM platform code.
+ * DTS for Luxul XAP-1510
+ *
+ * Copyright 2015 Luxul Inc.
+ *
+ * Licensed under the GNU/GPL. See COPYING for details.
+ */
+
+/dts-v1/;
+
+#include "bcm4708.dtsi"
+
+/ {
+	compatible = "luxul,xap-1510v1", "brcm,bcm4708";
+	model = "Luxul XAP-1510 V1";
+
+	chosen {
+		bootargs = "console=ttyS0,115200 earlycon";
+	};
+
+	memory {
+		reg = <0x00000000 0x08000000>;
+	};
+
+	leds {
+		compatible = "gpio-leds";
+
+		5ghz {
+			label = "bcm53xx:blue:5ghz";
+			gpios = <&chipcommon 13 GPIO_ACTIVE_LOW>;
+			linux,default-trigger = "none";
+		};
+
+		2ghz {
+			label = "bcm53xx:blue:2ghz";
+			gpios = <&chipcommon 14 GPIO_ACTIVE_LOW>;
+			linux,default-trigger = "none";
+		};
+
+		status {
+			label = "bcm53xx:green:status";
+			gpios = <&chipcommon 15 GPIO_ACTIVE_LOW>;
+			linux,default-trigger = "timer";
+		};
+	};
+
+	gpio-keys {
+		compatible = "gpio-keys";
+		#address-cells = <1>;
+		#size-cells = <0>;
+
+		restart {
+			label = "Reset";
+			linux,code = <KEY_RESTART>;
+			gpios = <&chipcommon 11 GPIO_ACTIVE_LOW>;
+		};
+	};
+};
+
+&uart0 {
+	status = "okay";
+};
+
+&spi_nor {
+	status = "okay";
+};
-- 
1.7.9.5

[toc] | [next] | [standalone]

#1394628

FromFlorian Fainelli <f.fainelli@gmail.com>
Date2016-05-04 21:00 +0200
Message-ID<rvabD-Yb-3@gated-at.bofh.it>
In reply to#1393654
On 03/05/16 10:28, Dan Haab wrote:
> Luxul XAP-1510 is an AP device based on BCM4708 SoC. It uses flash
> memory connected to the SPI controller.

Looks fine, except one nit:

> diff --git a/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
> new file mode 100644
> index 0000000..f4460b5
> --- /dev/null
> +++ b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
> @@ -0,0 +1,67 @@
> +/*
> + * Broadcom BCM470X / BCM5301X ARM platform code.
> + * DTS for Luxul XAP-1510
> + *
> + * Copyright 2015 Luxul Inc.
> + *
> + * Licensed under the GNU/GPL. See COPYING for details.

The ARM SoC maintainer have been asking to utilize a license which is
also BSD compatible, so something along the lines of this one:

arch/arm/boot/dts/bcm911360k.dts

Do you mind respining the patch to include that kind of license header?
Thanks!
-- 
Florian

[toc] | [prev] | [next] | [standalone]

#1394760

FromRafał Miłecki <zajec5@gmail.com>
Date2016-05-05 00:40 +0200
Message-ID<rvdCx-46c-11@gated-at.bofh.it>
In reply to#1394628
On 4 May 2016 at 20:53, Florian Fainelli <f.fainelli@gmail.com> wrote:
> On 03/05/16 10:28, Dan Haab wrote:
>> Luxul XAP-1510 is an AP device based on BCM4708 SoC. It uses flash
>> memory connected to the SPI controller.
>
> Looks fine, except one nit:
>
>> diff --git a/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
>> new file mode 100644
>> index 0000000..f4460b5
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
>> @@ -0,0 +1,67 @@
>> +/*
>> + * Broadcom BCM470X / BCM5301X ARM platform code.
>> + * DTS for Luxul XAP-1510
>> + *
>> + * Copyright 2015 Luxul Inc.
>> + *
>> + * Licensed under the GNU/GPL. See COPYING for details.
>
> The ARM SoC maintainer have been asking to utilize a license which is
> also BSD compatible, so something along the lines of this one:
>
> arch/arm/boot/dts/bcm911360k.dts
>
> Do you mind respining the patch to include that kind of license header?

I wasn't aware of this neither. Shall we re-license existing DTS files
as well? I'm fine with changing them to BSD compatible.

I'm a bit confused by using BSD license only. Kernel is licensed under
GNU GPL version 2 and if there are some BSD compatible modules, they
use dual licensing model (BSD/GPL). Shouldn't we use BSD/GPL in DTS
files as well then?

-- 
Rafał

[toc] | [prev] | [next] | [standalone]

#1395389

FromHauke Mehrtens <hauke@hauke-m.de>
Date2016-05-05 23:50 +0200
Message-ID<rvzjJ-80Y-13@gated-at.bofh.it>
In reply to#1394760
On 05/05/2016 12:36 AM, Rafał Miłecki wrote:
> On 4 May 2016 at 20:53, Florian Fainelli <f.fainelli@gmail.com> wrote:
>> On 03/05/16 10:28, Dan Haab wrote:
>>> Luxul XAP-1510 is an AP device based on BCM4708 SoC. It uses flash
>>> memory connected to the SPI controller.
>>
>> Looks fine, except one nit:
>>
>>> diff --git a/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
>>> new file mode 100644
>>> index 0000000..f4460b5
>>> --- /dev/null
>>> +++ b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
>>> @@ -0,0 +1,67 @@
>>> +/*
>>> + * Broadcom BCM470X / BCM5301X ARM platform code.
>>> + * DTS for Luxul XAP-1510
>>> + *
>>> + * Copyright 2015 Luxul Inc.
>>> + *
>>> + * Licensed under the GNU/GPL. See COPYING for details.
>>
>> The ARM SoC maintainer have been asking to utilize a license which is
>> also BSD compatible, so something along the lines of this one:
>>
>> arch/arm/boot/dts/bcm911360k.dts
>>
>> Do you mind respining the patch to include that kind of license header?
> 
> I wasn't aware of this neither. Shall we re-license existing DTS files
> as well? I'm fine with changing them to BSD compatible.
> 
> I'm a bit confused by using BSD license only. Kernel is licensed under
> GNU GPL version 2 and if there are some BSD compatible modules, they
> use dual licensing model (BSD/GPL). Shouldn't we use BSD/GPL in DTS
> files as well then?
> 

As far as I understand the License part you can take code which is
licensed under BSD license and use it in GPL code because you can comply
with the BSD license and also handle it in a way to be GPL compliant at
the same time. You can also make a change to BSD code and license this
change under the GPL, then the complete code (the old one + your change)
is now GPL.

Are the device tree files licensed under BSD license to make it easier
to integrate them into a proprietary boot loader or what is the purpose?

Hauke

[toc] | [prev] | [next] | [standalone]

#1395391

FromFlorian Fainelli <f.fainelli@gmail.com>
Date2016-05-05 23:50 +0200
Message-ID<rvzjJ-80Y-15@gated-at.bofh.it>
In reply to#1395389
On 05/05/16 14:47, Hauke Mehrtens wrote:
> On 05/05/2016 12:36 AM, Rafał Miłecki wrote:
>> On 4 May 2016 at 20:53, Florian Fainelli <f.fainelli@gmail.com> wrote:
>>> On 03/05/16 10:28, Dan Haab wrote:
>>>> Luxul XAP-1510 is an AP device based on BCM4708 SoC. It uses flash
>>>> memory connected to the SPI controller.
>>>
>>> Looks fine, except one nit:
>>>
>>>> diff --git a/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
>>>> new file mode 100644
>>>> index 0000000..f4460b5
>>>> --- /dev/null
>>>> +++ b/arch/arm/boot/dts/bcm4708-luxul-xap-1510.dts
>>>> @@ -0,0 +1,67 @@
>>>> +/*
>>>> + * Broadcom BCM470X / BCM5301X ARM platform code.
>>>> + * DTS for Luxul XAP-1510
>>>> + *
>>>> + * Copyright 2015 Luxul Inc.
>>>> + *
>>>> + * Licensed under the GNU/GPL. See COPYING for details.
>>>
>>> The ARM SoC maintainer have been asking to utilize a license which is
>>> also BSD compatible, so something along the lines of this one:
>>>
>>> arch/arm/boot/dts/bcm911360k.dts
>>>
>>> Do you mind respining the patch to include that kind of license header?
>>
>> I wasn't aware of this neither. Shall we re-license existing DTS files
>> as well? I'm fine with changing them to BSD compatible.
>>
>> I'm a bit confused by using BSD license only. Kernel is licensed under
>> GNU GPL version 2 and if there are some BSD compatible modules, they
>> use dual licensing model (BSD/GPL). Shouldn't we use BSD/GPL in DTS
>> files as well then?
>>
> 
> As far as I understand the License part you can take code which is
> licensed under BSD license and use it in GPL code because you can comply
> with the BSD license and also handle it in a way to be GPL compliant at
> the same time. You can also make a change to BSD code and license this
> change under the GPL, then the complete code (the old one + your change)
> is now GPL.
> 
> Are the device tree files licensed under BSD license to make it easier
> to integrate them into a proprietary boot loader or what is the purpose?

Yes, there could be that, or just making the license compatible with
other OSes like FreeBSD for instace.
-- 
Florian

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web