Skip to contentSkip to conversation

MAINTAINERS: brcmstb entry updates

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

v1 of 2 Superseded Latest is v2 →

lore
Share & tools
lore

Florian Fainelli

author
#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
author
#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
#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

author
#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

#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.

g.

author
#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

#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

author
#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
author
#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