Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp10973163rwd; Thu, 22 Jun 2023 07:14:28 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7kDoPIIkf+vPCMgPCavf9yJrJ9Ke4PVRPbygzKog4NqxpG8Xm9O+5t73tu0AKPauliSuAZ X-Received: by 2002:a17:90a:43e4:b0:25b:dc4b:611d with SMTP id r91-20020a17090a43e400b0025bdc4b611dmr18596523pjg.26.1687443268480; Thu, 22 Jun 2023 07:14:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1687443268; cv=none; d=google.com; s=arc-20160816; b=Pnd76cJNupEaZK3wfDnlFtcsAcD9zm1N9PRyQFTCweMSHuHmtqmfKXqI7RJh8fMO+l LLhdJVJoddDXJxvTRX+6lHG79DwPFTqCI4pPWTCADYyQmB/Piq1pWN14op2yk9cbXfH7 4b57VAl0yztLZDgio+hXvqTlQLiFlZEBwXktFpqG91dViqYzXcbshLzUCrP4gR646lv3 krwHa9A4irA04iVXv2yjuaJBGVTrfFQhEevkg6z/EBQ5smb9Y8FtafLSoEiLi74B5a0j hFtfq/7cszaCLYxV0+OR82eezGSs6xeJAwUQ4Yi9icTB4tA8g8hTmFs2Wp/FquktJWmP c72g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:user-agent:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=eKa4rTnfwF98HJy9xMz2GqxxsZRxQxHD1jhJK3ZmGag=; b=kTiNf9BB8R0LotfVc8eu4mK4QdrRWThdC5RGZJXZu3o7gJ0YmwAA3iI13hidc7Ruf6 ag//Emk1O39oQGfK3sgVkA3f/mkEghQNkd8UOxuzxmZVQS/t5cf/crqhfxo5x9vbrJLO 0fo0IZvDdeAYhjVZRNUshex/30DUdQhHzj16CYCNJo0QKVVfAC2NhkoTBJKY38v5amgr fV+eItDmEZXLmfyZufl12HLuNW0G71ifnwh81wxQNw/jBV+nqc2Xx2iH25CHoV7j4+T9 TxA2/hDgCT8XQVuhPclR4v4sIFutEFt1aZdQ4hlacL73P1FIf3n8YXHL2KhkDbSP/CTI rpRw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sberdevices.ru header.s=mail header.b=TpqNDkeM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=sberdevices.ru Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id w12-20020a17090aad4c00b002528f40700dsi83716pjv.153.2023.06.22.07.14.15; Thu, 22 Jun 2023 07:14:28 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@sberdevices.ru header.s=mail header.b=TpqNDkeM; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=sberdevices.ru Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230460AbjFVOAM (ORCPT + 99 others); Thu, 22 Jun 2023 10:00:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52590 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229813AbjFVOAJ (ORCPT ); Thu, 22 Jun 2023 10:00:09 -0400 Received: from mx.sberdevices.ru (mx.sberdevices.ru [45.89.227.171]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 50CDB118; Thu, 22 Jun 2023 07:00:05 -0700 (PDT) Received: from s-lin-edge02.sberdevices.ru (localhost [127.0.0.1]) by mx.sberdevices.ru (Postfix) with ESMTP id 472875FD0F; Thu, 22 Jun 2023 17:00:03 +0300 (MSK) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sberdevices.ru; s=mail; t=1687442403; bh=eKa4rTnfwF98HJy9xMz2GqxxsZRxQxHD1jhJK3ZmGag=; h=Date:From:To:Subject:Message-ID:MIME-Version:Content-Type; b=TpqNDkeM6DSevkhRsXA9uBCCfER0M1IfMbcxKTGMp8ZueswEO8pKBPNMJk3dBjYiG BzlklzNLD3aNSa3J7lGJBXOR/VDQKT5gUl04Ld4FCtY/SBmhSn9gaqE5rnYsnw7Yls MG5nyMM8hPH8RkbdsxvMySJ9qb6DR1+h8vytdBc+1QNcsAflp7S0jEJEuqlO666GOm /QzCpvqHo3s1r3BjtwHWIVzcc7Jbu8zj4wuNytb+UtmA+ZGQLS41P2W9q3ypL2u25j cLRy24dsPABH3dz9zS9xONHfOa7Y3PFpoZwoynEOBUtv5Io3Ob/CVMS3wZuRp3z7oK gVOT8NVmxjY0A== Received: from p-i-exch-sc-m01.sberdevices.ru (p-i-exch-sc-m01.sberdevices.ru [172.16.192.107]) by mx.sberdevices.ru (Postfix) with ESMTP; Thu, 22 Jun 2023 17:00:02 +0300 (MSK) Date: Thu, 22 Jun 2023 17:00:02 +0300 From: Dmitry Rokosov To: Neil Armstrong CC: Jerome Brunet , Michael Turquette , Stephen Boyd , Kevin Hilman , Martin Blumenstingl , Rob Herring , Krzysztof Kozlowski , Conor Dooley , , , , , , Subject: Re: [PATCH v2 04/19] clk: meson: migrate a1 clock drivers out of hw_onecell_data to drop NR_CLKS Message-ID: <20230622140002.e3ekm7joi4kxv53v@CAB-WSD-L081021> References: <20230607-topic-amlogic-upstream-clkid-public-migration-v2-0-38172d17c27a@linaro.org> <20230607-topic-amlogic-upstream-clkid-public-migration-v2-4-38172d17c27a@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20230607-topic-amlogic-upstream-clkid-public-migration-v2-4-38172d17c27a@linaro.org> User-Agent: NeoMutt/20220415 X-Originating-IP: [100.64.160.123] X-ClientProxiedBy: p-i-exch-sc-m01.sberdevices.ru (172.16.192.107) To p-i-exch-sc-m01.sberdevices.ru (172.16.192.107) X-KSMG-Rule-ID: 4 X-KSMG-Message-Action: clean X-KSMG-AntiSpam-Status: not scanned, disabled by settings X-KSMG-AntiSpam-Interceptor-Info: not scanned X-KSMG-AntiPhishing: not scanned, disabled by settings X-KSMG-AntiVirus: Kaspersky Secure Mail Gateway, version 1.1.2.30, bases: 2023/06/22 09:26:00 #21554371 X-KSMG-AntiVirus-Status: Clean, skipped X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_NONE, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jun 12, 2023 at 11:57:21AM +0200, Neil Armstrong wrote: > The way hw_onecell_data is declared: > struct clk_hw_onecell_data { > unsigned int num; > struct clk_hw *hws[]; > }; > > makes it impossible to have the clk_hw table declared outside while > using ARRAY_SIZE() to determine ".num" due to ".hws" being a flexible > array member. > > Completely move out of hw_onecell_data and add a custom > devm_of_clk_add_hw_provider() "get" callback to retrieve the clk_hw > in order to finally get rid on the NR_CLKS define. > > Signed-off-by: Neil Armstrong > --- > drivers/clk/meson/Kconfig | 2 + > drivers/clk/meson/a1-peripherals.c | 323 +++++++++++++++++++------------------ > drivers/clk/meson/a1-peripherals.h | 1 - > drivers/clk/meson/a1-pll.c | 36 +++-- > drivers/clk/meson/a1-pll.h | 1 - > 5 files changed, 183 insertions(+), 180 deletions(-) > > diff --git a/drivers/clk/meson/Kconfig b/drivers/clk/meson/Kconfig > index caadaf973317..7ae076cd9645 100644 > --- a/drivers/clk/meson/Kconfig > +++ b/drivers/clk/meson/Kconfig > @@ -108,6 +108,7 @@ config COMMON_CLK_A1_PLL > tristate "Amlogic A1 SoC PLL controller support" > depends on ARM64 > select COMMON_CLK_MESON_REGMAP > + select COMMON_CLK_MESON_CLKC_UTILS > select COMMON_CLK_MESON_PLL > help > Support for the PLL clock controller on Amlogic A113L based > @@ -119,6 +120,7 @@ config COMMON_CLK_A1_PERIPHERALS > depends on ARM64 > select COMMON_CLK_MESON_DUALDIV > select COMMON_CLK_MESON_REGMAP > + select COMMON_CLK_MESON_CLKC_UTILS > help > Support for the Peripherals clock controller on Amlogic A113L based > device, A1 SoC Family. Say Y if you want A1 Peripherals clock > diff --git a/drivers/clk/meson/a1-peripherals.c b/drivers/clk/meson/a1-peripherals.c > index b320134fefeb..a5cab418736a 100644 > --- a/drivers/clk/meson/a1-peripherals.c > +++ b/drivers/clk/meson/a1-peripherals.c > @@ -13,6 +13,7 @@ > #include "a1-peripherals.h" > #include "clk-dualdiv.h" > #include "clk-regmap.h" > +#include "meson-clkc-utils.h" > > static struct clk_regmap xtal_in = { > .data = &(struct clk_regmap_gate_data){ > @@ -1866,165 +1867,161 @@ static MESON_GATE(rom, AXI_CLK_EN, 11); > static MESON_GATE(prod_i2c, AXI_CLK_EN, 12); > > /* Array of all clocks registered by this provider */ > -static struct clk_hw_onecell_data a1_periphs_clks = { > - .hws = { > - [CLKID_XTAL_IN] = &xtal_in.hw, > - [CLKID_FIXPLL_IN] = &fixpll_in.hw, > - [CLKID_USB_PHY_IN] = &usb_phy_in.hw, > - [CLKID_USB_CTRL_IN] = &usb_ctrl_in.hw, > - [CLKID_HIFIPLL_IN] = &hifipll_in.hw, > - [CLKID_SYSPLL_IN] = &syspll_in.hw, > - [CLKID_DDS_IN] = &dds_in.hw, > - [CLKID_SYS] = &sys.hw, > - [CLKID_CLKTREE] = &clktree.hw, > - [CLKID_RESET_CTRL] = &reset_ctrl.hw, > - [CLKID_ANALOG_CTRL] = &analog_ctrl.hw, > - [CLKID_PWR_CTRL] = &pwr_ctrl.hw, > - [CLKID_PAD_CTRL] = &pad_ctrl.hw, > - [CLKID_SYS_CTRL] = &sys_ctrl.hw, > - [CLKID_TEMP_SENSOR] = &temp_sensor.hw, > - [CLKID_AM2AXI_DIV] = &am2axi_dev.hw, > - [CLKID_SPICC_B] = &spicc_b.hw, > - [CLKID_SPICC_A] = &spicc_a.hw, > - [CLKID_MSR] = &msr.hw, > - [CLKID_AUDIO] = &audio.hw, > - [CLKID_JTAG_CTRL] = &jtag_ctrl.hw, > - [CLKID_SARADC_EN] = &saradc_en.hw, > - [CLKID_PWM_EF] = &pwm_ef.hw, > - [CLKID_PWM_CD] = &pwm_cd.hw, > - [CLKID_PWM_AB] = &pwm_ab.hw, > - [CLKID_CEC] = &cec.hw, > - [CLKID_I2C_S] = &i2c_s.hw, > - [CLKID_IR_CTRL] = &ir_ctrl.hw, > - [CLKID_I2C_M_D] = &i2c_m_d.hw, > - [CLKID_I2C_M_C] = &i2c_m_c.hw, > - [CLKID_I2C_M_B] = &i2c_m_b.hw, > - [CLKID_I2C_M_A] = &i2c_m_a.hw, > - [CLKID_ACODEC] = &acodec.hw, > - [CLKID_OTP] = &otp.hw, > - [CLKID_SD_EMMC_A] = &sd_emmc_a.hw, > - [CLKID_USB_PHY] = &usb_phy.hw, > - [CLKID_USB_CTRL] = &usb_ctrl.hw, > - [CLKID_SYS_DSPB] = &sys_dspb.hw, > - [CLKID_SYS_DSPA] = &sys_dspa.hw, > - [CLKID_DMA] = &dma.hw, > - [CLKID_IRQ_CTRL] = &irq_ctrl.hw, > - [CLKID_NIC] = &nic.hw, > - [CLKID_GIC] = &gic.hw, > - [CLKID_UART_C] = &uart_c.hw, > - [CLKID_UART_B] = &uart_b.hw, > - [CLKID_UART_A] = &uart_a.hw, > - [CLKID_SYS_PSRAM] = &sys_psram.hw, > - [CLKID_RSA] = &rsa.hw, > - [CLKID_CORESIGHT] = &coresight.hw, > - [CLKID_AM2AXI_VAD] = &am2axi_vad.hw, > - [CLKID_AUDIO_VAD] = &audio_vad.hw, > - [CLKID_AXI_DMC] = &axi_dmc.hw, > - [CLKID_AXI_PSRAM] = &axi_psram.hw, > - [CLKID_RAMB] = &ramb.hw, > - [CLKID_RAMA] = &rama.hw, > - [CLKID_AXI_SPIFC] = &axi_spifc.hw, > - [CLKID_AXI_NIC] = &axi_nic.hw, > - [CLKID_AXI_DMA] = &axi_dma.hw, > - [CLKID_CPU_CTRL] = &cpu_ctrl.hw, > - [CLKID_ROM] = &rom.hw, > - [CLKID_PROC_I2C] = &prod_i2c.hw, > - [CLKID_DSPA_SEL] = &dspa_sel.hw, > - [CLKID_DSPB_SEL] = &dspb_sel.hw, > - [CLKID_DSPA_EN] = &dspa_en.hw, > - [CLKID_DSPA_EN_NIC] = &dspa_en_nic.hw, > - [CLKID_DSPB_EN] = &dspb_en.hw, > - [CLKID_DSPB_EN_NIC] = &dspb_en_nic.hw, > - [CLKID_RTC] = &rtc.hw, > - [CLKID_CECA_32K] = &ceca_32k_out.hw, > - [CLKID_CECB_32K] = &cecb_32k_out.hw, > - [CLKID_24M] = &clk_24m.hw, > - [CLKID_12M] = &clk_12m.hw, > - [CLKID_FCLK_DIV2_DIVN] = &fclk_div2_divn.hw, > - [CLKID_GEN] = &gen.hw, > - [CLKID_SARADC_SEL] = &saradc_sel.hw, > - [CLKID_SARADC] = &saradc.hw, > - [CLKID_PWM_A] = &pwm_a.hw, > - [CLKID_PWM_B] = &pwm_b.hw, > - [CLKID_PWM_C] = &pwm_c.hw, > - [CLKID_PWM_D] = &pwm_d.hw, > - [CLKID_PWM_E] = &pwm_e.hw, > - [CLKID_PWM_F] = &pwm_f.hw, > - [CLKID_SPICC] = &spicc.hw, > - [CLKID_TS] = &ts.hw, > - [CLKID_SPIFC] = &spifc.hw, > - [CLKID_USB_BUS] = &usb_bus.hw, > - [CLKID_SD_EMMC] = &sd_emmc.hw, > - [CLKID_PSRAM] = &psram.hw, > - [CLKID_DMC] = &dmc.hw, > - [CLKID_SYS_A_SEL] = &sys_a_sel.hw, > - [CLKID_SYS_A_DIV] = &sys_a_div.hw, > - [CLKID_SYS_A] = &sys_a.hw, > - [CLKID_SYS_B_SEL] = &sys_b_sel.hw, > - [CLKID_SYS_B_DIV] = &sys_b_div.hw, > - [CLKID_SYS_B] = &sys_b.hw, > - [CLKID_DSPA_A_SEL] = &dspa_a_sel.hw, > - [CLKID_DSPA_A_DIV] = &dspa_a_div.hw, > - [CLKID_DSPA_A] = &dspa_a.hw, > - [CLKID_DSPA_B_SEL] = &dspa_b_sel.hw, > - [CLKID_DSPA_B_DIV] = &dspa_b_div.hw, > - [CLKID_DSPA_B] = &dspa_b.hw, > - [CLKID_DSPB_A_SEL] = &dspb_a_sel.hw, > - [CLKID_DSPB_A_DIV] = &dspb_a_div.hw, > - [CLKID_DSPB_A] = &dspb_a.hw, > - [CLKID_DSPB_B_SEL] = &dspb_b_sel.hw, > - [CLKID_DSPB_B_DIV] = &dspb_b_div.hw, > - [CLKID_DSPB_B] = &dspb_b.hw, > - [CLKID_RTC_32K_IN] = &rtc_32k_in.hw, > - [CLKID_RTC_32K_DIV] = &rtc_32k_div.hw, > - [CLKID_RTC_32K_XTAL] = &rtc_32k_xtal.hw, > - [CLKID_RTC_32K_SEL] = &rtc_32k_sel.hw, > - [CLKID_CECB_32K_IN] = &cecb_32k_in.hw, > - [CLKID_CECB_32K_DIV] = &cecb_32k_div.hw, > - [CLKID_CECB_32K_SEL_PRE] = &cecb_32k_sel_pre.hw, > - [CLKID_CECB_32K_SEL] = &cecb_32k_sel.hw, > - [CLKID_CECA_32K_IN] = &ceca_32k_in.hw, > - [CLKID_CECA_32K_DIV] = &ceca_32k_div.hw, > - [CLKID_CECA_32K_SEL_PRE] = &ceca_32k_sel_pre.hw, > - [CLKID_CECA_32K_SEL] = &ceca_32k_sel.hw, > - [CLKID_DIV2_PRE] = &fclk_div2_divn_pre.hw, > - [CLKID_24M_DIV2] = &clk_24m_div2.hw, > - [CLKID_GEN_SEL] = &gen_sel.hw, > - [CLKID_GEN_DIV] = &gen_div.hw, > - [CLKID_SARADC_DIV] = &saradc_div.hw, > - [CLKID_PWM_A_SEL] = &pwm_a_sel.hw, > - [CLKID_PWM_A_DIV] = &pwm_a_div.hw, > - [CLKID_PWM_B_SEL] = &pwm_b_sel.hw, > - [CLKID_PWM_B_DIV] = &pwm_b_div.hw, > - [CLKID_PWM_C_SEL] = &pwm_c_sel.hw, > - [CLKID_PWM_C_DIV] = &pwm_c_div.hw, > - [CLKID_PWM_D_SEL] = &pwm_d_sel.hw, > - [CLKID_PWM_D_DIV] = &pwm_d_div.hw, > - [CLKID_PWM_E_SEL] = &pwm_e_sel.hw, > - [CLKID_PWM_E_DIV] = &pwm_e_div.hw, > - [CLKID_PWM_F_SEL] = &pwm_f_sel.hw, > - [CLKID_PWM_F_DIV] = &pwm_f_div.hw, > - [CLKID_SPICC_SEL] = &spicc_sel.hw, > - [CLKID_SPICC_DIV] = &spicc_div.hw, > - [CLKID_SPICC_SEL2] = &spicc_sel2.hw, > - [CLKID_TS_DIV] = &ts_div.hw, > - [CLKID_SPIFC_SEL] = &spifc_sel.hw, > - [CLKID_SPIFC_DIV] = &spifc_div.hw, > - [CLKID_SPIFC_SEL2] = &spifc_sel2.hw, > - [CLKID_USB_BUS_SEL] = &usb_bus_sel.hw, > - [CLKID_USB_BUS_DIV] = &usb_bus_div.hw, > - [CLKID_SD_EMMC_SEL] = &sd_emmc_sel.hw, > - [CLKID_SD_EMMC_DIV] = &sd_emmc_div.hw, > - [CLKID_SD_EMMC_SEL2] = &sd_emmc_sel2.hw, > - [CLKID_PSRAM_SEL] = &psram_sel.hw, > - [CLKID_PSRAM_DIV] = &psram_div.hw, > - [CLKID_PSRAM_SEL2] = &psram_sel2.hw, > - [CLKID_DMC_SEL] = &dmc_sel.hw, > - [CLKID_DMC_DIV] = &dmc_div.hw, > - [CLKID_DMC_SEL2] = &dmc_sel2.hw, > - [NR_CLKS] = NULL, > - }, > - .num = NR_CLKS, > +static struct clk_hw *a1_periphs_hw_clks[] = { > + [CLKID_XTAL_IN] = &xtal_in.hw, > + [CLKID_FIXPLL_IN] = &fixpll_in.hw, > + [CLKID_USB_PHY_IN] = &usb_phy_in.hw, > + [CLKID_USB_CTRL_IN] = &usb_ctrl_in.hw, > + [CLKID_HIFIPLL_IN] = &hifipll_in.hw, > + [CLKID_SYSPLL_IN] = &syspll_in.hw, > + [CLKID_DDS_IN] = &dds_in.hw, > + [CLKID_SYS] = &sys.hw, > + [CLKID_CLKTREE] = &clktree.hw, > + [CLKID_RESET_CTRL] = &reset_ctrl.hw, > + [CLKID_ANALOG_CTRL] = &analog_ctrl.hw, > + [CLKID_PWR_CTRL] = &pwr_ctrl.hw, > + [CLKID_PAD_CTRL] = &pad_ctrl.hw, > + [CLKID_SYS_CTRL] = &sys_ctrl.hw, > + [CLKID_TEMP_SENSOR] = &temp_sensor.hw, > + [CLKID_AM2AXI_DIV] = &am2axi_dev.hw, > + [CLKID_SPICC_B] = &spicc_b.hw, > + [CLKID_SPICC_A] = &spicc_a.hw, > + [CLKID_MSR] = &msr.hw, > + [CLKID_AUDIO] = &audio.hw, > + [CLKID_JTAG_CTRL] = &jtag_ctrl.hw, > + [CLKID_SARADC_EN] = &saradc_en.hw, > + [CLKID_PWM_EF] = &pwm_ef.hw, > + [CLKID_PWM_CD] = &pwm_cd.hw, > + [CLKID_PWM_AB] = &pwm_ab.hw, > + [CLKID_CEC] = &cec.hw, > + [CLKID_I2C_S] = &i2c_s.hw, > + [CLKID_IR_CTRL] = &ir_ctrl.hw, > + [CLKID_I2C_M_D] = &i2c_m_d.hw, > + [CLKID_I2C_M_C] = &i2c_m_c.hw, > + [CLKID_I2C_M_B] = &i2c_m_b.hw, > + [CLKID_I2C_M_A] = &i2c_m_a.hw, > + [CLKID_ACODEC] = &acodec.hw, > + [CLKID_OTP] = &otp.hw, > + [CLKID_SD_EMMC_A] = &sd_emmc_a.hw, > + [CLKID_USB_PHY] = &usb_phy.hw, > + [CLKID_USB_CTRL] = &usb_ctrl.hw, > + [CLKID_SYS_DSPB] = &sys_dspb.hw, > + [CLKID_SYS_DSPA] = &sys_dspa.hw, > + [CLKID_DMA] = &dma.hw, > + [CLKID_IRQ_CTRL] = &irq_ctrl.hw, > + [CLKID_NIC] = &nic.hw, > + [CLKID_GIC] = &gic.hw, > + [CLKID_UART_C] = &uart_c.hw, > + [CLKID_UART_B] = &uart_b.hw, > + [CLKID_UART_A] = &uart_a.hw, > + [CLKID_SYS_PSRAM] = &sys_psram.hw, > + [CLKID_RSA] = &rsa.hw, > + [CLKID_CORESIGHT] = &coresight.hw, > + [CLKID_AM2AXI_VAD] = &am2axi_vad.hw, > + [CLKID_AUDIO_VAD] = &audio_vad.hw, > + [CLKID_AXI_DMC] = &axi_dmc.hw, > + [CLKID_AXI_PSRAM] = &axi_psram.hw, > + [CLKID_RAMB] = &ramb.hw, > + [CLKID_RAMA] = &rama.hw, > + [CLKID_AXI_SPIFC] = &axi_spifc.hw, > + [CLKID_AXI_NIC] = &axi_nic.hw, > + [CLKID_AXI_DMA] = &axi_dma.hw, > + [CLKID_CPU_CTRL] = &cpu_ctrl.hw, > + [CLKID_ROM] = &rom.hw, > + [CLKID_PROC_I2C] = &prod_i2c.hw, > + [CLKID_DSPA_SEL] = &dspa_sel.hw, > + [CLKID_DSPB_SEL] = &dspb_sel.hw, > + [CLKID_DSPA_EN] = &dspa_en.hw, > + [CLKID_DSPA_EN_NIC] = &dspa_en_nic.hw, > + [CLKID_DSPB_EN] = &dspb_en.hw, > + [CLKID_DSPB_EN_NIC] = &dspb_en_nic.hw, > + [CLKID_RTC] = &rtc.hw, > + [CLKID_CECA_32K] = &ceca_32k_out.hw, > + [CLKID_CECB_32K] = &cecb_32k_out.hw, > + [CLKID_24M] = &clk_24m.hw, > + [CLKID_12M] = &clk_12m.hw, > + [CLKID_FCLK_DIV2_DIVN] = &fclk_div2_divn.hw, > + [CLKID_GEN] = &gen.hw, > + [CLKID_SARADC_SEL] = &saradc_sel.hw, > + [CLKID_SARADC] = &saradc.hw, > + [CLKID_PWM_A] = &pwm_a.hw, > + [CLKID_PWM_B] = &pwm_b.hw, > + [CLKID_PWM_C] = &pwm_c.hw, > + [CLKID_PWM_D] = &pwm_d.hw, > + [CLKID_PWM_E] = &pwm_e.hw, > + [CLKID_PWM_F] = &pwm_f.hw, > + [CLKID_SPICC] = &spicc.hw, > + [CLKID_TS] = &ts.hw, > + [CLKID_SPIFC] = &spifc.hw, > + [CLKID_USB_BUS] = &usb_bus.hw, > + [CLKID_SD_EMMC] = &sd_emmc.hw, > + [CLKID_PSRAM] = &psram.hw, > + [CLKID_DMC] = &dmc.hw, > + [CLKID_SYS_A_SEL] = &sys_a_sel.hw, > + [CLKID_SYS_A_DIV] = &sys_a_div.hw, > + [CLKID_SYS_A] = &sys_a.hw, > + [CLKID_SYS_B_SEL] = &sys_b_sel.hw, > + [CLKID_SYS_B_DIV] = &sys_b_div.hw, > + [CLKID_SYS_B] = &sys_b.hw, > + [CLKID_DSPA_A_SEL] = &dspa_a_sel.hw, > + [CLKID_DSPA_A_DIV] = &dspa_a_div.hw, > + [CLKID_DSPA_A] = &dspa_a.hw, > + [CLKID_DSPA_B_SEL] = &dspa_b_sel.hw, > + [CLKID_DSPA_B_DIV] = &dspa_b_div.hw, > + [CLKID_DSPA_B] = &dspa_b.hw, > + [CLKID_DSPB_A_SEL] = &dspb_a_sel.hw, > + [CLKID_DSPB_A_DIV] = &dspb_a_div.hw, > + [CLKID_DSPB_A] = &dspb_a.hw, > + [CLKID_DSPB_B_SEL] = &dspb_b_sel.hw, > + [CLKID_DSPB_B_DIV] = &dspb_b_div.hw, > + [CLKID_DSPB_B] = &dspb_b.hw, > + [CLKID_RTC_32K_IN] = &rtc_32k_in.hw, > + [CLKID_RTC_32K_DIV] = &rtc_32k_div.hw, > + [CLKID_RTC_32K_XTAL] = &rtc_32k_xtal.hw, > + [CLKID_RTC_32K_SEL] = &rtc_32k_sel.hw, > + [CLKID_CECB_32K_IN] = &cecb_32k_in.hw, > + [CLKID_CECB_32K_DIV] = &cecb_32k_div.hw, > + [CLKID_CECB_32K_SEL_PRE] = &cecb_32k_sel_pre.hw, > + [CLKID_CECB_32K_SEL] = &cecb_32k_sel.hw, > + [CLKID_CECA_32K_IN] = &ceca_32k_in.hw, > + [CLKID_CECA_32K_DIV] = &ceca_32k_div.hw, > + [CLKID_CECA_32K_SEL_PRE] = &ceca_32k_sel_pre.hw, > + [CLKID_CECA_32K_SEL] = &ceca_32k_sel.hw, > + [CLKID_DIV2_PRE] = &fclk_div2_divn_pre.hw, > + [CLKID_24M_DIV2] = &clk_24m_div2.hw, > + [CLKID_GEN_SEL] = &gen_sel.hw, > + [CLKID_GEN_DIV] = &gen_div.hw, > + [CLKID_SARADC_DIV] = &saradc_div.hw, > + [CLKID_PWM_A_SEL] = &pwm_a_sel.hw, > + [CLKID_PWM_A_DIV] = &pwm_a_div.hw, > + [CLKID_PWM_B_SEL] = &pwm_b_sel.hw, > + [CLKID_PWM_B_DIV] = &pwm_b_div.hw, > + [CLKID_PWM_C_SEL] = &pwm_c_sel.hw, > + [CLKID_PWM_C_DIV] = &pwm_c_div.hw, > + [CLKID_PWM_D_SEL] = &pwm_d_sel.hw, > + [CLKID_PWM_D_DIV] = &pwm_d_div.hw, > + [CLKID_PWM_E_SEL] = &pwm_e_sel.hw, > + [CLKID_PWM_E_DIV] = &pwm_e_div.hw, > + [CLKID_PWM_F_SEL] = &pwm_f_sel.hw, > + [CLKID_PWM_F_DIV] = &pwm_f_div.hw, > + [CLKID_SPICC_SEL] = &spicc_sel.hw, > + [CLKID_SPICC_DIV] = &spicc_div.hw, > + [CLKID_SPICC_SEL2] = &spicc_sel2.hw, > + [CLKID_TS_DIV] = &ts_div.hw, > + [CLKID_SPIFC_SEL] = &spifc_sel.hw, > + [CLKID_SPIFC_DIV] = &spifc_div.hw, > + [CLKID_SPIFC_SEL2] = &spifc_sel2.hw, > + [CLKID_USB_BUS_SEL] = &usb_bus_sel.hw, > + [CLKID_USB_BUS_DIV] = &usb_bus_div.hw, > + [CLKID_SD_EMMC_SEL] = &sd_emmc_sel.hw, > + [CLKID_SD_EMMC_DIV] = &sd_emmc_div.hw, > + [CLKID_SD_EMMC_SEL2] = &sd_emmc_sel2.hw, > + [CLKID_PSRAM_SEL] = &psram_sel.hw, > + [CLKID_PSRAM_DIV] = &psram_div.hw, > + [CLKID_PSRAM_SEL2] = &psram_sel2.hw, > + [CLKID_DMC_SEL] = &dmc_sel.hw, > + [CLKID_DMC_DIV] = &dmc_div.hw, > + [CLKID_DMC_SEL2] = &dmc_sel2.hw, > }; > > /* Convenience table to populate regmap in .probe */ > @@ -2190,6 +2187,11 @@ static struct regmap_config a1_periphs_regmap_cfg = { > .reg_stride = 4, > }; > > +static struct meson_clk_hw_data a1_periphs_clks = { > + .hws = a1_periphs_hw_clks, > + .num = ARRAY_SIZE(a1_periphs_hw_clks), > +}; > + > static int meson_a1_periphs_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > @@ -2219,8 +2221,7 @@ static int meson_a1_periphs_probe(struct platform_device *pdev) > clkid); > } > > - return devm_of_clk_add_hw_provider(dev, of_clk_hw_onecell_get, > - &a1_periphs_clks); > + return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, &a1_periphs_clks); > } > > static const struct of_device_id a1_periphs_clkc_match_table[] = { > diff --git a/drivers/clk/meson/a1-peripherals.h b/drivers/clk/meson/a1-peripherals.h > index 526fc9ba5c9f..4d60456a95a9 100644 > --- a/drivers/clk/meson/a1-peripherals.h > +++ b/drivers/clk/meson/a1-peripherals.h > @@ -108,6 +108,5 @@ > #define CLKID_DMC_SEL 151 > #define CLKID_DMC_DIV 152 > #define CLKID_DMC_SEL2 153 > -#define NR_CLKS 154 > > #endif /* __A1_PERIPHERALS_H */ > diff --git a/drivers/clk/meson/a1-pll.c b/drivers/clk/meson/a1-pll.c > index bd2f1d1ec6e4..25e6b567afd5 100644 > --- a/drivers/clk/meson/a1-pll.c > +++ b/drivers/clk/meson/a1-pll.c > @@ -12,6 +12,7 @@ > #include > #include "a1-pll.h" > #include "clk-regmap.h" > +#include "meson-clkc-utils.h" > > static struct clk_regmap fixed_pll_dco = { > .data = &(struct meson_clk_pll_data){ > @@ -268,22 +269,18 @@ static struct clk_regmap fclk_div7 = { > }; > > /* Array of all clocks registered by this provider */ > -static struct clk_hw_onecell_data a1_pll_clks = { > - .hws = { > - [CLKID_FIXED_PLL_DCO] = &fixed_pll_dco.hw, > - [CLKID_FIXED_PLL] = &fixed_pll.hw, > - [CLKID_FCLK_DIV2_DIV] = &fclk_div2_div.hw, > - [CLKID_FCLK_DIV3_DIV] = &fclk_div3_div.hw, > - [CLKID_FCLK_DIV5_DIV] = &fclk_div5_div.hw, > - [CLKID_FCLK_DIV7_DIV] = &fclk_div7_div.hw, > - [CLKID_FCLK_DIV2] = &fclk_div2.hw, > - [CLKID_FCLK_DIV3] = &fclk_div3.hw, > - [CLKID_FCLK_DIV5] = &fclk_div5.hw, > - [CLKID_FCLK_DIV7] = &fclk_div7.hw, > - [CLKID_HIFI_PLL] = &hifi_pll.hw, > - [NR_PLL_CLKS] = NULL, > - }, > - .num = NR_PLL_CLKS, > +static struct clk_hw *a1_pll_hw_clks[] = { > + [CLKID_FIXED_PLL_DCO] = &fixed_pll_dco.hw, > + [CLKID_FIXED_PLL] = &fixed_pll.hw, > + [CLKID_FCLK_DIV2_DIV] = &fclk_div2_div.hw, > + [CLKID_FCLK_DIV3_DIV] = &fclk_div3_div.hw, > + [CLKID_FCLK_DIV5_DIV] = &fclk_div5_div.hw, > + [CLKID_FCLK_DIV7_DIV] = &fclk_div7_div.hw, > + [CLKID_FCLK_DIV2] = &fclk_div2.hw, > + [CLKID_FCLK_DIV3] = &fclk_div3.hw, > + [CLKID_FCLK_DIV5] = &fclk_div5.hw, > + [CLKID_FCLK_DIV7] = &fclk_div7.hw, > + [CLKID_HIFI_PLL] = &hifi_pll.hw, > }; > > static struct clk_regmap *const a1_pll_regmaps[] = { > @@ -302,6 +299,11 @@ static struct regmap_config a1_pll_regmap_cfg = { > .reg_stride = 4, > }; > > +static struct meson_clk_hw_data a1_pll_clks = { > + .hws = a1_pll_hw_clks, > + .num = ARRAY_SIZE(a1_pll_hw_clks), > +}; > + > static int meson_a1_pll_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > @@ -332,7 +334,7 @@ static int meson_a1_pll_probe(struct platform_device *pdev) > clkid); > } > > - return devm_of_clk_add_hw_provider(dev, of_clk_hw_onecell_get, > + return devm_of_clk_add_hw_provider(dev, meson_clk_hw_get, > &a1_pll_clks); > } > > diff --git a/drivers/clk/meson/a1-pll.h b/drivers/clk/meson/a1-pll.h > index 29726651b056..82570759e6a2 100644 > --- a/drivers/clk/meson/a1-pll.h > +++ b/drivers/clk/meson/a1-pll.h > @@ -42,6 +42,5 @@ > #define CLKID_FCLK_DIV3_DIV 3 > #define CLKID_FCLK_DIV5_DIV 4 > #define CLKID_FCLK_DIV7_DIV 5 > -#define NR_PLL_CLKS 11 > > #endif /* __A1_PLL_H */ > Reviewed-by: Dmitry Rokosov -- Thank you, Dmitry