Bug 426574 - appletouch does not recognize trackpad in macbook 3.1
Summary: appletouch does not recognize trackpad in macbook 3.1
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: 8
Hardware: x86_64
OS: Linux
low
high
Target Milestone: ---
Assignee: Kernel Maintainer List
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-12-22 12:26 UTC by Chris Nolan
Modified: 2008-03-27 10:58 UTC (History)
1 user (show)

Fixed In Version: 2.6.24.3-50.fc8
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-03-27 10:58:37 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Test kernel patch for 2.6.23 x86_64 (105.60 KB, patch)
2007-12-22 17:46 UTC, Chris Nolan
no flags Details | Diff
Updated Test kernel patch for 2.6.23 x86_64 (9.49 KB, patch)
2008-01-19 10:42 UTC, Chris Nolan
no flags Details | Diff
Updated kernel patch for 2.6.23 x86_64 (1.54 KB, patch)
2008-01-20 16:42 UTC, Chris Nolan
no flags Details | Diff
Updated kernel patch for 2.6.23 x86_64 (1.32 KB, patch)
2008-02-12 12:55 UTC, Chris Nolan
no flags Details | Diff

Description Chris Nolan 2007-12-22 12:26:02 UTC
Description of problem:

appletouch does not recognize trackpad in macbook 3.1

the macbook 3.1 ("Santa Rosa") has a trackpad that appears to be identical to
previous generations, but with new IDs.

How reproducible:

Very - every time.

Steps to Reproduce:
1. Install FC8 x86_64
2. Update everything
3. Load the appletouch kernel module
4. Use synaptic driver in xorg.conf for the touchpad input device
5. Restart X
  
Actual results:

X fails to start as it cannot find a synaptics device

Expected results:

Touchpad should be found as a synaptics controllable device, synaptics driver
should load and X should start as normal

Additional info:

Patch available for Ubuntu kerbel at
https://bugs.edge.launchpad.net/ubuntu/+source/linux-meta/+bug/162090

Comment 1 Chris Nolan 2007-12-22 17:46:48 UTC
Created attachment 290293 [details]
Test kernel patch for 2.6.23 x86_64

This is a kernel patch to fix both this bug (426574) and 426576 - the
appletouch.c part of this patch is relevant to this bug. It seems to work for
me, but needs testing. 

Based on patch from ubuntu at
https://bugs.edge.launchpad.net/ubuntu/+source/linux-meta/+bug/162090

Comment 2 Chuck Ebbert 2008-01-02 19:52:20 UTC
Your patch needs to be regenerated because it includes the .orig files in their
entirety as new files.

Comment 3 Chris Nolan 2008-01-05 14:34:29 UTC
(In reply to comment #2)
> Your patch needs to be regenerated because it includes the .orig files in their
> entirety as new files.

I'm new to all this and I followed the instructions here
(http://fedoraproject.org/wiki/Docs/CustomKernel#head-3e01c3430de111cd9ac1cf65266d44ae7e3f4fac)
to create the patch. Can you suggest a different diff command that would
generate the correct format of patch?

Comment 4 Chuck Ebbert 2008-01-08 18:30:47 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > Your patch needs to be regenerated because it includes the .orig files in their
> > entirety as new files.
> 
> I'm new to all this and I followed the instructions here
>
(http://fedoraproject.org/wiki/Docs/CustomKernel#head-3e01c3430de111cd9ac1cf65266d44ae7e3f4fac)
> to create the patch. Can you suggest a different diff command that would
> generate the correct format of patch?

You just need to remove the .orig files so the source tree is clean before
generating the patch.



Comment 5 Johan Kok 2008-01-19 08:45:27 UTC
*** Bug 426576 has been marked as a duplicate of this bug. ***

Comment 6 Chris Nolan 2008-01-19 09:37:45 UTC
I am trying to help here but I am still completely at a loss as to how to
generate the patch file without it containing the .orig files in their entirety.
I find it odd that the fedora project wiki doesn't give the correct instructions
for what to do here.

I have two hardlinked directories in BUILD/ that I am using to build the patch:

kernel-2.6.23.orig
kernel-2.6.23.new

The edited files are in .new

I then ran this diff command to create the patch:

diff -uNrp kernel-2.6.x.orig kernel-2.6.x.new >
../SOURCES/linux-2.6-my-new-patch.patch

All this is exactly how the fedoraproject wiki explains to do it. 

Comment #4 above doesn't make much sense to me. Remove which ".orig" files?

Comment 7 Chris Nolan 2008-01-19 10:42:18 UTC
Created attachment 292253 [details]
Updated Test kernel patch for 2.6.23 x86_64

This is an updated kernel patch to fix both this bug (426574) and 426576 - the
appletouch.c part of this patch is relevant to this bug. 

I have been running this patched kernel for 4 weeks now without any problems
whatsoever. Using this patched kernel the touchpad works perfectly with the
synaptics drivers and the function key works as desired. (This is tested on a
macbook 3,1 santa rosa.)

Comment 8 Chris Nolan 2008-01-19 10:47:06 UTC
Please ignore Comment #6 - I figured out what was meant by "remove the .orig
files" - must have been having a dim moment. 

The patch attached above in Comment #7 is the recreated patch which does not
include the .orig files in their entirety.

Comment 9 Chris Nolan 2008-01-20 16:42:43 UTC
Created attachment 292294 [details]
Updated kernel patch for 2.6.23 x86_64

This is an updated kernel patch containing proposed kernel fixes for this bug
only.

I have yet to test this individual patch but I will update this report once I
have done so. We could really use some more people with the macbook 3,1 to test

this also.

Comment 10 Chuck Ebbert 2008-01-24 00:32:42 UTC
Patch in 2.6.23.14-119. (Taken from the Mactel site.)

Comment 11 Chris Nolan 2008-01-30 11:55:16 UTC
That is great news. Any chance of getting the patch from bug #426576 included
soon too?

Comment 12 Fedora Update System 2008-02-11 03:34:04 UTC
kernel-2.6.23.15-137.fc8 has been submitted as an update for Fedora 8

Comment 13 Fedora Update System 2008-02-11 22:38:50 UTC
kernel-2.6.23.15-137.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 14 Chris Nolan 2008-02-12 09:38:56 UTC
kernel-2.6.23.15-137.fc8 does not fix this bug - using that kernel, the trackpad
on macbook 3,1 is not recognised by appletouch and cannot be configured using
the synaptics driver in Xorg.

Comment 15 Chris Nolan 2008-02-12 12:35:12 UTC
I have tested the new kernel some more, and found the following:

1) The stock kernel 2.6.23.15-137.fc8 from the f8 updates repository does not
fix this bug. The error is the same as in the first post of this bug report.

2) Building the stock kernel 2.6.23.15-137.fc8 from source does not fix this bug
despite their being a patch included that is supposed to fix it.

3) Building the kernel 2.6.23.15-137.fc8 from source and patching with the
latest patch from bug #426576 does fix this bug and also fixes bug #426576

Thus it seems that the patch submitted for this bug does not work correctly
without the patch for bug #426576 also being applied. I can't see why this would
be the case (as the patches seem distinct from one another), but it appears to
be so.

Comment 16 Chris Nolan 2008-02-12 12:55:41 UTC
Created attachment 294653 [details]
Updated kernel patch for 2.6.23 x86_64

Just a cleaner version of the same patch (corrected the file paths). Tested and
works with 2.6.23.15-137 but only works in combination with patch from bug
#426576.

Comment 17 Chris Nolan 2008-02-12 21:03:26 UTC
I now believe the reason the patch from bug #426576 (at least the part where the
new device IDs are added) is also required is to prevent the usbhid driver from
claiming the touchpad device ahead of the appletouch driver. The patch sets a
flag called HID_QUIRK_IGNORE_MOUSE for the Macbook keyboards.

Thus, the patch for this bug is dependent on the patch from bug #426576 and so
both patches should be included in the kernel.

Comment 18 Michael Tiemann 2008-03-09 09:02:45 UTC
I would love to see this merged so I can use a standard fedora kernel.

Comment 19 ryan roth 2008-03-10 20:57:19 UTC
Ditto, I would love to be able to use a stock fedora kernel, please merge this
patch soon

Comment 20 Chris Nolan 2008-03-27 10:58:37 UTC
Bug #425576 is now fixed which in-turn fixes this bug.

kernel-2.6.24.3-50.fc8 works just fine with the macbook 3,1 touchpad and keyboard.


Note You need to log in before you can comment on or make changes to this bug.