On Mon, 17 May 2021 13:55:50 +0100 Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx> > > Resent because for some reason this didn't reach the mailing list > (I'm not sure it reached anywhere at all!) > > A wrong use of one of these in > https://lore.kernel.org/linux-iio/20210514135927.2926482-1-arnd@xxxxxxxxxx/ > included a reference from Nathan to a patch discouraging the use of > these strings in general: > https://lore.kernel.org/lkml/CAHk-=wgoxnmsj8GEVFJSvTwdnWm8wVJthefNk2n6+4TC=20e0Q@xxxxxxxxxxxxxx/ > > I did a quick grep and established we only have a few instances of these in > IIO anyway, so in the interests of avoiding those existing cases getting > cut and paste into new drivers, let's just clear them out now. Series applied to the togreg branch of iio.git and pushed out as testing to let 0-day poke at it before I push it out for linux-next. Thanks, Jonathan > > Cc: Nathan Chancellor <nathan@xxxxxxxxxx> > Jonathan Cameron (4): > iio: imu: inv_mpu6050: Drop use of %hhx format string. > iio: light: si1133: Drop remaining uses of %hhx format string. > iio: light: si1145: Drop use of %hhx format specifier. > iio: chemical: sgp30: Drop use of %hx in format string. > > drivers/iio/chemical/sgp30.c | 2 +- > drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 5 ++--- > drivers/iio/light/si1133.c | 14 +++++++------- > drivers/iio/light/si1145.c | 10 +++++----- > 4 files changed, 15 insertions(+), 16 deletions(-) >
