* [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-16 10:35 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-16 10:35 UTC (permalink / raw) To: linux-kernel-u79uwXL29TY76Z2rM5mHXA Cc: Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, linux-pwm-u79uwXL29TY76Z2rM5mHXA, linux-fbdev-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA, Paul Kocialkowski 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 <contact-W9ppeneeCTY@public.gmane.org> --- drivers/video/backlight/pwm_bl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index d7efcb632f7d..e81b2b55be2b 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -304,7 +304,8 @@ static int pwm_backlight_probe(struct platform_device *pdev) * early. Leave the decision of the initial backlight state for later. */ if (pb->enable_gpio && - gpiod_get_direction(pb->enable_gpio) == GPIOF_DIR_IN) + (gpiod_get_direction(pb->enable_gpio) == GPIOF_DIR_IN || + gpiod_get_value(pb->enable_gpio) != 0)) gpiod_direction_output(pb->enable_gpio, 1); pb->power_supply = devm_regulator_get(&pdev->dev, "power"); -- 2.12.2 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-16 10:35 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-16 10:35 UTC (permalink / raw) To: linux-kernel-u79uwXL29TY76Z2rM5mHXA Cc: Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, linux-pwm-u79uwXL29TY76Z2rM5mHXA, linux-fbdev-u79uwXL29TY76Z2rM5mHXA, linux-tegra-u79uwXL29TY76Z2rM5mHXA, Paul Kocialkowski 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 <contact@paulk.fr> --- drivers/video/backlight/pwm_bl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index d7efcb632f7d..e81b2b55be2b 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -304,7 +304,8 @@ static int pwm_backlight_probe(struct platform_device *pdev) * early. Leave the decision of the initial backlight state for later. */ if (pb->enable_gpio && - gpiod_get_direction(pb->enable_gpio) = GPIOF_DIR_IN) + (gpiod_get_direction(pb->enable_gpio) = GPIOF_DIR_IN || + gpiod_get_value(pb->enable_gpio) != 0)) gpiod_direction_output(pb->enable_gpio, 1); pb->power_supply = devm_regulator_get(&pdev->dev, "power"); -- 2.12.2 ^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-16 10:35 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-16 10:35 UTC (permalink / raw) To: linux-kernel Cc: Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, linux-pwm, linux-fbdev, linux-tegra, Paul Kocialkowski 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 <contact@paulk.fr> --- drivers/video/backlight/pwm_bl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/video/backlight/pwm_bl.c b/drivers/video/backlight/pwm_bl.c index d7efcb632f7d..e81b2b55be2b 100644 --- a/drivers/video/backlight/pwm_bl.c +++ b/drivers/video/backlight/pwm_bl.c @@ -304,7 +304,8 @@ static int pwm_backlight_probe(struct platform_device *pdev) * early. Leave the decision of the initial backlight state for later. */ if (pb->enable_gpio && - gpiod_get_direction(pb->enable_gpio) == GPIOF_DIR_IN) + (gpiod_get_direction(pb->enable_gpio) == GPIOF_DIR_IN || + gpiod_get_value(pb->enable_gpio) != 0)) gpiod_direction_output(pb->enable_gpio, 1); pb->power_supply = devm_regulator_get(&pdev->dev, "power"); -- 2.12.2 ^ permalink raw reply related [flat|nested] 22+ messages in thread
[parent not found: <20170416103546.3021-1-contact-W9ppeneeCTY@public.gmane.org>]
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output 2017-04-16 10:35 ` Paul Kocialkowski (?) @ 2017-04-16 20:55 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-16 20:55 UTC (permalink / raw) To: Paul Kocialkowski Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA Hi Paul, On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org> 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 <contact-W9ppeneeCTY@public.gmane.org> Does "[PATCH v2] backlight: pwm_bl: Fix GPIO out for unimplemented .get_direction()" (https://lkml.org/lkml/2017/4/4/225) fix your issue? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert-Td1EMuHUCqxL1ZNQvxDV9g@public.gmane.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-16 20:55 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-16 20:55 UTC (permalink / raw) To: Paul Kocialkowski Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA Hi Paul, On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> 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 <contact@paulk.fr> Does "[PATCH v2] backlight: pwm_bl: Fix GPIO out for unimplemented .get_direction()" (https://lkml.org/lkml/2017/4/4/225) fix your issue? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-16 20:55 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-16 20:55 UTC (permalink / raw) To: Paul Kocialkowski Cc: linux-kernel, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra Hi Paul, On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> 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 <contact@paulk.fr> Does "[PATCH v2] backlight: pwm_bl: Fix GPIO out for unimplemented .get_direction()" (https://lkml.org/lkml/2017/4/4/225) fix your issue? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAMuHMdV1+caNwbuk=Kw7cRUxcp1iC=YArnQXf2mcn+qnc6+OOQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output 2017-04-16 20:55 ` Geert Uytterhoeven (?) @ 2017-04-17 15:38 ` Paul Kocialkowski -1 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-17 15:38 UTC (permalink / raw) To: Geert Uytterhoeven Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 1214 bytes --] Hi, Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org> 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 <contact-W9ppeneeCTY@public.gmane.org> > > 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? Cheers! -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-17 15:38 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-17 15:38 UTC (permalink / raw) To: Geert Uytterhoeven Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 1174 bytes --] Hi, Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> 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 <contact@paulk.fr> > > 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? Cheers! -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-17 15:38 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-17 15:38 UTC (permalink / raw) To: Geert Uytterhoeven Cc: linux-kernel, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra [-- Attachment #1: Type: text/plain, Size: 1174 bytes --] Hi, Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> 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 <contact@paulk.fr> > > 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? Cheers! -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output 2017-04-17 15:38 ` Paul Kocialkowski @ 2017-04-17 19:36 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-17 19:36 UTC (permalink / raw) To: Paul Kocialkowski Cc: linux-kernel, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra Hi Paul, On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact@paulk.fr> wrote: > Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> 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 <contact@paulk.fr> >> >> 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! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-17 19:36 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-17 19:36 UTC (permalink / raw) To: Paul Kocialkowski Cc: linux-kernel, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra Hi Paul, On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact@paulk.fr> wrote: > Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> 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 <contact@paulk.fr> >> >> 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! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <CAMuHMdVLUmT_NJt86tspYEZnsFEiEwiNu_19RQu0qhtcyP1SdQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output 2017-04-17 19:36 ` Geert Uytterhoeven (?) @ 2017-04-18 8:46 ` Paul Kocialkowski -1 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-18 8:46 UTC (permalink / raw) To: Geert Uytterhoeven Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 1566 bytes --] Hi, Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : > On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org> wrote: > > Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > > > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org> > > > 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 <contact-W9ppeneeCTY@public.gmane.org> > > > > > > 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! Cheers, -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-18 8:46 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-18 8:46 UTC (permalink / raw) To: Geert Uytterhoeven Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 1506 bytes --] Hi, Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : > On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact@paulk.fr> wrote: > > Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > > > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> > > > 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 <contact@paulk.fr> > > > > > > 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! Cheers, -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-18 8:46 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-18 8:46 UTC (permalink / raw) To: Geert Uytterhoeven Cc: linux-kernel, Thierry Reding, Lee Jones, Daniel Thompson, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra [-- Attachment #1: Type: text/plain, Size: 1506 bytes --] Hi, Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : > On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact@paulk.fr> wrote: > > Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > > > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> > > > 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 <contact@paulk.fr> > > > > > > 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! Cheers, -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <1492505196.2014.0.camel-W9ppeneeCTY@public.gmane.org>]
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output 2017-04-18 8:46 ` Paul Kocialkowski (?) @ 2017-04-20 9:55 ` Daniel Thompson -1 siblings, 0 replies; 22+ messages in thread From: Daniel Thompson @ 2017-04-20 9:55 UTC (permalink / raw) To: Paul Kocialkowski, Geert Uytterhoeven Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA 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 <contact-W9ppeneeCTY@public.gmane.org> wrote: >>> Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >>>> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org> >>>> 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 <contact-W9ppeneeCTY@public.gmane.org> >>>> >>>> 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. ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-20 9:55 ` Daniel Thompson 0 siblings, 0 replies; 22+ messages in thread From: Daniel Thompson @ 2017-04-20 9:55 UTC (permalink / raw) To: Paul Kocialkowski, Geert Uytterhoeven Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA 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 <contact@paulk.fr> wrote: >>> Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >>>> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> >>>> 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 <contact@paulk.fr> >>>> >>>> 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. ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-20 9:55 ` Daniel Thompson 0 siblings, 0 replies; 22+ messages in thread From: Daniel Thompson @ 2017-04-20 9:55 UTC (permalink / raw) To: Paul Kocialkowski, Geert Uytterhoeven Cc: linux-kernel, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra 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 <contact@paulk.fr> wrote: >>> Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >>>> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> >>>> 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 <contact@paulk.fr> >>>> >>>> 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. ^ permalink raw reply [flat|nested] 22+ messages in thread
[parent not found: <e5a9e6ce-b1dd-96ae-c051-60be7cbfbe1d-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output 2017-04-20 9:55 ` Daniel Thompson (?) @ 2017-04-20 10:16 ` Geert Uytterhoeven -1 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-20 10:16 UTC (permalink / raw) To: Daniel Thompson Cc: Paul Kocialkowski, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA Hi Daniel, On Thu, Apr 20, 2017 at 11:55 AM, Daniel Thompson <daniel.thompson-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> wrote: > On 18/04/17 09:46, Paul Kocialkowski wrote: >> Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : >>> On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org> >>> wrote: >>>> Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >>>>> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact-W9ppeneeCTY@public.gmane.org> >>>>> 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 <contact-W9ppeneeCTY@public.gmane.org> >>>>> >>>>> 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). Thank you, this morning I had already noticed it went upstream. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-20 10:16 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-20 10:16 UTC (permalink / raw) To: Daniel Thompson Cc: Paul Kocialkowski, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra-u79uwXL29TY76Z2rM5mHXA Hi Daniel, On Thu, Apr 20, 2017 at 11:55 AM, Daniel Thompson <daniel.thompson@linaro.org> wrote: > On 18/04/17 09:46, Paul Kocialkowski wrote: >> Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : >>> On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact@paulk.fr> >>> wrote: >>>> Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >>>>> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> >>>>> 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 <contact@paulk.fr> >>>>> >>>>> 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). Thank you, this morning I had already noticed it went upstream. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-20 10:16 ` Geert Uytterhoeven 0 siblings, 0 replies; 22+ messages in thread From: Geert Uytterhoeven @ 2017-04-20 10:16 UTC (permalink / raw) To: Daniel Thompson Cc: Paul Kocialkowski, linux-kernel, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra Hi Daniel, On Thu, Apr 20, 2017 at 11:55 AM, Daniel Thompson <daniel.thompson@linaro.org> wrote: > On 18/04/17 09:46, Paul Kocialkowski wrote: >> Le lundi 17 avril 2017 à 21:36 +0200, Geert Uytterhoeven a écrit : >>> On Mon, Apr 17, 2017 at 5:38 PM, Paul Kocialkowski <contact@paulk.fr> >>> wrote: >>>> Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : >>>>> On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> >>>>> 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 <contact@paulk.fr> >>>>> >>>>> 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). Thank you, this morning I had already noticed it went upstream. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output 2017-04-20 9:55 ` Daniel Thompson @ 2017-04-22 13:00 ` Paul Kocialkowski -1 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-22 13:00 UTC (permalink / raw) To: Daniel Thompson, Geert Uytterhoeven Cc: linux-kernel, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra [-- Attachment #1: Type: text/plain, Size: 2022 bytes --] Hi, Le jeudi 20 avril 2017 à 10:55 +0100, Daniel Thompson a écrit : > 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 <contact@paulk.fr> > > > wrote: > > > > Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > > > > > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> > > > > > 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 <contact@paulk.fr> > > > > > > > > > > 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). Thanks for acting fast and getting this merged already. This is greatly appreciated! Cheers, -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output @ 2017-04-22 13:00 ` Paul Kocialkowski 0 siblings, 0 replies; 22+ messages in thread From: Paul Kocialkowski @ 2017-04-22 13:00 UTC (permalink / raw) To: Daniel Thompson, Geert Uytterhoeven Cc: linux-kernel, Thierry Reding, Lee Jones, Jingoo Han, Bartlomiej Zolnierkiewicz, Linux PWM List, Linux Fbdev development list, linux-tegra [-- Attachment #1: Type: text/plain, Size: 2022 bytes --] Hi, Le jeudi 20 avril 2017 à 10:55 +0100, Daniel Thompson a écrit : > 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 <contact@paulk.fr> > > > wrote: > > > > Le dimanche 16 avril 2017 à 22:55 +0200, Geert Uytterhoeven a écrit : > > > > > On Sun, Apr 16, 2017 at 12:35 PM, Paul Kocialkowski <contact@paulk.fr> > > > > > 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 <contact@paulk.fr> > > > > > > > > > > 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). Thanks for acting fast and getting this merged already. This is greatly appreciated! Cheers, -- Paul Kocialkowski, developer of free digital technology at the lower levels Website: https://www.paulk.fr/ Coding blog: https://code.paulk.fr/ Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/ [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2017-04-22 13:00 UTC | newest] Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-04-16 10:35 [PATCH] backlight: pwm_bl: Fix condition to set enable gpio as output Paul Kocialkowski 2017-04-16 10:35 ` Paul Kocialkowski 2017-04-16 10:35 ` Paul Kocialkowski [not found] ` <20170416103546.3021-1-contact-W9ppeneeCTY@public.gmane.org> 2017-04-16 20:55 ` Geert Uytterhoeven 2017-04-16 20:55 ` Geert Uytterhoeven 2017-04-16 20:55 ` Geert Uytterhoeven [not found] ` <CAMuHMdV1+caNwbuk=Kw7cRUxcp1iC=YArnQXf2mcn+qnc6+OOQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-04-17 15:38 ` Paul Kocialkowski 2017-04-17 15:38 ` Paul Kocialkowski 2017-04-17 15:38 ` Paul Kocialkowski 2017-04-17 19:36 ` Geert Uytterhoeven 2017-04-17 19:36 ` Geert Uytterhoeven [not found] ` <CAMuHMdVLUmT_NJt86tspYEZnsFEiEwiNu_19RQu0qhtcyP1SdQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2017-04-18 8:46 ` Paul Kocialkowski 2017-04-18 8:46 ` Paul Kocialkowski 2017-04-18 8:46 ` Paul Kocialkowski [not found] ` <1492505196.2014.0.camel-W9ppeneeCTY@public.gmane.org> 2017-04-20 9:55 ` Daniel Thompson 2017-04-20 9:55 ` Daniel Thompson 2017-04-20 9:55 ` Daniel Thompson [not found] ` <e5a9e6ce-b1dd-96ae-c051-60be7cbfbe1d-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> 2017-04-20 10:16 ` Geert Uytterhoeven 2017-04-20 10:16 ` Geert Uytterhoeven 2017-04-20 10:16 ` Geert Uytterhoeven 2017-04-22 13:00 ` Paul Kocialkowski 2017-04-22 13:00 ` Paul Kocialkowski
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.