On Sat, Oct 12, 2013 at 1:19 AM, Jesse Barnes wrote:
> But I don't want this fix (once I fix the locking) blocked on
> those tests, since they'll probably take me a few days and people are
> already using the original version, which is missing the locks for the
> backlight class and ASLE call site
On Sat, Oct 12, 2013 at 1:55 AM, Daniel Vetter wrote:
> On Sat, Oct 12, 2013 at 1:19 AM, Jesse Barnes
> wrote:
>> On Fri, 11 Oct 2013 14:34:35 -0700
>> Jesse Barnes wrote:
>>
>>> > Ideas:
>>> > - Make sure all lvds/edp connectors are enabled and bash on all backlight
>>> > interfaces (with ig
On Sat, Oct 12, 2013 at 1:19 AM, Jesse Barnes wrote:
> On Fri, 11 Oct 2013 14:34:35 -0700
> Jesse Barnes wrote:
>
>> > Ideas:
>> > - Make sure all lvds/edp connectors are enabled and bash on all backlight
>> > interfaces (with igt_fork it's easy to do that concurrently).
>> > - Race the above w
On Fri, 11 Oct 2013 14:34:35 -0700
Jesse Barnes wrote:
> > Ideas:
> > - Make sure all lvds/edp connectors are enabled and bash on all backlight
> > interfaces (with igt_fork it's easy to do that concurrently).
> > - Race the above with output changes: dpms on/off and changing the crtc
> > aro
On Fri, 11 Oct 2013 23:31:10 +0200
Daniel Vetter wrote:
> On Fri, Oct 11, 2013 at 11:31:53AM -0700, Jesse Barnes wrote:
>
> [snip]
>
> > static int intel_panel_get_brightness(struct backlight_device *bd)
> > {
> > - struct drm_device *dev = bl_get_data(bd);
> > - return intel_panel_get_ba
On Fri, Oct 11, 2013 at 11:31:53AM -0700, Jesse Barnes wrote:
[snip]
> static int intel_panel_get_brightness(struct backlight_device *bd)
> {
> - struct drm_device *dev = bl_get_data(bd);
> - return intel_panel_get_backlight(dev);
> + struct intel_connector *connector = bl_get_data(
On VLV/BYT, backlight controls a per-pipe, so when adjusting the
backlight we need to pass the correct info. So make the externally
visible backlight functions take a connector argument, which can be used
internally to figure out the pipe backlight to adjust.
v2: make connector pipe lookup check
On Sat, 5 Oct 2013 13:04:08 +0200
Daniel Vetter wrote:
> On Fri, Oct 4, 2013 at 9:42 PM, Jesse Barnes wrote:
> > On VLV/BYT, backlight controls a per-pipe, so when adjusting the
> > backlight we need to pass the correct info. So make the externally
> > visible backlight functions take a connect
On Fri, Oct 4, 2013 at 9:42 PM, Jesse Barnes wrote:
> On VLV/BYT, backlight controls a per-pipe, so when adjusting the
> backlight we need to pass the correct info. So make the externally
> visible backlight functions take a connector argument, which can be used
> internally to figure out the pip
On VLV/BYT, backlight controls a per-pipe, so when adjusting the
backlight we need to pass the correct info. So make the externally
visible backlight functions take a connector argument, which can be used
internally to figure out the pipe backlight to adjust.
v2: make connector pipe lookup check
On Mon, 30 Sep 2013 15:15:12 +0300
Ville Syrjälä wrote:
> On Mon, Sep 30, 2013 at 11:18:28AM +0300, Jani Nikula wrote:
> > On Sat, 28 Sep 2013, Jesse Barnes wrote:
> > > On VLV/BYT, backlight controls a per-pipe, so when adjusting the
> > > backlight we need to pass the correct info. So make th
On Mon, 30 Sep 2013 11:18:28 +0300
Jani Nikula wrote:
> On Sat, 28 Sep 2013, Jesse Barnes wrote:
> > On VLV/BYT, backlight controls a per-pipe, so when adjusting the
> > backlight we need to pass the correct info. So make the externally
> > visible backlight functions take a connector argument,
On Mon, Sep 30, 2013 at 2:15 PM, Ville Syrjälä
wrote:
> I was discussing things a bit with Jani, and I'll repeat my thoughts
> here for others to see. Feel free to kick me if I get things totally
> wrong as I don't know the backlight code, nor do I really want to.
>
> We may need to handle backlig
On Mon, Sep 30, 2013 at 11:18:28AM +0300, Jani Nikula wrote:
> On Sat, 28 Sep 2013, Jesse Barnes wrote:
> > On VLV/BYT, backlight controls a per-pipe, so when adjusting the
> > backlight we need to pass the correct info. So make the externally
> > visible backlight functions take a connector argu
On Sat, 28 Sep 2013, Jesse Barnes wrote:
> On VLV/BYT, backlight controls a per-pipe, so when adjusting the
> backlight we need to pass the correct info. So make the externally
> visible backlight functions take a connector argument, which can be used
> internally to figure out the pipe backlight
On VLV/BYT, backlight controls a per-pipe, so when adjusting the
backlight we need to pass the correct info. So make the externally
visible backlight functions take a connector argument, which can be used
internally to figure out the pipe backlight to adjust.
Signed-off-by: Jesse Barnes
---
dri
16 matches
Mail list logo
