From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Thompson Date: Thu, 20 Apr 2017 09:55:34 +0000 Subject: Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output Message-Id: List-Id: References: <20170416103546.3021-1-contact@paulk.fr> <1492443526.12267.4.camel@paulk.fr> <1492505196.2014.0.camel@paulk.fr> In-Reply-To: <1492505196.2014.0.camel-W9ppeneeCTY@public.gmane.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: Paul Kocialkowski , Geert Uytterhoeven Cc: "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Thierry Reding , Lee Jones , Jingoo Han , Bartlomiej Zolnierkiewicz , Linux PWM List , Linux Fbdev development list , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org On 18/04/17 09:46, Paul Kocialkowski wrote: > Hi, > > Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : >> On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski wrote: >>> Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >>>> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski >>>> wrote: >>>>> The move to a dedicated pwm_backlight_initial_power_state function in >>>>> commit 7613c922315e308a6486d802abed2eb74443dffd modified the condition >>>>> to set the enable gpio as output. This breaks specific use cases using >>>>> that GPIO, such as tegra124-based nyan Chromebooks where backlight >>>>> stopped working. >>>>> >>>>> This puts the condition back to the way it was before the move. >>>>> >>>>> Signed-off-by: Paul Kocialkowski >>>> >>>> Does "[PATCH v2] backlight: pwm_bl: Fix GPIO out for unimplemented >>>> .get_direction()" >>>> (https://lkml.org/lkml/2017/4/4/225) fix your issue? >>> >>> This definitely solves the issue! >>> >>> Since this fixes broken backlight (that, in turn, makes systems unusable), >>> perhaps this should be included in the next rc cycle? >> >> You mean v4.11 final, as there probably won't be another rc? >> Yes! > > I'm all for it! Thanks for keeping our attention on this. I sent a PR yesterday (and hope my GPG key has attracted enough signatures). Daniel.