Skip to content

Commit e37868f

Browse files
Dan Carpentergeertu
authored andcommitted
clk: renesas: rzg2l: Avoid mixing error pointers and NULL
These functions accidentally return both error pointers and NULL when there is an error. It doesn't cause a problem but it is confusing and seems unintentional. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Link: https://lore.kernel.org/r/YMtY7nOtqEvTokh7@mwanda Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
1 parent 36aaa3a commit e37868f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/clk/renesas/renesas-rzg2l-cpg.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -125,7 +125,7 @@ rzg2l_cpg_div_clk_register(const struct cpg_core_clk *core,
125125
core->flag, &priv->rmw_lock);
126126

127127
if (IS_ERR(clk_hw))
128-
return NULL;
128+
return ERR_CAST(clk_hw);
129129

130130
return clk_hw->clk;
131131
}

0 commit comments

Comments
 (0)