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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 431B7C54F51 for ; Tue, 28 Jul 2026 13:53:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=BCxdxuc2t1F0shcsjrQHcCCpmuNiX52smi5jUY1dE5I=; b=rTGaSxLiBD+nQQ sIyhwk2V+evYCRqD8LIupBIAFWsOw65Pr/XVEI16v3i/7VOITy457gTwf1mwXmut+SrIE3oGUgosr NhnBPCUX27AgSBleMSOE31R6lMvEhN6q1Uk0gFcui9gsBEOFCZVcDaa2RzEOP9R38dvLxlj0dIH+B 9gJJVUftd923Wx9ZkjjOOw8JrPzT4BnqcCsPWcP2RpAPUp1f+kZfzjHsmDi5XcS8n8iy/2EWpy4Dq +0uV4Rl/f+YgCeYZlhfOYkeFjQYNxy6l9DRO/GlfgC12pHr+1RFmDuhrF9VnFuwQDCi3xMM5WF7KC Ym5vD9oDIrAXerWVYPgw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1woiFM-00000005OyT-3puC; Tue, 28 Jul 2026 13:53:28 +0000 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1woiFJ-00000005Owz-3L6T for linux-phy@lists.infradead.org; Tue, 28 Jul 2026 13:53:27 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1785246805; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=57/s/XNhO9Wt95H/WMpqEA5nKekp47X7yp5AMK8PfyU=; b=ZZTbjTjDiKOWzurlKhq9n+IMn9+OBl65LfQv7Ag72b15lKpI8xNcKaadhjJu9Zk0a8V3oD ui0clB0ewC97BJgPm4NQeWnj7TmnuFNqWurryYCEBAYcaDlwuKOLhl+mgOC6emWPtGjF/o Nvik653j+oYotQfksW4g4UdqAskN9T0= Received: from mail-qt1-f200.google.com (mail-qt1-f200.google.com [209.85.160.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-619-axbxhr2GNqyevE9rU98X5w-1; Tue, 28 Jul 2026 09:53:21 -0400 X-MC-Unique: axbxhr2GNqyevE9rU98X5w-1 X-Mimecast-MFC-AGG-ID: axbxhr2GNqyevE9rU98X5w_1785246801 Received: by mail-qt1-f200.google.com with SMTP id d75a77b69052e-51c0f0a0760so88216741cf.2 for ; Tue, 28 Jul 2026 06:53:21 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785246801; x=1785851601; h=user-agent:in-reply-to:content-transfer-encoding :content-disposition:content-type:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to:content-type; bh=57/s/XNhO9Wt95H/WMpqEA5nKekp47X7yp5AMK8PfyU=; b=bEGft8bRh72Hb4x/06xz6rN9hZZfIXm2GenkohRbg1qmQhjOq/Y1btbgtyWVQgC3CO i6dNRunM4Sf8aCCvs4NmVBDs2BZOZ95cF4fDc9/Nkh1A+UbyVNSy2imqrlti5PuAPSgH EREszRc7/bK68v06e+ki87yboKELT65T1yc040vFdYNfl6+WC6VdhhXzEDTL5lCggUlp TweLzQbx7iF9M5yKLNHWjW6014TwDTDUtcfNVDUiURuWEPcSt4o0F225zm0v3nKUZlB3 PbD3wDTh3V6tqxpyOLkch3sCdcDGUlPnKhQbcP6j3ZYNh69OpVUvs92YvwxzTgRJz+7R fnLQ== X-Forwarded-Encrypted: i=1; AHgh+RqDF/1DodqRjuBRN04z54jjH0pGn/dMqEKZrrf84MndQp/fj0SZ9RQ5lyzHO8QOlpktfSky9GPpuTw=@lists.infradead.org X-Gm-Message-State: AOJu0Yx0rTL0ugbMhragkeqTKtBxJ3je9oVHksV6uRjoxS8o03B7Re/o BrHv17VUzUE3zQ8dSLpPtNgBkWc9mKAsN0SbQ3KuMwHHu7HiQBcL/6AlZJQp7e7IM0IyFOAL1YW mZQMPZul6IK916CZigu8ZfWwyTZ/lh1MQw4gvLUYTKegDB/PTZUKt+GgS8NAxX1+vTwU= X-Gm-Gg: AR+sD12aJQALMZrfglN581jiIOpyrodHOqwWEvvHCQjunNZEHvLkgG5JV0va1DrcIZW fmpak5pj5Dn9N3nStGdxnWKgxS28lamN95QuQAZE4mcuPTjslIJjtfy7WwveZrMfOi2xC/YGcJv GoTvMmeNb31MDzuJjaFW7Kq/mpgShhV9y1T71QYExt7XfDXr/a3O8pjyreK+DrlmQe/NkG03Pqn /3vqJc+pmkE3SQzL/58VYUsa90SHzSn2xZ0QfYlZuKqlhu3fXoLJBG5oyZ5JBx56lF1bT6Tv42y i9htqMAkJy2mS/oUfLRSm1gF1uBAWLx9xOdPxruZvhkRCl4YB2Iq4Fmrf6xpa0CITwScDHJmdrM zrhFY76uRONKLjU48q48N68YU8oTjJ63I8HU= X-Received: by 2002:a05:622a:192a:b0:51c:c67:2042 with SMTP id d75a77b69052e-529d71da27dmr20270721cf.53.1785246800483; Tue, 28 Jul 2026 06:53:20 -0700 (PDT) X-Received: by 2002:a05:622a:192a:b0:51c:c67:2042 with SMTP id d75a77b69052e-529d71da27dmr20270051cf.53.1785246799669; Tue, 28 Jul 2026 06:53:19 -0700 (PDT) Received: from redhat.com (c-73-183-53-213.hsd1.pa.comcast.net. [73.183.53.213]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-529a2989b0bsm73997611cf.16.2026.07.28.06.53.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 28 Jul 2026 06:53:18 -0700 (PDT) Date: Tue, 28 Jul 2026 09:53:17 -0400 From: Brian Masney To: Stefan =?iso-8859-1?Q?D=F6singer?= Cc: Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , Vinod Koul , Neil Armstrong , Russell King , Lee Jones , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org, mfd@lists.linux.dev Subject: Re: [PATCH v8 05/12] clk: zte: Add Clock registration infrastructure Message-ID: References: <20260727-zx29clk-v8-0-7a107b00f1dd@gmail.com> <20260727-zx29clk-v8-5-7a107b00f1dd@gmail.com> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/2.4.0 (2026-06-19) X-Mimecast-Spam-Score: 0 X-Mimecast-MFC-PROC-ID: tDy_NUDrR7yE1y3EFrVsYQSeeTsEbTHjvMfXFsuqxY4_1785246801 X-Mimecast-Originator: redhat.com Content-Disposition: inline X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260728_065325_988801_5A7688BC X-CRM114-Status: GOOD ( 50.53 ) X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org Hi Stefan, On Tue, Jul 28, 2026 at 09:38:54AM -0400, Brian Masney wrote: > On Mon, Jul 27, 2026 at 09:24:20PM +0300, Stefan D=F6singer wrote: > > The next patches will implement the regmap clocks and PLL driver. The > > actual hardware specific clock listing will live in a separate module. > > = > > Signed-off-by: Stefan D=F6singer > > = > > --- > > = > > Version 8: > > Use ZX297520V3_CLK_NO_EXPORT=3D(~0u) for unexported clocks. While using= 0, > > and starting clock indices at 1, is a common pattern in existing > > drivers, it exposes a driver implementation detail in the hardware > > binding interface. > > = > > If desired, I can change the special index to a separate field in the > > structs. > > = > > Fix the return value if the ->init() callback fails. (Sashiko) > > = > > Version 7: > > *) Add fixed dividers to handle PLL subdivisions > > *) Never register PLLs directly as exported clocks - everything on this > > SoC goes through a gate before it leaves a controller. > > = > > Version 6: > > *) Remove auxdev now that LSP clocks also use MFD > > *) Error codepath fixes pointed out by Sashiko. > > = > > Version 5: > > = > > *) Pass the static clk data instead of calling get_match_data to prepare > > for operating as an MFD child. > > = > > *) Don't use devm_kzalloc to allocate the auxiliary_device > > structure. I guess Sashiko is right, and that's what "Because once the > > device is placed on the bus the parent driver can not tell what other > > code may have a reference to this data" is trying to tell me. > > = > > *) Fix error check for device_node_to_regmap. > > --- > > MAINTAINERS | 1 + > > drivers/clk/Kconfig | 1 + > > drivers/clk/Makefile | 1 + > > drivers/clk/zte/Kconfig | 16 +++++ > > drivers/clk/zte/Makefile | 5 ++ > > drivers/clk/zte/clk-regmap.c | 34 ++++++++++ > > drivers/clk/zte/clk-zx.c | 155 +++++++++++++++++++++++++++++++++++= ++++++++ > > drivers/clk/zte/clk-zx.h | 91 +++++++++++++++++++++++++ > > drivers/clk/zte/pll-zx.c | 16 +++++ > > 9 files changed, 320 insertions(+) > > = > > diff --git a/MAINTAINERS b/MAINTAINERS > > index b04e9d43cfb0..692f005cb2c4 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -3883,6 +3883,7 @@ F: Documentation/devicetree/bindings/clock/zte,zx= 297520v3-matrixcrm.yaml > > F: Documentation/devicetree/bindings/clock/zte,zx297520v3-topcrm.yaml > > F: arch/arm/boot/dts/zte/ > > F: arch/arm/mach-zte/ > > +F: drivers/clk/zte/ > > F: drivers/mfd/zte-zx297520v3-crm.c > > F: include/dt-bindings/clock/zte,zx297520v3-clk.h > > F: include/dt-bindings/phy/zte,zx297520v3-topcrm.h > > diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig > > index 1717ce75a907..6f0a863951ca 100644 > > --- a/drivers/clk/Kconfig > > +++ b/drivers/clk/Kconfig > > @@ -545,6 +545,7 @@ source "drivers/clk/uniphier/Kconfig" > > source "drivers/clk/visconti/Kconfig" > > source "drivers/clk/x86/Kconfig" > > source "drivers/clk/xilinx/Kconfig" > > +source "drivers/clk/zte/Kconfig" > > source "drivers/clk/zynqmp/Kconfig" > > = > > # Kunit test cases > > diff --git a/drivers/clk/Makefile b/drivers/clk/Makefile > > index cc108a75a900..13a5478f1112 100644 > > --- a/drivers/clk/Makefile > > +++ b/drivers/clk/Makefile > > @@ -167,5 +167,6 @@ ifeq ($(CONFIG_COMMON_CLK), y) > > obj-$(CONFIG_X86) +=3D x86/ > > endif > > obj-y +=3D xilinx/ > > +obj-$(CONFIG_COMMON_CLK_ZTE) +=3D zte/ > > obj-$(CONFIG_ARCH_ZYNQ) +=3D zynq/ > > obj-$(CONFIG_COMMON_CLK_ZYNQMP) +=3D zynqmp/ > > diff --git a/drivers/clk/zte/Kconfig b/drivers/clk/zte/Kconfig > > new file mode 100644 > > index 000000000000..0222549dd211 > > --- /dev/null > > +++ b/drivers/clk/zte/Kconfig > > @@ -0,0 +1,16 @@ > > +# SPDX-License-Identifier: GPL-2.0-only > > +# > > +# ZTE Clock Drivers > > +# > > + > > +config COMMON_CLK_ZTE > > + tristate "Clock driver for ZTE SoCs" > > + depends on ARCH_ZTE || COMPILE_TEST > > + default ARCH_ZTE > > + select MFD_SYSCON > > + help > > + This option selects common clock infrastructure for ZTE based SoCs. > > + You will need to enable one or more SoC specific drivers to make use > > + of this. > > + > > + Enable this if you are building a kernel for a ZTE designed board. > > diff --git a/drivers/clk/zte/Makefile b/drivers/clk/zte/Makefile > > new file mode 100644 > > index 000000000000..27db07293165 > > --- /dev/null > > +++ b/drivers/clk/zte/Makefile > > @@ -0,0 +1,5 @@ > > +# SPDX-License-Identifier: GPL-2.0-only > > + > > +obj-$(CONFIG_COMMON_CLK_ZTE) +=3D clk-zte.o > > + > > +clk-zte-y +=3D clk-zx.o pll-zx.o clk-regmap.o > > diff --git a/drivers/clk/zte/clk-regmap.c b/drivers/clk/zte/clk-regmap.c > > new file mode 100644 > > index 000000000000..984abeb45ab2 > > --- /dev/null > > +++ b/drivers/clk/zte/clk-regmap.c > > @@ -0,0 +1,34 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Copyright (c) 2014 MediaTek Inc. > > + * Copyright (c) 2018 BayLibre, SAS. > > + * Copyright (c) 2026 Stefan D=F6singer. > > + * Author: Stefan D=F6singer > > + */ > > + > > +#include > > +#include > > +#include > > +#include > = > Sort the headers. > = > > + > > +#include "clk-zx.h" > > + > > +int zx_clk_register_gates(struct device *dev, struct regmap *regmap, > > + const struct zx_gate_desc *desc, unsigned int num, > > + struct clk_hw_onecell_data *clocks) > > +{ > > + return -ENODEV; > > +} > > + > > +int zx_clk_register_dividers(struct device *dev, struct regmap *regmap, > > + const struct zx_div_desc *desc, unsigned int num) > > +{ > > + return -ENODEV; > > +} > > + > > +int zx_clk_register_muxes(struct device *dev, struct regmap *regmap, > > + const struct zx_mux_desc *desc, unsigned int num, > > + struct clk_hw_onecell_data *clocks) > > +{ > > + return -ENODEV; > > +} > > diff --git a/drivers/clk/zte/clk-zx.c b/drivers/clk/zte/clk-zx.c > > new file mode 100644 > > index 000000000000..c9e9048d3ade > > --- /dev/null > > +++ b/drivers/clk/zte/clk-zx.c > > @@ -0,0 +1,155 @@ > > +// SPDX-License-Identifier: GPL-2.0-only > > +/* > > + * Copyright (C) 2026 Stefan D=F6singer > > + */ > > + > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > = > Also sort the headers. > = > > + > > +#include "clk-zx.h" > > + > > +static int zx_clk_register_fixed_dividers(struct device *dev, struct r= egmap *regmap, > > + const struct zx_fixed_divider_desc *desc, > > + unsigned int num) > > +{ > > + struct clk_hw *clk; > > + unsigned int i; > > + > > + for (i =3D 0; i < num; ++i) { > > + clk =3D devm_clk_hw_register_fixed_factor(dev, desc[i].name, desc[i]= .parent, > > + CLK_SET_RATE_PARENT, 1, desc[i].div); > > + if (IS_ERR(clk)) { > = > The { } is not needed here. > = > > + return dev_err_probe(dev, PTR_ERR(clk), "Failed to register clk %s\= n", > > + desc[i].name); > > + } > > + } > > + > > + return 0; > > +} > > + > > +static void zx_delete_clk_provider(void *data) > > +{ > > + of_clk_del_provider(data); > > +} > > + > > +static void zx_clk_disable_unprepare_put(void *data) > > +{ > > + clk_disable_unprepare(data); > > + clk_put(data); > > +} > > + > > +int zx_clk_common_probe(struct device *dev, struct device_node *of_nod= e, > > + const struct zx_clk_data *data) > > +{ > > + unsigned int public_clk_count =3D 0, highest_id =3D 0; > > + struct clk_hw_onecell_data *clocks; > > + struct regmap *map; > > + struct clk *clk; > > + unsigned int i; > > + int res; > > + > > + map =3D device_node_to_regmap(of_node); > > + if (IS_ERR(map)) > > + return PTR_ERR(map); > > + > > + for (i =3D 0; i < data->num_muxes; ++i) { > > + if (data->muxes[i].id !=3D ZX297520V3_CLK_NO_EXPORT) { > > + if (data->muxes[i].id > highest_id) > > + highest_id =3D data->muxes[i].id; > > + public_clk_count++; > > + } > > + } > > + for (i =3D 0; i < data->num_gates; ++i) { > > + if (data->gates[i].id !=3D ZX297520V3_CLK_NO_EXPORT) { > > + if (data->gates[i].id > highest_id) > > + highest_id =3D data->gates[i].id; > > + public_clk_count++; > > + } > > + } > > + > > + if (WARN_ON(public_clk_count !=3D highest_id + 1)) > > + return -EINVAL; > > + > > + clocks =3D devm_kzalloc(dev, struct_size(clocks, hws, public_clk_coun= t), GFP_KERNEL); > > + if (!clocks) > > + return -ENOMEM; > > + clocks->num =3D public_clk_count; > > + > > + for (i =3D 0; i < data->num_inputs_enable; ++i) { > > + clk =3D of_clk_get_by_name(of_node, data->inputs_enable[i]); > > + if (IS_ERR(clk)) { > > + return dev_err_probe(dev, PTR_ERR(clk), "Input clk %s failure\n", > > + data->inputs_enable[i]); > > + } > > + > > + res =3D clk_prepare_enable(clk); > > + if (res) { > > + clk_put(clk); > > + return dev_err_probe(dev, res, "Input clk %s enable failure\n", > > + data->inputs_enable[i]); > > + } > > + res =3D devm_add_action_or_reset(dev, zx_clk_disable_unprepare_put, = clk); > > + if (res) > > + return res; > > + } > > + for (i =3D 0; i < data->num_inputs; ++i) { > > + /* FIXME: devm_get_clk_from_child doesn't do any tree traversal, so = it works here > > + * whether "of_node" belongs to "dev" or a parent of "dev". Is it su= pposed to be > > + * used that way though? > > + */ > > + clk =3D devm_get_clk_from_child(dev, of_node, data->inputs[i]); > = > I don't know the intention of the original function without digging into > the history. Can you just use of_clk_get_by_name(of_node, data->inputs[i]) > like you do in the for loop above for consistency? Looking into the other patches. There's more mixing of the clk provider calling the clk consumer APIs here. It looks like this just takes a reference and holds them. Would moving to parent_data address this? Brian -- = linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy