Hi Sachin, > Hi Lukasz, > > Thanks for adding this. Some comments inline. > > On 19 April 2013 22:08, Lukasz Majewski <l.majewski@xxxxxxxxxxx> > wrote: > > Proper description for Exynos4 bindings added to > > Documentation/devicetree/ bindings > > > > Signed-off-by: Lukasz Majewski <l.majewski@xxxxxxxxxxx> > > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > > --- > > .../devicetree/bindings/thermal/exynos-thermal.txt | 22 > > ++++++++++++++++++++ 1 file changed, 22 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/thermal/exynos-thermal.txt > > > > diff --git > > a/Documentation/devicetree/bindings/thermal/exynos-thermal.txt > > b/Documentation/devicetree/bindings/thermal/exynos-thermal.txt new > > file mode 100644 index 0000000..e994e1e --- /dev/null > > +++ b/Documentation/devicetree/bindings/thermal/exynos-thermal.txt > > @@ -0,0 +1,22 @@ > > +* Exynos Thermal > > How about "Exynos Thermal Management Unit (TMU)"? > > > + > > +Required properties: > > +- compatible : "samsung,exynos4412-tmu" > Should be one of the following: > "samsung,exynos4210-tmu" > "samsung,exynos4412-tmu" So all thee names shall be added? Right? > "samsung,exynos5250-tmu" > > > > +- interrupts-parent : The phandle for the interrupt controller > > s/interrupts-parent /interrupt-parent > > > > +- reg : Address range of the thermal registers > > +- interrupts : Should contain interrupt for thermal system > > +- clocks : The main clock for TMU device > > +- clocks-names : Thermal system clock name > > s/clocks-names /clock-names > You may also choose to add "from common clock binding" for clocks and > clock-names properties above. > > > +- status : Initial state of the device > You may remove this as it is quite obvious now. Even if you want to > retain this, it should be under 'Optional properties:' > > Ok, I will correct those names. -- Best regards, Lukasz Majewski Samsung R&D Poland (SRPOL) | Linux Platform Group -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html