From: yakui_zhao <yakui.zhao@intel.com>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Jesse Barnes <jbarnes@virtuousgeek.org>,
"intel-gfx@lists.freedesktop.org"
<intel-gfx@lists.freedesktop.org>,
"dri-devel@lists.sourceforge.net"
<dri-devel@lists.sourceforge.net>,
"sonne@debian.org" <sonne@debian.org>,
"Zhang, Rui" <rui.zhang@intel.com>,
"rpurdie@linux.intel.com" <rpurdie@linux.intel.com>,
Thomas Renninger <trenn@suse.de>, Len Brown <lenb@kernel.org>,
linux-acpi <linux-acpi@vger.kernel.org>
Subject: Re: [RFC] [intel-gfx] :The backlight issue when KMS is used
Date: Wed, 08 Apr 2009 08:56:19 +0800 [thread overview]
Message-ID: <1239152179.3635.8.camel@localhost.localdomain> (raw)
In-Reply-To: <20090407073845.GA15635@srcf.ucam.org>
On Tue, 2009-04-07 at 15:38 +0800, Matthew Garrett wrote:
> On Tue, Apr 07, 2009 at 03:25:23PM +0800, yakui_zhao wrote:
> > On Tue, 2009-04-07 at 13:57 +0800, Matthew Garrett wrote:
> > > No, since in UMS mode the X server handles the backlight registers.
> > If so, there is no change about the backlight flowchart in UMS mode.
> >
> > And only when the KMS mode is used, a new backlight I/F is registered.
> > Right?
>
> Right.
>
> > > The acpi case is uninteresting - acpi_video_backlight_support() doesn't
> > > require the acpi video driver. The platform driver case is more
> > > interesting, but the easiest solution is probably to add a notifier
> > > chain for backlight device add and have i915 unregister when a platform
> > > device registers.
> > Yes. We will have to create the communication channel between backlight
> > device and i915 driver. And when a backlight I/F is registered, we will
> > have to check whether the backlight I/F in 915 should be unregistered.
> > Right?
> > Does this make the problem complex?
>
> Not really. It's not a lot of code.
When a new backlight I/F is registered, it should unregister the
backlight I/F registered in 915 driver if it exists. And before i915
driver registers a backlight I/F, it will have to check whether the
backlight I/F is already registered by other code. If it exists, it will
give up.
And this code seems hack.
If so, we will have to do so even for the boxes based on non-intel
platforms.
>
> > > I'm not sure what you mean by arbiter order?
> > What I said is which backlight I/F should be selected if there exist
> > multiple backlight I/F?
>
> The platform-specific one.
>
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2009-04-08 0:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1238737778.3582.163.camel@localhost.localdomain>
[not found] ` <20090403092408.66809996@hobbes>
[not found] ` <20090403162906.GB23243@srcf.ucam.org>
[not found] ` <1239075911.3582.206.camel@localhost.localdomain>
[not found] ` <20090407055717.GA14710@srcf.ucam.org>
[not found] ` <1239089123.3582.228.camel@localhost.localdomain>
[not found] ` <1239092434.7661.87.camel@rzhang-dt>
[not found] ` <20090407082256.GA17012@srcf.ucam.org>
2009-04-07 8:48 ` [RFC] [intel-gfx] :The backlight issue when KMS is used Zhang Rui
2009-04-07 23:27 ` Jesse Barnes
2009-04-08 0:59 ` yakui_zhao
2009-04-08 7:54 ` Thomas Renninger
[not found] ` <20090407073845.GA15635@srcf.ucam.org>
2009-04-08 0:56 ` yakui_zhao [this message]
2009-04-08 10:17 ` Thomas Renninger
2009-04-08 10:50 ` Richard Purdie
2009-04-09 1:12 ` yakui_zhao
2009-04-09 7:42 ` Thomas Renninger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1239152179.3635.8.camel@localhost.localdomain \
--to=yakui.zhao@intel.com \
--cc=dri-devel@lists.sourceforge.net \
--cc=intel-gfx@lists.freedesktop.org \
--cc=jbarnes@virtuousgeek.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=rpurdie@linux.intel.com \
--cc=rui.zhang@intel.com \
--cc=sonne@debian.org \
--cc=trenn@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.