Toralf, thanks for reporting these inconsistencies in our drivers. Patches are welcome, if you can spend the time. Otherwise we will take care about them. I suggest to keep the "unsigned int" format in lcs.c and ctcm_sysfs.c, but move to a "u8" format in qeth_l3_main.c. Regards, Ursula Braun > From: Toralf Förster <toralf.foerster@xxxxxx> > To: linux-s390@xxxxxxxxxxxxxxx, > Date: 19/05/2014 17:42 > Subject: format string mismatch > Sent by: linux-s390-owner@xxxxxxxxxxxxxxx > > > FWIW a current cppcheck blamed these issues : > > [drivers/s390/net/ctcm_sysfs.c:46]: (warning) %u in format string (no. > 1) requires 'unsigned int *' but the argument type is 'int *'. > [drivers/s390/net/ctcm_sysfs.c:151]: (warning) %u in format string > (no. 1) requires 'unsigned int *' but the argument type is 'int *'. > [drivers/s390/net/lcs.c:1953]: (warning) %u in format string (no. 1) > requires 'unsigned int *' but the argument type is 'int *'. > [drivers/s390/net/lcs.c:2007]: (warning) %u in format string (no. 1) > requires 'unsigned int *' but the argument type is 'int *'. > [drivers/s390/net/qeth_l3_main.c:69]: (warning) %u in format string > (no. 1) requires 'unsigned int *' but the argument type is 'int *'. > [drivers/s390/net/qeth_l3_main.c:69]: (warning) %u in format string > (no. 2) requires 'unsigned int *' but the argument type is 'int *'. > [drivers/s390/net/qeth_l3_main.c:69]: (warning) %u in format string > (no. 3) requires 'unsigned int *' but the argument type is 'int *'. > [drivers/s390/net/qeth_l3_main.c:69]: (warning) %u in format string > (no. 4) requires 'unsigned int *' but the argument type is 'int *'. > > patches welcome ? > -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
