Re: [PATCH] clk: rk808: fix OF node reference imbalance
From: Sebastian Reichel
Date: Tue Apr 07 2026 - 06:46:46 EST
Hi,
On Tue, Apr 07, 2026 at 11:50:27AM +0200, Johan Hovold wrote:
> The driver reuses the OF node of the parent multi-function device but
> fails to take another reference to balance the one dropped by the
> platform bus code when unbinding the MFD and deregistering the child
> devices.
>
> Fix this by using the intended helper for reusing OF nodes.
>
> Fixes: 2dc51ca822e4 ("clk: RK808: Reduce 'struct rk808' usage")
> Cc: stable@xxxxxxxxxxxxxxx # 6.5
> Cc: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>
> Signed-off-by: Johan Hovold <johan@xxxxxxxxxx>
> ---
Reviewed-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>
Greetings,
-- Sebastian
> drivers/clk/clk-rk808.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/clk/clk-rk808.c b/drivers/clk/clk-rk808.c
> index f7412b137e5e..5a75b5c91555 100644
> --- a/drivers/clk/clk-rk808.c
> +++ b/drivers/clk/clk-rk808.c
> @@ -153,7 +153,7 @@ static int rk808_clkout_probe(struct platform_device *pdev)
> struct rk808_clkout *rk808_clkout;
> int ret;
>
> - dev->of_node = pdev->dev.parent->of_node;
> + device_set_of_node_from_dev(dev, dev->parent);
>
> rk808_clkout = devm_kzalloc(dev,
> sizeof(*rk808_clkout), GFP_KERNEL);
> --
> 2.52.0
>
Attachment:
signature.asc
Description: PGP signature