]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
USB: omap_udc: use resource_size
authorJulia Lawall <Julia.Lawall@inria.fr>
Wed, 1 Jan 2020 17:49:41 +0000 (18:49 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Jan 2020 16:55:25 +0000 (17:55 +0100)
Use resource_size rather than a verbose computation on
the end and start fields.

The semantic patch that makes these changes is as follows:
(http://coccinelle.lip6.fr/)

<smpl>
@@ struct resource ptr; @@
- (ptr.end - ptr.start + 1)
+ resource_size(&ptr)
</smpl>

Signed-off-by: Julia Lawall <Julia.Lawall@inria.fr>
Link: https://lore.kernel.org/r/1577900990-8588-2-git-send-email-Julia.Lawall@inria.fr
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/gadget/udc/omap_udc.c

index f36f0730afabde6a790e177d8189ab6a47346abf..bd12417996db300e0a89a4a89ad867e913dbe1b1 100644 (file)
@@ -2757,7 +2757,7 @@ static int omap_udc_probe(struct platform_device *pdev)
 
        /* NOTE:  "knows" the order of the resources! */
        if (!request_mem_region(pdev->resource[0].start,
-                       pdev->resource[0].end - pdev->resource[0].start + 1,
+                       resource_size(&pdev->resource[0]),
                        driver_name)) {
                DBG("request_mem_region failed\n");
                return -EBUSY;
@@ -2934,7 +2934,7 @@ static int omap_udc_probe(struct platform_device *pdev)
        }
 
        release_mem_region(pdev->resource[0].start,
-                       pdev->resource[0].end - pdev->resource[0].start + 1);
+                          resource_size(&pdev->resource[0]));
 
        return status;
 }
@@ -2950,7 +2950,7 @@ static int omap_udc_remove(struct platform_device *pdev)
        wait_for_completion(&done);
 
        release_mem_region(pdev->resource[0].start,
-                       pdev->resource[0].end - pdev->resource[0].start + 1);
+                          resource_size(&pdev->resource[0]));
 
        return 0;
 }