diff mbox

atmel_mxt: simplify mxt_dump_message

Message ID 1348657555-8226-1-git-send-email-andriy.shevchenko@linux.intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Andy Shevchenko Sept. 26, 2012, 11:05 a.m. UTC
This patch applies %*ph to print small buffer and marks function as inline.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 drivers/input/touchscreen/atmel_mxt_ts.c |   10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

Comments

Daniel Kurtz Sept. 26, 2012, 2:18 p.m. UTC | #1
On Wed, Sep 26, 2012 at 7:05 PM, Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> This patch applies %*ph to print small buffer and marks function as inline.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

Reviewed-by: Daniel Kurtz <djkurtz@chromium.org>

>
> ---
>  drivers/input/touchscreen/atmel_mxt_ts.c |   10 ++++------
>  1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c
> index e92615d..211bfb4 100644
> --- a/drivers/input/touchscreen/atmel_mxt_ts.c
> +++ b/drivers/input/touchscreen/atmel_mxt_ts.c
> @@ -317,13 +317,11 @@ static bool mxt_object_writable(unsigned int type)
>         }
>  }
>
> -static void mxt_dump_message(struct device *dev,
> -                            struct mxt_message *message)
> +static inline void mxt_dump_message(struct device *dev,
> +                                   struct mxt_message *message)
>  {
> -       dev_dbg(dev, "reportid: %u\tmessage: %02x %02x %02x %02x %02x %02x %02x\n",
> -               message->reportid, message->message[0], message->message[1],
> -               message->message[2], message->message[3], message->message[4],
> -               message->message[5], message->message[6]);
> +       dev_dbg(dev, "reportid: %u\tmessage: %*ph\n",
> +               message->reportid, 7, message->message);
>  }
>
>  static int mxt_check_bootloader(struct i2c_client *client,
> --
> 1.7.10.4
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-input" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c
index e92615d..211bfb4 100644
--- a/drivers/input/touchscreen/atmel_mxt_ts.c
+++ b/drivers/input/touchscreen/atmel_mxt_ts.c
@@ -317,13 +317,11 @@  static bool mxt_object_writable(unsigned int type)
 	}
 }
 
-static void mxt_dump_message(struct device *dev,
-			     struct mxt_message *message)
+static inline void mxt_dump_message(struct device *dev,
+				    struct mxt_message *message)
 {
-	dev_dbg(dev, "reportid: %u\tmessage: %02x %02x %02x %02x %02x %02x %02x\n",
-		message->reportid, message->message[0], message->message[1],
-		message->message[2], message->message[3], message->message[4],
-		message->message[5], message->message[6]);
+	dev_dbg(dev, "reportid: %u\tmessage: %*ph\n",
+		message->reportid, 7, message->message);
 }
 
 static int mxt_check_bootloader(struct i2c_client *client,