On Tue, Jun 4, 2013 at 9:11 AM, Linus Walleij <[email protected]> wrote: > On Fri, May 31, 2013 at 12:13 PM, Hebbar Gururaja > <[email protected]> wrote: > >> Amend the hsmmc controller to optionally take a pin control handle and >> set the state of the pins to: >> >> - "default" on boot, resume and before performing a mmc transfer >> - "idle" after initial default, after resume default, and after each >> mmc/sd card access >> - "sleep" on suspend() >> >> By optionally putting the pins into sleep state in the suspend callback >> we can accomplish two things. >> - One is to minimize current leakage from pins and thus save power, >> - second, we can prevent the IP from driving pins output in an >> uncontrolled manner, which may happen if the power domain drops the >> domain regulator. >> >> If any of the above pin states are missing in dt, a warning message >> about the missing state is displayed. >> If certain pin-states are not available, to remove this warning message >> pass respective state name with null phandler. >> >> Signed-off-by: Hebbar Gururaja <[email protected]> >> Cc: Balaji T K <[email protected]> >> Cc: Chris Ball <[email protected]> >> Cc: [email protected] >> Cc: [email protected] > > This is perfectly correct. > Acked-by: Linus Walleij <[email protected]>
So please consider my other option given in patch 2 instead. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
