Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp636743pxb; Wed, 15 Sep 2021 09:40:16 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzlBespjShpljzLSGJzAr2qPH/NC+XBo6MMtMGSjjGO24RV1yoxpTWeIlii5E5iwacJa6fd X-Received: by 2002:a17:906:558f:: with SMTP id y15mr943619ejp.572.1631724016376; Wed, 15 Sep 2021 09:40:16 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631724016; cv=none; d=google.com; s=arc-20160816; b=IGmgmJQccKuaBZ+fkCVUX9XxjebU8OnMDLswuCpnUocefEbbWI+XNXOoVU4uMkbRgm t7GcAfNg/ObNlaeDfmcj+thQ+O6by3IVZdGDhj++dvBl0+qKvM4DYNEK6SwrpPAxsAMJ W24EYcpj011v38hEcj2pOmHfbfR1xj24fGEtmOjkg84ejxyadLaGEfs9zqIpHYRbkVbn vZRvbn8vmRHceciypEL8AGdXPGbCQk9l66HEGYMQe+0gDDaAkZFRDQEIEwjacGB/h8nQ OVrUhFeK8f6bkgHb5YbTDotrpc0JYX2WMYbaQNKIIzBw8kIDzPdZoCE2MWw/5PoG+7Qj XtVg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=iOr6zcjMaLs0KTnNS9eDKxY/oi2/iWBDb8WhSTtIqNc=; b=nfE9+ZM4h3PK53g0SrIxYtuIoXZepnolm9PA3KtULp4S/EX/U6NvYY1Z/i0xamgFuN IqbPRNmoDLNUl1hKs/8f4qzujaBrtdfuJkNojKQiqy93UDfm8L0OzVVydkeBc/Dp53kR DY3FxAtyBTF5KSKy/ZO5Vqc0b0hwbLJSdJhzYV7REF/o0fSfoB3kKh1Hz5h60ApF9VeN 4YNvcNMIAbn7FWZqVdC3oGBWPJtaJh/nj7cS9qqoJtDBF7GfpBdO2GRMzzZimbXTVZGj Z0uxXKAWmF5nRFwEcaec3xpTUqRTDBMDS0VCtu1yZi1IgHlv/5ergYCqxTg3WwZhsV2z TT/g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=TwImoFin; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w18si431405edc.532.2021.09.15.09.39.51; Wed, 15 Sep 2021 09:40:16 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=TwImoFin; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229893AbhIOQjQ (ORCPT + 99 others); Wed, 15 Sep 2021 12:39:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33566 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229690AbhIOQjP (ORCPT ); Wed, 15 Sep 2021 12:39:15 -0400 Received: from mail-pg1-x530.google.com (mail-pg1-x530.google.com [IPv6:2607:f8b0:4864:20::530]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0374CC061574; Wed, 15 Sep 2021 09:37:57 -0700 (PDT) Received: by mail-pg1-x530.google.com with SMTP id q68so3262509pga.9; Wed, 15 Sep 2021 09:37:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=iOr6zcjMaLs0KTnNS9eDKxY/oi2/iWBDb8WhSTtIqNc=; b=TwImoFinOGFJOCMBCRwV0eIxPyqA+atDqHF9BZ6hf+a1adRn+2HGrq/D/pCbBbMbxR A6e7OAm/LpFYCeIXr1QQi96L3XsCAiX4874o/d1ffYzMiwQKEL5Xsot5tkMMzLzfCk05 +LGoeWTQY271gJhSpGHlfOaNpfizJ8vAXgLvWjyKBzGilmx2c8P0M8bZpcjgFTLARqz4 BmkDwD4UpPjPYdO22/zhUzarlv3EzKCzPzvPogPurJWLKUPJaMpF1L93svPXqjSFSeRW 0yAc4EnZS5+oekvBMKgWrS5JhONVjD2DOlH5Mv7+ORRkt8EUk2eb3b6nfxhQDYZci8bb dsTQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=iOr6zcjMaLs0KTnNS9eDKxY/oi2/iWBDb8WhSTtIqNc=; b=LXkrLLlum2MIXgMdGEa2U7IBGIPmzvKazRq78cmedK33L53lQP7wzqsavZDs7TSA2u aaYyN+mLNgtHYqJC+mImgwvoqOiuoF2Vr+AZDm48ueZaxStCvHc1B2/OGmbj5POW9Bz8 BEyxnQVAwQVSGRD6IaLEDQTrL7dDNaIOO4MhSTCxeNTz08dvIM/dP57FgIWtb0tAXBvZ 2uJubm9ashkz76Ek9RtNNYAu7bstygejpi3vldlKngeS3FGwfzyxqUwre+2HGkR30zPL 6hxI54ChqusHFqvaVDmgoFVipJUNxmGtCQ6dqIaAIMTGrrLcca5cSjmWtUisS2le81Hy uwQA== X-Gm-Message-State: AOAM531hRFB2j01vQF7GqcfmRfUvzeCUZ5JJvQUoIH0wjF2U8YZ5Q83D pz0qGfgjwB5G0FYTobYXGgmqtpJuUWo= X-Received: by 2002:aa7:8d58:0:b0:439:d68:b405 with SMTP id s24-20020aa78d58000000b004390d68b405mr598896pfe.2.1631723875989; Wed, 15 Sep 2021 09:37:55 -0700 (PDT) Received: from [172.30.1.2] ([14.32.163.5]) by smtp.gmail.com with ESMTPSA id b29sm478926pgn.53.2021.09.15.09.37.50 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 15 Sep 2021 09:37:55 -0700 (PDT) Subject: Re: [PATCH 4/6] dt-bindings: clock: Add bindings definitions for Exynos850 CMU To: Sam Protsenko , Krzysztof Kozlowski , Sylwester Nawrocki , =?UTF-8?Q?Pawe=c5=82_Chmiel?= , Chanwoo Choi , Tomasz Figa , Rob Herring , Stephen Boyd , Michael Turquette Cc: Ryu Euiyoul , Tom Gall , Sumit Semwal , John Stultz , Amit Pundir , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org References: <20210914155607.14122-1-semen.protsenko@linaro.org> <20210914155607.14122-5-semen.protsenko@linaro.org> From: Chanwoo Choi Message-ID: <96e5587e-aca7-248e-6448-8edfc70784b7@gmail.com> Date: Thu, 16 Sep 2021 01:37:49 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.13.0 MIME-Version: 1.0 In-Reply-To: <20210914155607.14122-5-semen.protsenko@linaro.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, You don't add clock ids for the all defined clocks in clk-exynos850.c. I recommend that add all clock ids for the defined clocks if possible. If you want to change the parent clock of mux or change the clock rate of div rate for some clocks, you have to touch the files as following: - include/dt-bindings/clock/exynos850.h - drivers/clk/samsung/clk-exynos850.c - exynos850 dt files If you define the clock ids for all clocks added to this patchset, you can change the parent or rate by just editing the dt files. But, I have no strongly objection about just keeping this patch. On 21. 9. 15. 오전 12:56, Sam Protsenko wrote: > Clock controller driver is designed to have separate instances for each > particular CMU. So clock IDs in this bindings header also start from 1 > for each CMU. > > Signed-off-by: Sam Protsenko > --- > include/dt-bindings/clock/exynos850.h | 72 +++++++++++++++++++++++++++ > 1 file changed, 72 insertions(+) > create mode 100644 include/dt-bindings/clock/exynos850.h > > diff --git a/include/dt-bindings/clock/exynos850.h b/include/dt-bindings/clock/exynos850.h > new file mode 100644 > index 000000000000..2f0a7f619627 > --- /dev/null > +++ b/include/dt-bindings/clock/exynos850.h > @@ -0,0 +1,72 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +/* > + * Copyright (C) 2021 Linaro Ltd. > + * Author: Sam Protsenko > + * > + * Device Tree binding constants for Exynos850 clock controller. > + */ > + > +#ifndef _DT_BINDINGS_CLOCK_EXYNOS_850_H > +#define _DT_BINDINGS_CLOCK_EXYNOS_850_H > + > +/* CMU_TOP */ > +#define DOUT_HSI_BUS 1 > +#define DOUT_HSI_MMC_CARD 2 > +#define DOUT_HSI_USB20DRD 3 > +#define DOUT_PERI_BUS 4 > +#define DOUT_PERI_UART 5 > +#define DOUT_PERI_IP 6 > +#define DOUT_CORE_BUS 7 > +#define DOUT_CORE_CCI 8 > +#define DOUT_CORE_MMC_EMBD 9 > +#define DOUT_CORE_SSS 10 > +#define TOP_NR_CLK 11 > + > +/* CMU_HSI */ > +#define GOUT_USB_RTC_CLK 1 > +#define GOUT_USB_REF_CLK 2 > +#define GOUT_USB_PHY_REF_CLK 3 > +#define GOUT_USB_PHY_ACLK 4 > +#define GOUT_USB_BUS_EARLY_CLK 5 > +#define GOUT_GPIO_HSI_PCLK 6 > +#define GOUT_MMC_CARD_ACLK 7 > +#define GOUT_MMC_CARD_SDCLKIN 8 > +#define GOUT_SYSREG_HSI_PCLK 9 > +#define HSI_NR_CLK 10 > + > +/* CMU_PERI */ > +#define GOUT_GPIO_PERI_PCLK 1 > +#define GOUT_HSI2C0_IPCLK 2 > +#define GOUT_HSI2C0_PCLK 3 > +#define GOUT_HSI2C1_IPCLK 4 > +#define GOUT_HSI2C1_PCLK 5 > +#define GOUT_HSI2C2_IPCLK 6 > +#define GOUT_HSI2C2_PCLK 7 > +#define GOUT_I2C0_PCLK 8 > +#define GOUT_I2C1_PCLK 9 > +#define GOUT_I2C2_PCLK 10 > +#define GOUT_I2C3_PCLK 11 > +#define GOUT_I2C4_PCLK 12 > +#define GOUT_I2C5_PCLK 13 > +#define GOUT_I2C6_PCLK 14 > +#define GOUT_MCT_PCLK 15 > +#define GOUT_PWM_MOTOR_PCLK 16 > +#define GOUT_SPI0_IPCLK 17 > +#define GOUT_SPI0_PCLK 18 > +#define GOUT_SYSREG_PERI_PCLK 19 > +#define GOUT_UART_IPCLK 20 > +#define GOUT_UART_PCLK 21 > +#define GOUT_WDT0_PCLK 22 > +#define GOUT_WDT1_PCLK 23 > +#define PERI_NR_CLK 24 > + > +/* CMU_CORE */ > +#define GOUT_CCI_ACLK 1 > +#define GOUT_GIC_CLK 2 > +#define GOUT_MMC_EMBD_ACLK 3 > +#define GOUT_MMC_EMBD_SDCLKIN 4 > +#define GOUT_SSS_ACLK 5 > +#define GOUT_SSS_PCLK 6 > +#define CORE_NR_CLK 7 > + > +#endif /* _DT_BINDINGS_CLOCK_EXYNOS_850_H */ > -- Best Regards, Samsung Electronics Chanwoo Choi