Messages in this thread | | | Date | Thu, 22 Feb 2007 01:35:05 +0000 (GMT) | From | James Simmons <> | Subject | Re: [Linux-fbdev-devel] [PATCH] |
| |
> On Wed, 2007-02-21 at 21:23 +0000, James Simmons wrote: > > This is the new display intreface. Its goal is to provide a standard > > interface to various types of displays. Currently we have auxdisplay, > > output acpi device and the now defunct lcd class in the backlight directory. > > Please apply. > > Is this an attempt to consolidate all display hardware and drivers?
All display types. Its LCD panels, LCD teletypes, TVs, CRTs etc. Right now we have several solutions to the same problem (acpi output.c driver, auxdisplay directory). A good idea what this aimed at take a look at
http://lcd4linux.sf.net
The bonus is the acpi and fbdev layer coudl use this as well.
> > + } else { > > + new_dev->dev = NULL; > > + kfree(new_dev); > > Set new_dev to NULL on failure.
Hum let me look. Okay, from what I see kfree doesn't set the the pointer to null. Will fix.
> > + mutex_lock(&ddev->lock); > > + device_del(ddev->dev); > > + ddev->driver = NULL; > > + index--; > > display0 > display1 > index = 2 > unregister display0 > index = 1 > display_device_register() as device1 > device1 <-- BUG, already used.
Ug. Will fix.
> If this is an attempt to consolidate, I don't see the 'brightness' > hook of backlight and lcd. > > If this is not a consolidation, why don't we just extend the lcd class?
Its is a replacement for lcd. Is brightness universal for all display types? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |