* [PATCH 6/6] drivers/mtd/nand/txx9ndfmc.c: use devm_request_and_ioremap [not found] <1324994491-21828-1-git-send-email-julia@diku.dk> @ 2011-12-27 14:01 ` Julia Lawall 2011-12-27 14:45 ` Artem Bityutskiy 0 siblings, 1 reply; 3+ messages in thread From: Julia Lawall @ 2011-12-27 14:01 UTC (permalink / raw) To: David Woodhouse; +Cc: linux-mtd, kernel-janitors, linux-kernel From: Julia Lawall <julia@diku.dk> Reimplement a call to devm_request_mem_region followed by a call to ioremap or ioremap_nocache by a call to devm_request_and_ioremap. The semantic patch that makes this transformation is as follows: (http://coccinelle.lip6.fr/) // <smpl> @nm@ expression myname; identifier i; @@ struct platform_driver i = { .driver = { .name = myname } }; @@ expression dev,res,size; expression nm.myname; @@ -if (!devm_request_mem_region(dev, res->start, size, - \(res->name\|dev_name(dev)\|myname\))) { - ... - return ...; -} ... when != res->start ( -devm_ioremap(dev,res->start,size) +devm_request_and_ioremap(dev,res) | -devm_ioremap_nocache(dev,res->start,size) +devm_request_and_ioremap(dev,res) ) ... when any when != res->start // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> --- drivers/mtd/nand/txx9ndfmc.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/mtd/nand/txx9ndfmc.c b/drivers/mtd/nand/txx9ndfmc.c index ace46fd..c7c4f1d 100644 --- a/drivers/mtd/nand/txx9ndfmc.c +++ b/drivers/mtd/nand/txx9ndfmc.c @@ -298,11 +298,7 @@ static int __init txx9ndfmc_probe(struct platform_device *dev) drvdata = devm_kzalloc(&dev->dev, sizeof(*drvdata), GFP_KERNEL); if (!drvdata) return -ENOMEM; - if (!devm_request_mem_region(&dev->dev, res->start, - resource_size(res), dev_name(&dev->dev))) - return -EBUSY; - drvdata->base = devm_ioremap(&dev->dev, res->start, - resource_size(res)); + drvdata->base = devm_request_and_ioremap(&dev->dev, res); if (!drvdata->base) return -EBUSY; ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 6/6] drivers/mtd/nand/txx9ndfmc.c: use devm_request_and_ioremap 2011-12-27 14:01 ` [PATCH 6/6] drivers/mtd/nand/txx9ndfmc.c: use devm_request_and_ioremap Julia Lawall @ 2011-12-27 14:45 ` Artem Bityutskiy 2011-12-27 14:46 ` Julia Lawall 0 siblings, 1 reply; 3+ messages in thread From: Artem Bityutskiy @ 2011-12-27 14:45 UTC (permalink / raw) To: Julia Lawall; +Cc: kernel-janitors, David Woodhouse, linux-kernel, linux-mtd [-- Attachment #1: Type: text/plain, Size: 1285 bytes --] On Tue, 2011-12-27 at 15:01 +0100, Julia Lawall wrote: > From: Julia Lawall <julia@diku.dk> > > Reimplement a call to devm_request_mem_region followed by a call to ioremap > or ioremap_nocache by a call to devm_request_and_ioremap. > > The semantic patch that makes this transformation is as follows: > (http://coccinelle.lip6.fr/) > > // <smpl> > @nm@ > expression myname; > identifier i; > @@ > > struct platform_driver i = { .driver = { .name = myname } }; > > @@ > expression dev,res,size; > expression nm.myname; > @@ > > -if (!devm_request_mem_region(dev, res->start, size, > - \(res->name\|dev_name(dev)\|myname\))) { > - ... > - return ...; > -} > ... when != res->start > ( > -devm_ioremap(dev,res->start,size) > +devm_request_and_ioremap(dev,res) > | > -devm_ioremap_nocache(dev,res->start,size) > +devm_request_and_ioremap(dev,res) > ) > ... when any > when != res->start > // </smpl> > > Signed-off-by: Julia Lawall <julia@diku.dk> Pushed to l2-mtd-2.6.git, thanks! I think this semantic patch is also useful and deserves to be added to scripts/coccinelle. In general, it would be great if you sumbitted useful spatches there for inclusion. -- Best Regards, Artem Bityutskiy [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 836 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 6/6] drivers/mtd/nand/txx9ndfmc.c: use devm_request_and_ioremap 2011-12-27 14:45 ` Artem Bityutskiy @ 2011-12-27 14:46 ` Julia Lawall 0 siblings, 0 replies; 3+ messages in thread From: Julia Lawall @ 2011-12-27 14:46 UTC (permalink / raw) To: Artem Bityutskiy Cc: kernel-janitors, David Woodhouse, linux-kernel, linux-mtd On Tue, 27 Dec 2011, Artem Bityutskiy wrote: > On Tue, 2011-12-27 at 15:01 +0100, Julia Lawall wrote: >> From: Julia Lawall <julia@diku.dk> >> >> Reimplement a call to devm_request_mem_region followed by a call to ioremap >> or ioremap_nocache by a call to devm_request_and_ioremap. >> >> The semantic patch that makes this transformation is as follows: >> (http://coccinelle.lip6.fr/) >> >> // <smpl> >> @nm@ >> expression myname; >> identifier i; >> @@ >> >> struct platform_driver i = { .driver = { .name = myname } }; >> >> @@ >> expression dev,res,size; >> expression nm.myname; >> @@ >> >> -if (!devm_request_mem_region(dev, res->start, size, >> - \(res->name\|dev_name(dev)\|myname\))) { >> - ... >> - return ...; >> -} >> ... when != res->start >> ( >> -devm_ioremap(dev,res->start,size) >> +devm_request_and_ioremap(dev,res) >> | >> -devm_ioremap_nocache(dev,res->start,size) >> +devm_request_and_ioremap(dev,res) >> ) >> ... when any >> when != res->start >> // </smpl> >> >> Signed-off-by: Julia Lawall <julia@diku.dk> > > Pushed to l2-mtd-2.6.git, thanks! > > I think this semantic patch is also useful and deserves to be added to > scripts/coccinelle. In general, it would be great if you sumbitted > useful spatches there for inclusion. OK, I will add this one as well. julia ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-12-27 14:46 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1324994491-21828-1-git-send-email-julia@diku.dk>
2011-12-27 14:01 ` [PATCH 6/6] drivers/mtd/nand/txx9ndfmc.c: use devm_request_and_ioremap Julia Lawall
2011-12-27 14:45 ` Artem Bityutskiy
2011-12-27 14:46 ` Julia Lawall
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox