Hi, On Wed, Nov 27, 2013 at 8:34 PM, Leela Krishna Amudala <[email protected]> wrote: > This patch adds pmusysreg node to exynos5250 and exynos5420 dtsi files to > handle PMU register accesses in a centralized way using syscon driver > > Signed-off-by: Leela Krishna Amudala <[email protected]> > Reviewed-by: Tomasz Figa <[email protected]> > Reviewed-by: Doug Anderson <[email protected]> > Tested-by: Doug Anderson <[email protected]> > --- > Documentation/devicetree/bindings/arm/samsung/pmu.txt | 16 ++++++++++++++++ > arch/arm/boot/dts/exynos5250.dtsi | 5 +++++ > arch/arm/boot/dts/exynos5420.dtsi | 5 +++++ > 3 files changed, 26 insertions(+) > create mode 100644 Documentation/devicetree/bindings/arm/samsung/pmu.txt > > diff --git a/Documentation/devicetree/bindings/arm/samsung/pmu.txt > b/Documentation/devicetree/bindings/arm/samsung/pmu.txt > new file mode 100644 > index 0000000..307e727 > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/samsung/pmu.txt > @@ -0,0 +1,16 @@ > +SAMSUNG Exynos SoC series PMU Registers > + > +Properties: > + - name : should be 'syscon';
It's common to use a non-abbreviated name, such as 'system-controller' here. Or, given that it's referring to "pmusysreg" then maybe something like "pmu-system-registers". > + - compatible : should contain two values. First value must be one from > following list: > + - "samsung,exynos5250-pmu" - for Exynos5250 SoC, > + - "samsung,exynos5420-pmu" - for Exynos5420 SoC. > + second value must be always "syscon". -Olof -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
