Thread (60 messages) 60 messages, 6 authors, 2018-11-26

Re: [RCF PATCH,v2,2/2] pwm: imx: Configure output to GPIO in disabled state

From: Uwe Kleine-König <hidden>
Date: 2018-11-19 08:48:32
Also in: linux-pwm, lkml

Hello,

On Sun, Nov 18, 2018 at 09:08:15PM +0100, Uwe Kleine-König wrote:
On Fri, Nov 16, 2018 at 01:24:45PM +0100, Thierry Reding wrote:
quoted
On Fri, Nov 16, 2018 at 11:39:29AM +0100, Uwe Kleine-König wrote:
quoted
Also note that you don't include the poor souls where there is no
hardware pullup into the right direction.
The poor souls should speak up and then we can look into finding a good
solution for them. I'm pretty sure there must be some equivalent that
can be used for other users.
I think one of the boards that I'm working with has an enabled backlight
at power on until the bootloader actively drives the related pwm to
the right level (either using the pwm or the gpio function of the pin).
I will double check that but assuming that is right, just disabling the
output won't work here.
I just confirmed this. On the board I have in mind the backlight really
goes on during powerup. One of the first things the bootloader does is
to ensure the backlight goes off again by actively driving the
respective pin.

I fully agree with you this is unfortunate and the hardware team could
have made the situation smoother if there were a pull up. I guess asking
them to go back and fix that won't be accepted[1]. So please whatever
solution you force through, please consider it should work without a
sane pull and also with the imx SoCs before i.MX6 which cannot disable
the output driver via pinmux.

Best regards
Uwe

[1] Additionally this is a bought display+backlight+touch combo, so the
    right place to put the pull would be in this unit, because the board
    designer rightfully says: I won't put a pull up on my board because
    for the next batch we're maybe switching to the similar display
    combo that needs an active high signal for the backlight and I won't
    create another factor 2 for the board variants for stuff that can be
    fixed in software.

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help