|
From: | Richard Henderson |
Subject: | Re: [Qemu-devel] [qemu-s390x] [RFC 06/15] s390-bios: Clean up cio.h |
Date: | Tue, 17 Jul 2018 11:22:50 -0700 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 07/17/2018 11:15 AM, Thomas Huth wrote: >> - __u32 isc:3; >> - __u32 ena:1; >> - __u32 mme:2; >> - __u32 mp:1; >> - __u32 csense:1; >> - __u32 mbfc:1; >> + __u32 isc : 3; >> + __u32 ena : 1; >> + __u32 mme : 2; >> + __u32 mp : 1; >> + __u32 csense : 1; >> + __u32 mbfc : 1; > > checkpatch.pl does not like these spaces, so I think it would be better > to not make these changes (otherwise we'll have to deal with the output > of checkpatch each time again we want to change something here). AFAICT, this is a checkpatch bug confusing switch case labels. r~
[Prev in Thread] | Current Thread | [Next in Thread] |