All of lore.kernel.org
 help / color / mirror / Atom feed
From: f.fainelli@gmail.com (Florian Fainelli)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510
Date: Wed, 4 May 2016 11:53:50 -0700	[thread overview]
Message-ID: <572A453E.6010000@gmail.com> (raw)
In-Reply-To: <1462296501-5209-1-git-send-email-dhaab@luxul.com>

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

WARNING: multiple messages have this Message-ID (diff)
From: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Dan Haab <dhaab-RSK6WAzFx3QAvxtiuMwx3w@public.gmane.org>
Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"moderated list:ARM PORT"
	<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
	open list <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	zajec5-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	hauke-5/S+JYg5SzeELgA04lAiVw@public.gmane.org
Subject: Re: [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510
Date: Wed, 4 May 2016 11:53:50 -0700	[thread overview]
Message-ID: <572A453E.6010000@gmail.com> (raw)
In-Reply-To: <1462296501-5209-1-git-send-email-dhaab-RSK6WAzFx3QAvxtiuMwx3w@public.gmane.org>

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

WARNING: multiple messages have this Message-ID (diff)
From: Florian Fainelli <f.fainelli@gmail.com>
To: Dan Haab <dhaab@luxul.com>
Cc: "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
	<devicetree@vger.kernel.org>,
	"moderated list:ARM PORT" <linux-arm-kernel@lists.infradead.org>,
	open list <linux-kernel@vger.kernel.org>,
	zajec5@gmail.com, hauke@hauke-m.de
Subject: Re: [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510
Date: Wed, 4 May 2016 11:53:50 -0700	[thread overview]
Message-ID: <572A453E.6010000@gmail.com> (raw)
In-Reply-To: <1462296501-5209-1-git-send-email-dhaab@luxul.com>

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

  reply	other threads:[~2016-05-04 18:53 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-03 17:28 [PATCH] ARM: BCM5301X: Add DT for Luxul XAP-1510 Dan Haab
2016-05-03 17:28 ` Dan Haab
2016-05-03 17:28 ` Dan Haab
2016-05-04 18:53 ` Florian Fainelli [this message]
2016-05-04 18:53   ` Florian Fainelli
2016-05-04 18:53   ` Florian Fainelli
2016-05-04 22:36   ` Rafał Miłecki
2016-05-04 22:36     ` Rafał Miłecki
2016-05-04 22:36     ` Rafał Miłecki
2016-05-05 21:47     ` Hauke Mehrtens
2016-05-05 21:47       ` Hauke Mehrtens
2016-05-05 21:47       ` Hauke Mehrtens
2016-05-05 21:48       ` Florian Fainelli
2016-05-05 21:48         ` Florian Fainelli
2016-05-05 21:48         ` Florian Fainelli

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=572A453E.6010000@gmail.com \
    --to=f.fainelli@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.