On Thu, Feb 10, 2011 at 02:00:58PM +0200, Felipe Balbi wrote: > checkpatch.pl prefers us to use __packed instead > of the more lengthy __attribute__ ((packed)), so > a simple search and replace fixed all occurrences > on all USB headers. > > This patch was conceived with the following: > > $ for i in $(git grep -e "__attribute__.((packed))" \ > include/linux/usb/ | cut -d: -f1 | uniq); \ > do sed -i 's/__attribute__ ((packed))/__packed/g' \ > $i; done > > It fixes checkpatch.pl warnings such as: > > WARNING: __packed is preferred over __attribute__((packed)) > #838: FILE: linux/usb/ch9.h:838: > +} __attribute__((packed)); > > Signed-off-by: Felipe Balbi <balbi@xxxxxx> Normally I'm all for checkpatch cleanups, but this is ridiculous. Why is __packed better than "__attribute__ ((packed))"? This seems way unnecessary to me, so I'll just ignore it for now if you don't mind. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
