On Mon, Sep 12, 2011 at 08:59:01AM -0400, Chris Peterson wrote: > Some function prototypes declare an 'int register' parameter. gcc -W warns > "‘register’ is not at beginning of declaration" because the compiler thinks > the parameter has been marked as a 'register' variable but the author > just wanted to name the parameter "register" (which is a reserved keyword). > > Signed-off-by: Chris Peterson <cpeterso@xxxxxxxxxxxx> This needs to be broken up into separate patches by file. The problem is that you have four maintainers and they each can only take the chunk that affects their subsystem. > --- a/drivers/isdn/hardware/eicon/platform.h > +++ b/drivers/isdn/hardware/eicon/platform.h > @@ -206,7 +206,7 @@ void PCIread (byte bus, byte func, int offset, void* data, int length, void* pci > /* > ** I/O Port utilities > */ > -int diva_os_register_io_port (void *adapter, int register, unsigned long port, > +int diva_os_register_io_port (void *adapter, int reg, unsigned long port, > unsigned long length, const char* name, int id); Instead of using "reg" for this one, it probably would be better to use "on". That's how it's named in the .c file. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
