MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs
PATCH 5/5 MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs
Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me to the list of people responsible for it.
--- MAINTAINERS | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-)
MAINTAINERS +10 -2
@@ -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 <[email protected]> M: Scott Branden <[email protected]> +M: Jon Mason <[email protected]> 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 <[email protected]>
Signature · 1 line
1.9.1
PATCH 5/5 MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs
Quoted 2 lines · Jon Mason
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? :)
Quoted 2 lines
Signed-off-by Jon Mason <[email protected]>Reviewed-by Scott Branden <[email protected]>
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
Quoted 10 lines · Olof Johansson
Quoted 2 lines · Jon Mason
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 ;)
Quoted 11 lines
Quoted 3 lines
Signed-off-by Jon Mason <[email protected]>Reviewed-by Scott Branden <[email protected]>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
Signature · 1 line
Florian
PATCH 5/5 MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs
In reply to Olof JohanssonQuoted 4 lines · Olof Johansson
Quoted 2 lines · Jon Mason
Modify the iProc MAINTAINER to cover the Nortstar Plus SoCs, and add me to the list of people responsible for it.
Quoted 9 lines
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? :)
Quoted 2 lines
Signed-off-by Jon Mason <[email protected]>Reviewed-by Scott Branden <[email protected]>
Quoted 8 lines
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
In reply to Olof JohanssonQuoted 6 lines · Olof Johansson
Quoted 2 lines · Jon Mason
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.
Quoted 4 lines
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.
Quoted 7 lines
Quoted 3 lines
Signed-off-by Jon Mason <[email protected]>Reviewed-by Scott Branden <[email protected]>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
Quoted 3 lines
-Olof
PATCH 5/5 MAINTAINERS: add entry for the Broadcom Northstar Plus SoCs
Quoted 11 lines · Jon Mason
Quoted 6 lines · Olof Johansson
Quoted 2 lines · Jon Mason
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.
Quoted 5 lines
Quoted 3 lines
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