diff mbox series

[v7,31/37] MAINTAINERS: Add nds32

Message ID f7758221d49fd24fdff7c900aa5951a6c1c6a3cd.1518505384.git.greentime@andestech.com
State Not Applicable, archived
Delegated to: David Miller
Headers show
Series Andes(nds32) Linux Kernel Port | expand

Commit Message

Greentime Hu Feb. 13, 2018, 9:09 a.m. UTC 
Add a maintainer information for the nds32(Andes) architecture.

Signed-off-by: Greentime Hu <greentime@andestech.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
---
 MAINTAINERS | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Joe Perches Feb. 13, 2018, 4:02 p.m. UTC | #1 
On Tue, 2018-02-13 at 17:09 +0800, Greentime Hu wrote:
> Add a maintainer information for the nds32(Andes) architecture.
[]
> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -868,6 +868,17 @@ X:	drivers/iio/*/adjd*
>  F:	drivers/staging/iio/*/ad*
>  F:	drivers/staging/iio/trigger/iio-trig-bfin-timer.c
>  
> +ANDES ARCHITECTURE
> +M:	Greentime Hu <green.hu@gmail.com>
> +M:	Vincent Chen <deanbo422@gmail.com>
> +T:	git https://github.com/andestech/linux.git
> +S:	Supported
> +F:	arch/nds32

This should have a trailing /

F:	arch/nds32/

> +F:	Documentation/devicetree/bindings/interrupt-controller/andestech,ativic32.txt
> +F:	Documentation/devicetree/bindings/nds32
> +K:	nds32

Perhaps this should be

K:	\bnds32

as there are some existing uses of nds32 in the current tree.

or maybe case insensitive like

K:	(?i:\bnds32)
or
K:	(?:\bnds32|\bNDS32)

> +N:	nds32
> +
>  ANDROID CONFIG FRAGMENTS
>  M:	Rob Herring <robh@kernel.org>
>  S:	Supported
Greentime Hu Feb. 14, 2018, 5:54 a.m. UTC | #2 
2018-02-14 0:02 GMT+08:00 Joe Perches <joe@perches.com>:
> On Tue, 2018-02-13 at 17:09 +0800, Greentime Hu wrote:
>> Add a maintainer information for the nds32(Andes) architecture.
> []
>> diff --git a/MAINTAINERS b/MAINTAINERS
> []
>> @@ -868,6 +868,17 @@ X:       drivers/iio/*/adjd*
>>  F:   drivers/staging/iio/*/ad*
>>  F:   drivers/staging/iio/trigger/iio-trig-bfin-timer.c
>>
>> +ANDES ARCHITECTURE
>> +M:   Greentime Hu <green.hu@gmail.com>
>> +M:   Vincent Chen <deanbo422@gmail.com>
>> +T:   git https://github.com/andestech/linux.git
>> +S:   Supported
>> +F:   arch/nds32
>
> This should have a trailing /
>
> F:      arch/nds32/

Thank you Joe.
I will add this trailing /
>
>> +F:   Documentation/devicetree/bindings/interrupt-controller/andestech,ativic32.txt
>> +F:   Documentation/devicetree/bindings/nds32

And here

>> +K:   nds32
>
> Perhaps this should be
>
> K:      \bnds32
>
> as there are some existing uses of nds32 in the current tree.
>
> or maybe case insensitive like
>
> K:      (?i:\bnds32)
> or
> K:      (?:\bnds32|\bNDS32)
>

I think it might be better to keep it "nds32" becaue some intrinsic
functions are defined with __nds32__xxx.
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 3bdc260e36b7..bf7c2ac6bb5a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -868,6 +868,17 @@  X:	drivers/iio/*/adjd*
 F:	drivers/staging/iio/*/ad*
 F:	drivers/staging/iio/trigger/iio-trig-bfin-timer.c
 
+ANDES ARCHITECTURE
+M:	Greentime Hu <green.hu@gmail.com>
+M:	Vincent Chen <deanbo422@gmail.com>
+T:	git https://github.com/andestech/linux.git
+S:	Supported
+F:	arch/nds32
+F:	Documentation/devicetree/bindings/interrupt-controller/andestech,ativic32.txt
+F:	Documentation/devicetree/bindings/nds32
+K:	nds32
+N:	nds32
+
 ANDROID CONFIG FRAGMENTS
 M:	Rob Herring <robh@kernel.org>
 S:	Supported