Re: [PATCH 29/31] MAINTAINERS: Add nds32

From: Greentime Hu
Date: Thu Nov 16 2017 - 07:23:50 EST


2017-11-14 23:39 GMT+08:00 Joe Perches <joe@xxxxxxxxxxx>:
> On Thu, 2017-11-09 at 11:36 +0100, Arnd Bergmann wrote:
>> On Thu, Nov 9, 2017 at 10:46 AM, Greentime Hu <green.hu@xxxxxxxxx> wrote:
>> > 2017-11-08 21:31 GMT+08:00 Rob Herring <robh+dt@xxxxxxxxxx>:
>> > > On Tue, Nov 7, 2017 at 11:55 PM, Greentime Hu <green.hu@xxxxxxxxx> wrote:
>> > > > From: Greentime Hu <greentime@xxxxxxxxxxxxx>
>> > > >
>> > > > Signed-off-by: Greentime Hu <greentime@xxxxxxxxxxxxx>
>> > > > ---
>> > > > MAINTAINERS | 9 +++++++++
>> > > > 1 file changed, 9 insertions(+)
>> > > >
>> > > > diff --git a/MAINTAINERS b/MAINTAINERS
>> > > > index 2f4e462..bce1181 100644
>> > > > --- a/MAINTAINERS
>> > > > +++ b/MAINTAINERS
>> > > > @@ -857,6 +857,15 @@ 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@xxxxxxxxx>
>> > > > +M: Vincent Chen <deanbo422@xxxxxxxxx>
>> > > > +T: git https://github.com/andestech/linux.git
>> > > > +S: Supported
>> > > > +F: arch/nds32
>> > >
>> > > DT binding files?
>> >
>> > Thanks.
>> > I should add
>> > F: Documentation/devicetree/bindings/interrupt-controller/andestech,ativic32.txt
>> > F: Documentation/devicetree/bindings/nds32/cpus.txt
>> >
>> > I will fix it in the next version patch.
>>
>> Better make the second one the directory, in case you add more files
>> there later.
>>
Thanks. I will add the second one directory.

>> I would also add a "K: nds32" line to catch all files that have this
>> in their names.
>
> That's probably not what would be desired
>
> K: looks for keywords inside all file contents
> N: looks for filenames matching a regex pattern
>
> See the header in MAINTAINERS for more details
>

Thanks.
I will add K and N both.