Re: [PATCH 1/5] Backlight: Add backlight type

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2010-11-22 at 12:30 +0000, Matthew Garrett wrote:
> On Mon, Nov 22, 2010 at 10:17:00AM +0000, Richard Purdie wrote:
> 
> > I also want to make sure you think this patch is going to scale with
> > multiple GPU output machines? Thats the main reason I've held off any
> > patch like this as it doesn't help solve that problem as far as I can
> > tell. Yes, we have the device parent information and I see later in the
> > patch series you ensure the backlight is registered against the
> > connector which is good. If you have an ACPI "firmware" control that you
> > say should always be preferred, how do we know which connector device
> > that corresponds to in the multiple output case? From that point of view
> > this model falls apart?
> 
> The ACPI device will point at the correct PCI device. Associating it 
> with the appropriate connector is theoretically possible in the case of 
> open drivers, but I hadn't seen it as a high priority since (in 
> practice) there's no situations where an ACPI interface will be able to 
> control more than one backlight.

Its the reverse situation I worry about. Are there situations where
there are multiple connectors on the PCI device and the ACPI interface
just controls one of them but controls for the other connectors may
exist?

I'm typing this with an external monitor plugged into my laptop with
i915 graphics...

Cheers,

Richard





--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux