iio: imu: inv_mpu6050: Drop use of %hhx format string.
authorJonathan Cameron <Jonathan.Cameron@huawei.com>
Thu, 3 Jun 2021 18:06:11 +0000 (19:06 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Wed, 9 Jun 2021 17:31:03 +0000 (18:31 +0100)
Since:
commit cbacb5ab0aa0 ("docs: printk-formats: Stop encouraging use of
unnecessary %h[xudi] and %hh[xudi]")
use of these format strings has been discouraged.

Use the 0x02x form as the length specifier when used with # includes
the 0x prefix and is very unlikely to be what was intended by the author.

Part of a series removing all uses from IIO in the interestings of
avoiding providing bad examples for people to copy.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Nathan Chancellor <nathan@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
Link: https://lore.kernel.org/r/20210603180612.3635250-4-jic23@kernel.org
drivers/iio/imu/inv_mpu6050/inv_mpu_core.c

index 64704b5..b7254d9 100644 (file)
@@ -1314,8 +1314,7 @@ static int inv_check_and_setup_chip(struct inv_mpu6050_state *st)
                for (i = 0; i < INV_NUM_PARTS; ++i) {
                        if (regval == hw_info[i].whoami) {
                                dev_warn(regmap_get_device(st->map),
-                                       "whoami mismatch got %#02x (%s)"
-                                       "expected %#02hhx (%s)\n",
+                                       "whoami mismatch got 0x%02x (%s) expected 0x%02x (%s)\n",
                                        regval, hw_info[i].name,
                                        st->hw->whoami, st->hw->name);
                                break;
@@ -1323,7 +1322,7 @@ static int inv_check_and_setup_chip(struct inv_mpu6050_state *st)
                }
                if (i >= INV_NUM_PARTS) {
                        dev_err(regmap_get_device(st->map),
-                               "invalid whoami %#02x expected %#02hhx (%s)\n",
+                               "invalid whoami 0x%02x expected 0x%02x (%s)\n",
                                regval, st->hw->whoami, st->hw->name);
                        return -ENODEV;
                }