diff mbox

[v7,1/4] MAINTAINERS: Add entry for APM X-Gene SoC PMU driver

Message ID 1467850045-10584-2-git-send-email-ttnguyen@apm.com (mailing list archive)
State New, archived
Headers show

Commit Message

Tai Nguyen July 7, 2016, 12:07 a.m. UTC
Signed-off-by: Tai Nguyen <ttnguyen@apm.com>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Joe Perches July 7, 2016, 12:19 a.m. UTC | #1
On Wed, 2016-07-06 at 17:07 -0700, Tai Nguyen wrote:
> Signed-off-by: Tai Nguyen <ttnguyen@apm.com>
> ---
>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -8593,6 +8593,14 @@ L:	linux-scsi@vger.kernel.org
>  S:	Supported
>  F:	drivers/scsi/pm8001/
>  
> +PMU-XGENE
> +APPLIED MICRO (APM) X-GENE SOC PMU

One line section header please.

> +M:	Tai Nguyen <ttnguyen@apm.com>
> +S:	Supported
> +F:	drivers/perf/xgene_pmu.c
> +F:	Documentation/perf/xgene-pmu.txt
> +F:	Documentation/devicetree/bindings/perf/apm-xgene-pmu.txt
> +
>  POSIX CLOCKS and TIMERS
>  M:	Thomas Gleixner <tglx@linutronix.de>
>  L:	linux-kernel@vger.kernel.org
Tai Nguyen July 7, 2016, 12:35 a.m. UTC | #2
Hi Joe,

Thanks.

On Wed, Jul 6, 2016 at 5:19 PM, Joe Perches <joe@perches.com> wrote:
> On Wed, 2016-07-06 at 17:07 -0700, Tai Nguyen wrote:
>> Signed-off-by: Tai Nguyen <ttnguyen@apm.com>
>> ---
>>  MAINTAINERS | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
> []
>> @@ -8593,6 +8593,14 @@ L:     linux-scsi@vger.kernel.org
>>  S:   Supported
>>  F:   drivers/scsi/pm8001/
>>
>> +PMU-XGENE
>> +APPLIED MICRO (APM) X-GENE SOC PMU
>
> One line section header please.

Is this strictly required?
I think I can just drop the first line "PMU-XGENE" if it's required.

>
>> +M:   Tai Nguyen <ttnguyen@apm.com>
>> +S:   Supported
>> +F:   drivers/perf/xgene_pmu.c
>> +F:   Documentation/perf/xgene-pmu.txt
>> +F:   Documentation/devicetree/bindings/perf/apm-xgene-pmu.txt
>> +
>>  POSIX CLOCKS and TIMERS
>>  M:   Thomas Gleixner <tglx@linutronix.de>
>>  L:   linux-kernel@vger.kernel.org

Regards,
Joe Perches July 7, 2016, 12:43 a.m. UTC | #3
On Wed, 2016-07-06 at 17:35 -0700, Tai Tri Nguyen wrote:
> Hi Joe,

Chào Tai.

> On Wed, Jul 6, 2016 at 5:19 PM, Joe Perches <joe@perches.com> wrote:
> > On Wed, 2016-07-06 at 17:07 -0700, Tai Nguyen wrote:
[]
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > []
> > > 
> > > @@ -8593,6 +8593,14 @@ L:     linux-scsi@vger.kernel.org
> > >  S:   Supported
> > >  F:   drivers/scsi/pm8001/
> > > 
> > > +PMU-XGENE
> > > +APPLIED MICRO (APM) X-GENE SOC PMU
> > One line section header please.
> Is this strictly required?

Yes.

get_maintainer.pl doesn't like multiple line section headers.
Tai Nguyen July 7, 2016, 1:01 a.m. UTC | #4
Hi Joe,

On Wed, Jul 6, 2016 at 5:43 PM, Joe Perches <joe@perches.com> wrote:
> On Wed, 2016-07-06 at 17:35 -0700, Tai Tri Nguyen wrote:
>> Hi Joe,
>
> Chào Tai.
>

Thanks for the greeting!

>> On Wed, Jul 6, 2016 at 5:19 PM, Joe Perches <joe@perches.com> wrote:
>> > On Wed, 2016-07-06 at 17:07 -0700, Tai Nguyen wrote:
> []
>> > > diff --git a/MAINTAINERS b/MAINTAINERS
>> > []
>> > >
>> > > @@ -8593,6 +8593,14 @@ L:     linux-scsi@vger.kernel.org
>> > >  S:   Supported
>> > >  F:   drivers/scsi/pm8001/
>> > >
>> > > +PMU-XGENE
>> > > +APPLIED MICRO (APM) X-GENE SOC PMU
>> > One line section header please.
>> Is this strictly required?
>
> Yes.
>
> get_maintainer.pl doesn't like multiple line section headers.
>

Okay, I'll drop the first line and move it accordingly.

Regards,
Tai
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 4978dc1..3bd4143 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8593,6 +8593,14 @@  L:	linux-scsi@vger.kernel.org
 S:	Supported
 F:	drivers/scsi/pm8001/
 
+PMU-XGENE
+APPLIED MICRO (APM) X-GENE SOC PMU
+M:	Tai Nguyen <ttnguyen@apm.com>
+S:	Supported
+F:	drivers/perf/xgene_pmu.c
+F:	Documentation/perf/xgene-pmu.txt
+F:	Documentation/devicetree/bindings/perf/apm-xgene-pmu.txt
+
 POSIX CLOCKS and TIMERS
 M:	Thomas Gleixner <tglx@linutronix.de>
 L:	linux-kernel@vger.kernel.org