On 07/04/13 05:25, Tobias Diedrich wrote:
++static struct of_gpio_flag of_gpio_flags[] = {
++      { "in", GPIOF_DIR_IN },
++      { "out_init_low", GPIOF_OUT_INIT_LOW },
++      { "out_init_high", GPIOF_OUT_INIT_HIGH },
++      { "open_drain", GPIOF_OPEN_DRAIN },
++      { "open_source", GPIOF_OPEN_SOURCE },
++      { "active_low", GPIOF_ACTIVE_LOW },
++      { "changeable", GPIOF_EXPORT_CHANGEABLE },
++      { /* sentinel */ }


this seems overkill. the opendrain is defined via pinctrl. all other options are defianble via the existing API afaict.

active low should be set via the gpio flags

gpios = &<&gpio0 2 1>   <--- will use gpio 2 as active_low

changeable == pio-export,direction_may_change


if you look here -> you see the available options -> https://dev.openwrt.org/browser/trunk/target/linux/ramips/patches-3.8/0209-owrt-GPIO-add-gpio_export_with_name.patch

correct me if i am wrong but i dont htink that big patch is needed to make your use case work

        John
_______________________________________________
openwrt-devel mailing list
[email protected]
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Reply via email to