From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752246AbcLGL2v (ORCPT ); Wed, 7 Dec 2016 06:28:51 -0500 Received: from mailout1.samsung.com ([203.254.224.24]:37496 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751675AbcLGL2s (ORCPT ); Wed, 7 Dec 2016 06:28:48 -0500 MIME-version: 1.0 Content-type: text/plain; charset=utf-8 X-AuditID: cbfee61b-f796f6d000004092-b9-5847efdc67ae Content-transfer-encoding: 8BIT Message-id: <5847EFDC.8050702@samsung.com> Date: Wed, 07 Dec 2016 20:17:48 +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: Krzysztof Kozlowski Cc: javier@osg.samsung.com, kgene@kernel.org, robh+dt@kernel.org, s.nawrocki@samsung.com, tomasz.figa@gmail.com, 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 Subject: Re: [PATCH 4/5] arm64: dts: exynos5433: Add bus dt node using VDD_INT for Exynos5433 References: <1480663087-4590-1-git-send-email-cw00.choi@samsung.com> <1480663087-4590-5-git-send-email-cw00.choi@samsung.com> <20161206192104.GB12683@kozik-lap> In-reply-to: <20161206192104.GB12683@kozik-lap> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrJIsWRmVeSWpSXmKPExsVy+t9jAd07790jDN42aFnMP3KO1eLN2zVM Fv2PXzNbnD+/gd3ibNMbdotNj6+xWlzeNYfNYsb5fUwWtxtXsFm07j3CbnH4TTurxapdfxgd eDx2zrrL7rFpVSebx+Yl9R5b+oG8vi2rGD0+b5ILYItys8lITUxJLVJIzUvOT8nMS7dVCg1x 07VQUshLzE21VYrQ9Q0JUlIoS8wpBfKMDNCAg3OAe7CSvl2CW8bRNY/YC26oV3w8foqlgXGl fBcjJ4eEgInEryvtzBC2mMSFe+vZuhi5OIQEljJKLOjtZAVJ8AoISvyYfI+li5GDg1lAXuLI pWwIU11iypRckAohgQeMEo3fsiGqtSSaZ91hArFZBFQlHt+cyQ5iswHF97+4wQZi8wsoSlz9 8ZgRZIyoQIRE94lKkLCIgKbE9b/fWUEuYBZYzyQxpXMCWL2wQLzEwkc3mCF2/WCUOHLcG8Tm FNCXuDlxAcsERsFZSA6dhXDoLIRDFzAyr2KUSC1ILihOSs81ykst1ytOzC0uzUvXS87P3cQI jtZn0jsYD+9yP8QowMGoxMMbweEeIcSaWFZcmXuIUYKDWUmE989boBBvSmJlVWpRfnxRaU5q 8SFGU6BXJzJLiSbnAxNJXkm8oYm5ibmxgYW5paWJkZI4b+PsZ+FCAumJJanZqakFqUUwfUwc nFINjHFt3ZVXeT5EdLRNYLpbPuGSfvDD/zOYr25eECa97ZNdjNxmkeqWHK4Fid+cWZSNGgsu Gn7lve/SOO335Lwn5+qv77l6V65l+eE0oXdP3ovciZd6++8/8/UHwmc6XV9NrPzStElF9uNz +5Jb4dmP1mqFn128cmX2XtXU6Fzdf2cZWla3TzryyUqJpTgj0VCLuag4EQCkf40N7AIAAA== X-MTR: 20000000000000000@CPGS Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016년 12월 07일 04:21, Krzysztof Kozlowski wrote: > On Fri, Dec 02, 2016 at 04:18:06PM +0900, Chanwoo Choi wrote: >> This patch adds the bus nodes using VDD_INT for Exynos5433 SoC. >> Exynos5433 has the following AMBA AXI buses to translate data >> between DRAM and sub-blocks. >> >> Following list specify the detailed correlation between sub-block and clock: >> - CLK_ACLK_G2D_{400|266} : Bus clock for G2D >> - CLK_ACLK_MSCL_400 : Bus clock for MSCL (Mobile Scaler) >> - CLK_ACLK_GSCL_333 : Bus clock for GSCL (General Scaler) >> - CLK_SCLK_JPEG_MSCL : Bus clock for JPEG >> - CLK_ACLK_MFC_400 : Bus clock for MFC (Multi Format Codec) >> - CLK_ACLK_HEVC_400 : Bus clock for HEVC (High Effective Video Codec) >> - CLK_ACLK_BUS0_400 : NoC(Network On Chip)'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 >> >> Signed-off-by: Chanwoo Choi >> --- >> arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi | 208 +++++++++++++++++++++++++ >> arch/arm64/boot/dts/exynos/exynos5433.dtsi | 1 + >> 2 files changed, 209 insertions(+) >> create mode 100644 arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi >> >> diff --git a/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi b/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi >> new file mode 100644 >> index 000000000000..b1e1d9c622e1 >> --- /dev/null >> +++ b/arch/arm64/boot/dts/exynos/exynos5433-bus.dtsi >> @@ -0,0 +1,208 @@ >> +/* >> + * Samsung's Exynos5433 SoC Memory interface and AMBA bus device tree source >> + * >> + * Copyright (c) 2016 Samsung Electronics Co., Ltd. >> + * Chanwoo Choi >> + * >> + * Samsung's Exynos5433 SoC Memory interface and AMBA buses are listed >> + * as device tree nodes are listed in this file. > > This duplicates the introduction line and does not make sense. I'll remove it. > >> + * >> + * This program is free software; you can redistribute it and/or modify >> + * it under the terms of the GNU General Public License version 2 as >> + * published by the Free Software Foundation. >> + */ >> + >> +/ { > > Shouldn't these be under soc node? It looks like property of SoC itself. OK. Move to them under SoC. - "/ {" -> "&soc {" > >> + /* INT (Internal) block using VDD_INT */ >> + bus_g2d_400: bus_g2d_400 { > > In node name, the dash '-' is preferred. The name should describe > general class of device so probably this should be just "bus"... but I > don't see a way how to do it reasonable anyway. I'll change them as following with 'busX'. The each dt node has the unique number('X') because each dt node does not have the base address and then need to identify oneself. bus_g2d_400: bus0 { bus_g2d_266: bus1 { bus_gscl: bus2 { bus_hevc: bus3 { bus_jpeg: bus4 { bus_mfc: bus5 { bus_mscl: bus6 { bus_noc0: bus7 { bus_noc1: bus8 { bus_noc2: bus9 { > >> + compatible = "samsung,exynos-bus"; >> + clocks = <&cmu_top CLK_ACLK_G2D_400>; >> + clock-names = "bus"; >> + operating-points-v2 = <&bus_g2d_400_opp_table>; >> + status ="disable"; > > Hm? I'll fix it. disable -> disabled > > >> + }; >> + >> + bus_mscl: bus_mscl { >> + compatible = "samsung,exynos-bus"; >> + clocks = <&cmu_top CLK_ACLK_MSCL_400>; >> + clock-names = "bus"; >> + operating-points-v2 = <&bus_g2d_400_opp_table>; >> + status ="disable"; >> + }; >> + >> + bus_jpeg: bus_jpeg { >> + compatible = "samsung,exynos-bus"; >> + clocks = <&cmu_top CLK_SCLK_JPEG_MSCL>; >> + clock-names = "bus"; >> + operating-points-v2 = <&bus_g2d_400_opp_table>; >> + status ="disable"; >> + }; >> + >> + bus_mfc: bus_mfc { >> + compatible = "samsung,exynos-bus"; >> + clocks = <&cmu_top CLK_ACLK_MFC_400>; >> + >> + clock-names = "bus"; >> + operating-points-v2 = <&bus_g2d_400_opp_table>; >> + status ="disable"; >> + }; >> + >> + bus_g2d_266: bus_g2d_266 { >> + compatible = "samsung,exynos-bus"; >> + clocks = <&cmu_top CLK_ACLK_G2D_266>; >> + clock-names = "bus"; >> + operating-points-v2 = <&bus_g2d_266_opp_table>; >> + status ="disable"; >> + }; >> + >> + bus_gscl: bus_gscl { >> + compatible = "samsung,exynos-bus"; >> + clocks = <&cmu_top CLK_ACLK_GSCL_333>; >> + clock-names = "bus"; >> + operating-points-v2 = <&bus_gscl_opp_table>; >> + status ="disable"; >> + }; >> + >> + bus_hevc: bus_hevc { >> + compatible = "samsung,exynos-bus"; >> + clocks = <&cmu_top CLK_ACLK_HEVC_400>; >> + clock-names = "bus"; >> + operating-points-v2 = <&bus_hevc_opp_table>; >> + status ="disable"; >> + }; >> + >> + bus_bus0: bus_bus0 { > > bus, bus, bus, bus, jackpot! Let's try to find better name and label for > these. :) I'll change the name with 'noc' prefix because this bus is used for NoC (Network On Chip)'s bus clock as commit msg. - old : bus_bus0 - new : bus_noc0 Best Regards, Chanwoo Choi