Skip to contentSkip to conversation

MAINTAINERS: update BCM SoC entries

PATCH In review Jon Mason 4 participants 11 messages Jun 20, 2016 – Jul 5, 2016

lore
Share & tools
lore

Jon Mason

author
#0 lore raw

cover MAINTAINERS: update BCM SoC entries

An overall clean-up to the MAINTAINER entries for a few Broadcom SoCs.

Jon Mason (4):
MAINTAINERS: Fix nsp false-positives
MAINTAINERS: Add NS2 entry
MAINTAINERS: Update BCM63XX entry
MAINTAINERS: Update BCM281XX/BCM11XXX/BCM216XX entry

 MAINTAINERS | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)
Signature · 1 line
1.9.1
author
#1 lore raw

PATCH 1/4 MAINTAINERS: Fix nsp false-positives

The 'nsp' regex pattern in the "BROADCOM IPROC ARM ARCHITECTURE" section is getting unintended hits due to the common frequence of these letters appearing in sequence. To change the regex expression to be more specific to the files we care about, add a "bcm" prefix to the regex and add file entries for those that do not naturally match this new regex.

Signed-off-by Jon Mason <[email protected]>
---
 MAINTAINERS | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
MAINTAINERS +3 -1
index 7304d2e..18b235a 100644
@@ -2601,7 +2601,7 @@ T:	git git://github.com/broadcom/cygnus-linux.git
 S:	Maintained
 N:	iproc
 N:	cygnus
-N:	nsp
+N:	bcm[-_]nsp
 N:	bcm9113*
 N:	bcm9583*
 N:	bcm9585*

@@ -2612,6 +2612,8 @@ N:	bcm583*
 N:	bcm585*
 N:	bcm586*
 N:	bcm88312
+F:	drivers/clk/bcm/clk-ns*
+F:	drivers/pinctrl/bcm/pinctrl-ns*
 
 BROADCOM BRCMSTB GPIO DRIVER
 M:	Gregory Fong <[email protected]>
Signature · 1 line
1.9.1
author
#2 lore raw

PATCH 2/4 MAINTAINERS: Add NS2 entry

In reply to Jon Mason

Add NS2 to the Broadcom iProc Subsystem maintainers entry. Since most of the NS2 entries are already covered via the ns* already present there, all that is currently needed is to reference the device tree files.

Signed-off-by Jon Mason <[email protected]>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)
MAINTAINERS +1 -0
index 18b235a..81be402 100644
@@ -2612,6 +2612,7 @@ N:	bcm583*
 N:	bcm585*
 N:	bcm586*
 N:	bcm88312
+F:	arch/arm64/boot/dts/broadcom/ns2*
 F:	drivers/clk/bcm/clk-ns*
 F:	drivers/pinctrl/bcm/pinctrl-ns*
Signature · 1 line
1.9.1
author
#3 lore raw

PATCH 3/4 MAINTAINERS: Update BCM63XX entry

In reply to Jon Mason

There are more files being supported by the BCM63XX than simply "arch/arm/mach-bcm/bcm63xx.c" and "arch/arm/include/debug/bcm63xx.S". Add a regex of "bcm63xx" to catch all the other files that are out there.

Signed-off-by Jon Mason <[email protected]>
---
 MAINTAINERS | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
MAINTAINERS +1 -2
index 81be402..28a35f4 100644
@@ -2523,8 +2523,7 @@ 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/stblinux.git
 S:	Maintained
-F:	arch/arm/mach-bcm/bcm63xx.c
-F:	arch/arm/include/debug/bcm63xx.S
+N:	bcm63xx
 
 BROADCOM BCM63XX/BCM33XX UDC DRIVER
 M:	Kevin Cernekee <[email protected]>
Signature · 1 line
1.9.1
author
#4 lore raw

PATCH 4/4 MAINTAINERS: Update BCM281XX/BCM11XXX/BCM216XX entry

In reply to Jon Mason

There are many clock, pinctl, and misc others that should be covered under the BCM281XX/BCM11XXX/BCM216XX ARM listing. Change the entry to use regex's that should cover all the files.

Also, remove the bcm_defconfig entry (as the file is being removed), and arch/arm64/boot/dts/broadcom reference (as that is not accurate for this group of maintainers and all the device trees under it should now be covered by other maintainer entries).

Signed-off-by Jon Mason <[email protected]>
---
 MAINTAINERS | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)
MAINTAINERS +4 -7
index 28a35f4..01382ce 100644
@@ -2481,14 +2481,11 @@ M:	Scott Branden <[email protected]>
 L:	bcm-kernel-feedback-list at broadcom.com
 T:	git git://github.com/broadcom/mach-bcm
 S:	Maintained
+N:	bcm281*
+N:	bcm113*
+N:	bcm216*
+N:	kona
 F:	arch/arm/mach-bcm/
-F:	arch/arm/boot/dts/bcm113*
-F:	arch/arm/boot/dts/bcm216*
-F:	arch/arm/boot/dts/bcm281*
-F:	arch/arm64/boot/dts/broadcom/
-F:	arch/arm/configs/bcm_defconfig
-F:	drivers/mmc/host/sdhci-bcm-kona.c
-F:	drivers/clocksource/bcm_kona_timer.c
 
 BROADCOM BCM2835 ARM ARCHITECTURE
 M:	Stephen Warren <[email protected]>
Signature · 1 line
1.9.1
#5 lore raw

PATCH 4/4 MAINTAINERS: Update BCM281XX/BCM11XXX/BCM216XX entry

Quoted 19 lines · Jon Mason

There are many clock, pinctl, and misc others that should be covered under the BCM281XX/BCM11XXX/BCM216XX ARM listing. Change the entry to use regex's that should cover all the files.

Also, remove the bcm_defconfig entry (as the file is being removed), and arch/arm64/boot/dts/broadcom reference (as that is not accurate for this group of maintainers and all the device trees under it should now be covered by other maintainer entries).

Signed-off-by Jon Mason <[email protected]>
---
 MAINTAINERS | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)
MAINTAINERS +0 -0
index 28a35f4..01382ce 100644

MAINTAINERS M: Scott Branden <[email protected]> · line 2487

Quoted 7 lines
MAINTAINERS +4 -0
@@ -2481,14 +2481,11 @@ M:      Scott Branden <[email protected]>
 L:     bcm-kernel-feedback-list at broadcom.com
 T:     git git://github.com/broadcom/mach-bcm
 S:     Maintained
+N:     bcm281*
+N:     bcm113*
+N:     bcm216*
+N:     kona

Hmmm. We should probably add bcm23550 here, too.

MAINTAINERS M: Scott Branden <[email protected]> · line 2491

Quoted 11 lines
MAINTAINERS +0 -7
 F:     arch/arm/mach-bcm/
-F:     arch/arm/boot/dts/bcm113*
-F:     arch/arm/boot/dts/bcm216*
-F:     arch/arm/boot/dts/bcm281*
-F:     arch/arm64/boot/dts/broadcom/
-F:     arch/arm/configs/bcm_defconfig
-F:     drivers/mmc/host/sdhci-bcm-kona.c
-F:     drivers/clocksource/bcm_kona_timer.c
 
 BROADCOM BCM2835 ARM ARCHITECTURE
 M:     Stephen Warren <[email protected]>

Chris

author
#6 lore raw

PATCH 4/4 MAINTAINERS: Update BCM281XX/BCM11XXX/BCM216XX entry

Quoted 15 lines · Chris Brand
Quoted 14 lines · Jon Mason

There are many clock, pinctl, and misc others that should be covered under the BCM281XX/BCM11XXX/BCM216XX ARM listing. Change the entry to use regex's that should cover all the files.

Also, remove the bcm_defconfig entry (as the file is being removed), and arch/arm64/boot/dts/broadcom reference (as that is not accurate for this group of maintainers and all the device trees under it should now be covered by other maintainer entries).

Signed-off-by Jon Mason <[email protected]>
---
 MAINTAINERS | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)
Quoted 14 lines

MAINTAINERS M: Scott Branden <[email protected]> · line 2487

Quoted 12 lines
MAINTAINERS +4 -0
index 28a35f4..01382ce 100644
@@ -2481,14 +2481,11 @@ M:      Scott Branden <[email protected]>
 L:     bcm-kernel-feedback-list at broadcom.com
 T:     git git://github.com/broadcom/mach-bcm
 S:     Maintained
+N:     bcm281*
+N:     bcm113*
+N:     bcm216*
+N:     kona

Hmmm. We should probably add bcm23550 here, too.

Sure, I'll be happy to ammend this patch to add "N: bcm23550",
which appears to cover all of the newly added files. Also, does it make sense to add you to the list of maintainers above?

Thanks,
Jon

Quoted 13 lines

MAINTAINERS M: Scott Branden <[email protected]> · line 2491

Quoted 11 lines
MAINTAINERS +0 -7
 F:     arch/arm/mach-bcm/
-F:     arch/arm/boot/dts/bcm113*
-F:     arch/arm/boot/dts/bcm216*
-F:     arch/arm/boot/dts/bcm281*
-F:     arch/arm64/boot/dts/broadcom/
-F:     arch/arm/configs/bcm_defconfig
-F:     drivers/mmc/host/sdhci-bcm-kona.c
-F:     drivers/clocksource/bcm_kona_timer.c
 
 BROADCOM BCM2835 ARM ARCHITECTURE
 M:     Stephen Warren <[email protected]>

Chris

#7 lore raw

PATCH 4/4 MAINTAINERS: Update BCM281XX/BCM11XXX/BCM216XX entry

On Tue, Jun 21, 2016 at 7:34 AM, Jon Mason <[email protected]> wrote:
[snip]

Quoted 6 lines
Quoted 2 lines

Hmmm. We should probably add bcm23550 here, too.

Sure, I'll be happy to ammend this patch to add "N: bcm23550",
which appears to cover all of the newly added files. Also, does it make sense to add you to the list of maintainers above?

I guess so, yes.

Chris

#8 lore raw

PATCH 4/4 MAINTAINERS: Update BCM281XX/BCM11XXX/BCM216XX entry

Le 21/06/2016 09:07, Chris Brand a ?crit :

Quoted 10 lines · Jon Mason

[snip]

Quoted 6 lines
Quoted 2 lines

Hmmm. We should probably add bcm23550 here, too.

Sure, I'll be happy to ammend this patch to add "N: bcm23550",
which appears to cover all of the newly added files. Also, does it make sense to add you to the list of maintainers above?

I guess so, yes.

Do you have a specific email address that would you want to use?

Signature · 1 line
Florian
Acked-by on the v1 series
#9 lore raw
In reply to Jon Mason

Patch series looks good.

Acked-by Scott Branden <[email protected]>
Quoted 11 lines · Jon Mason

An overall clean-up to the MAINTAINER entries for a few Broadcom SoCs.

Jon Mason (4):
MAINTAINERS: Fix nsp false-positives
MAINTAINERS: Add NS2 entry
MAINTAINERS: Update BCM63XX entry
MAINTAINERS: Update BCM281XX/BCM11XXX/BCM216XX entry

  MAINTAINERS | 19 +++++++++----------
  1 file changed, 9 insertions(+), 10 deletions(-)