Re: [PATCH] ARM: OMAP2+: Fixed inverted OMAP_OFFOUT_EN

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* Ben Tucker <benjamint@xxxxxxxxxxx> [151009 10:27]:
> On Fri, Oct 09, 2015 at 09:16:28AM -0700, Tony Lindgren wrote:
> > 
> > Hmm so what's the use case for PIN_OFF_INPUT* then?
> On the face of it if the peripheral behind the pin is powered off which
> will be the case when we have powered the domains off, then it cannot
> process any input values.
> 
> However there are two uses for input off mode I can think of:
> 
> 1- the pin is not connected, in which case you want to make it an input
> so that there is no driver enabled for it. I am guessing this would save
> a little power even though the pin has no load. Note an internal pull
> resister should be enabled in this case to avoid the pin acting like a
> aerial and switching internal logic, which again would be a source of
> power draw.
> (Having just read the TRM, I see that the safe-mode can be used for most
> N/C pins)

Right.. So that has not been used again.

> 2- we want to enable a wakeup of the system on the pin

For the IO chain wake-up PIN_OFF_INPUT is not needed either..

FYI, the IO chain wake-up can be done just by enabling the WAKEUPENABLE
bit for the pin. All that's now automatically handled with the Linux
generic dedicated wakeirq. For the serial drivers all you need to do
is specify is something like what we do in omap3-beagle-xm.dts for
example:

&uart3 {
	interrupts-extended = <&intc 74 &omap3_pmx_core OMAP3_UART3_RX>;
	pinctrl-names = "default";
	pinctrl-0 = <&uart3_pins>;
};

> Something else you may want to look out for is the fact that the omap4
> mux config is has the same active low OFFOUTENABLE as omap3. I do not
> know if the same problem exists in the omap4 mux control code. I do not
> think it is shared with the omap3 mux control code.
> (Just my 2p worth)

And we also need to update include/dt-bindings/pinctrl/omap.h too.

Regards,

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux