On 13 May 2015 at 14:02, Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> wrote: > 2015-05-13 17:21 GMT+09:00 Anand Moon <linux.amoon@xxxxxxxxx>: >> On 13 May 2015 at 12:51, Krzysztof Kozlowski <k.kozlowski@xxxxxxxxxxx> wrote: >>> 2015-05-13 15:36 GMT+09:00 Anand Moon <linux.amoon@xxxxxxxxx>: >>>> This changes enables TMU IP block on the Exynos5422 Odroid-XU3 >>>> device. >>>> >>>> Tested-by: Markus Reichl <m.reichl@xxxxxxxxxxxxx> >>>> Acked-by: Lukasz Majewski <l.majewski@xxxxxxxxxxx> >>>> Signed-off-by: Anand Moon <linux.amoon@xxxxxxxxx> >>>> --- >>>> arch/arm/boot/dts/exynos5422-odroidxu3.dts | 25 +++++++++++++++++++++++++ >>>> 1 file changed, 25 insertions(+) >>>> >>>> diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3.dts b/arch/arm/boot/dts/exynos5422-odroidxu3.dts >>>> index 9446e28..cd78816 100644 >>>> --- a/arch/arm/boot/dts/exynos5422-odroidxu3.dts >>>> +++ b/arch/arm/boot/dts/exynos5422-odroidxu3.dts >>>> @@ -319,6 +319,31 @@ >>>> #cooling-cells = <2>; >>>> cooling-levels = <0 130 170 230>; >>>> }; >>>> + >>>> + tmu@10060000 { >>> >>> Here and for other overrides please use label notation, like: >>> >>> &tmu_cpu0 { >>> ... >>> }; >>> >>>> + vtmu-supply = <&ldo10_reg>; >>> >>> I am curious, how did you find that LDO10 supplies TMU unit? >>> >>> Best regards, >>> Krzysztof >> >> Hi Krzysztof, >> >> I have re-base my work on earlier Lukasz Majewski patches. >> >> https://patchwork.kernel.org/patch/5693201/ > > NAK. > I am sorry, but this is not sufficient explanation. Actually such > explanation could mean that you did just blindly copied everything > instead of developing it. > > You cannot use some regulator here just because some Exynos4 boards > use it. You have to be sure that this regulator supplies this part of > SoC or device. Hi Krzysztof, After going through the schematics, I came to understanding that their is missing regulator related to TEMP SE in the exynos5422-odroidxu3.dts. Below is the schematic of the board. http://dn.odroid.com/5422/ODROID-XU3/Schematics/XU3_MAIN_REV0.2.PDF ldo18_reg: LDO18 { regulator-name = "vdd_ldo18"; regulator-min-microvolt = <1800000>; regulator-max-microvolt = <1800000>; regulator-always-on; }; Please share your thought on this. -Anand Moon -- 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