Re: [PATCH] MAINTAINERS: rename EEH from "enhanced" to "extended" error handling

[Russell Currey <ruscur@russell.cc>]

On Wed, 2016-02-17 at 10:49 +1100, Russell Currey wrote:
> On Wed, 2016-02-17 at 09:58 +1100, Gavin Shan wrote:
> > 
> > On Wed, Feb 17, 2016 at 09:49:08AM +1100, Gavin Shan wrote:
> > > 
> > > 
> > > On Tue, Feb 16, 2016 at 07:52:00PM +1100, Russell Currey wrote:
> > > > 
> > > > 
> > > > Incredibly, IBM online documentation for EEH uses "extended error
> > > > handling"
> > > > and "enhanced error handling" to refer to the same thing, in
> > > > different
> > > > places.  In other parts of the kernel, namely the EEH documentation
> > > > (found
> > > > in Documentation/powerpc/eeh-pci-error-recovery.txt), it's referred
> > > > to as
> > > > "extended", and in my opinion "extended" makes more sense for what
> > > > EEH
> > > > does.
> > > > 
> > > > The only place "enhanced error handling" shows up in the kernel is
> > > > in
> > > > MAINTAINERS, so fix it.
> > > > 
> > > Russell, Thanks for fixing it up. Since you're at it, Please replace
> > > the
> > > maintainer to yourself. Also, the components mentioned in this file
> > > are
> > > listed in alphabetic order according to their names. As the name of
> > > EEH
> > > is changed, it would be put in front of "Extended Verification Module
> > > (EVM)".
> > > 
> > I agree with Michael as discussed in another thread: we're going to use
> > "enhanced", not "extended" though some chip datasheet (P7IOC) talks
> > about
> > "extended error handling".
> > 
> > Russell, please change the maintainer to you and repost.
> Sure.
> 
> While I'm changing MAINTAINERS, what are your thoughts on renaming the
> EEH
> entry from "ENHANCED ERROR HANDLING (EEH)" to something like "PCI
> EXTENDED
> ERROR HANDLING (EEH)" or "EXTENDED ERROR HANDLING (EEH) FOR PCI" - so
> it's
> clear it refers to PCI and not something more generic?

and of course, s/EXTENDED/ENHANCED...-ENOCOFFEE
> > 
> > 
> > > 
> > > 
> > > > 
> > > > 
> > > > Signed-off-by: Russell Currey <ruscur@russell.cc>
> > > With above issues fixed:
> > > 
> > > Acked-by: Gavin Shan <gwshan@linux.vnet.ibm.com>
> > > 
> > Thanks,
> > Gavin
> > 
> > > 
> > > 
> > > > 
> > > > 
> > > > ---
> > > > I don't know what kind of things parse MAINTAINERS, and if there's
> > > > a
> > > > chance
> > > > this will break them.  Also, the powerpc tree *is* the right place
> > > > to
> > > > send
> > > > this, right?
> > > > ---
> > > > MAINTAINERS | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > 
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index 28eb61b..e5d9134 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -4222,7 +4222,7 @@ M:	Maxim Levitsky <maximlevitsky@gmail
> > > > .c
> > > > om>
> > > > S:	Maintained
> > > > F:	drivers/media/rc/ene_ir.*
> > > > 
> > > > -ENHANCED ERROR HANDLING (EEH)
> > > > +EXTENDED ERROR HANDLING (EEH)
> > > > M:	Gavin Shan <shangw@linux.vnet.ibm.com>
> > > > L:	linuxppc-dev@lists.ozlabs.org
> > > > S:	Supported
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev