2009-01-11 18:39:32

by Valdis Klētnieks

[permalink] [raw]
Subject: 2.6.29-mmotm0109 Odd prompt for Sharp Corgi backlight.

So I'm building mmotm-0109, doing a 'make oldconfig', and out pops:

Generic (aka Sharp Corgi) Backlight Driver (BACKLIGHT_GENERIC) [Y/n/m/?] (NEW) ?

Say y to enable the generic platform backlight driver previously
known as the Corgi backlight driver. If you have a Sharp Zaurus
SL-C7xx, SL-Cxx00 or SL-6000x say y.


Two questions:

1) Is it a Corgi driver, or truly a generic one? It can't be *too* generic
if it only applies to 3 Sharp Zaurus models? Or do I want to specify it for
my Dell laptop that has a backlight too (in which case that help text needs
fixing)...

2) Why does this default to 'Y'? (Probably depends on the answer to (1) - if
it's for one vendor, it probably should be N, but Y may make sense if it's
proper for the vast majority of all laptops and similar that have a backlight.


Attachments:
(No filename) (226.00 B)

2009-01-11 23:59:26

by Richard Purdie

[permalink] [raw]
Subject: Re: 2.6.29-mmotm0109 Odd prompt for Sharp Corgi backlight.

On Sun, 2009-01-11 at 13:38 -0500, [email protected] wrote:
> So I'm building mmotm-0109, doing a 'make oldconfig', and out pops:
>
> Generic (aka Sharp Corgi) Backlight Driver (BACKLIGHT_GENERIC) [Y/n/m/?] (NEW) ?
>
> Say y to enable the generic platform backlight driver previously
> known as the Corgi backlight driver. If you have a Sharp Zaurus
> SL-C7xx, SL-Cxx00 or SL-6000x say y.
>
>
> Two questions:
>
> 1) Is it a Corgi driver, or truly a generic one? It can't be *too* generic
> if it only applies to 3 Sharp Zaurus models? Or do I want to specify it for
> my Dell laptop that has a backlight too (in which case that help text needs
> fixing)...

It is truly generic code now. Your Dell laptop won't use it but it won't
hurt it either.

> 2) Why does this default to 'Y'? (Probably depends on the answer to (1) - if
> it's for one vendor, it probably should be N, but Y may make sense if it's
> proper for the vast majority of all laptops and similar that have a backlight.

It defaults to Y since its harmless if not used and will become the
common case for a large number of devices.

Cheers,

Richard


2009-01-13 06:26:05

by Dmitry Torokhov

[permalink] [raw]
Subject: Re: 2.6.29-mmotm0109 Odd prompt for Sharp Corgi backlight.

On Sun, Jan 11, 2009 at 11:59:59PM +0000, Richard Purdie wrote:
> On Sun, 2009-01-11 at 13:38 -0500, [email protected] wrote:
> > So I'm building mmotm-0109, doing a 'make oldconfig', and out pops:
> >
> > Generic (aka Sharp Corgi) Backlight Driver (BACKLIGHT_GENERIC) [Y/n/m/?] (NEW) ?
> >
> > Say y to enable the generic platform backlight driver previously
> > known as the Corgi backlight driver. If you have a Sharp Zaurus
> > SL-C7xx, SL-Cxx00 or SL-6000x say y.
> >
> >
> > Two questions:
> >
> > 1) Is it a Corgi driver, or truly a generic one? It can't be *too* generic
> > if it only applies to 3 Sharp Zaurus models? Or do I want to specify it for
> > my Dell laptop that has a backlight too (in which case that help text needs
> > fixing)...
>
> It is truly generic code now. Your Dell laptop won't use it but it won't
> hurt it either.
>
> > 2) Why does this default to 'Y'? (Probably depends on the answer to (1) - if
> > it's for one vendor, it probably should be N, but Y may make sense if it's
> > proper for the vast majority of all laptops and similar that have a backlight.
>
> It defaults to Y since its harmless if not used and will become the
> common case for a large number of devices.
>

While lots of the drivers are harmless if not used the dafault ofr them
is still N. Can we add some arch dependency or something?

--
Dmitry