Input: elan_i2c - fix wrong %p extension

Villemoes · dtor · commit 98a4a59ee31a · 2015-02-06T15:32:18.000-08:00
There's no %px extension. From the context I think the intention was to
dump the five bytes which were not as expected, and for that one should use
%ph.

Signed-off-by: Rasmus Villemoes &lt;linux@rasmusvillemoes.dk&gt;
Signed-off-by: Dmitry Torokhov &lt;dmitry.torokhov@gmail.com&gt;
diff --git a/drivers/input/mouse/elan_i2c_smbus.c b/drivers/input/mouse/elan_i2c_smbus.c
@@ -70,7 +70,7 @@ static int elan_smbus_initialize(struct i2c_client *client)
 
 	/* compare hello packet */
 	if (memcmp(values, check, ETP_SMBUS_HELLOPACKET_LEN)) {
-		dev_err(&client->dev, "hello packet fail [%*px]\n",
+		dev_err(&client->dev, "hello packet fail [%*ph]\n",
 			ETP_SMBUS_HELLOPACKET_LEN, values);
 		return -ENXIO;
 	}

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ static int elan_smbus_initialize(struct i2c_client *client)`
`70`	`70`
`71`	`71`	`/* compare hello packet */`
`72`	`72`	`if (memcmp(values, check, ETP_SMBUS_HELLOPACKET_LEN)) {`
`73`		`- dev_err(&client->dev, "hello packet fail [%*px]\n",`
	`73`	`+ dev_err(&client->dev, "hello packet fail [%*ph]\n",`
`74`	`74`	`ETP_SMBUS_HELLOPACKET_LEN, values);`
`75`	`75`	`return -ENXIO;`
`76`	`76`	`}`