27 Jul
2013
27 Jul
'13
2:05 p.m.
On Sat, Jul 27, 2013 at 01:31:50PM +0200, Markus Pargmann wrote:
This is a really nice approach to the problem, just one thing I'd like to see changed in the bindings:
+ssi {
- ...
- pinctrl-names = "default", "ac97-running", "ac97-reset", "ac97-warm-reset";
I'd like to see some documentation of this "default" state - I'd expect this corresponds to the bus being idle and waiting for a wakeup from the CODEC which is definitely a useful state to have for power optimisation so it'd be good to bake it into the bindings.
Alternatively it could be the bus being totally idle with no possibility of wakeup in which case it'd be good to say that too then someone can add the waiting for wakeup state later.
Please also note the new list devicetree@vger.kernel.org.