From nobody Mon Feb 9 16:47:06 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2C23FC001E0 for ; Tue, 1 Aug 2023 10:36:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232533AbjHAKgk (ORCPT ); Tue, 1 Aug 2023 06:36:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52484 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232242AbjHAKgd (ORCPT ); Tue, 1 Aug 2023 06:36:33 -0400 Received: from mx1.tq-group.com (mx1.tq-group.com [93.104.207.81]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0864ADB; Tue, 1 Aug 2023 03:36:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1690886192; x=1722422192; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=AyHQfuQ25AxPTEIiRutOr2ogGOuarDxm7sZRM0h1amw=; b=m9toAB7sFDDXjnctND8mrDeSnmwujAIEamfSFOwXWFt7jsxmLkBZ250Q x1sMXdZlYASWCmP2nZTmy8tkAGdmZFRepqemD20Hv1j4MI26dA+TcmS9R 81fFpgk/BKtPSUncUkyvHMCKp8t3mEPcONiQHvbVraA6rWQpmdZMunTTw 3ZYcCnEAHXdkHu1hCiEhO/svWh8xmfdxdmQRyfyWRO+r0W00vOfL5lBOB 9bwLFRyoIizGBl/YGQQjbiDBPqu3EmrSDx0W3unwOJU2AO/fyfR7d0cOi jmCW4FwNHjQBQWYN08V7pOQE+ZCOhpeKiH5ZA+dJBxvg8uqHE76Jc8b5M g==; X-IronPort-AV: E=Sophos;i="6.01,246,1684792800"; d="scan'208";a="32226639" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 01 Aug 2023 12:36:25 +0200 Received: from localhost.localdomain (SCHIFFERM-M2.tq-net.de [10.121.49.20]) by vtuxmail01.tq-net.de (Postfix) with ESMTPA id D6DFC280084; Tue, 1 Aug 2023 12:36:24 +0200 (CEST) From: Matthias Schiffer To: Santosh Shilimkar , Michael Turquette , Stephen Boyd Cc: Jai Luthra , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux@ew.tq-group.com, Matthias Schiffer Subject: [PATCH 1/3] clk: keystone: syscon-clk: use struct instead of array for match data Date: Tue, 1 Aug 2023 12:36:07 +0200 Message-Id: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Allow to store information with the match data that is not specific to the individual clock entries. Counting the number of elements to a sentinel at runtime is replaced with the usual ARRAY_SIZE() approach. No functional change intended. Fixes: 6acab96ee337 ("clk: keystone: syscon-clk: Add support for audio refc= lk") Signed-off-by: Matthias Schiffer --- drivers/clk/keystone/syscon-clk.c | 55 +++++++++++++++++++------------ 1 file changed, 34 insertions(+), 21 deletions(-) diff --git a/drivers/clk/keystone/syscon-clk.c b/drivers/clk/keystone/sysco= n-clk.c index d33f74119488..68ac536f564e 100644 --- a/drivers/clk/keystone/syscon-clk.c +++ b/drivers/clk/keystone/syscon-clk.c @@ -18,12 +18,17 @@ struct ti_syscon_gate_clk_priv { u32 idx; }; =20 -struct ti_syscon_gate_clk_data { +struct ti_syscon_gate_clk_entry { char *name; u32 offset; u32 bit_idx; }; =20 +struct ti_syscon_gate_clk_data { + const struct ti_syscon_gate_clk_entry *clks; + size_t num_clks; +}; + static struct ti_syscon_gate_clk_priv *to_ti_syscon_gate_clk_priv(struct clk_hw *hw) { @@ -64,7 +69,7 @@ static const struct clk_ops ti_syscon_gate_clk_ops =3D { static struct clk_hw *ti_syscon_gate_clk_register(struct device *dev, struct regmap *regmap, const char *parent_name, - const struct ti_syscon_gate_clk_data *data) + const struct ti_syscon_gate_clk_entry *data) { struct ti_syscon_gate_clk_priv *priv; struct clk_init_data init; @@ -107,10 +112,10 @@ static struct clk_hw =20 static int ti_syscon_gate_clk_probe(struct platform_device *pdev) { - const struct ti_syscon_gate_clk_data *data, *p; + const struct ti_syscon_gate_clk_data *data; struct clk_hw_onecell_data *hw_data; struct device *dev =3D &pdev->dev; - int num_clks, num_parents, i; + int num_parents, i; const char *parent_name; struct regmap *regmap; =20 @@ -123,10 +128,6 @@ static int ti_syscon_gate_clk_probe(struct platform_de= vice *pdev) return dev_err_probe(dev, PTR_ERR(regmap), "failed to get regmap\n"); =20 - num_clks =3D 0; - for (p =3D data; p->name; p++) - num_clks++; - num_parents =3D of_clk_get_parent_count(dev->of_node); if (of_device_is_compatible(dev->of_node, "ti,am62-audio-refclk") && num_parents =3D=3D 0) { @@ -134,21 +135,21 @@ static int ti_syscon_gate_clk_probe(struct platform_d= evice *pdev) "must specify a parent clock\n"); } =20 - hw_data =3D devm_kzalloc(dev, struct_size(hw_data, hws, num_clks), + hw_data =3D devm_kzalloc(dev, struct_size(hw_data, hws, data->num_clks), GFP_KERNEL); if (!hw_data) return -ENOMEM; =20 - hw_data->num =3D num_clks; + hw_data->num =3D data->num_clks; =20 parent_name =3D of_clk_get_parent_name(dev->of_node, 0); - for (i =3D 0; i < num_clks; i++) { + for (i =3D 0; i < data->num_clks; i++) { hw_data->hws[i] =3D ti_syscon_gate_clk_register(dev, regmap, parent_name, - &data[i]); + &data->clks[i]); if (IS_ERR(hw_data->hws[i])) dev_warn(dev, "failed to register %s\n", - data[i].name); + data->clks[i].name); } =20 return devm_of_clk_add_hw_provider(dev, of_clk_hw_onecell_get, @@ -162,17 +163,20 @@ static int ti_syscon_gate_clk_probe(struct platform_d= evice *pdev) .bit_idx =3D (_bit_idx), \ } =20 -static const struct ti_syscon_gate_clk_data am654_clk_data[] =3D { +static const struct ti_syscon_gate_clk_entry am654_clks[] =3D { TI_SYSCON_CLK_GATE("ehrpwm_tbclk0", 0x0, 0), TI_SYSCON_CLK_GATE("ehrpwm_tbclk1", 0x4, 0), TI_SYSCON_CLK_GATE("ehrpwm_tbclk2", 0x8, 0), TI_SYSCON_CLK_GATE("ehrpwm_tbclk3", 0xc, 0), TI_SYSCON_CLK_GATE("ehrpwm_tbclk4", 0x10, 0), TI_SYSCON_CLK_GATE("ehrpwm_tbclk5", 0x14, 0), - { /* Sentinel */ }, +}; +static const struct ti_syscon_gate_clk_data am654_clk_data =3D { + .clks =3D am654_clks, + .num_clks =3D ARRAY_SIZE(am654_clks), }; =20 -static const struct ti_syscon_gate_clk_data am64_clk_data[] =3D { +static const struct ti_syscon_gate_clk_entry am64_clks[] =3D { TI_SYSCON_CLK_GATE("epwm_tbclk0", 0x0, 0), TI_SYSCON_CLK_GATE("epwm_tbclk1", 0x0, 1), TI_SYSCON_CLK_GATE("epwm_tbclk2", 0x0, 2), @@ -182,19 +186,28 @@ static const struct ti_syscon_gate_clk_data am64_clk_= data[] =3D { TI_SYSCON_CLK_GATE("epwm_tbclk6", 0x0, 6), TI_SYSCON_CLK_GATE("epwm_tbclk7", 0x0, 7), TI_SYSCON_CLK_GATE("epwm_tbclk8", 0x0, 8), - { /* Sentinel */ }, +}; +static const struct ti_syscon_gate_clk_data am64_clk_data =3D { + .clks =3D am64_clks, + .num_clks =3D ARRAY_SIZE(am64_clks), }; =20 -static const struct ti_syscon_gate_clk_data am62_clk_data[] =3D { +static const struct ti_syscon_gate_clk_entry am62_clks[] =3D { TI_SYSCON_CLK_GATE("epwm_tbclk0", 0x0, 0), TI_SYSCON_CLK_GATE("epwm_tbclk1", 0x0, 1), TI_SYSCON_CLK_GATE("epwm_tbclk2", 0x0, 2), - { /* Sentinel */ }, +}; +static const struct ti_syscon_gate_clk_data am62_clk_data =3D { + .clks =3D am62_clks, + .num_clks =3D ARRAY_SIZE(am62_clks), }; =20 -static const struct ti_syscon_gate_clk_data am62_audio_clk_data[] =3D { +static const struct ti_syscon_gate_clk_entry am62_audio_clks[] =3D { TI_SYSCON_CLK_GATE("audio_refclk", 0x0, 15), - { /* Sentinel */ }, +}; +static const struct ti_syscon_gate_clk_data am62_audio_clk_data =3D { + .clks =3D am62_audio_clks, + .num_clks =3D ARRAY_SIZE(am62_audio_clks), }; =20 static const struct of_device_id ti_syscon_gate_clk_ids[] =3D { --=20 TQ-Systems GmbH | M=C3=BChlstra=C3=9Fe 2, Gut Delling | 82229 Seefeld, Germ= any Amtsgericht M=C3=BCnchen, HRB 105018 Gesch=C3=A4ftsf=C3=BChrer: Detlef Schneider, R=C3=BCdiger Stahl, Stefan Sch= neider https://www.tq-group.com/