On Sat, Oct 20, 2012 at 3:13 AM, Tony Lindgren [off-list ref] wrote:

* Haojian Zhuang [off-list ref] [121018 02:08]:

quoted

Add pinconf generic support with POWER SOURCE, BIAS PULL.

...

quoted

+     case PIN_CONFIG_POWER_SOURCE:
+             if (pcs->psmask == PCS_OFF_DISABLED
+                     || pcs->psshift == PCS_OFF_DISABLED)
+                     return -ENOTSUPP;
+             data &= pcs->psmask;
+             data = data >> pcs->psshift;
+             *config = data;
+             return 0;
+             break;

Hmm, only slightly related to this patch, mostly a generic
question to others: Do others have any mux registers with
status bits for things like PIN_CONFIG_POWER_SOURCE?

I could use PIN_CONFIG_POWER_SOURCE for controlling the PBIAS
for omap MMC. But there's also a status bit that needs to be
checked for that. I think there was some other similar mux
register for USB PHY that has a status register.

So I'm wondering should the checking for status bit be handled
in the pinctrl consume driver? Or should we have some bindings
for that?

Do you mean that the status register only exists in USB PHY controller or
MMC controller?

If so, could we use regulator framework in USB PHY or MMC driver?

Regards
Haojian