[PATCH 0/2] MAINTAINERS: brcmstb entry updates

[PATCH 0/2] MAINTAINERS: brcmstb entry updates

[Florian Fainelli]

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(-)

-- 
1.9.1

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[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 <f.fainelli@gmail.com>
---
 MAINTAINERS | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index dab92a78d1d5..33a8b31e03f0 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -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 <marc.ceeeee@gmail.com>
 M:	Brian Norris <computersforpeace@gmail.com>
+M:	Gregory Fong <gregory.0xf0@gmail.com>
+M:	Florian Fainelli <f.fainelli@gmail.com>
 L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
 S:	Maintained
 F:	arch/arm/mach-bcm/*brcmstb*
-- 
1.9.1

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

[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 <f.fainelli@gmail.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 33a8b31e03f0..e9fe70cb525a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -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 <prashant@broadcom.com>
-- 
1.9.1

[PATCH 0/2] MAINTAINERS: brcmstb entry updates

[Brian Norris]

On Tue, Oct 28, 2014 at 11:44:53AM -0700, Florian Fainelli wrote:
> 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 <computersforpeace@gmail.com>

for both patches.

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[Gregory Fong]

On Tue, Oct 28, 2014 at 11:44 AM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> 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 <f.fainelli@gmail.com>

Acked-by: Gregory Fong <gregory.0xf0@gmail.com>

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[Olof Johansson]

Hi,

On Tue, Oct 28, 2014 at 11:44:54AM -0700, Florian Fainelli wrote:
> 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 <f.fainelli@gmail.com>
> ---
>  MAINTAINERS | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index dab92a78d1d5..33a8b31e03f0 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -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 <marc.ceeeee@gmail.com>
>  M:	Brian Norris <computersforpeace@gmail.com>
> +M:	Gregory Fong <gregory.0xf0@gmail.com>
> +M:	Florian Fainelli <f.fainelli@gmail.com>

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

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[Florian Fainelli]

On 11/04/2014 12:45 PM, Olof Johansson wrote:
> Hi,
> 
> On Tue, Oct 28, 2014 at 11:44:54AM -0700, Florian Fainelli wrote:
>> 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 <f.fainelli@gmail.com>
>> ---
>>  MAINTAINERS | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index dab92a78d1d5..33a8b31e03f0 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -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 <marc.ceeeee@gmail.com>
>>  M:	Brian Norris <computersforpeace@gmail.com>
>> +M:	Gregory Fong <gregory.0xf0@gmail.com>
>> +M:	Florian Fainelli <f.fainelli@gmail.com>
> 
> 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

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[Grant Likely]

On Tue, Nov 4, 2014 at 10:48 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> On 11/04/2014 12:45 PM, Olof Johansson wrote:
>> Hi,
>>
>> On Tue, Oct 28, 2014 at 11:44:54AM -0700, Florian Fainelli wrote:
>>> 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 <f.fainelli@gmail.com>
>>> ---
>>>  MAINTAINERS | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index dab92a78d1d5..33a8b31e03f0 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -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 <marc.ceeeee@gmail.com>
>>>  M:  Brian Norris <computersforpeace@gmail.com>
>>> +M:  Gregory Fong <gregory.0xf0@gmail.com>
>>> +M:  Florian Fainelli <f.fainelli@gmail.com>
>>
>> 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?

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.

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[Florian Fainelli]

On 11/04/2014 02:56 PM, Grant Likely wrote:
> On Tue, Nov 4, 2014 at 10:48 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>> On 11/04/2014 12:45 PM, Olof Johansson wrote:
>>> Hi,
>>>
>>> On Tue, Oct 28, 2014 at 11:44:54AM -0700, Florian Fainelli wrote:
>>>> 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 <f.fainelli@gmail.com>
>>>> ---
>>>>  MAINTAINERS | 3 ++-
>>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>> index dab92a78d1d5..33a8b31e03f0 100644
>>>> --- a/MAINTAINERS
>>>> +++ b/MAINTAINERS
>>>> @@ -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 <marc.ceeeee@gmail.com>
>>>>  M:  Brian Norris <computersforpeace@gmail.com>
>>>> +M:  Gregory Fong <gregory.0xf0@gmail.com>
>>>> +M:  Florian Fainelli <f.fainelli@gmail.com>
>>>
>>> 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?
> 
> 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

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[Brian Norris]

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

On Tue, Nov 4, 2014 at 3:23 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> On 11/04/2014 02:56 PM, Grant Likely wrote:
>> On Tue, Nov 4, 2014 at 10:48 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>>> On 11/04/2014 12:45 PM, Olof Johansson wrote:
>>>> On Tue, Oct 28, 2014 at 11:44:54AM -0700, Florian Fainelli wrote:
>>>>> 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 <f.fainelli@gmail.com>
>>>>> ---
>>>>>  MAINTAINERS | 3 ++-
>>>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>>> index dab92a78d1d5..33a8b31e03f0 100644
>>>>> --- a/MAINTAINERS
>>>>> +++ b/MAINTAINERS
>>>>> @@ -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 <marc.ceeeee@gmail.com>
>>>>>  M:  Brian Norris <computersforpeace@gmail.com>
>>>>> +M:  Gregory Fong <gregory.0xf0@gmail.com>
>>>>> +M:  Florian Fainelli <f.fainelli@gmail.com>
>>>>
>>>> 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?
>>
>> 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

[PATCH 1/2] MAINTAINERS: update brcmstb entries

[Florian Fainelli]

On 14/01/15 16:13, Brian Norris wrote:
> Since I found this in my inbox during garbage collection...
> 
> On Tue, Nov 4, 2014 at 3:23 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>> On 11/04/2014 02:56 PM, Grant Likely wrote:
>>> On Tue, Nov 4, 2014 at 10:48 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
>>>> On 11/04/2014 12:45 PM, Olof Johansson wrote:
>>>>> On Tue, Oct 28, 2014 at 11:44:54AM -0700, Florian Fainelli wrote:
>>>>>> 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 <f.fainelli@gmail.com>
>>>>>> ---
>>>>>>  MAINTAINERS | 3 ++-
>>>>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>>>>
>>>>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>>>>> index dab92a78d1d5..33a8b31e03f0 100644
>>>>>> --- a/MAINTAINERS
>>>>>> +++ b/MAINTAINERS
>>>>>> @@ -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 <marc.ceeeee@gmail.com>
>>>>>>  M:  Brian Norris <computersforpeace@gmail.com>
>>>>>> +M:  Gregory Fong <gregory.0xf0@gmail.com>
>>>>>> +M:  Florian Fainelli <f.fainelli@gmail.com>
>>>>>
>>>>> 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?
>>>
>>> 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.

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?
-- 
Florian