qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] net: Add SunGEM device emulation as found on Ap


From: Benjamin Herrenschmidt
Subject: Re: [Qemu-devel] [PATCH] net: Add SunGEM device emulation as found on Apple UniNorth
Date: Wed, 21 Sep 2016 20:40:19 +1000

On Wed, 2016-09-21 at 11:16 +0100, Alex Bennée wrote:
> 
> > > 
> > > total: 428 errors, 73 warnings, 1950 lines checked
> > > 
> > > Your patch has style problems, please review.If any of these
> > > errors
> > > are false positives report them to the maintainer, see
> > > CHECKPATCH in MAINTAINERS.
> 
> It seems very unlikely all of the reported errors here can be
> justified
> along the lines of internal consistency, especially for adding a new
> file.

It's actually an old thread ;-) I've already fixed some of these, just
haven't had a chance to finish and submit a new version.

However *most* of the above are intentional and I won't fix, it's about
the .h file describing the registers that is essentially lifted from
Linux and that I want to keep largely identical for obvious reasons
(thus including the use of tabs).

Cheers,
Ben.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]