On Wed, Oct 28, 2015 at 3:11 AM, Michel Dänzer <michel at daenzer.net> wrote: > On 28.10.2015 01:43, Alex Deucher wrote: >> On Tue, Oct 27, 2015 at 12:23 PM, Michael Burian <michael.burian at sbg.at> >> wrote: >>> On 10/27/15 16:10, Alex Deucher wrote: >>> >>>> >>>> It would appear that your system does not use the gpu backlight >>>> controller. Either it's lying or messing with the GPU backlight >>>> controller causes some bad interaction with whatever does control it. >>>> Does the attached radeon patch help? I'm also attaching an amdgpu >>>> patch for reference in case the same problem appears on amdgpu. >>>> >>> >>> no, still no backlight when applied against current current mainline >>> >>> 858e904bd71dd0057a548d6785d94ce5ec4aeabd (Merge tag 'iommu-fixes-v4.3-rc7' >>> of git://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu) >>> >> >> I see the problem. We don't enable native backlight control on older >> asics like yours by default. Does the attached patch help? > > My only doubt about this patch is: Should we also fall back to the old > beahviour in the !(rdev->mode_info.firmware_flags & > ATOM_BIOS_INFO_BL_CONTROLLED_BY_GPU) case?
Theoretically, it's not necessary, but I guess better safe than sorry. Updated patches attached. Alex -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-drm-radeon-move-bl-encoder-assignment-into-bl-init.patch Type: text/x-patch Size: 2094 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151028/a56f6f45/attachment.bin> -------------- next part -------------- A non-text attachment was scrubbed... Name: 0002-drm-radeon-fix-dpms-when-driver-backlight-control-is.patch Type: text/x-patch Size: 2355 bytes Desc: not available URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151028/a56f6f45/attachment-0001.bin>
