ata: pata_platform: Make use of platform_get_mem_or_io()
authorLad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Mon, 17 Jan 2022 02:01:34 +0000 (02:01 +0000)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Mon, 31 Jan 2022 00:58:31 +0000 (09:58 +0900)
Make use of platform_get_mem_or_io() to simplify the code.

While at it, drop use of unlikely() from pata_platform_probe()
as it isn't a hotpath.

Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Reviewed-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/pata_platform.c

index 87c7c90..21fb059 100644 (file)
@@ -200,22 +200,16 @@ static int pata_platform_probe(struct platform_device *pdev)
        /*
         * Get the I/O base first
         */
-       io_res = platform_get_resource(pdev, IORESOURCE_IO, 0);
-       if (io_res == NULL) {
-               io_res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-               if (unlikely(io_res == NULL))
-                       return -EINVAL;
-       }
+       io_res = platform_get_mem_or_io(pdev, 0);
+       if (!io_res)
+               return -EINVAL;
 
        /*
         * Then the CTL base
         */
-       ctl_res = platform_get_resource(pdev, IORESOURCE_IO, 1);
-       if (ctl_res == NULL) {
-               ctl_res = platform_get_resource(pdev, IORESOURCE_MEM, 1);
-               if (unlikely(ctl_res == NULL))
-                       return -EINVAL;
-       }
+       ctl_res = platform_get_mem_or_io(pdev, 1);
+       if (!ctl_res)
+               return -EINVAL;
 
        /*
         * And the IRQ