On Wed, 2008-04-30 at 17:29 -0500, James Bottomley wrote: > On Wed, 2008-04-30 at 15:03 -0700, Harvey Harrison wrote: > > drivers/scsi/FlashPoint.c:906:9: warning: Using plain integer as NULL pointer > > drivers/scsi/FlashPoint.c:907:53: warning: Using plain integer as NULL pointer > > drivers/scsi/FlashPoint.c:922:1: warning: Using plain integer as NULL pointer > The flashpoint initialiser changes look wrong: {0} is a conventional > notation for zero initialise this structure. Sparse should probably be > taught to recognise that. Actually, looking closer, should this be initialized at all? Can we just remove the initialization here and let them end up in .bss? Harvey -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html