On Wed, Apr 09, 2014 at 11:41:17AM +0800, Shawn Guo wrote: > On Wed, Apr 09, 2014 at 12:44:18AM +0100, Russell King - ARM Linux wrote: > > On Wed, Apr 09, 2014 at 01:16:54AM +0200, Alexandre Belloni wrote: > > > I'm not home, else I'll try on my i.mx28 boards... > > > > I just tried heartbeat here, on iMX6, and it works fine. That's with this > > in the DT: > > FYI. i.MX28 is known as 'mxs' sub-architecture and uses pwm-mxs driver > rather than pwm-imx which is the one for i.MX6.
My point being by doing the above test is that the pwm_disable() call in leds-pwm works correctly with iMX6 and the "troublesome" active-low flag set at the leds-pwm level. It wouldn't work on iMX6 if you tried to use the broken inversion on the PWM output - it would have exactly the effects being reported on iMX28 as setting a duty of 100% would pull the signal low and disabling the PWM will also pull the signal low (which are the two states heartbeat uses.) -- FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly improving, and getting towards what was expected from it. -- To unsubscribe from this list: send the line "unsubscribe linux-pwm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
