Op 1 sep. 2012 om 09:01 heeft "AnilKumar, Chimata" <anilkumar@xxxxxx> het volgende geschreven: > Hi Koen, > > On Fri, Aug 31, 2012 at 21:23:18, Koen Kooi wrote: >> >> Op 30 aug. 2012, om 22:35 heeft Tony Lindgren <tony@xxxxxxxxxxx> het volgende geschreven: >> >>> * AnilKumar Ch <anilkumar@xxxxxx> [120828 01:11]: >>>> Adds GPIO pinctrl nodes to am3358_pinmux master node to control >>>> user leds (USR0, USR1, USR2 and USR3) present on BeagleBone. >>>> >>>> Signed-off-by: AnilKumar Ch <anilkumar@xxxxxx> >>>> --- >>>> arch/arm/boot/dts/am335x-bone.dts | 14 ++++++++++++++ >>>> 1 file changed, 14 insertions(+) >>>> >>>> diff --git a/arch/arm/boot/dts/am335x-bone.dts b/arch/arm/boot/dts/am335x-bone.dts >>>> index a7906cb..58f5042 100644 >>>> --- a/arch/arm/boot/dts/am335x-bone.dts >>>> +++ b/arch/arm/boot/dts/am335x-bone.dts >>>> @@ -18,6 +18,20 @@ >>>> reg = <0x80000000 0x10000000>; /* 256 MB */ >>>> }; >>>> >>>> + am3358_pinmux: pinmux@44E10800 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&userled_pins>; >>>> + >>>> + userled_pins: pinmux_userled_pins { >>>> + pinctrl-single,pins = < >>>> + 0x54 0x7 /* gpmc_a5.gpio1_21, OUTPUT | MODE7 */ >>>> + 0x58 0x17 /* gpmc_a6.gpio1_22, OUTPUT_PULLUP | MODE7 */ >>>> + 0x5C 0x7 /* gpmc_a7.gpio1_23, OUTPUT | MODE7 */ >>>> + 0x60 0x17 /* gpmc_a8.gpio1_24, OUTPUT_PULLUP | MODE7 */ >>>> + >; >>>> + }; >>>> + }; >>>> + >>> >>> Looks like this patch should also claim these pins by the led driver. >>> Then the led driver should just do pinctrl_get_select_default(&pdev->dev) >>> in it's probe function to set the pins. >> >> FWIW, I've been using this for a while now: >> >> + leds { >> + compatible = "gpio-leds"; >> + heartbeat { >> + label = "beaglebone::usr0"; >> + gpios = <&gpio2 21 0>; >> + linux,default-trigger = "heartbeat"; >> + }; >> + >> + mmc { >> + label = "beaglebone:usr1"; >> + gpios = <&gpio2 22 0>; >> + linux,default-trigger = "mmc0"; >> + }; > > Thanks for the inputs, similar data but not exact is added to > v5 series. > > + gpio-leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&userled_pins>; > + > + led0 { > + label = "status:green:user0"; > + gpios = <&gpio2 21 0>; > + default-state = "off"; > + }; > + > + led1 { > + label = "status:green:user1"; > + gpios = <&gpio2 22 0>; > + default-state = "off"; > + }; > + > + led2 { > + label = "status:green:user2"; > + gpios = <&gpio2 23 0>; > + default-state = "off"; > + }; > + > + led3 { > + label = "status:green:user3"; > + gpios = <&gpio2 24 0>; > + default-state = "off"; > + }; > + }; > + > > Can you review my v5 patches and tell me if there are any > modifications/changes required? It would be nice to keep de heartbeat and mmc triggers like we have in the 3.2 kernel that ships with the bone > > Thanks > AnilKumar -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html