* [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
* Re: [PATCH v2] Input: elan_i2c - always output the device information
[not found] ` <CAAZ5spAq7_yEZe9eYnSPenzLWw4sZgkmJZRmXkoe72mJKeKRWg@mail.gmail.com>
@ 2016-10-13 12:16 ` Benjamin Tissoires
0 siblings, 0 replies; 3+ messages in thread
From: Benjamin Tissoires @ 2016-10-13 12:16 UTC (permalink / raw)
To: Michel Hermier; +Cc: linux-input, linux-kernel, Dmitry Torokhov, KT Liao
On Oct 13 2016 or thereabouts, Michel Hermier wrote:
> Le 13 oct. 2016 13:49, "Benjamin Tissoires" <benjamin.tissoires@redhat.com>
> a écrit :
> >
> > 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"
>
> Looks like there is an indent issue here.
I can't seem to see it. Could you explain a little bit more?
Cheers,
Benjamin
>
> > " 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
> >
> > --
> > 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
^ permalink raw reply [flat|nested] 3+ messages in thread
* 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).