On Wed, Nov 20, 2013 at 5:14 PM, boris brezillon <b.brezillon@xxxxxxxxxxx> wrote: > On 20/11/2013 15:59, Jean-Christophe PLAGNIOL-VILLARD wrote: >> On 13:06 Wed 28 Aug , Boris BREZILLON wrote: >>> mmc0_slot0_switch-0 { >>> + atmel,pins = >>> + <AT91_PIOB 22 >>> AT91_PERIPH_GPIO (AT91_PINCTRL_OUTPUT | AT91_PINCTRL_OUTPUT_VAL(1))>; >>> + }; >> >> nack this is a regulator the pinctrl API is not done for gpio default >> value > > I know this can be represented as a regulator, but looking at the datasheet, > this looks more like a switch than a regulator. No matter whether it's a switch or a GPIO regulator it seems we are in violent agreement that it should not be controlled by the pin control states at least. Start with making it a GPIO then you can figure out whether a GPIO regulator or drivers/extcon/extcon-gpio.c should be used. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html