On Thu, 2009-06-18 at 14:14 +0200, Roel Kluin wrote: > - dc390_laststatus |= bval << 24; > + dc390_laststatus |= (unsigned)bval << 24; bval is already u8, thus unsigned, so the cast is a nop. James -- 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