[PATCH 1/2] hardy-lbm: bcm5974: remove dynamic quirks, instead post-install quirkline

Henrik Rydberg rydberg at euromail.se
Fri Jul 25 00:03:29 UTC 2008


Ben Collins wrote:
> On Fri, 2008-07-25 at 01:56 +0200, Henrik Rydberg wrote:
>> 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.
> 
> In our case, you cannot boot from disk, you have to boot from initramfs
> (all the storage drivers are modular).
> 

I was beginning to suspect as much - thanks for the quick reply!

Henrik





More information about the kernel-team mailing list