2007/3/17, Nicolas Boichat <[EMAIL PROTECTED]>:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Hi,
>
> Guillaume Pujol wrote:
> > Hi,
> >
> > Attached to this mail you'll can find clean mactel patches for linux
> > kernel 2.6.20.
> > They apply against linux 2.6.20 + gentoo patchset + suspend2 without
> > offset nor fuzz (should apply to unpatched official kernel as well).
>
> Ok cool. Personally, when I did this job, I edited the patchfiles
> manually... Then it was easier for me to do a diff between the files,
> and check I didn't remove a part of the patch, or modify something...
> With your patchset it's hard for me to compare...

Actually I did almost all modifications manually directly to the
patchfiles. I had an hard time tracking and fixing all tab/spaces
mixings :). Apart from that and usbhid.patch, I did not change
anything in the code.

>
> > Also:
> > - rewritten usbhid.patch which was partly applied to official 2.6.20,
> > and added HID_QUIRK_IGNORE_MOUSE everywhere it was useful.
>
> Just beware of this in include/linux/hid.h:
> +#define HID_QUIRK_IGNORE_MOUSE                 0x00010000
>  #define HID_QUIRK_BAD_RELATIVE_KEYS            0x00010000
>
> These two constants should have a different values.
You are right, but this is the value figuring in the original
patchfile in mactel SVN. Maybe you'd like to change this value to
0x00020000 like in upstream 2.6.21 ?

> > - removed a part of applesmc.patch which was redundant with coretemp.patch
> Which part exactly? I cannot see what you changed because you changed
> all the tabs in spaces... Also I don't think there is anything in
> applesmc redundant with coretemp...

Oops sorry nervermind... I just checked again after a fresh checkout
and obviously there is nothing redundant. I guess it was the result of
a wrong manipulation on my local repo :|
>
> Thanks,
Thanks to you,
>
> Best regards,
>

P.S: also I just checked and these patches apply just fine against
linux-2.6.20.3 straight from kernel.org.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
Mactel-linux-devel mailing list
Mactel-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mactel-linux-devel

Reply via email to