| Message ID | d7078eb7-a7d6-4753-b453-8fce15245c34@stanley.mountain (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | [next] clk: mmp: pxa1908-apbc: Fix NULL vs IS_ERR() check in probe | expand |
On Wednesday 20 November 2024 18:18:38 Central European Standard Time Dan Carpenter wrote: > The devm_kzalloc() function returns NULL on error, not error pointers. > Fix the check. > > Fixes: 51ce55919273 ("clk: mmp: Add Marvell PXA1908 APBC driver") > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> > --- Acked-by: Duje Mihanović <duje.mihanovic@skole.hr> Regards,
Quoting Dan Carpenter (2024-11-20 09:18:38) > The devm_kzalloc() function returns NULL on error, not error pointers. > Fix the check. > > Fixes: 51ce55919273 ("clk: mmp: Add Marvell PXA1908 APBC driver") > Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> > --- Applied to clk-next
diff --git a/drivers/clk/mmp/clk-pxa1908-apbc.c b/drivers/clk/mmp/clk-pxa1908-apbc.c index b93d08466198..3fd7b5e644f3 100644 --- a/drivers/clk/mmp/clk-pxa1908-apbc.c +++ b/drivers/clk/mmp/clk-pxa1908-apbc.c @@ -96,8 +96,8 @@ static int pxa1908_apbc_probe(struct platform_device *pdev) struct pxa1908_clk_unit *pxa_unit; pxa_unit = devm_kzalloc(&pdev->dev, sizeof(*pxa_unit), GFP_KERNEL); - if (IS_ERR(pxa_unit)) - return PTR_ERR(pxa_unit); + if (!pxa_unit) + return -ENOMEM; pxa_unit->base = devm_platform_ioremap_resource(pdev, 0); if (IS_ERR(pxa_unit->base))
The devm_kzalloc() function returns NULL on error, not error pointers. Fix the check. Fixes: 51ce55919273 ("clk: mmp: Add Marvell PXA1908 APBC driver") Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org> --- drivers/clk/mmp/clk-pxa1908-apbc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)