[lkml]   [2007]   [Feb]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 1/1] PXAFB: Support for backlight control
    On Thu, 2007-02-22 at 09:28 +0100, Rodolfo Giometti wrote:
    > On Thu, Feb 22, 2007 at 12:59:06AM +0000, Richard Purdie wrote:
    > >
    > > Reading through the patch its:
    > >
    > > 1) Not against any mainline kernel

    Sorry, I'd missed a patch entering mainline,

    > > 2) Not against a recent kernel
    > I'm sorry, but the patch applay against the latest kernel. Please, try
    > it.

    but this still applies as there were a number of backlight changes
    merged just before 2.6.21-rc1 and your driver will not work with

    > > As mentioned by others, there is no need to tie the backlight driver
    > > into the framebuffer any more. Have a look at
    > > drivers/video/backlight/corgi_bl.c for an example (its used by PXA
    > > devices).
    > z
    > That driver uses the backlight class support as my patch does into
    > pxafb.

    Yes, my point is that you shouldn't need to touch pxafb if you use the
    backlight class. I know pxafb has backlight hooks but they are probably
    going to get removed at some point as they should no longer be needed.

    > > I have said elsewhere I will take patches to make corgi_bl a more
    > > generic driver (or maybe create a simple generic backlight driver) along
    > > the lines of what Paul mentioned.
    > I see.
    > I suppose you are the backlight support mantainer, so what do you
    > suggest to do to "make corgi_bl a more generic driver"?

    What changes do you need to it to be able to use it as a generic driver?

    The main issue is that the structure definition is in one of the
    sharpsl.h files at the moment so most drivers can't get to it. Fix that
    and it should make a good generic driver.

    Ideally I'd prefer to leave the name alone since there is broken
    userspace on the Zaurus that uses that name but I can see why people
    want it renamed if its to be used as a generic driver...


    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-02-22 10:31    [W:0.031 / U:8.352 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site