Hi Abhilash, Please see my comments inline. Also CCing some DT maintainers. On 15.07.2014 20:34, Abhilash Kesavan wrote: > PPMU is required by the exynos5250 devfreq driver. Add a device > tree node for it. > > Signed-off-by: Abhilash Kesavan <a.kesavan@xxxxxxxxxxx> > --- > .../bindings/arm/exynos/ppmu-exynos5.txt | 26 ++++++++++++++++++++ > arch/arm/boot/dts/exynos5250.dtsi | 8 ++++++ > 2 files changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/arm/exynos/ppmu-exynos5.txt > > diff --git a/Documentation/devicetree/bindings/arm/exynos/ppmu-exynos5.txt b/Documentation/devicetree/bindings/arm/exynos/ppmu-exynos5.txt > new file mode 100644 > index 0000000..a6a2eba > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/exynos/ppmu-exynos5.txt > @@ -0,0 +1,26 @@ > +Exynos PPMU driver > +------------------- > + > +Performance events are primitive values used to get performance data. These > +events provide information about the behavior of the SoC that can be used > +when analyzing system performance. These events are made visible using the > +PPMU logic. > +Exynos PPMU driver is used by the exynos5 devfreq drivers to control the > +bus frequency/voltage. > + > +Required properties: > +- compatible: "samsung,exynos5250-int-busfreq", "samsung,exynos5420-int-busfreq" If this is a binding for PPMU, shouldn't the compatible string contain the "ppmu" string too? E.g. "samsung,exynos5250-ppmu", "samsung,exynos5420-ppmu". > +- reg: > + * physical base address of the PPMUs (e.g DDR, Right Bus, Left bus etc) > + and length of memory mapped region. Since PPMU are separate IP blocks, they should have their own device nodes with only address ranges of their own. Representing all PPMUs in the SoC with a single node is incorrect. Best regards, Tomasz -- 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