Received-SPF: pass (google.com: domain of linux-kernel+bounces-75969-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <654a33c729af90d8d5bd70d320fa6f6f.sboyd@kernel.org>
Content-Type: text/plain; charset="utf-8"
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
In-Reply-To: <20240221-mbly-clk-v7-2-31d4ce3630c3@bootlin.com>
References: <20240221-mbly-clk-v7-0-31d4ce3630c3@bootlin.com> <20240221-mbly-clk-v7-2-31d4ce3630c3@bootlin.com>
Subject: Re: [PATCH v7 02/14] clk: fixed-factor: add fwname-based constructor functions
From: Stephen Boyd <sboyd@kernel.org>
Cc: Vladimir Kondratiev <vladimir.kondratiev@mobileye.com>, linux-mips@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Thomas Petazzoni <thomas.petazzoni@bootlin.com>, Tawfik Bayouk <tawfik.bayouk@mobileye.com>, linux-gpio@vger.kernel.org, =?utf-8?q?Th=C3=A9o?= Lebrun <theo.lebrun@bootlin.com>
To: Conor Dooley <conor+dt@kernel.org>, Gregory CLEMENT <gregory.clement@bootlin.com>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>, Linus Walleij <linus.walleij@linaro.org>, Michael Turquette <mturquette@baylibre.com>, Philipp Zabel <p.zabel@pengutronix.de>, =?utf-8?q?Rafa=C5=82_Mi=C5=82ecki?= <rafal@milecki.pl>, Rob Herring <robh+dt@kernel.org>, Thomas Bogendoerfer <tsbogend@alpha.franken.de>, =?utf-8?q?Th=C3=A9o?= Lebrun <theo.lebrun@bootlin.com>
Date: Wed, 21 Feb 2024 22:13:16 -0800
User-Agent: alot/0.10

Quoting Th=C3=A9o Lebrun (2024-02-21 10:22:10)
> Add four functions to register clk_hw based on the fw_name field in
> clk_parent_data, ie the value in the DT property `clock-names`.
>=20
> There are variants for devm or not and passing an accuracy or not
> passing one:
>=20
>  - clk_hw_register_fixed_factor_fwname
>  - clk_hw_register_fixed_factor_with_accuracy_fwname
>  - devm_clk_hw_register_fixed_factor_fwname
>  - devm_clk_hw_register_fixed_factor_with_accuracy_fwname
>=20
> The `struct clk_parent_data` init is extracted from
> __clk_hw_register_fixed_factor to each calling function. It is required
> to allow each function to pass whatever field they want, not only index.
>=20
> Signed-off-by: Th=C3=A9o Lebrun <theo.lebrun@bootlin.com>
> ---

Applied to clk-next

Hopefully these are all going to be used.