Re: Lenovo X220 Touchpad + Trackpoint BIOS issue workaround ?

[Manoj <manoj.iyer@canonical.com>]

I think I got excited too soon :) rmmod and insmod of the module an 
couple of time and it still fails in psmouse_probe() even with the 
psmouse_reset() in probe.

Manoj Iyer
Ubuntu/Canonical
Hardware Enablement

On 04/05/2011 05:19 PM, Manoj wrote:
> I just found that calling psmouse_reset() in the psmouse_probe()
> function causes the probe to work, enabling the trackpoint.. is this a
> correct fix? Or just a side effect due to some firmware bug?
>
> static int psmouse_probe(struct psmouse *psmouse)
> {
> struct ps2dev *ps2dev = &psmouse->ps2dev;
> unsigned char param[2];
>
> /*
> * First, we check if it's a mouse. It should send 0x00 or 0x03
> * in case of an IntelliMouse in 4-byte mode or 0x04 for IM Explorer.
> * Sunrex K8561 IR Keyboard/Mouse reports 0xff on second and subsequent
> * ID queries, probably due to a firmware bug.
> */
>
> param[0] = 0xa5;
> psmouse_reset(psmouse); <<<< === called psmouse_reset() here
> if (ps2_command(ps2dev, param, PSMOUSE_CMD_GETID))
> return -1;
>
> if (param[0] != 0x00 && param[0] != 0x03 &&
> param[0] != 0x04 && param[0] != 0xff)
> return -1;
>
> /*
> * Then we reset and disable the mouse so that it doesn't generate events.
> */
>
> if (ps2_command(ps2dev, NULL, PSMOUSE_CMD_RESET_DIS))
> printk(KERN_WARNING "psmouse.c: Failed to reset mouse on %s\n",
> ps2dev->serio->phys);
>
> return 0;
> }
>
>
> Manoj Iyer
> Ubuntu/Canonical
> Hardware Enablement
>
> On 04/05/2011 05:02 PM, Manoj wrote:
>> Dmitry,
>>
>> I turned on i8042.debug and it produced a lot of noise, and I couldint
>> make out what it was trying to tell me. So I put some debug prints in
>> synaptic.c and psmouse-base.c. In the case where both touchpad and
>> trackpoint are enabled in the BIOS I get the following failure in
>> psmouse_probe(). I have attached the psmouse-base.c and synaptic.c for
>> you reference as to where the prints are coming from. Dont know why
>> psmouse_probe() PSMOUSE_CMD_GETID would fail on the pass-through device.
>>
>> [ 726.431405] DEBUG: psmouse_probe:931 probing for isa0060/serio1
>> [ 726.436949] DEBUG: psmouse_probe:937 got PSMOUSE_CMD_GETID 00
>> [ 727.349407] Synaptics Touchpad, model: 1, fw: 8.0, id: 0x1e2b1, caps:
>> 0xd001a3/0x940300/0x120c00
>> [ 727.349415] Synaptics: Clickpad mode enabled
>> [ 727.349423] serio: Synaptics pass-through port at isa0060/serio1/input0
>> [ 727.349438] DEBUG: Synaptics: synaptics_init:828: psmouse->rate = 0x64
>> [ 727.349444] DEBUG: psmouse_switch_protocol:1266 selected_proto->type
>> = 7
>> [ 727.407740] input: SynPS/2 Synaptics TouchPad as
>> /devices/platform/i8042/serio1/input/input9
>> [ 727.419607] DEBUG: psmouse_connect:1318 pass-though device detected
>> [ 727.425287] DEBUG: psmouse_deactivate:1028 deacivating isa0060/serio1
>> [ 727.425307] DEBUG: psmouse_probe:931 probing for synaptics-pt/serio0
>> [ 727.672729] DEBUG: psmouse_probe:934 failed to get PSMOUSE_CMD_GETID a5
>> [ 727.672738] DEBUG: psmouse_connect:1342 failed in psmouse_probe()
>> [ 727.672747] DEBUG: psmouse_connect:1376 psmouse_activate() called on
>> parent
>>
>>
>> Manoj Iyer
>> Ubuntu/Canonical
>> Hardware Enablement
>>
>> On 04/05/2011 01:30 PM, Dmitry Torokhov wrote:
>>> On Tue, Apr 05, 2011 at 12:57:27PM -0500, Manoj wrote:
>>>>
>>>>> When both are enabled in BIOS what protocol does the kernel use to
>>>>> talk
>>>>> to the touchpad? Does it use synaptics protocol?
>>>>
>>>> Yes it uses Synaptics TouchPad.
>>>>
>>>> if (max_proto> PSMOUSE_PS2&& synaptics_detect(p .. etc...
>>>>
>>>> I should say here that the trackpoint is not detected. And sometimes
>>>> (very random) the trackpad is detected as PS/2 Mouse when touchpad
>>>> and trackpoint are enabled in the BIOS... here is what it looks like
>>>> in dmesg when it shows up as PS/2 Mouse instead of TPPS/2 IBM
>>>> TrackPoint.
>>>>
>>>> [ 9.797624] input: ThinkPad Extra Buttons as
>>>> /devices/platform/thinkpad_acpi/input/input8
>>>> [ 10.480034] serio: Synaptics pass-through port at
>>>> isa0060/serio1/input0
>>>> [ 10.536122] input: SynPS/2 Synaptics TouchPad as
>>>> /devices/platform/i8042/serio1/input/input9
>>>> [ 34.507201] input: PS/2 Generic Mouse as
>>>> /devices/platform/i8042/serio1/serio2/input/input10
>>>>
>>>
>>> Ah, so we do know that the touchpad has a pass-through port.
>>>
>>> I guess you can try booting with i8042.debug with touchpad enabled and
>>> disabled and see where exactly we fail to enable trackpoint the the
>>> former case.
>>>
> --
> 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
>