[PATCH][RFC] Backlight addon for dell laptops

Andrey Borzenkov arvidjaar at mail.ru
Thu Jan 11 09:37:59 PST 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Thursday 11 January 2007 07:42, Bill Nottingham wrote:
> David Zeuthen (david at fubar.dk) said:
> > Not at this point; it's distro independent right now. For example, in
> > Fedora we simply load /lib/modules/$unamer/kernel/drivers/acpi/*
> > in /etc/rc.sysint. That's, uh, ugly as hell. I think also sonypi suffers
> > from this.
> >
> > I think the wrong answer is to make the HAL addon load the module.
> > Probably the right answer to loading such modules belongs somewhere in
> > udev perhaps. Or maybe in HAL, I don't really know but I do think stuff
> > like this is important to fix.... Kay, Bill (and others, of course, feel
> > free to jump in), what's your take on this?
>
> I'm all for any way that lets us autoload it via udev/uevents;
> the issue is there's no good way to do matching, AFAIK. For things
> like ibm_acpi, toshiba_acpi, etc., you could theoretically do
> DMI matching, if that was exported somewhere.
>
> Of course, you could always just build them all in. <waits for
> bricks from the kernel people>
>

We still use this hack to load platform-specific ACPI modules:

    for f in /lib/modules/`uname -r`/kernel/drivers/acpi/*;do
        basename="${f##*/}"
        mod="${basename%%.*}"
        case $mod in
            *_acpi)
                dmidecode | grep -i Vendor | egrep -qi "${mod%%_acpi}" 
2>/dev/null || continue
                ;;
        esac
        message="Loading ACPI module $mod:"
        modprobe $mod 2>/dev/null >/dev/null

something like this could probably be used in this case too.

- -andrey
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD8DBQFFpnX9R6LMutpd94wRAlPXAJ9WFE6ANi6WxQyV8qoZdkUWEHkT1gCeMCoz
7EbcUrUodiNdZuuf3xMV9u0=
=I2yY
-----END PGP SIGNATURE-----


More information about the hal mailing list