2020-08-12 07:54:43

by Álvaro Fernández Rojas

Subject: [PATCH v4 0/5] MIPS: BCM63xx: board improvements

v4: Split license change to a new patch.
v3: Reword DWV-S0 board commit description to avoid possible confusions.
v2: switch to SPDX license identifier.

Álvaro Fernández Rojas (5):
MIPS: BCM63xx: remove duplicated new lines
MIPS: BCM63xx: remove EHCI from BCM6348 boards
MIPS: BCM63xx: enable EHCI for DWV-S0 board
MIPS: BCM63xx: refactor board declarations
MIPS: BCM63xx: switch to SPDX license identifier

arch/mips/bcm63xx/boards/board_bcm963xx.c | 625 +++++++++++-----------
1 file changed, 306 insertions(+), 319 deletions(-)

--
2.28.0

2020-08-12 07:56:06

by Álvaro Fernández Rojas

Subject: [PATCH v4 5/5] MIPS: BCM63xx: switch to SPDX license identifier

Use SPDX license indentifier instead of local reference to COPYING.

Signed-off-by: Álvaro Fernández Rojas <[email protected]>
---
v4: add new patch with SPDX license change.

arch/mips/bcm63xx/boards/board_bcm963xx.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/arch/mips/bcm63xx/boards/board_bcm963xx.c b/arch/mips/bcm63xx/boards/board_bcm963xx.c
index fa0acf003a46..01aff80a5967 100644
--- a/arch/mips/bcm63xx/boards/board_bcm963xx.c
+++ b/arch/mips/bcm63xx/boards/board_bcm963xx.c
@@ -1,8 +1,5 @@
+// SPDX-License-Identifier: GPL-2.0-only
/*
- * This file is subject to the terms and conditions of the GNU General Public
- * License. See the file "COPYING" in the main directory of this archive
- * for more details.
- *
* Copyright (C) 2008 Maxime Bizon <[email protected]>
* Copyright (C) 2008 Florian Fainelli <[email protected]>
*/
--
2.28.0

2020-08-12 07:56:34

by Álvaro Fernández Rojas

Subject: [PATCH v4 2/5] MIPS: BCM63xx: remove EHCI from BCM6348 boards

There's no EHCI controller on BCM6348.

Signed-off-by: Álvaro Fernández Rojas <[email protected]>
Acked-by: Florian Fainelli <[email protected]>
---
v4: no changes.
v3: no changes.
v2: no changes.

arch/mips/bcm63xx/boards/board_bcm963xx.c | 3 ---
1 file changed, 3 deletions(-)

diff --git a/arch/mips/bcm63xx/boards/board_bcm963xx.c b/arch/mips/bcm63xx/boards/board_bcm963xx.c
index 744aa16bab12..45f1bc437245 100644
--- a/arch/mips/bcm63xx/boards/board_bcm963xx.c
+++ b/arch/mips/bcm63xx/boards/board_bcm963xx.c
@@ -285,7 +285,6 @@ static struct board_info __initdata board_96348gw_10 = {

.has_ohci0 = 1,
.has_pccard = 1,
- .has_ehci0 = 1,

.leds = {
{
@@ -338,7 +337,6 @@ static struct board_info __initdata board_96348gw_11 = {

.has_ohci0 = 1,
.has_pccard = 1,
- .has_ehci0 = 1,

.leds = {
{
@@ -441,7 +439,6 @@ static struct board_info __initdata board_FAST2404 = {

.has_ohci0 = 1,
.has_pccard = 1,
- .has_ehci0 = 1,
};

static struct board_info __initdata board_rta1025w_16 = {
--
2.28.0

2020-08-12 13:06:32

by Philippe Mathieu-Daudé

Subject: Re: [PATCH v4 2/5] MIPS: BCM63xx: remove EHCI from BCM6348 boards

On Wed, Aug 12, 2020 at 9:53 AM Álvaro Fernández Rojas
<[email protected]> wrote:
>
> There's no EHCI controller on BCM6348.
>
> Signed-off-by: Álvaro Fernández Rojas <[email protected]>
> Acked-by: Florian Fainelli <[email protected]>

v3 already reviewed, again:
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>

> ---
> v4: no changes.
> v3: no changes.
> v2: no changes.
>
> arch/mips/bcm63xx/boards/board_bcm963xx.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/arch/mips/bcm63xx/boards/board_bcm963xx.c b/arch/mips/bcm63xx/boards/board_bcm963xx.c
> index 744aa16bab12..45f1bc437245 100644
> --- a/arch/mips/bcm63xx/boards/board_bcm963xx.c
> +++ b/arch/mips/bcm63xx/boards/board_bcm963xx.c
> @@ -285,7 +285,6 @@ static struct board_info __initdata board_96348gw_10 = {
>
> .has_ohci0 = 1,
> .has_pccard = 1,
> - .has_ehci0 = 1,
>
> .leds = {
> {
> @@ -338,7 +337,6 @@ static struct board_info __initdata board_96348gw_11 = {
>
> .has_ohci0 = 1,
> .has_pccard = 1,
> - .has_ehci0 = 1,
>
> .leds = {
> {
> @@ -441,7 +439,6 @@ static struct board_info __initdata board_FAST2404 = {
>
> .has_ohci0 = 1,
> .has_pccard = 1,
> - .has_ehci0 = 1,
> };
>
> static struct board_info __initdata board_rta1025w_16 = {
> --
> 2.28.0
>

2020-08-12 14:25:31

by Florian Fainelli

Subject: Re: [PATCH v4 5/5] MIPS: BCM63xx: switch to SPDX license identifier

On 8/12/2020 12:52 AM, Álvaro Fernández Rojas wrote:
> Use SPDX license indentifier instead of local reference to COPYING.
>
> Signed-off-by: Álvaro Fernández Rojas <[email protected]>

Acked-by: Florian Fainelli <[email protected]>
--
Florian

2020-08-17 11:36:25

by Thomas Bogendoerfer

Subject: Re: [PATCH v4 0/5] MIPS: BCM63xx: board improvements

On Wed, Aug 12, 2020 at 09:52:30AM +0200, ?lvaro Fern?ndez Rojas wrote:
> v4: Split license change to a new patch.
> v3: Reword DWV-S0 board commit description to avoid possible confusions.
> v2: switch to SPDX license identifier.
>
> ?lvaro Fern?ndez Rojas (5):
> MIPS: BCM63xx: remove duplicated new lines
> MIPS: BCM63xx: remove EHCI from BCM6348 boards
> MIPS: BCM63xx: enable EHCI for DWV-S0 board
> MIPS: BCM63xx: refactor board declarations
> MIPS: BCM63xx: switch to SPDX license identifier
>
> arch/mips/bcm63xx/boards/board_bcm963xx.c | 625 +++++++++++-----------
> 1 file changed, 306 insertions(+), 319 deletions(-)

series applied to mips-next.

Thomas.

--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]