On Tuesday, April 28, 2015 05:23:56 PM Yingjoe Chen wrote: > From: Hongzhou Yang <hongzhou.yang@xxxxxxxxxxxx> > > > Matthias, > > Since mt8135 pinctrl node patch is not in v4.1-rc1, I'd like to make some > minor change to follow dts convention. > > Let me know what you think. > Thanks. > > Joe.C > > ---8<-------------------------------------------------------------------- > Add pinctrl,GPIO and EINT node to mt8135.dtsi. > > Signed-off-by: Yingjoe Chen <yingjoe.chen@xxxxxxxxxxxx> > Signed-off-by: Hongzhou Yang <hongzhou.yang@xxxxxxxxxxxx> > Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > arch/arm/boot/dts/mt8135-pinfunc.h | 1302 > ++++++++++++++++++++++++++++++++++++ arch/arm/boot/dts/mt8135.dtsi | > 31 + > 2 files changed, 1333 insertions(+) > create mode 100644 arch/arm/boot/dts/mt8135-pinfunc.h > > diff --git a/arch/arm/boot/dts/mt8135-pinfunc.h > b/arch/arm/boot/dts/mt8135-pinfunc.h new file mode 100644 > index 0000000..5a60987 > --- /dev/null > +++ b/arch/arm/boot/dts/mt8135-pinfunc.h [...] > diff --git a/arch/arm/boot/dts/mt8135.dtsi b/arch/arm/boot/dts/mt8135.dtsi > index 2c5c4da..d6c1f14 100644 > --- a/arch/arm/boot/dts/mt8135.dtsi > +++ b/arch/arm/boot/dts/mt8135.dtsi > @@ -17,6 +17,7 @@ > #include <dt-bindings/interrupt-controller/arm-gic.h> > #include <dt-bindings/reset-controller/mt8135-resets.h> > #include "skeleton64.dtsi" > +#include "mt8135-pinfunc.h" > > / { > compatible = "mediatek,mt8135"; > @@ -141,6 +142,36 @@ > clock-names = "spi", "wrap"; > }; > > + /* Provide regmap for pinctrl driver */ I think from the DTS it is quite clear for what the node is used for. Please delete the comment. > + syscfg_pctl_a: syscfg_pctl_a@10005000 { > + compatible = "mediatek,mt8135-pctl-a-syscfg", "syscon"; > + reg = <0 0x10005000 0 0x1000>; > + }; > + > + /* Provide regmap for pinctrl driver */ Same here. > + syscfg_pctl_b: syscfg_pctl_b@1020c000 { > + compatible = "mediatek,mt8135-pctl-b-syscfg", "syscon"; > + reg = <0 0x1020c000 0 0x1000>; > + }; Please sort the nodes by it's address. Cheers, Matthias -- 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