On Fri, 2008-11-07 at 09:58 -0800, Reinette Chatre wrote: > From: Kolekar, Abhijeet <abhijeet.kolekar@xxxxxxxxx> > > Patch fixes sparse warnings. No need to convert u8 variable > to 32. > > Signed-off-by: Abhijeet Kolekar <abhijeet.kolekar@xxxxxxxxx> > Signed-off-by: Reinette Chatre <reinette.chatre@xxxxxxxxx> > --- It's customary to put the actual sparse warnings being fixed in the commit message. In this case I assume it was this: drivers/net/wireless/iwlwifi/iwl3945-base.c:4838:18: warning: invalid assignment: |= drivers/net/wireless/iwlwifi/iwl3945-base.c:4838:18: left side has type unsigned char drivers/net/wireless/iwlwifi/iwl3945-base.c:4838:18: right side has type restricted __le32 Anyways, looks fine. Cheers, Harvey -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html