On Tue, Oct 3, 2017 at 4:42 AM, Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx> wrote: > On Mon, Oct 02, 2017 at 12:08:54PM +0000, Quentin Schulz wrote: >> On AXP813/818, GPIO0 and GPIO1 can be used as LDO as (respectively) >> ldo_io0 and ldo_io1. >> >> Let's add the pinctrl properties to the said regulators. >> >> Signed-off-by: Quentin Schulz <quentin.schulz@xxxxxxxxxxxxxxxxxx> >> --- >> arch/arm/boot/dts/axp81x.dtsi | 14 ++++++++++++++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/arch/arm/boot/dts/axp81x.dtsi b/arch/arm/boot/dts/axp81x.dtsi >> index f90f257130d5..099b0ddc1bbb 100644 >> --- a/arch/arm/boot/dts/axp81x.dtsi >> +++ b/arch/arm/boot/dts/axp81x.dtsi >> @@ -52,6 +52,16 @@ >> compatible = "x-powers,axp813-gpio"; >> gpio-controller; >> #gpio-cells = <2>; >> + >> + gpio0_ldo: gpio0_ldo { >> + pins = "GPIO0"; >> + function = "ldo"; >> + }; >> + >> + gpio1_ldo: gpio1_ldo { >> + pins = "GPIO1"; >> + function = "ldo"; >> + }; > > The node names are not supposed to contain any hyphens. Hmm, I was under the impression that hyphens were preferred in node names, and a warning would be added to dtc later on. I might be wrong though. ChenYu -- 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