[PATCH 5/5] MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs

[PATCH 5/5] MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs

[Jon Mason]

Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me
to the list of people responsible for it.

Signed-off-by: Jon Mason <jonmason@broadcom.com>
Reviewed-by: Scott Branden <sbranden@broadcom.com>
---
 MAINTAINERS | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index a9ae6c1..4b851f5 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2315,19 +2315,27 @@ L:	linux-scsi at vger.kernel.org
 S:	Supported
 F:	drivers/scsi/bnx2i/
 
-BROADCOM CYGNUS/IPROC ARM ARCHITECTURE
+BROADCOM IPROC ARM ARCHITECTURE
 M:	Ray Jui <rjui@broadcom.com>
 M:	Scott Branden <sbranden@broadcom.com>
+M:	Jon Mason <jonmason@broadcom.com>
 L:	linux-arm-kernel at lists.infradead.org (moderated for non-subscribers)
 L:	bcm-kernel-feedback-list at broadcom.com
 T:	git git://github.com/broadcom/cygnus-linux.git
 S:	Maintained
 N:	iproc
 N:	cygnus
+N:	nsp
 N:	bcm9113*
 N:	bcm9583*
-N:	bcm583*
+N:	bcm9585*
+N:	bcm9586*
+N:	bcm988312
 N:	bcm113*
+N:	bcm583*
+N:	bcm585*
+N:	bcm586*
+N:	bcm88312
 
 BROADCOM BRCMSTB GPIO DRIVER
 M:	Gregory Fong <gregory.0xf0@gmail.com>
-- 
1.9.1

[PATCH 5/5] MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs

[Olof Johansson]

On Thu, Aug 20, 2015 at 10:47 AM, Jon Mason <jonmason@broadcom.com> wrote:
> Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me
> to the list of people responsible for it.

That's not what this patch does. It removes CYGNUS from the
description as well. Hmm?

I'm also not a fan of the complex patterns, but that's more of a
bikeshedding comment. Does Broadcom really not have a bit more
systematic naming than this? :)

> Signed-off-by: Jon Mason <jonmason@broadcom.com>
> Reviewed-by: Scott Branden <sbranden@broadcom.com>

For maintainer updates, we usuallu like to see a public ack of this
and not just a pre-acked patch. You _could_ have added it yourself
without Scott's actual approval, after all.


-Olof

[PATCH 5/5] MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs

[Florian Fainelli]

On 25/08/15 16:42, Olof Johansson wrote:
> On Thu, Aug 20, 2015 at 10:47 AM, Jon Mason <jonmason@broadcom.com> wrote:
>> Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me
>> to the list of people responsible for it.
> 
> That's not what this patch does. It removes CYGNUS from the
> description as well. Hmm?
> 
> I'm also not a fan of the complex patterns, but that's more of a
> bikeshedding comment. Does Broadcom really not have a bit more
> systematic naming than this? :)

The naming conventions are fairly simple actually:

* BCM is just a common prefix for internal names/projects/whatever
* 9 indicates that this is a development/reference board
* 4 to 5 digits designate the chip identifier

Now, if you are trying to get some sense about how the chip ids are
actually chosen, I am afraid none of us have that knowledge ;)

> 
>> Signed-off-by: Jon Mason <jonmason@broadcom.com>
>> Reviewed-by: Scott Branden <sbranden@broadcom.com>
> 
> For maintainer updates, we usuallu like to see a public ack of this
> and not just a pre-acked patch. You _could_ have added it yourself
> without Scott's actual approval, after all.
> 
> 
> -Olof
> 


-- 
Florian

[PATCH 5/5] MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs

[Scott Branden]

On 15-08-25 04:42 PM, Olof Johansson wrote:
> On Thu, Aug 20, 2015 at 10:47 AM, Jon Mason <jonmason@broadcom.com> wrote:
>> Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me
>> to the list of people responsible for it.
>
> That's not what this patch does. It removes CYGNUS from the
> description as well. Hmm?
>
> I'm also not a fan of the complex patterns, but that's more of a
> bikeshedding comment. Does Broadcom really not have a bit more
> systematic naming than this? :)
>
>> Signed-off-by: Jon Mason <jonmason@broadcom.com>
>> Reviewed-by: Scott Branden <sbranden@broadcom.com>
Acked-by: Scott Branden <sbranden@broadcom.com>

>
> For maintainer updates, we usuallu like to see a public ack of this
> and not just a pre-acked patch. You _could_ have added it yourself
> without Scott's actual approval, after all.
>
>
> -Olof
>

[PATCH 5/5] MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs

[Jon Mason]

On Tue, Aug 25, 2015 at 04:42:42PM -0700, Olof Johansson wrote:
> On Thu, Aug 20, 2015 at 10:47 AM, Jon Mason <jonmason@broadcom.com> wrote:
> > Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me
> > to the list of people responsible for it.
> 
> That's not what this patch does. It removes CYGNUS from the
> description as well. Hmm?

iProc is a family of SoCs that includes Cygnus, Northstar (BCM5301X),
Northstar Plus, Northstar 2, and a number of other platforms.  I
removed the "Cygnus" for that reason.

> 
> I'm also not a fan of the complex patterns, but that's more of a
> bikeshedding comment. Does Broadcom really not have a bit more
> systematic naming than this? :)

Per Florian's comment, it's a bit of a mess internally.

> 
> > Signed-off-by: Jon Mason <jonmason@broadcom.com>
> > Reviewed-by: Scott Branden <sbranden@broadcom.com>
> 
> For maintainer updates, we usuallu like to see a public ack of this
> and not just a pre-acked patch. You _could_ have added it yourself
> without Scott's actual approval, after all.

Thanks,
Jon

> 
> 
> -Olof

[PATCH 5/5] MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs

[Olof Johansson]

On Wed, Aug 26, 2015 at 12:00 PM, Jon Mason <jonmason@broadcom.com> wrote:
> On Tue, Aug 25, 2015 at 04:42:42PM -0700, Olof Johansson wrote:
>> On Thu, Aug 20, 2015 at 10:47 AM, Jon Mason <jonmason@broadcom.com> wrote:
>> > Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me
>> > to the list of people responsible for it.
>>
>> That's not what this patch does. It removes CYGNUS from the
>> description as well. Hmm?
>
> iProc is a family of SoCs that includes Cygnus, Northstar (BCM5301X),
> Northstar Plus, Northstar 2, and a number of other platforms.  I
> removed the "Cygnus" for that reason.

Ok, feel free to add that info to the commit message in case others
see the same and wonder what happened.

>> I'm also not a fan of the complex patterns, but that's more of a
>> bikeshedding comment. Does Broadcom really not have a bit more
>> systematic naming than this? :)
>
> Per Florian's comment, it's a bit of a mess internally.

Unfortunate. Hopefully the chaotic product naming doesn't have to leak
quite everywhere in the tree and things can be referred to by family,
etc. :)


-Olof