From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934100AbcLOEdd (ORCPT ); Wed, 14 Dec 2016 23:33:33 -0500 Received: from mailout1.samsung.com ([203.254.224.24]:55572 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932150AbcLOEda (ORCPT ); Wed, 14 Dec 2016 23:33:30 -0500 MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 X-AuditID: cbfee61b-f796f6d000004092-48-58521d0fbf0c Content-transfer-encoding: 8BIT Message-id: <58521D0F.5040704@samsung.com> Date: Thu, 15 Dec 2016 13:33:19 +0900 From: Chanwoo Choi Organization: Samsung Electronics User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 To: krzk@kernel.org, javier@osg.samsung.com, kgene@kernel.org, robh+dt@kernel.org, s.nawrocki@samsung.com, tomasz.figa@gmail.com Cc: myungjoo.ham@samsung.com, kyungmin.park@samsung.com, devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Michael Turquette , Stephen Boyd Subject: Re: [PATCH v2 1/5] clk: samsung: exynos5433: Set NoC (Network On Chip) clocks as critical References: <1481173091-9728-1-git-send-email-cw00.choi@samsung.com> <1481173091-9728-2-git-send-email-cw00.choi@samsung.com> In-reply-to: <1481173091-9728-2-git-send-email-cw00.choi@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrPIsWRmVeSWpSXmKPExsVy+t9jAV1+2aAIgytrdS3mHznHavHm7Rom i/7Hr5ktzp/fwG5xtukNu8Wmx9dYLS7vmsNmMeP8PiaLi6dcLW43rmCzaN17hN3i8Jt2Vosf Z7pZLFbt+sPowOfx/kYru8flvl4mj52z7rJ7bFrVyeaxeUm9x5Z+IK9vyypGj8+b5AI4otxs MlITU1KLFFLzkvNTMvPSbZVCQ9x0LZQU8hJzU22VInR9Q4KUFMoSc0qBPCMDNODgHOAerKRv l+CWseHlBeaCx6IV2yd0MTYwThTqYuTkkBAwkTh2YioLhC0mceHeerYuRi4OIYGljBLTWqex gyR4BQQlfky+B1TEwcEsIC9x5FI2SJhZQF1i0rxFzCC2kMADRolzJ1ghyrUk+s5NZwSxWQRU JWasOcoGYrMBxfe/uAFm8wsoSlz98ZgRZKSoQIRE94lKkLUiAk2MEg0X5jKCOMwCU5gkrq7Z wwZSJCyQIvHjlwjEbZeBdj24wwQyiFPAVeLdrU2sExgFZyE5dRbCqbOQnLqAkXkVo0RqQXJB cVJ6rlFearlecWJucWleul5yfu4mRnBkP5PewXh4l/shRgEORiUe3oSAwAgh1sSy4srcQ4wS HMxKIrx2EkERQrwpiZVVqUX58UWlOanFhxhNgZ6dyCwlmpwPTDp5JfGGJuYm5sYGFuaWliZG SuK8jbOfhQsJpCeWpGanphakFsH0MXFwSjUwzqnPKDpsksqRV1ifeb1xo/SWRcuuznaWc56Q LaCTJrp3t5m09f4fWYev9cTLK30U7Xyi1Wrwc8070YiUVZ0SjA8t9Yzv3BRK5PuVIqjzeo/a pz2hW3ZaZ/5jEpm5NL07xaTu/slFf2RMqqX2fxT4IPclZYnUk14VP/4lCgo7755m6/dq925X YinOSDTUYi4qTgQA+/p6XQIDAAA= X-MTR: 20000000000000000@CPGS Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dear Sylwester, Could you please review this patch? -- Regards, Chanwoo Choi On 2016년 12월 08일 13:58, Chanwoo Choi wrote: > The ACLK_BUS0/1/2 are used for NoC (Network on Chip). If NoC's clocks are > disabled, the system halt happen. Following clock must be always enabled. > - CLK_ACLK_BUS0_400 : NoC's bus clock for PERIC/PERIS/FSYS/MSCL > - CLK_ACLK_BUS1_400 : NoC's bus clock for MFC/HEVC/G3D > - CLK_ACLK_BUS2_400 : NoC's bus clock for GSCL/DISP/G2D/CAM0/CAM1/ISP > > Also, this patch adds the CLK_SET_RATE_PARENT flag to the CLK_SCLK_JPEG_MSCL > because this clock should be used for bus frequency scaling. This clock need to > be changed on the fly with CLK_SET_RATE_PARENT flag. > > Cc: Sylwester Nawrocki > Cc: Tomasz Figa > Cc: Chanwoo Choi > Cc: Michael Turquette > Cc: Stephen Boyd > Cc:linux-clk@vger.kernel.org > Signed-off-by: Chanwoo Choi > --- > drivers/clk/samsung/clk-exynos5433.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/clk/samsung/clk-exynos5433.c b/drivers/clk/samsung/clk-exynos5433.c > index f096bd7df40c..0db5204c307c 100644 > --- a/drivers/clk/samsung/clk-exynos5433.c > +++ b/drivers/clk/samsung/clk-exynos5433.c > @@ -549,10 +549,10 @@ > 29, CLK_IGNORE_UNUSED, 0), > GATE(CLK_ACLK_BUS0_400, "aclk_bus0_400", "div_aclk_bus0_400", > ENABLE_ACLK_TOP, 26, > - CLK_IGNORE_UNUSED | CLK_SET_RATE_PARENT, 0), > + CLK_IS_CRITICAL | CLK_SET_RATE_PARENT, 0), > GATE(CLK_ACLK_BUS1_400, "aclk_bus1_400", "div_aclk_bus1_400", > ENABLE_ACLK_TOP, 25, > - CLK_IGNORE_UNUSED | CLK_SET_RATE_PARENT, 0), > + CLK_IS_CRITICAL | CLK_SET_RATE_PARENT, 0), > GATE(CLK_ACLK_IMEM_200, "aclk_imem_200", "div_aclk_imem_266", > ENABLE_ACLK_TOP, 24, > CLK_IS_CRITICAL | CLK_SET_RATE_PARENT, 0), > @@ -616,7 +616,7 @@ > > /* ENABLE_SCLK_TOP_MSCL */ > GATE(CLK_SCLK_JPEG_MSCL, "sclk_jpeg_mscl", "div_sclk_jpeg", > - ENABLE_SCLK_TOP_MSCL, 0, 0, 0), > + ENABLE_SCLK_TOP_MSCL, 0, CLK_SET_RATE_PARENT, 0), > > /* ENABLE_SCLK_TOP_CAM1 */ > GATE(CLK_SCLK_ISP_SENSOR2, "sclk_isp_sensor2", "div_sclk_isp_sensor2_b", > @@ -1382,7 +1382,7 @@ static void __init exynos5433_cmu_cpif_init(struct device_node *np) > /* ENABLE_ACLK_MIF3 */ > GATE(CLK_ACLK_BUS2_400, "aclk_bus2_400", "div_aclk_bus2_400", > ENABLE_ACLK_MIF3, 4, > - CLK_IGNORE_UNUSED | CLK_SET_RATE_PARENT, 0), > + CLK_IS_CRITICAL | CLK_SET_RATE_PARENT, 0), > GATE(CLK_ACLK_DISP_333, "aclk_disp_333", "div_aclk_disp_333", > ENABLE_ACLK_MIF3, 1, > CLK_IS_CRITICAL | CLK_SET_RATE_PARENT, 0), >