From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1755292AbcCNIf0 (ORCPT <rfc822;w@1wt.eu>);
	Mon, 14 Mar 2016 04:35:26 -0400
Received: from mailout4.w1.samsung.com ([210.118.77.14]:63622 "EHLO
	mailout4.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1755248AbcCNIfH (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Mon, 14 Mar 2016 04:35:07 -0400
X-AuditID: cbfec7f4-f79026d00000418a-b0-56e677b7db99
Subject: Re: [PATCH 10/10] ARM: dts: Add PPMU node for exynos3250-artik5 module
To: Chanwoo Choi <cw00.choi@samsung.com>, kgene@kernel.org,
        s.nawrocki@samsung.com, tomasz.figa@gmail.com
References: <1457921056-21257-1-git-send-email-cw00.choi@samsung.com>
 <1457921056-21257-12-git-send-email-cw00.choi@samsung.com>
Cc: jh80.chung@samsung.com, andi.shyti@samsung.com, inki.dae@samsung.com,
        sw0312.kim@samsung.com, pankaj.dubey@samsung.com,
        linux-samsung-soc@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        devicetree@vger.kernel.org
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Message-id: <56E677B2.3080304@samsung.com>
Date: Mon, 14 Mar 2016 17:34:58 +0900
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.5.1
MIME-version: 1.0
In-reply-to: <1457921056-21257-12-git-send-email-cw00.choi@samsung.com>
Content-type: text/plain; charset=windows-1252
Content-transfer-encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrLLMWRmVeSWpSXmKPExsVy+t/xK7rby5+FGTy6bmix/cgzVovrX56z
	Wsw/co7VYtL9CSwWN361sVq8fmFo0f/4NbPFpsfXWC0u75rDZjHj/D4mi0Vbv7BbHH7Tzmox
	Y/JLNotVu/4wOvB57Jx1l91j06pONo/NS+o9+rasYvT4vEkugDWKyyYlNSezLLVI3y6BK+PO
	sgnMBS94Ks4dW8TYwNjL1cXIySEhYCLRvKOTHcIWk7hwbz1bFyMXh5DAUkaJPVffMEE4Txkl
	nk86xwpSJSzgL9F2bAYjiC0ikC0xY8lMqI4mRolfaxYzgzjMAs1MEnfXrGICqWITMJbYvHwJ
	G4jNK6Alsf35OrB9LAKqEh+v3QGzRQUiJA53drFD1AhK/Jh8jwXE5hRwl1h14SzQNg6goXoS
	9y9qgYSZBeQlNq95yzyBUWAWko5ZCFWzkFQtYGRexSiaWppcUJyUnmuoV5yYW1yal66XnJ+7
	iRESKV92MC4+ZnWIUYCDUYmH98eap2FCrIllxZW5hxglOJiVRHh9HZ+FCfGmJFZWpRblxxeV
	5qQWH2KU5mBREuedu+t9iJBAemJJanZqakFqEUyWiYNTqoFx7e+n/gwMy2K6cmVNtxs/zLBw
	3ZU2x9gg+7CH5Yunz6Kz2EyvByUZXXtc2W/216vPzTJhwyy1g9/ld3xMfj5PikFtyhL1Xcyl
	Pb2L7ni3LY3cpPbz4+HvczqySkzeKtU+nZA/W+Xootc3lT5dfHb+ePeZ+1388xZN6O9dn77P
	y+K2uez54P/ySizFGYmGWsxFxYkA5iBn6JACAAA=
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 14.03.2016 11:04, Chanwoo Choi wrote:
> This patch adds PPMU (Platform Performance Monitoring Unit) Device Tree node
> for Exynos3250 based ARTIK5 module. The PPMU node is used to get the
> utilization of DMC0/DMC1/LEFTBUS/RIGHTBUS Block.
> 
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---
>  arch/arm/boot/dts/exynos3250-artik5.dtsi | 36 ++++++++++++++++++++++++++++++++
>  1 file changed, 36 insertions(+)

What is the status with your devfreq patches? Instead of this one, there
should be just:
	#include "exynos4412-ppmu-common.dtsi"
(squashed into patch 5/10).

Best regards,
Krzysztof

> diff --git a/arch/arm/boot/dts/exynos3250-artik5.dtsi b/arch/arm/boot/dts/exynos3250-artik5.dtsi
> index 673883fda480..995356708b7d 100644
> --- a/arch/arm/boot/dts/exynos3250-artik5.dtsi
> +++ b/arch/arm/boot/dts/exynos3250-artik5.dtsi
> @@ -321,6 +321,42 @@
>  	status = "okay";
>  };
>  
> +&ppmu_dmc0 {
> +	status = "okay";
> +	events {
> +		ppmu_dmc0_3: ppmu-event3-dmc0 {
> +			event-name = "ppmu-event3-dmc0";
> +		};
> +	};
> +};
> +
> +&ppmu_dmc1 {
> +	status = "okay";
> +	events {
> +		ppmu_dmc1_3: ppmu-event3-dmc1 {
> +			event-name = "ppmu-event3-dmc1";
> +		};
> +	};
> +};
> +
> +&ppmu_leftbus {
> +	status = "okay";
> +	events {
> +		ppmu_leftbus_3: ppmu-event3-leftbus {
> +			event-name = "ppmu-event3-leftbus";
> +		};
> +	};
> +};
> +
> +&ppmu_rightbus {
> +	status = "okay";
> +	events {
> +		ppmu_rightbus_3: ppmu-event3-rightbus {
> +			event-name = "ppmu-event3-rightbus";
> +		};
> +	};
> +};
> +
>  &tmu {
>  	status = "okay";
>  };
>