On Wed, Nov 07, 2007 at 10:58:12AM -0500, Salyzyn, Mark wrote: > Good point, thanks. The intent of the management applications > utilization of this AIF report is to observe the LSB of the value of > integer value in BlinkLED. The actions of the cpu_to_le32 actually > breaks this and reports the wrong content in swapped architectures. > > This attached follow-up patch is against current scsi-misc-2.6 *after* > the application of the 'don't assign cpu_to_le32(constant) to u8' patch > submitted by Stephen Rothwell which has already been taken by the -mm > tree. Inspection of other areas of the aacraid driver came up blank for > similar style bugs. Did anyone run the driver through sparse to see if we have more issues like this? - 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