diff options
| author |   Viresh Kumar <viresh.kumar@linaro.org> | 2015-12-04 21:30:09 +0530 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@google.com> | 2015-12-04 16:23:36 -0800 |
| commit | 2f3db927cdf7627aa5359ff46c80ab72f7971980 (patch) | |
| tree | 62cb682c89f4981053412bc149b9fef27750f50a /drivers/staging/greybus/loopback.c | |
| parent | greybus: svc: Use -EIO instead of -EINVAL for unipro errors (diff) | |
| download | linux-dev-2f3db927cdf7627aa5359ff46c80ab72f7971980.tar.xz linux-dev-2f3db927cdf7627aa5359ff46c80ab72f7971980.zip | |
greybus: don't use %h and %hh for printing short and char variables
Because the width of our fields is already known, we can use %0Nx (for
hex) to print N bytes and %u (for unsigned decimal), instead of using %h
and %hh, which isn't that readable.
This patch makes following changes:
- s/%hx/%04x
- s/%04hx/%04x
- s/%hhx/%02x
- s/%02hhx/%02x
- s/%hhu/%u
- s/%hu/%u
- s/%x/%02x for u8 value (only at a single place)
Suggested-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/loopback.c')
| -rw-r--r-- | drivers/staging/greybus/loopback.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greybus/loopback.c index 8524ce128738..689ebfdb456e 100644 --- a/drivers/staging/greybus/loopback.c +++ b/drivers/staging/greybus/loopback.c @@ -487,7 +487,7 @@ static int gb_loopback_request_recv(u8 type, struct gb_operation *operation) return 0; default: - dev_err(dev, "unsupported request: %hhu\n", type); + dev_err(dev, "unsupported request: %u\n", type); return -EINVAL; } } |