No need to have a struct resource. Gets rid of the TODO.
Signed-off-by: Rosen Penev <rosenp@gmail.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Reviewed-by: Shannon Nelson <shannon.nelson@amd.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
 
        ndev->irq = dev->emac_irq;
 
-       /* Map EMAC regs */
-       // TODO : platform_get_resource() and devm_ioremap_resource()
-       dev->emacp = devm_of_iomap(&ofdev->dev, np, 0, NULL);
-       if (!dev->emacp) {
+       dev->emacp = devm_platform_ioremap_resource(ofdev, 0);
+       if (IS_ERR(dev->emacp)) {
                dev_err(&ofdev->dev, "can't map device registers");
-               err = -ENOMEM;
+               err = PTR_ERR(dev->emacp);
                goto err_gone;
        }