[PATCH 1/2] hardy-lbm: bcm5974: remove dynamic quirks, instead post-install quirkline
Henrik Rydberg
rydberg at euromail.se
Thu Jul 24 23:56:09 UTC 2008
Ben Collins wrote:
> On Thu, 2008-07-24 at 12:18 +0200, Henrik Rydberg wrote:
>> This patch aims at resolving the problem of initializing
>> the bcm5974 by installing a quirkline in /etc/modprobe.d/bcm5974,
>> which makes usbhid ignore the trackpad.
>>
>> The install/remove scripts are only added, not linked into
>> postinst; hoping for some help to do it properly.
>
> NACK in a few ways:
>
> - Completely against policy. Creating files like this is just broken.
> - Wont work on LiveCD installs since postinst is never run in this case.
> - Causes linux-lbm to fail to install on systems where bcm5974 doesn't
> exist.
>
> Put the quirks in the code where it belongs. This is the wrong way to
> handle this.
Very well - you can scratch patch 2/2 also, since it wont apply cleanly
without 1/2. I will resend it as a single upgrade patch.
Regarding the quirks, I'll simply wait and see how it works in the
next proposed ppa upload before doing anything about it. I already
reported that there are severe problems with the current approach
if booting from disk and not from the initrd.
Henrik
More information about the kernel-team
mailing list