Sorry it took me so long to test on my GoFlex Net.  Working fine.<div><br></div><div>tested by: Joshua Coombs <<a href="mailto:josh.coombs@gmail.com">josh.coombs@gmail.com</a>></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sun, Oct 28, 2012 at 12:57 PM, Andrew Lunn <span dir="ltr"><<a href="mailto:andrew@lunn.ch" target="_blank">andrew@lunn.ch</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Wed, Oct 24, 2012 at 04:53:45PM +0200, Andrew Lunn wrote:<br>
> This patchset converts all DT kirkwood boards to using pinctrl.<br>
<br>
</div>Thanks for the feedback everybody gave.<br>
<br>
Progress has been made:<br>
<br>
The bug causing the lockups has been found, fixed and passed upstream.<br>
<br>
I factored out the repeated pinmux definitions for sata0, sata1, spi,<br>
twsi0, uart0, uart1.<br>
<br>
I fixed a few types reported by testers.<br>
<br>
I found a solution to many of the gpio_request() failures. The ones<br>
for controlling power to SATA or USB i've replaced with a regulator in<br>
DT. The regulator will grab the GPIO, set it to output, and driver it<br>
high/low as required. I just hope we don't have an ordering issue.<br>
The regulator framework seems to load quite early and probes all the<br>
regulators. This fails, because pinctrl has not yet loaded, so the<br>
gpio framework returns EPROBE_DEFER. This is not fatal to the<br>
regulator framework, it tries again later. But this later is after<br>
most of the rest of the drivers have loaded. So we might get into<br>
issues where uboot has not enabled power to USB/SATA, the driver is<br>
probed and fails because of the lack of power, and once things of gone<br>
wrong, we turn the power on.<br>
<br>
I still expect problems with GPIOs used for power off. I've not yet<br>
touched them. Again, a regulator makes sense, but there is no 'out of<br>
the box' regulator type or property which adds itself to pm_power_off.<br>
<br>
The last problem is:<br>
<br>
        /* Set NAS to turn back on after a power failure */<br>
        dnskw_gpio_register(37, "dnskw:power:recover", 1);<br>
<br>
I think its a bit of a stretch calling this a regulator.<br>
<br>
Since the current patchset is 18 patches, plus a few other bug fixes<br>
and dependencies, i'm not going to post them all here. I think we are<br>
still at the smoke/no-smoke stage of testing. Please pull from:<br>
<br>
git://<a href="http://github.com/lunn/linux.git" target="_blank">github.com/lunn/linux.git</a> v3.7-rc2-pinctrl-v2<br>
<br>
and let me know if things work, what typo's need fixing, where the<br>
smoke comes out, etc.<br>
<br>
      Thanks<br>
<span class="HOEnZb"><font color="#888888">        Andrew<br>
</font></span></blockquote></div><br></div>