lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <CA+V-a8tPjhVU2zWSTcyj+nLTvmTB6q8cTMaw3KriQwBqcVg_bg@mail.gmail.com> Date: Sun, 22 Jun 2014 12:54:13 +0100 From: Prabhakar Lad <prabhakar.csengg@...il.com> To: Marc Kleine-Budde <mkl@...gutronix.de> Cc: linux-can@...r.kernel.org, netdev <netdev@...r.kernel.org>, Wolfgang Grandegger <wg@...ndegger.com>, LKML <linux-kernel@...r.kernel.org> Subject: Re: [PATCH] can: c_can: convert to use devm * api On Sun, Jun 22, 2014 at 11:56 AM, Marc Kleine-Budde <mkl@...gutronix.de> wrote: > On 06/20/2014 01:59 PM, Lad, Prabhakar wrote: >> From: "Lad, Prabhakar" <prabhakar.csengg@...il.com> >> >> this patch uses devm_* APIs as they are device managed >> and make code simpler. >> >> Signed-off-by: Lad, Prabhakar <prabhakar.csengg@...il.com> > > Thanks for the patch. Applied to can-next with minor change (see inline). > >> --- >> Note: This patch is compile tested only. >> >> drivers/net/can/c_can/c_can_platform.c | 41 ++++++++-------------------------- >> 1 file changed, 9 insertions(+), 32 deletions(-) >> >> diff --git a/drivers/net/can/c_can/c_can_platform.c b/drivers/net/can/c_can/c_can_platform.c >> index 1df0b32..e0dcbcf 100644 >> --- a/drivers/net/can/c_can/c_can_platform.c >> +++ b/drivers/net/can/c_can/c_can_platform.c >> @@ -157,40 +157,31 @@ static int c_can_plat_probe(struct platform_device *pdev) >> } >> >> /* get the appropriate clk */ >> - clk = clk_get(&pdev->dev, NULL); >> + clk = devm_clk_get(&pdev->dev, NULL); >> if (IS_ERR(clk)) { >> - dev_err(&pdev->dev, "no clock defined\n"); >> - ret = -ENODEV; >> + ret = PTR_ERR(clk); >> goto exit; >> } >> >> /* get the platform data */ >> mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); >> irq = platform_get_irq(pdev, 0); >> - if (!mem || irq <= 0) { >> - ret = -ENODEV; >> - goto exit_free_clk; >> - } >> - >> - if (!request_mem_region(mem->start, resource_size(mem), >> - KBUILD_MODNAME)) { >> - dev_err(&pdev->dev, "resource unavailable\n"); >> + if (irq <= 0) { >> ret = -ENODEV; >> - goto exit_free_clk; >> + goto exit; >> } >> >> - addr = ioremap(mem->start, resource_size(mem)); >> - if (!addr) { >> - dev_err(&pdev->dev, "failed to map can port\n"); >> - ret = -ENOMEM; >> - goto exit_release_mem; > > I've moved the mem = platform_get_resource() for slightly better > readability here.... > Thanks makes sense. Regards, --Prabhakar Lad >> + addr = devm_ioremap_resource(&pdev->dev, mem); >> + if (IS_ERR(addr)) { >> + ret = PTR_ERR(addr); >> + goto exit; >> } > > Thanks, > Marc > > -- > Pengutronix e.K. | Marc Kleine-Budde | > Industrial Linux Solutions | Phone: +49-231-2826-924 | > Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | > Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists