MAINTAINERS: brcmstb entry updates

PATCH Superseded Florian Fainelli 5 participants 11 messages Oct 28, 2014 – Jan 15, 2015

lore

Share & tools

lore

Florian Fainelli

author

2014-10-28 18:44 UTC #0 lore raw

cover MAINTAINERS: brcmstb entry updates

Arnd, Olof,

Here are two small updates to the brcmstb MAINTAINERS entry for our platform.

I would appreciate if you could queue these for an upcoming 3.18-rc, thanks!

Thanks!

Florian Fainelli (2):
MAINTAINERS: update brcmstb entries
MAINTAINERS: add entry for the GISB arbiter driver

 MAINTAINERS | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Signature · 1 line

1.9.1

Florian Fainelli

author

2014-10-28 18:44 UTC #1 lore raw

PATCH 1/2 MAINTAINERS: update brcmstb entries

Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.

Signed-off-by Florian Fainelli <[email protected]>

---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

MAINTAINERS +2 -1

index dab92a78d1d5..33a8b31e03f0 100644
@@ -2093,8 +2093,9 @@ F:	arch/arm/mach-bcm/bcm63xx.c
 F:	arch/arm/include/debug/bcm63xx.S
 
 BROADCOM BCM7XXX ARM ARCHITECTURE
-M:	Marc Carino <[email protected]>
 M:	Brian Norris <[email protected]>
+M:	Gregory Fong <[email protected]>
+M:	Florian Fainelli <[email protected]>
 L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
 S:	Maintained
 F:	arch/arm/mach-bcm/*brcmstb*

Signature · 1 line

1.9.1

Gregory Fong

2014-10-31 20:14 UTC #2 lore raw

PATCH 1/2 MAINTAINERS: update brcmstb entries

Quoted 4 lines · Florian Fainelli

Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.
Signed-off-by Florian Fainelli <[email protected]>

Acked-by Gregory Fong <[email protected]>

Olof Johansson

2014-11-04 20:45 UTC #3 lore raw

PATCH 1/2 MAINTAINERS: update brcmstb entries

In reply to Florian Fainelli

Hi,

Quoted 13 lines · Florian Fainelli

Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.
Signed-off-by Florian Fainelli <[email protected]>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
MAINTAINERS +0 -0
index dab92a78d1d5..33a8b31e03f0 100644

MAINTAINERS F: arch/arm/mach-bcm/bcm63xx.c · line 2098

Quoted 7 lines

MAINTAINERS +2 -1

@@ -2093,8 +2093,9 @@ F:	arch/arm/mach-bcm/bcm63xx.c
 F:	arch/arm/include/debug/bcm63xx.S
 
 BROADCOM BCM7XXX ARM ARCHITECTURE
-M:	Marc Carino <[email protected]>
 M:	Brian Norris <[email protected]>
+M:	Gregory Fong <[email protected]>
+M:	Florian Fainelli <[email protected]>

You're removing Marc here, but I don't see an ack from him, nor has he been cc:d on the patch. It's common practice for people to agree to being removed. Can you please make sure we get an ack from him on this, maybe by reposting and cc:ing him?

Thanks!

-Olof

Florian Fainelli

author

2014-11-04 22:48 UTC #4 lore raw

PATCH 1/2 MAINTAINERS: update brcmstb entries

Quoted 18 lines · Olof Johansson

Hi,
Quoted 15 lines · Florian Fainelli
Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.
Signed-off-by Florian Fainelli <[email protected]>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
MAINTAINERS +0 -0
index dab92a78d1d5..33a8b31e03f0 100644
@@ -2093,8 +2093,9 @@ F:	arch/arm/mach-bcm/bcm63xx.c
 F:	arch/arm/include/debug/bcm63xx.S
 

Quoted 10 lines

MAINTAINERS F: arch/arm/mach-bcm/bcm63xx.c · line 2098
Quoted 5 lines
MAINTAINERS +2 -1
 BROADCOM BCM7XXX ARM ARCHITECTURE
-M:	Marc Carino <[email protected]>
 M:	Brian Norris <[email protected]>
+M:	Gregory Fong <[email protected]>
+M:	Florian Fainelli <[email protected]>
You're removing Marc here, but I don't see an ack from him, nor has he been cc:d on the patch. It's common practice for people to agree to being removed. Can you please make sure we get an ack from him on this, maybe by reposting and cc:ing him?

Marc is no longer with Broadcom, do you still want him to ack this? --
Florian

Grant Likely

2014-11-04 22:56 UTC #5 lore raw

PATCH 1/2 MAINTAINERS: update brcmstb entries

Quoted 30 lines · Florian Fainelli

Quoted 18 lines · Olof Johansson
Hi,
Quoted 15 lines · Florian Fainelli
Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.
Signed-off-by Florian Fainelli <[email protected]>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
MAINTAINERS +0 -0
index dab92a78d1d5..33a8b31e03f0 100644
@@ -2093,8 +2093,9 @@ F:     arch/arm/mach-bcm/bcm63xx.c
 F:  arch/arm/include/debug/bcm63xx.S
 
Quoted 10 lines
MAINTAINERS F: arch/arm/mach-bcm/bcm63xx.c · line 2098
Quoted 5 lines
MAINTAINERS +2 -1
 BROADCOM BCM7XXX ARM ARCHITECTURE
-M:  Marc Carino <[email protected]>
 M:  Brian Norris <[email protected]>
+M:  Gregory Fong <[email protected]>
+M:  Florian Fainelli <[email protected]>
You're removing Marc here, but I don't see an ack from him, nor has he been cc:d on the patch. It's common practice for people to agree to being removed. Can you please make sure we get an ack from him on this, maybe by reposting and cc:ing him?

Quoted 1 line

Marc is no longer with Broadcom, do you still want him to ack this?

Yes, absolutely. Maintainership is entrusted to an individual, not their employer. We've had many situations where someone has continued to maintain a platform even after changing employment. So, yes, there needs to be an ack from Marc.

Florian Fainelli

author

2014-11-04 23:23 UTC #6 lore raw

PATCH 1/2 MAINTAINERS: update brcmstb entries

Quoted 31 lines · Grant Likely

Quoted 30 lines · Florian Fainelli
Quoted 28 lines · Olof Johansson
Hi,
Quoted 13 lines · Florian Fainelli
Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.
Signed-off-by Florian Fainelli <[email protected]>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
MAINTAINERS +0 -0
index dab92a78d1d5..33a8b31e03f0 100644
MAINTAINERS F: arch/arm/mach-bcm/bcm63xx.c · line 2098
Quoted 7 lines
MAINTAINERS +2 -1
@@ -2093,8 +2093,9 @@ F:     arch/arm/mach-bcm/bcm63xx.c
 F:  arch/arm/include/debug/bcm63xx.S
 
 BROADCOM BCM7XXX ARM ARCHITECTURE
-M:  Marc Carino <[email protected]>
 M:  Brian Norris <[email protected]>
+M:  Gregory Fong <[email protected]>
+M:  Florian Fainelli <[email protected]>
You're removing Marc here, but I don't see an ack from him, nor has he been cc:d on the patch. It's common practice for people to agree to being removed. Can you please make sure we get an ack from him on this, maybe by reposting and cc:ing him?

Quoted 6 lines

Quoted 1 line
Marc is no longer with Broadcom, do you still want him to ack this?
Yes, absolutely. Maintainership is entrusted to an individual, not their employer. We've had many situations where someone has continued to maintain a platform even after changing employment. So, yes, there needs to be an ack from Marc.

He is CC'd, now. Marc, please Ack this if you are okay with the change.

Thanks
--
Florian

Brian Norris

2015-01-15 00:13 UTC #7 lore raw

Re: [PATCH 1/2] MAINTAINERS: update brcmstb entries

Since I found this in my inbox during garbage collection...

Quoted 30 lines · Florian Fainelli

Quoted 29 lines · Grant Likely
Quoted 28 lines · Florian Fainelli
Quoted 26 lines · Olof Johansson
Quoted 13 lines · Florian Fainelli
Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.
Signed-off-by Florian Fainelli <[email protected]>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
MAINTAINERS +0 -0
index dab92a78d1d5..33a8b31e03f0 100644
MAINTAINERS F: arch/arm/mach-bcm/bcm63xx.c · line 2098
Quoted 7 lines
MAINTAINERS +2 -1
@@ -2093,8 +2093,9 @@ F:     arch/arm/mach-bcm/bcm63xx.c
 F:  arch/arm/include/debug/bcm63xx.S
 
 BROADCOM BCM7XXX ARM ARCHITECTURE
-M:  Marc Carino <[email protected]>
 M:  Brian Norris <[email protected]>
+M:  Gregory Fong <[email protected]>
+M:  Florian Fainelli <[email protected]>
You're removing Marc here, but I don't see an ack from him, nor has he been cc:d on the patch. It's common practice for people to agree to being removed. Can you please make sure we get an ack from him on this, maybe by reposting and cc:ing him?

Quoted 8 lines

Quoted 6 lines
Quoted 1 line
Marc is no longer with Broadcom, do you still want him to ack this?
Yes, absolutely. Maintainership is entrusted to an individual, not their employer. We've had many situations where someone has continued to maintain a platform even after changing employment. So, yes, there needs to be an ack from Marc.
He is CC'd, now. Marc, please Ack this if you are okay with the change.

Can we add a "Not-nacked-by" tag and proceed? Marc is obviously not involved here any more. I figure removing expired maintainers should be a little easier than removing SCOTUS judges.

Brian

Florian Fainelli

author

2015-01-15 01:52 UTC #8 lore raw

Re: [PATCH 1/2] MAINTAINERS: update brcmstb entries

Quoted 40 lines · Brian Norris

Since I found this in my inbox during garbage collection...
Quoted 37 lines · Florian Fainelli
Quoted 35 lines · Grant Likely
Quoted 28 lines · Florian Fainelli
Quoted 16 lines · Olof Johansson
Quoted 15 lines · Florian Fainelli
Remove Marc Carino from the brcmstb maintainers list and add Gregory Fong and myself as new maintainers for the brcmstb platform.
Signed-off-by Florian Fainelli <[email protected]>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
MAINTAINERS +0 -0
index dab92a78d1d5..33a8b31e03f0 100644
@@ -2093,8 +2093,9 @@ F:     arch/arm/mach-bcm/bcm63xx.c
 F:  arch/arm/include/debug/bcm63xx.S
 
Quoted 10 lines
MAINTAINERS F: arch/arm/mach-bcm/bcm63xx.c · line 2098
Quoted 5 lines
MAINTAINERS +2 -1
 BROADCOM BCM7XXX ARM ARCHITECTURE
-M:  Marc Carino <[email protected]>
 M:  Brian Norris <[email protected]>
+M:  Gregory Fong <[email protected]>
+M:  Florian Fainelli <[email protected]>
You're removing Marc here, but I don't see an ack from him, nor has he been cc:d on the patch. It's common practice for people to agree to being removed. Can you please make sure we get an ack from him on this, maybe by reposting and cc:ing him?
Quoted 1 line
Marc is no longer with Broadcom, do you still want him to ack this?
Yes, absolutely. Maintainership is entrusted to an individual, not their employer. We've had many situations where someone has continued to maintain a platform even after changing employment. So, yes, there needs to be an ack from Marc.

Quoted 5 lines

Quoted 1 line
He is CC'd, now. Marc, please Ack this if you are okay with the change.
Can we add a "Not-nacked-by" tag and proceed? Marc is obviously not involved here any more. I figure removing expired maintainers should be a little easier than removing SCOTUS judges.

You'd think so right? How about this plan: unless Marc replies otherwise, I submit a MAINTAINERS update removing him for the v3.21 kernel release?

Signature · 1 line

Florian

Florian Fainelli

author

2014-10-28 18:44 UTC #9 lore raw

PATCH 2/2 MAINTAINERS: add entry for the GISB arbiter driver

In reply to Florian Fainelli

The GISB bus arbiter driver is relevant for BCM7xxx (brcmstb) platforms, add an entry to cover this file.

Signed-off-by Florian Fainelli <[email protected]>

---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

MAINTAINERS +1 -0

index 33a8b31e03f0..e9fe70cb525a 100644
@@ -2100,6 +2100,7 @@ L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
 S:	Maintained
 F:	arch/arm/mach-bcm/*brcmstb*
 F:	arch/arm/boot/dts/bcm7*.dts*
+F:	drivers/bus/brcmstb_gisb.c
 
 BROADCOM TG3 GIGABIT ETHERNET DRIVER
 M:	Prashant Sreedharan <[email protected]>

Signature · 1 line

1.9.1

Brian Norris

2014-10-29 05:03 UTC #10 lore raw

cover MAINTAINERS: brcmstb entry updates

In reply to Florian Fainelli

Quoted 4 lines · Florian Fainelli

Here are two small updates to the brcmstb MAINTAINERS entry for our platform.
I would appreciate if you could queue these for an upcoming 3.18-rc, thanks!

In case it helps:

Acked-by Brian Norris <[email protected]>

for both patches.