diff mbox

i8042: Add noloop quirk for Asus X750LN

Message ID 1413032946-19748-1-git-send-email-hdegoede@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hans de Goede Oct. 11, 2014, 1:09 p.m. UTC
Without this the aux port does not get detected, and consequently the touchpad
will not work.

https://bugzilla.redhat.com/show_bug.cgi?id=1110011

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/input/serio/i8042-x86ia64io.h | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Dmitry Torokhov Oct. 11, 2014, 6:35 p.m. UTC | #1
On Sat, Oct 11, 2014 at 03:09:06PM +0200, Hans de Goede wrote:
> Without this the aux port does not get detected, and consequently the touchpad
> will not work.
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1110011
> 
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

Applied, thank you.

> ---
>  drivers/input/serio/i8042-x86ia64io.h | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
> index 40b7d6c..4a6d05d 100644
> --- a/drivers/input/serio/i8042-x86ia64io.h
> +++ b/drivers/input/serio/i8042-x86ia64io.h
> @@ -101,6 +101,12 @@ static const struct dmi_system_id __initconst i8042_dmi_noloop_table[] = {
>  	},
>  	{
>  		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "X750LN"),
> +		},
> +	},
> +	{
> +		.matches = {
>  			DMI_MATCH(DMI_SYS_VENDOR, "Compaq"),
>  			DMI_MATCH(DMI_PRODUCT_NAME , "ProLiant"),
>  			DMI_MATCH(DMI_PRODUCT_VERSION, "8500"),
> -- 
> 2.1.0
>
Dmitry Torokhov Oct. 30, 2014, 6:34 p.m. UTC | #2
On Sat, Oct 11, 2014 at 11:35:23AM -0700, Dmitry Torokhov wrote:
> On Sat, Oct 11, 2014 at 03:09:06PM +0200, Hans de Goede wrote:
> > Without this the aux port does not get detected, and consequently the touchpad
> > will not work.
> > 
> > https://bugzilla.redhat.com/show_bug.cgi?id=1110011
> > 
> > Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> 
> Applied, thank you.

Hans,

Do you still have dmidecode for this laptop around? Can I see it? I am
just wondering what part of AUX_LOOP was causing issues. Was it aux
delivery test or loop test itself?

Thanks.
Hans de Goede Oct. 31, 2014, 8:21 a.m. UTC | #3
Hi,

On 10/30/2014 07:34 PM, Dmitry Torokhov wrote:
> On Sat, Oct 11, 2014 at 11:35:23AM -0700, Dmitry Torokhov wrote:
>> On Sat, Oct 11, 2014 at 03:09:06PM +0200, Hans de Goede wrote:
>>> Without this the aux port does not get detected, and consequently the touchpad
>>> will not work.
>>>
>>> https://bugzilla.redhat.com/show_bug.cgi?id=1110011
>>>
>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>
>> Applied, thank you.
> 
> Hans,
> 
> Do you still have dmidecode for this laptop around?

No I took the necessary identification strings from the dmesg, which is here:

https://bugzilla.redhat.com/attachment.cgi?id=944389

I'll drop you a private mail with the users email address, then you
can get into contact with him directly, I expect him to be willing to
help debug this further.

> Can I see it? I am
> just wondering what part of AUX_LOOP was causing issues. Was it aux
> delivery test or loop test itself?

I was wondering myself, which is why the above dmesg is with:

"dyndbg=file drivers/input/serio/* +p"

But that does not seem to print anything extra ..., is the i8042 code
properly using one of the foo_dbg macro's ?

Regards,

Hans
--
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
Hans de Goede Oct. 31, 2014, 8:23 a.m. UTC | #4
Hi,

On 10/31/2014 09:21 AM, Hans de Goede wrote:
> Hi,
> 
> On 10/30/2014 07:34 PM, Dmitry Torokhov wrote:
>> On Sat, Oct 11, 2014 at 11:35:23AM -0700, Dmitry Torokhov wrote:
>>> On Sat, Oct 11, 2014 at 03:09:06PM +0200, Hans de Goede wrote:
>>>> Without this the aux port does not get detected, and consequently the touchpad
>>>> will not work.
>>>>
>>>> https://bugzilla.redhat.com/show_bug.cgi?id=1110011
>>>>
>>>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>>>
>>> Applied, thank you.
>>
>> Hans,
>>
>> Do you still have dmidecode for this laptop around?
> 
> No I took the necessary identification strings from the dmesg, which is here:
> 
> https://bugzilla.redhat.com/attachment.cgi?id=944389
> 
> I'll drop you a private mail with the users email address, then you
> can get into contact with him directly, I expect him to be willing to
> help debug this further.
> 
>> Can I see it? I am
>> just wondering what part of AUX_LOOP was causing issues. Was it aux
>> delivery test or loop test itself?
> 
> I was wondering myself, which is why the above dmesg is with:
> 
> "dyndbg=file drivers/input/serio/* +p"
> 
> But that does not seem to print anything extra ..., is the i8042 code
> properly using one of the foo_dbg macro's ?

Ah, wait I just realized that even though the user was asking for help
in Red Hat bugzilla he is actually a ubuntu user, I guess their kernels
just don't have dyndbg turned on.

Regards,

Hans
--
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
Dmitry Torokhov Oct. 31, 2014, 4:46 p.m. UTC | #5
On Fri, Oct 31, 2014 at 09:21:53AM +0100, Hans de Goede wrote:
> Hi,
> 
> On 10/30/2014 07:34 PM, Dmitry Torokhov wrote:
> > On Sat, Oct 11, 2014 at 11:35:23AM -0700, Dmitry Torokhov wrote:
> >> On Sat, Oct 11, 2014 at 03:09:06PM +0200, Hans de Goede wrote:
> >>> Without this the aux port does not get detected, and consequently the touchpad
> >>> will not work.
> >>>
> >>> https://bugzilla.redhat.com/show_bug.cgi?id=1110011
> >>>
> >>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> >>
> >> Applied, thank you.
> > 
> > Hans,
> > 
> > Do you still have dmidecode for this laptop around?
> 
> No I took the necessary identification strings from the dmesg, which is here:
> 
> https://bugzilla.redhat.com/attachment.cgi?id=944389
> 
> I'll drop you a private mail with the users email address, then you
> can get into contact with him directly, I expect him to be willing to
> help debug this further.

Thanks!

> 
> > Can I see it? I am
> > just wondering what part of AUX_LOOP was causing issues. Was it aux
> > delivery test or loop test itself?
> 
> I was wondering myself, which is why the above dmesg is with:
> 
> "dyndbg=file drivers/input/serio/* +p"
> 
> But that does not seem to print anything extra ..., is the i8042 code
> properly using one of the foo_dbg macro's ?
>

The debug facilities in i8042 were done way before dynamic debug so if
you want to see the data stream you need i8042.debug. That works on all
kernels regardless of config.
 
Thanks.
diff mbox

Patch

diff --git a/drivers/input/serio/i8042-x86ia64io.h b/drivers/input/serio/i8042-x86ia64io.h
index 40b7d6c..4a6d05d 100644
--- a/drivers/input/serio/i8042-x86ia64io.h
+++ b/drivers/input/serio/i8042-x86ia64io.h
@@ -101,6 +101,12 @@  static const struct dmi_system_id __initconst i8042_dmi_noloop_table[] = {
 	},
 	{
 		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
+			DMI_MATCH(DMI_PRODUCT_NAME, "X750LN"),
+		},
+	},
+	{
+		.matches = {
 			DMI_MATCH(DMI_SYS_VENDOR, "Compaq"),
 			DMI_MATCH(DMI_PRODUCT_NAME , "ProLiant"),
 			DMI_MATCH(DMI_PRODUCT_VERSION, "8500"),