From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.1 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A9FF4C4CEC9 for ; Wed, 18 Sep 2019 09:42:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6524E218AF for ; Wed, 18 Sep 2019 09:42:28 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="AWzmCZ1O"; dkim=fail reason="key not found in DNS" (0-bit key) header.d=codeaurora.org header.i=@codeaurora.org header.b="Wu+r07eo" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727814AbfIRJm2 (ORCPT ); Wed, 18 Sep 2019 05:42:28 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:54106 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727146AbfIRJm1 (ORCPT ); Wed, 18 Sep 2019 05:42:27 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 98B3C6156F; Wed, 18 Sep 2019 09:42:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1568799745; bh=WLXIi4f1QNdWbc3tOmtATiZAcgqtcMbzXOV035jMKsc=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=AWzmCZ1O4gq/kfkzzGCR3kTvKCM9TMFyZEYhIe9GTiWBcfV13gLs4jfNuyxJojudS Tzu8NVNPDOq6mRoataBimukIEM05a8iDGqiSlLh+P2+UuZyg/CUgnZao4WDlsyfQH9 5fp8WMfyTQwEvHAevb0XAZHc7jvZVEhwjo16joak= Received: from [10.206.28.9] (blr-c-bdr-fw-01_globalnat_allzones-outside.qualcomm.com [103.229.19.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: tdas@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 1DBE76156F; Wed, 18 Sep 2019 09:42:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1568799743; bh=WLXIi4f1QNdWbc3tOmtATiZAcgqtcMbzXOV035jMKsc=; h=Subject:To:Cc:References:From:Date:In-Reply-To:From; b=Wu+r07eoUwKAeOmMw3butCA6404IQ/olZjh5Ig868vMhj0Hk+8NB8chASbGvX0otd gRSaVQAYW4lvMEGFVOYokoDkVX75pV8SvkrR33r25/aRcmekWxmTBVHl2M1eaytmXK HLAFsPJYuE1MUte817KYjDgGgFHfEIo0ZcL4rXaU= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 1DBE76156F Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=tdas@codeaurora.org Subject: Re: [PATCH v2 2/3] dt-bindings: clk: qcom: Add YAML schemas for the GCC clock bindings To: Rob Herring Cc: Stephen Boyd , Michael Turquette , David Brown , Rajendra Nayak , linux-arm-msm , "open list:ARM/QUALCOMM SUPPORT" , linux-clk , "linux-kernel@vger.kernel.org" , devicetree@vger.kernel.org References: <20190819163748.18318-1-tdas@codeaurora.org> <20190819163748.18318-3-tdas@codeaurora.org> From: Taniya Das Message-ID: Date: Wed, 18 Sep 2019 15:12:17 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-clk-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-clk@vger.kernel.org Hello Rob, Thank you for your review comments. On 8/20/2019 12:44 AM, Rob Herring wrote: > On Mon, Aug 19, 2019 at 11:38 AM Taniya Das wrote: >> >> The GCC clock provider have a bunch of generic properties that >> are needed in a device tree. Add a YAML schemas for those. Also update >> the compatible for SC7180 along with example for clocks & clock-names. >> >> Signed-off-by: Taniya Das >> --- >> .../devicetree/bindings/clock/qcom,gcc.yaml | 141 ++++++++++++++++ >> include/dt-bindings/clock/qcom,gcc-sc7180.h | 155 ++++++++++++++++++ >> 2 files changed, 296 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/clock/qcom,gcc.yaml >> create mode 100644 include/dt-bindings/clock/qcom,gcc-sc7180.h > > You need to remove the old doc. > Would remove it in the next patch. >> >> diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml >> new file mode 100644 >> index 000000000000..17c563a036c7 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/clock/qcom,gcc.yaml >> @@ -0,0 +1,141 @@ >> +# SPDX-License-Identifier: GPL-2.0 >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/clock/qcom,gcc.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Qualcomm Global Clock & Reset Controller Binding >> + >> +maintainers: >> + - Stephen Boyd >> + >> +properties: >> + "#clock-cells": >> + const: 1 >> + >> + "#reset-cells": >> + const: 1 >> + >> + reg: >> + maxItems: 1 >> + description: | >> + shall contain the base register location and length > > Don't need a description if there's only 1 entry. > Thanks, would remove. >> + >> + compatible : >> + enum: >> + - qcom,gcc-apq8064 >> + - qcom,gcc-apq8084 >> + - qcom,gcc-ipq8064 >> + - qcom,gcc-ipq4019 >> + - qcom,gcc-ipq8074 >> + - qcom,gcc-msm8660 >> + - qcom,gcc-msm8916 >> + - qcom,gcc-msm8960 >> + - qcom,gcc-msm8974 >> + - qcom,gcc-msm8974pro >> + - qcom,gcc-msm8974pro-ac >> + - qcom,gcc-msm8994 >> + - qcom,gcc-msm8996 >> + - qcom,gcc-msm8998 >> + - qcom,gcc-mdm9615 >> + - qcom,gcc-qcs404 >> + - qcom,gcc-sdm630 >> + - qcom,gcc-sdm660 >> + - qcom,gcc-sdm845 >> + - qcom,gcc-sc7180 >> + >> + clocks: >> + minItems: 1 >> + maxItems: 2 >> + items: >> + - description: Board XO source >> + - description: Board active XO source >> + >> + clock-names: >> + minItems: 1 >> + maxItems: 2 >> + items: >> + - const: bi_tcxo >> + - const: bi_tcxo_ao >> + >> + nvmem-cells: >> + $ref: /schemas/types.yaml#/definitions/phandle-array > > Standard property, you don't need the type here. What's needed is the size. > Would take care of it in the next patch. >> + description: >> + Qualcomm TSENS (thermal sensor device) on some devices can >> + be part of GCC and hence the TSENS properties can also be part >> + of the GCC/clock-controller node. >> + For more details on the TSENS properties please refer >> + Documentation/devicetree/bindings/thermal/qcom-tsens.txt >> + >> + nvmem-cell-names: >> + $ref: /schemas/types.yaml#/definitions/string-array > > Don't need the type (and this would have to be under an 'allOf' to > actually work). > Would take care of it in the next patch. >> + description: >> + Names for each nvmem-cells specified. >> + items: >> + - const: calib >> + - const: calib_backup >> + >> + "#thermal-sensor-cells": >> + const: 1 >> + >> + "#power-domain-cells": >> + const: 1 >> + >> + protected-clocks: >> + description: >> + Protected clock specifier list as per common clock binding >> + >> +required: >> + - "#clock-cells" >> + - "#reset-cells" >> + - compatible >> + - reg >> + - clocks >> + - clock-names >> + >> +examples: >> + - | >> + clock-controller@900000 { >> + compatible = "qcom,gcc-msm8960"; >> + reg = <0x900000 0x4000>; >> + #clock-cells = <1>; >> + #reset-cells = <1>; >> + #power-domain-cells = <1>; >> + }; >> + >> + >> + - | >> + // Example of GCC with TSENS properties: >> + clock-controller@900000 { >> + compatible = "qcom,gcc-apq8064"; >> + reg = <0x00900000 0x4000>; >> + nvmem-cells = <&tsens_calib>, <&tsens_backup>; >> + nvmem-cell-names = "calib", "calib_backup"; >> + #clock-cells = <1>; >> + #reset-cells = <1>; >> + #thermal-sensor-cells = <1>; >> + }; >> + >> + - | >> + //Example of GCC with protected-clocks properties: >> + clock-controller@100000 { >> + compatible = "qcom,gcc-sdm845"; >> + reg = <0x100000 0x1f0000>; >> + #clock-cells = <1>; >> + #reset-cells = <1>; >> + #power-domain-cells = <1>; >> + protected-clocks = <187>, <188>, <189>, <190>, <191>; >> + }; >> + >> + - | >> + //Example of GCC with clock nodes properties: >> + clock-controller@100000 { >> + compatible = "qcom,gcc-sc7180"; >> + reg = <0x100000 0x1f0000>; >> + clocks = <&rpmhcc 0>, <&rpmhcc 1>; >> + clock-names = "bi_tcxo", "bi_tcxo_ao"; >> + #clock-cells = <1>; >> + #reset-cells = <1>; >> + #power-domain-cells = <1>; >> + }; >> +... >> diff --git a/include/dt-bindings/clock/qcom,gcc-sc7180.h b/include/dt-bindings/clock/qcom,gcc-sc7180.h >> new file mode 100644 >> index 000000000000..d76b061f6a4e >> --- /dev/null >> +++ b/include/dt-bindings/clock/qcom,gcc-sc7180.h >> @@ -0,0 +1,155 @@ >> +/* SPDX-License-Identifier: GPL-2.0 */ >> +/* >> + * Copyright (c) 2019, The Linux Foundation. All rights reserved. >> + */ >> + >> +#ifndef _DT_BINDINGS_CLK_QCOM_GCC_SC7180_H >> +#define _DT_BINDINGS_CLK_QCOM_GCC_SC7180_H >> + >> +/* GCC clocks */ >> +#define GCC_GPLL0_MAIN_DIV_CDIV 0 >> +#define GPLL0 1 >> +#define GPLL0_OUT_EVEN 2 >> +#define GPLL1 3 >> +#define GPLL4 4 >> +#define GPLL6 5 >> +#define GPLL7 6 >> +#define GCC_AGGRE_UFS_PHY_AXI_CLK 7 >> +#define GCC_AGGRE_USB3_PRIM_AXI_CLK 8 >> +#define GCC_BOOT_ROM_AHB_CLK 9 >> +#define GCC_CAMERA_AHB_CLK 10 >> +#define GCC_CAMERA_HF_AXI_CLK 11 >> +#define GCC_CAMERA_THROTTLE_HF_AXI_CLK 12 >> +#define GCC_CAMERA_XO_CLK 13 >> +#define GCC_CE1_AHB_CLK 14 >> +#define GCC_CE1_AXI_CLK 15 >> +#define GCC_CE1_CLK 16 >> +#define GCC_CFG_NOC_USB3_PRIM_AXI_CLK 17 >> +#define GCC_CPUSS_AHB_CLK 18 >> +#define GCC_CPUSS_AHB_CLK_SRC 19 >> +#define GCC_CPUSS_GNOC_CLK 20 >> +#define GCC_CPUSS_RBCPR_CLK 21 >> +#define GCC_DDRSS_GPU_AXI_CLK 22 >> +#define GCC_DISP_AHB_CLK 23 >> +#define GCC_DISP_GPLL0_CLK_SRC 24 >> +#define GCC_DISP_GPLL0_DIV_CLK_SRC 25 >> +#define GCC_DISP_HF_AXI_CLK 26 >> +#define GCC_DISP_THROTTLE_HF_AXI_CLK 27 >> +#define GCC_DISP_XO_CLK 28 >> +#define GCC_GP1_CLK 29 >> +#define GCC_GP1_CLK_SRC 30 >> +#define GCC_GP2_CLK 31 >> +#define GCC_GP2_CLK_SRC 32 >> +#define GCC_GP3_CLK 33 >> +#define GCC_GP3_CLK_SRC 34 >> +#define GCC_GPU_CFG_AHB_CLK 35 >> +#define GCC_GPU_GPLL0_CLK_SRC 36 >> +#define GCC_GPU_GPLL0_DIV_CLK_SRC 37 >> +#define GCC_GPU_MEMNOC_GFX_CLK 38 >> +#define GCC_GPU_SNOC_DVM_GFX_CLK 39 >> +#define GCC_NPU_AXI_CLK 40 >> +#define GCC_NPU_BWMON_AXI_CLK 41 >> +#define GCC_NPU_BWMON_DMA_CFG_AHB_CLK 42 >> +#define GCC_NPU_BWMON_DSP_CFG_AHB_CLK 43 >> +#define GCC_NPU_CFG_AHB_CLK 44 >> +#define GCC_NPU_DMA_CLK 45 >> +#define GCC_NPU_GPLL0_CLK_SRC 46 >> +#define GCC_NPU_GPLL0_DIV_CLK_SRC 47 >> +#define GCC_PDM2_CLK 48 >> +#define GCC_PDM2_CLK_SRC 49 >> +#define GCC_PDM_AHB_CLK 50 >> +#define GCC_PDM_XO4_CLK 51 >> +#define GCC_PRNG_AHB_CLK 52 >> +#define GCC_QSPI_CNOC_PERIPH_AHB_CLK 53 >> +#define GCC_QSPI_CORE_CLK 54 >> +#define GCC_QSPI_CORE_CLK_SRC 55 >> +#define GCC_QUPV3_WRAP0_CORE_2X_CLK 56 >> +#define GCC_QUPV3_WRAP0_CORE_CLK 57 >> +#define GCC_QUPV3_WRAP0_S0_CLK 58 >> +#define GCC_QUPV3_WRAP0_S0_CLK_SRC 59 >> +#define GCC_QUPV3_WRAP0_S1_CLK 60 >> +#define GCC_QUPV3_WRAP0_S1_CLK_SRC 61 >> +#define GCC_QUPV3_WRAP0_S2_CLK 62 >> +#define GCC_QUPV3_WRAP0_S2_CLK_SRC 63 >> +#define GCC_QUPV3_WRAP0_S3_CLK 64 >> +#define GCC_QUPV3_WRAP0_S3_CLK_SRC 65 >> +#define GCC_QUPV3_WRAP0_S4_CLK 66 >> +#define GCC_QUPV3_WRAP0_S4_CLK_SRC 67 >> +#define GCC_QUPV3_WRAP0_S5_CLK 68 >> +#define GCC_QUPV3_WRAP0_S5_CLK_SRC 69 >> +#define GCC_QUPV3_WRAP1_CORE_2X_CLK 70 >> +#define GCC_QUPV3_WRAP1_CORE_CLK 71 >> +#define GCC_QUPV3_WRAP1_S0_CLK 72 >> +#define GCC_QUPV3_WRAP1_S0_CLK_SRC 73 >> +#define GCC_QUPV3_WRAP1_S1_CLK 74 >> +#define GCC_QUPV3_WRAP1_S1_CLK_SRC 75 >> +#define GCC_QUPV3_WRAP1_S2_CLK 76 >> +#define GCC_QUPV3_WRAP1_S2_CLK_SRC 77 >> +#define GCC_QUPV3_WRAP1_S3_CLK 78 >> +#define GCC_QUPV3_WRAP1_S3_CLK_SRC 79 >> +#define GCC_QUPV3_WRAP1_S4_CLK 80 >> +#define GCC_QUPV3_WRAP1_S4_CLK_SRC 81 >> +#define GCC_QUPV3_WRAP1_S5_CLK 82 >> +#define GCC_QUPV3_WRAP1_S5_CLK_SRC 83 >> +#define GCC_QUPV3_WRAP_0_M_AHB_CLK 84 >> +#define GCC_QUPV3_WRAP_0_S_AHB_CLK 85 >> +#define GCC_QUPV3_WRAP_1_M_AHB_CLK 86 >> +#define GCC_QUPV3_WRAP_1_S_AHB_CLK 87 >> +#define GCC_SDCC1_AHB_CLK 88 >> +#define GCC_SDCC1_APPS_CLK 89 >> +#define GCC_SDCC1_APPS_CLK_SRC 90 >> +#define GCC_SDCC1_ICE_CORE_CLK 91 >> +#define GCC_SDCC1_ICE_CORE_CLK_SRC 92 >> +#define GCC_SDCC2_AHB_CLK 93 >> +#define GCC_SDCC2_APPS_CLK 94 >> +#define GCC_SDCC2_APPS_CLK_SRC 95 >> +#define GCC_SYS_NOC_CPUSS_AHB_CLK 96 >> +#define GCC_UFS_MEM_CLKREF_CLK 97 >> +#define GCC_UFS_PHY_AHB_CLK 98 >> +#define GCC_UFS_PHY_AXI_CLK 99 >> +#define GCC_UFS_PHY_AXI_CLK_SRC 100 >> +#define GCC_UFS_PHY_ICE_CORE_CLK 101 >> +#define GCC_UFS_PHY_ICE_CORE_CLK_SRC 102 >> +#define GCC_UFS_PHY_PHY_AUX_CLK 103 >> +#define GCC_UFS_PHY_PHY_AUX_CLK_SRC 104 >> +#define GCC_UFS_PHY_RX_SYMBOL_0_CLK 105 >> +#define GCC_UFS_PHY_TX_SYMBOL_0_CLK 106 >> +#define GCC_UFS_PHY_UNIPRO_CORE_CLK 107 >> +#define GCC_UFS_PHY_UNIPRO_CORE_CLK_SRC 108 >> +#define GCC_USB30_PRIM_MASTER_CLK 109 >> +#define GCC_USB30_PRIM_MASTER_CLK_SRC 110 >> +#define GCC_USB30_PRIM_MOCK_UTMI_CLK 111 >> +#define GCC_USB30_PRIM_MOCK_UTMI_CLK_SRC 112 >> +#define GCC_USB30_PRIM_SLEEP_CLK 113 >> +#define GCC_USB3_PRIM_CLKREF_CLK 114 >> +#define GCC_USB3_PRIM_PHY_AUX_CLK 115 >> +#define GCC_USB3_PRIM_PHY_AUX_CLK_SRC 116 >> +#define GCC_USB3_PRIM_PHY_COM_AUX_CLK 117 >> +#define GCC_USB3_PRIM_PHY_PIPE_CLK 118 >> +#define GCC_USB_PHY_CFG_AHB2PHY_CLK 119 >> +#define GCC_VIDEO_AHB_CLK 120 >> +#define GCC_VIDEO_AXI_CLK 121 >> +#define GCC_VIDEO_GPLL0_DIV_CLK_SRC 122 >> +#define GCC_VIDEO_THROTTLE_AXI_CLK 123 >> +#define GCC_VIDEO_XO_CLK 124 >> + >> +/* GCC resets */ >> +#define GCC_QUSB2PHY_PRIM_BCR 0 >> +#define GCC_QUSB2PHY_SEC_BCR 1 >> +#define GCC_UFS_PHY_BCR 2 >> +#define GCC_USB30_PRIM_BCR 3 >> +#define GCC_USB3_DP_PHY_PRIM_BCR 4 >> +#define GCC_USB3_DP_PHY_SEC_BCR 5 >> +#define GCC_USB3_PHY_PRIM_BCR 6 >> +#define GCC_USB3_PHY_SEC_BCR 7 >> +#define GCC_USB3PHY_PHY_PRIM_BCR 8 >> +#define GCC_USB3PHY_PHY_SEC_BCR 9 >> +#define GCC_USB_PHY_CFG_AHB2PHY_BCR 10 >> + >> +/* GCC GDSCRs */ >> +#define UFS_PHY_GDSC 0 >> +#define USB30_PRIM_GDSC 1 >> +#define HLOS1_VOTE_MMNOC_MMU_TBU_HF0_GDSC 2 >> +#define HLOS1_VOTE_MMNOC_MMU_TBU_SF_GDSC 3 >> + >> +#endif >> -- >> Qualcomm INDIA, on behalf of Qualcomm Innovation Center, Inc.is a member >> of the Code Aurora Forum, hosted by the Linux Foundation. >> -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation. --