From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751936AbbLJAxR (ORCPT ); Wed, 9 Dec 2015 19:53:17 -0500 Received: from mailout4.w1.samsung.com ([210.118.77.14]:18623 "EHLO mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751084AbbLJAxP (ORCPT ); Wed, 9 Dec 2015 19:53:15 -0500 X-AuditID: cbfec7f4-f79026d00000418a-ba-5668ccf786c5 Subject: Re: [PATCH v2 04/19] ARM: dts: Add DMC bus frequency for exynos3250-rinato/monk To: Chanwoo Choi , myungjoo.ham@samsung.com, kgene@kernel.org References: <1449634091-1842-1-git-send-email-cw00.choi@samsung.com> <1449634091-1842-5-git-send-email-cw00.choi@samsung.com> Cc: kyungmin.park@samsung.com, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, linux@arm.linux.org.uk, tjakobi@math.uni-bielefeld.de, linux.amoon@gmail.com, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org From: Krzysztof Kozlowski Message-id: <5668CCF1.8080807@samsung.com> Date: Thu, 10 Dec 2015 09:53:05 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-version: 1.0 In-reply-to: <1449634091-1842-5-git-send-email-cw00.choi@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprNIsWRmVeSWpSXmKPExsVy+t/xq7rfz2SEGczulre4/uU5q8X8I+dY LfrfLGS1OPdqJaPF6xeGFv2PXzNbnG16w25xedccNovPvUcYLWac38dksW7jLXaL25d5LZZe v8hkcbtxBZvFhOlrWSxa9x5ht2hb/YHVQdBjzbw1jB4tzT1sHpf7epk8ds66y+6xcvkXNo9N qzrZPP4dY/fo27KK0ePzJrkAzigum5TUnMyy1CJ9uwSujMdfPjEVrOaruHIzvYHxBXcXIyeH hICJxMNLO9khbDGJC/fWs3UxcnEICSxllHi94gwrSEJI4CmjxJ/tViC2sECUxNzGiSwgtohA mMTsGT+YIWrqJRbf62EGaWYWuMskcfH9EzaQBJuAscTm5UvAbF4BLYnbP/8wgtgsAqoS685M AWrg4BAViJBYtCMTokRQ4sfke2DzOQVcJRp2LGEHKWEW0JO4f1ELJMwsIC+xec1b5gmMArOQ dMxCqJqFpGoBI/MqRtHU0uSC4qT0XEO94sTc4tK8dL3k/NxNjJAo+7KDcfExq0OMAhyMSjy8 F1zSw4RYE8uKK3MPMUpwMCuJ8OqczggT4k1JrKxKLcqPLyrNSS0+xCjNwaIkzjt31/sQIYH0 xJLU7NTUgtQimCwTB6dUA2NwvtHrdXb355wN+1nweaPE4b/Tkzet5vh9z1tCZo3qJr1wz9uH bmzm7+2NfGxsa+tcsXd3vegO4+Ilt/M1zD81njqwdHLOFeNJezmWPRXfk7YzXfmrrqaFcdbx PpO29yL1x6f7Hl+pw2qfJcOqU9q3T0D1aZjvz4ynAeE/U650sTa3FwocXaPEUpyRaKjFXFSc CAC3wUOvrgIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09.12.2015 13:07, Chanwoo Choi wrote: > This patch adds the DMC (Dynamic Memory Controller) bus frequency node > which includes the devfreq-events and regulator properties. The bus > frequency support the DVFS (Dynamic Voltage Frequency Scaling) feature > with ondemand governor. > > The devfreq-events (ppmu_dmc0*) can monitor the utilization of DMC bus > on runtime and the buck1_reg (VDD_MIF power line) supplies the power to > the DMC block. > > Signed-off-by: Chanwoo Choi > --- > arch/arm/boot/dts/exynos3250-monk.dts | 6 ++++++ > arch/arm/boot/dts/exynos3250-rinato.dts | 6 ++++++ > 2 files changed, 12 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos3250-monk.dts b/arch/arm/boot/dts/exynos3250-monk.dts > index 443a35085846..d982586a6533 100644 > --- a/arch/arm/boot/dts/exynos3250-monk.dts > +++ b/arch/arm/boot/dts/exynos3250-monk.dts > @@ -498,6 +498,12 @@ > }; > }; > > +&bus_dmc { > + devfreq-events = <&ppmu_dmc0_3>, <&ppmu_dmc1_3>; > + vdd-supply = <&buck1_reg>; > + status = "okay"; > +}; > + > &xusbxti { > clock-frequency = <24000000>; > }; > diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts > index 3e64d5dcdd60..61477943015b 100644 > --- a/arch/arm/boot/dts/exynos3250-rinato.dts > +++ b/arch/arm/boot/dts/exynos3250-rinato.dts > @@ -675,6 +675,12 @@ > }; > }; > > +&bus_dmc { > + devfreq-events = <&ppmu_dmc0_3>, <&ppmu_dmc1_3>; > + vdd-supply = <&buck1_reg>; > + status = "okay"; > +}; I would prefer to put this in alphabetical order... which could be tricky because the nodes are not entirely sorted. Maybe after the "&adc" node? Anyway the change looks good: Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof