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

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

[Dan Haab]

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

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

[Florian Fainelli]

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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Rafał Miłecki]

On 4 May 2016 at 20:53, Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 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ł
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

[Hauke Mehrtens]

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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

[Florian Fainelli]

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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html