Date: Wed, 11 Nov 2015 15:43:21 +0000
From: Luis Henriques <luis.henriques@canonical.com>
To: Michel =?iso-8859-1?Q?D=E4nzer?= <michel@daenzer.net>
Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org,
        kernel-team@lists.ubuntu.com, Alex Deucher <alexander.deucher@amd.com>
Subject: Re: [PATCH 3.16.y-ckt 072/104] drm/radeon: Restore LCD backlight
 level on resume (>= R5xx)
Message-ID: <20151111154321.GC8684@ares>
References: <1445867006-18048-1-git-send-email-luis.henriques@canonical.com>
 <1445867006-18048-73-git-send-email-luis.henriques@canonical.com>
 <562EDD15.8030205@daenzer.net>
 <20151028104559.GB2792@ares>
 <56371626.6050504@daenzer.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <56371626.6050504@daenzer.net>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2112
Lines: 51

On Mon, Nov 02, 2015 at 04:52:06PM +0900, Michel D?nzer wrote:
> On 28.10.2015 19:45, Luis Henriques wrote:
> > On Tue, Oct 27, 2015 at 11:10:29AM +0900, Michel D?nzer wrote:
> >> On 26.10.2015 22:42, Luis Henriques wrote:
> >>> 3.16.7-ckt19 -stable review patch.  If anyone has any objections, please let me know.
> >>>
> >>> ------------------
> >>>
> >>> From: =TF-8?q?Michel D?nzer?= <michel.daenzer@amd.com>
> >>>
> >>> commit 4281f46ef839050d2ef60348f661eb463c21cc2e upstream.
> >>>
> >>> Instead of only enabling the backlight (which seems to set it to max
> >>> brightness), just re-set the current backlight level, which also takes
> >>> care of enabling the backlight if necessary.
> >>>
> >>> Only the radeon_atom_encoder_dpms_dig part tested on a Kaveri laptop,
> >>> the radeon_atom_encoder_dpms_avivo part is only compile tested.
> >>>
> >>> Signed-off-by: Michel D?nzer <michel.daenzer@amd.com>
> >>> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
> >>> Signed-off-by: Luis Henriques <luis.henriques@canonical.com>
> >>
> >> We're currently investigating a regression which was bisected to this
> >> change. Please hold off on backporting this change until we have a
> >> solution for that.
> > 
> > Thanks, I'll drop this patch for this 3.16 release, and will wait for a
> > fix to be available.
> 
> The regression is fixed in commits
> 4cee6a9057d5e13911f0cb6e143d11dc1a3245dd and
> ae93580ee59c02395c1711d3e6b90546b8137b86 . Please only backport
> 4281f46ef839050d2ef60348f661eb463c21cc2e together with those two.
> 

Thanks (and sorry for the delay in my reply!).

I'll queue all these 3 commits for the 3.16 kernel.  Commit 4cee6a9057d5
("drm/radeon: move bl encoder assignment into bl init") required a bit of
backporting which I would appreciate if you could review it (you'll
received the email with the patch soon).

Cheers,
--
Lu?s
--
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/