2.6.29-mmotm0109 Odd prompt for Sharp Corgi backlight.

2.6.29-mmotm0109 Odd prompt for Sharp Corgi backlight.

[Valdis.Kletnieks]

[-- Attachment #1: Type: text/plain, Size: 811 bytes --]

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.

[-- Attachment #2: Type: application/pgp-signature, Size: 226 bytes --]

Re: 2.6.29-mmotm0109 Odd prompt for Sharp Corgi backlight.

[Richard Purdie]

On Sun, 2009-01-11 at 13:38 -0500, Valdis.Kletnieks@vt.edu 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

Re: 2.6.29-mmotm0109 Odd prompt for Sharp Corgi backlight.

[Dmitry Torokhov]

On Sun, Jan 11, 2009 at 11:59:59PM +0000, Richard Purdie wrote:
> On Sun, 2009-01-11 at 13:38 -0500, Valdis.Kletnieks@vt.edu 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