Re: [PATCH] drivers: scsi: #define missing include guards

2014-08-26 Thread Hannes Reinecke
On 08/22/2014 02:37 PM, Rasmus Villemoes wrote: The four files aha1542.h, aha1740.h, gvp11.h and mvme147.h under drivers/scsi/ contain two-thirds of an include guard, but do not elsewhere. Signed-off-by: Rasmus Villemoes --- drivers/scsi/aha1542.h | 1 + drivers/scsi/aha1740.h | 1 + driver

Re: [PATCH] drivers: scsi: #define missing include guards

2014-08-24 Thread Christoph Hellwig
On Fri, Aug 22, 2014 at 02:37:33PM +0200, Rasmus Villemoes wrote: > The four files aha1542.h, aha1740.h, gvp11.h and mvme147.h under > drivers/scsi/ contain two-thirds of an include guard, but do not > elsewhere. > > Signed-off-by: Rasmus Villemoes Looks reasonable to me, can I get a quick revie

Re: [PATCH] drivers: scsi: #define missing include guards

2014-08-22 Thread Rasmus Villemoes
Rasmus Villemoes writes: > The four files aha1542.h, aha1740.h, gvp11.h and mvme147.h under > drivers/scsi/ contain two-thirds of an include guard, but do not > elsewhere. > Argh, git commit ate a line because it happened to start with #. This was supposed to be something like "...but do not #de

[PATCH] drivers: scsi: #define missing include guards

2014-08-22 Thread Rasmus Villemoes
The four files aha1542.h, aha1740.h, gvp11.h and mvme147.h under drivers/scsi/ contain two-thirds of an include guard, but do not elsewhere. Signed-off-by: Rasmus Villemoes --- drivers/scsi/aha1542.h | 1 + drivers/scsi/aha1740.h | 1 + drivers/scsi/gvp11.h | 1 + drivers/scsi/mvme147.h | 1 +