Re: [Qemu-devel] [PATCH v5 04/46] checkpatch: Recognize IEC binary prefix definitions

2018-06-29 Thread Joe Perches
On Fri, 2018-06-29 at 15:41 -0500, Eric Blake wrote:
> On 06/25/2018 07:41 AM, Philippe Mathieu-Daudé wrote:
> > Do not match the IEC binary prefix as camelcase typedefs.
> > 
> > This fixes:
> > 
> >  ERROR: "foo * bar" should be "foo *bar"
> >  #310: FILE: hw/ppc/ppc440_uc.c:564:
> >  +size = 8 * MiB * sh;
> >  total: 1 errors, 0 warnings, 433 lines checked

Does the linux kernel use these?
I don't believe so.

> > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
[]
> > @@ -242,6 +242,7 @@ our $UTF8   = qr{
> >   # There are still some false positives, but this catches most
> >   # common cases.
> >   our $typeTypedefs = qr{(?x:
> > +(?![KMGTPE]iB)  # IEC binary prefix (do not 
> > match)



Re: [Qemu-devel] [PATCH v5 04/46] checkpatch: Recognize IEC binary prefix definitions

2018-07-02 Thread Joe Perches
On Mon, 2018-07-02 at 14:37 +0200, Paolo Bonzini wrote:
> On 29/06/2018 23:46, Joe Perches wrote:
> > > >  ERROR: "foo * bar" should be "foo *bar"
> > > >  #310: FILE: hw/ppc/ppc440_uc.c:564:
> > > >  +size = 8 * MiB * sh;
> > > >  total: 1 errors, 0 warnings, 433 lines checked
> > 
> > Does the linux kernel use these?
> > I don't believe so.
> > 
> No, I'm not sure why you were CCed. :)

Me neither but I am the kernel MAINTAINERS file as
checkpatch maintainer.

My guess is some MAINTAINERS file has me in it or qemu
uses git history for unmaintained files and qemu pulls
in the kernel checkpatch changes.