From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753457AbbLJHZK (ORCPT ); Thu, 10 Dec 2015 02:25:10 -0500 Received: from mailout4.samsung.com ([203.254.224.34]:57452 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753268AbbLJHZG (ORCPT ); Thu, 10 Dec 2015 02:25:06 -0500 MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 X-AuditID: cbfee691-f79766d0000012b6-0f-566928c859e3 Content-transfer-encoding: 8BIT Message-id: <566928C8.5080109@samsung.com> Date: Thu, 10 Dec 2015 16:24:56 +0900 From: Chanwoo Choi User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: Krzysztof Kozlowski , myungjoo.ham@samsung.com, kgene@kernel.org 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 Subject: Re: [PATCH v2 17/19] ARM: dts: Add support of bus frequency using VDD_INT for exynos3250-rinato References: <1449634091-1842-1-git-send-email-cw00.choi@samsung.com> <1449634091-1842-18-git-send-email-cw00.choi@samsung.com> <566922A4.5090807@samsung.com> In-reply-to: <566922A4.5090807@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrPIsWRmVeSWpSXmKPExsWyRsSkSPeERmaYwYNmI4v5R86xWvS/Wchq ce7VSkaL1y8MLfofv2a2ONv0ht3i8q45bBafe48wWsw4v4/JYt3GW+wWty/zWiy9fpHJ4nbj CjaLCdPXsli07j3CbtG2+gOrg4DHmnlrGD1amnvYPC739TJ57Jx1l91j5fIvbB6bVnWyefw7 xu7Rt2UVo8fnTXIBnFFcNimpOZllqUX6dglcGQfvPGcvWMlWsWvhLqYGxo8sXYwcHBICJhIH pqV3MXICmWISF+6tZ+ti5OIQEljBKPF4w1p2iISJxLq3PcwQiVmMEl+3b2QDSfAKCEr8mHwP bBCzgLzEkUvZIGFmAXWJSfMWQdU/YJS4+foeK0gNr4CWxKIVSSA1LAKqEssPPWUCsdmAwvtf 3GADKREViJDoPlEJEhYRiJfY9OIq2D3MAneZJC6+fwK2VlggS+Ld31+sEPMXMkrcmnoALMEp oC2xbM8EsA4JgR0cEjPmT2KH2CYg8W3yIaiPZSU2HWCGeExS4uCKGywTGMVmIXlnFsI7s5C8 s4CReRWjaGpBckFxUnqRqV5xYm5xaV66XnJ+7iZGYLyf/vds4g7G+wesDzEKcDAq8fC+kM4M E2JNLCuuzD3EaAp0xERmKdHkfGBSySuJNzQ2M7IwNTE1NjK3NFMS59WR/hksJJCeWJKanZpa kFoUX1Sak1p8iJGJg1OqgfE8i+UOpj+dnLV/c5bFGxgKSPdqV/M+ePzs26vNvp5n6kKuCTi7 TLvNmXyG31/Q8ci0Yj3HWY7hnYvvKAkyaX+2Vru44OBFSe5f884rfdsfXLzTqjI69FXEnKpp jzOkFxl4LF5VNv3gn3CuXpmJpunPpxW62l7L+LFerl1ZQvOtgfy66O6DSkosxRmJhlrMRcWJ AEfg+6ryAgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrLKsWRmVeSWpSXmKPExsVy+t9jQd0TGplhBm/vaFvMP3KO1aL/zUJW i3OvVjJavH5haNH/+DWzxdmmN+wWl3fNYbP43HuE0WLG+X1MFus23mK3uH2Z12Lp9YtMFrcb V7BZTJi+lsWide8Rdou21R9YHQQ81sxbw+jR0tzD5nG5r5fJY+esu+weK5d/YfPYtKqTzePf MXaPvi2rGD0+b5IL4IxqYLTJSE1MSS1SSM1Lzk/JzEu3VfIOjneONzUzMNQ1tLQwV1LIS8xN tVVy8QnQdcvMAfpCSaEsMacUKBSQWFyspG+HaUJoiJuuBUxjhK5vSBBcj5EBGkhYw5hx8M5z 9oKVbBW7Fu5iamD8yNLFyMkhIWAise5tDzOELSZx4d56ti5GLg4hgVmMEl+3b2QDSfAKCEr8 mHwPqIGDg1lAXuLIpWyQMLOAusSkeYuYIeofMErcfH2PFaSGV0BLYtGKJJAaFgFVieWHnjKB 2GxA4f0vbrCBlIgKREh0n6gECYsIxEtsenEVbC2zwF0miYvvn4CtFRbIknj39xcrxPyFjBK3 ph4AS3AKaEss2zOBbQIj0JUI581COG8WkvMWMDKvYpRILUguKE5KzzXMSy3XK07MLS7NS9dL zs/dxAhOKs+kdjAe3OV+iFGAg1GJh/eFdGaYEGtiWXFl7iFGCQ5mJRHeoz8zwoR4UxIrq1KL 8uOLSnNSiw8xmgI9OJFZSjQ5H5jw8kriDY1NzIwsjcwNLYyMzZXEeWsvRYYJCaQnlqRmp6YW pBbB9DFxcEo1MAb9O5XeuHvR3MVz2889fCiY/Gn9tG3/P35QrfpwR+F8tc/+Nz6H63n7Nu3K DFvC9WJVZ6v00QPHr7UZZ22Y9iJe61oBn2n3X14d9pRE1rbeCRwnnthKPOuYrabKLXBKpOtO 3dZft0Vm5KnGd55xU+FOPeDbc/9+jtttcbOjv3983xYitTLh0WElluKMREMt5qLiRAAM1HFi QAMAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2015년 12월 10일 15:58, Krzysztof Kozlowski wrote: > On 09.12.2015 13:08, Chanwoo Choi wrote: >> This patch adds the bus device-tree node of INT (internal) block >> to enable the bus frequency. The following sub-blocks share > > "to enable the bus frequency scaling" > >> the VDD_INT power source: >> - LEFTBUS (parent device) >> - RIGHTBUS >> - PERIL >> - LCD0 >> - FSYS >> - MCUISP / ISP >> - MFC >> >> The LEFTBUS is parent device with devfreq ondemand governor >> and the rest devices has the dependency on LEFTBUS bus. > > How about: > "and the rest of devices depend on the LEFTBUS device." > ? OK, I'll modify it. > > The patch is good: > > Reviewed-by: Krzysztof Kozlowski Thanks for your review. Best Regards, Chanwoo Choi