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 li...@rasmusvillemoes.dk --- drivers/scsi/aha1542.h | 1 +

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 li...@rasmusvillemoes.dk Looks reasonable to me, can

[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 li...@rasmusvillemoes.dk --- drivers/scsi/aha1542.h | 1 + drivers/scsi/aha1740.h | 1 + drivers/scsi/gvp11.h | 1 +

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

2014-08-22 Thread Rasmus Villemoes
Rasmus Villemoes li...@rasmusvillemoes.dk 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