On Wed, 2008-12-03 at 18:24 +0200, Vorobiev Dmitri wrote: > > This patch fixes the following compilation warning: > > > > CC [M] drivers/scsi/sgiwd93.o > > drivers/scsi/sgiwd93.c:314: warning: initialization from incompatible > > pointer type > > > > Hello James, > > Any news about this one? I think this patch should go via linux-scsi, > unless you would be insisting on pushing it via linux-mips, in which case > I'll politely bug Ralf about it. :) Looks OK for the local change. Globally, having driver->remove and platform_driver->remove return int instead of void looks wrong. Particularly when the only use cases are in drivers/base/ and they all ignore the return code. Greg and Kay ... shouldn't we simply redefine the return values for the remove methods in these structures to return void (and thus match the use case)? James
