Re: [PATCH] MAINTAINERS: Fix multiple line section header for XGENE

From: Joe Perches
Date: Fri Jul 21 2017 - 12:57:27 EST


On Fri, 2017-07-21 at 09:48 -0700, Loc Ho wrote:
> On Fri, Jul 21, 2017 at 9:38 AM, Joe Perches <joe@xxxxxxxxxxx> wrote:
> > Section headers should be single line.
> >
> > Fix it.
>
> Can we drop the "Fix it."

<shrug> I use that frequently.

> > diff --git a/MAINTAINERS b/MAINTAINERS
[]
> > @@ -5029,8 +5029,7 @@ L: linux-edac@xxxxxxxxxxxxxxx
> > S: Maintained
> > F: drivers/edac/skx_edac.c
> >
> > -EDAC-XGENE
> > -APPLIED MICRO (APM) X-GENE SOC EDAC
> > +EDAC-XGENE-APPLIED MICRO (APM) X-GENE SOC EDAC
>
> To be consistent with other, how about just:
>
> EDAC-XGENE (drop the Applied micro ... line)
>
> > M: Loc Ho <lho@xxxxxxx>
> > S: Supported
> > F: drivers/edac/xgene_edac.c

I don't care one way or another.