* [PATCH v2] Input: elan_i2c - always output the device information
@ 2016-10-13 11:23 Benjamin Tissoires
[not found] ` <CAAZ5spB5Y_6=xKTz7hbgGjS=yYEr0hA3KoY19WCy1fJ6BbY-VQ@mail.gmail.com>
2016-10-26 23:27 ` Dmitry Torokhov
0 siblings, 2 replies; 3+ messages in thread
From: Benjamin Tissoires @ 2016-10-13 11:23 UTC (permalink / raw)
To: Dmitry Torokhov, KT Liao; +Cc: linux-kernel, linux-input
it's always easier to retrieve these information in bug reports when
it is always printed in the dmesg.
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
---
changes in v2:
- split the version information message and the touchpad characteristics
- keep sending the versions as info (on one line)
- send the characteristics on debug only
Hi Dmitry,
This is a respin of one of the patches I sent for elan_i2c. Given that you
applied the fixed one for SMBus and that I don't currently have a way to
check from I2C if the trackstick is going to be used, this one is the only
one left (also given SMBus Host Notify might meld into client->irq).
So, well, yes, there it comes, a single patch which IMO is useful
when we receive bug reports.
Cheers,
Benjamin
---
drivers/input/mouse/elan_i2c_core.c | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/input/mouse/elan_i2c_core.c b/drivers/input/mouse/elan_i2c_core.c
index d15b338..dc7c94d 100644
--- a/drivers/input/mouse/elan_i2c_core.c
+++ b/drivers/input/mouse/elan_i2c_core.c
@@ -1093,19 +1093,18 @@ static int elan_probe(struct i2c_client *client,
if (error)
return error;
+ dev_info(&client->dev,
+ "Elan Touchpad: Module ID: 0x%04x, Firmware: 0x%04x, Sample: 0x%04x, IAP: 0x%04x\n",
+ data->product_id,
+ data->fw_version,
+ data->sm_version,
+ data->iap_version);
+
dev_dbg(&client->dev,
- "Elan Touchpad Information:\n"
- " Module product ID: 0x%04x\n"
- " Firmware Version: 0x%04x\n"
- " Sample Version: 0x%04x\n"
- " IAP Version: 0x%04x\n"
+ "Elan Touchpad Extra Information:\n"
" Max ABS X,Y: %d,%d\n"
" Width X,Y: %d,%d\n"
" Resolution X,Y: %d,%d (dots/mm)\n",
- data->product_id,
- data->fw_version,
- data->sm_version,
- data->iap_version,
data->max_x, data->max_y,
data->width_x, data->width_y,
data->x_res, data->y_res);
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread[parent not found: <CAAZ5spB5Y_6=xKTz7hbgGjS=yYEr0hA3KoY19WCy1fJ6BbY-VQ@mail.gmail.com>]
* Re: [PATCH v2] Input: elan_i2c - always output the device information
2016-10-13 11:23 [PATCH v2] Input: elan_i2c - always output the device information Benjamin Tissoires
[not found] ` <CAAZ5spB5Y_6=xKTz7hbgGjS=yYEr0hA3KoY19WCy1fJ6BbY-VQ@mail.gmail.com>
@ 2016-10-26 23:27 ` Dmitry Torokhov
1 sibling, 0 replies; 3+ messages in thread
From: Dmitry Torokhov @ 2016-10-26 23:27 UTC (permalink / raw)
To: Benjamin Tissoires; +Cc: KT Liao, linux-kernel, linux-input
On Thu, Oct 13, 2016 at 01:23:30PM +0200, Benjamin Tissoires wrote:
> it's always easier to retrieve these information in bug reports when
> it is always printed in the dmesg.
>
> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Applied, thank you.
>
> ---
>
> changes in v2:
> - split the version information message and the touchpad characteristics
> - keep sending the versions as info (on one line)
> - send the characteristics on debug only
>
> Hi Dmitry,
>
> This is a respin of one of the patches I sent for elan_i2c. Given that you
> applied the fixed one for SMBus and that I don't currently have a way to
> check from I2C if the trackstick is going to be used, this one is the only
> one left (also given SMBus Host Notify might meld into client->irq).
>
> So, well, yes, there it comes, a single patch which IMO is useful
> when we receive bug reports.
>
> Cheers,
> Benjamin
>
> ---
> drivers/input/mouse/elan_i2c_core.c | 17 ++++++++---------
> 1 file changed, 8 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/input/mouse/elan_i2c_core.c b/drivers/input/mouse/elan_i2c_core.c
> index d15b338..dc7c94d 100644
> --- a/drivers/input/mouse/elan_i2c_core.c
> +++ b/drivers/input/mouse/elan_i2c_core.c
> @@ -1093,19 +1093,18 @@ static int elan_probe(struct i2c_client *client,
> if (error)
> return error;
>
> + dev_info(&client->dev,
> + "Elan Touchpad: Module ID: 0x%04x, Firmware: 0x%04x, Sample: 0x%04x, IAP: 0x%04x\n",
> + data->product_id,
> + data->fw_version,
> + data->sm_version,
> + data->iap_version);
> +
> dev_dbg(&client->dev,
> - "Elan Touchpad Information:\n"
> - " Module product ID: 0x%04x\n"
> - " Firmware Version: 0x%04x\n"
> - " Sample Version: 0x%04x\n"
> - " IAP Version: 0x%04x\n"
> + "Elan Touchpad Extra Information:\n"
> " Max ABS X,Y: %d,%d\n"
> " Width X,Y: %d,%d\n"
> " Resolution X,Y: %d,%d (dots/mm)\n",
> - data->product_id,
> - data->fw_version,
> - data->sm_version,
> - data->iap_version,
> data->max_x, data->max_y,
> data->width_x, data->width_y,
> data->x_res, data->y_res);
> --
> 2.7.4
>
--
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-10-26 23:27 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-13 11:23 [PATCH v2] Input: elan_i2c - always output the device information Benjamin Tissoires
[not found] ` <CAAZ5spB5Y_6=xKTz7hbgGjS=yYEr0hA3KoY19WCy1fJ6BbY-VQ@mail.gmail.com>
[not found] ` <CAAZ5spAq7_yEZe9eYnSPenzLWw4sZgkmJZRmXkoe72mJKeKRWg@mail.gmail.com>
2016-10-13 12:16 ` Benjamin Tissoires
2016-10-26 23:27 ` Dmitry Torokhov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).