On 03/04/2014 11:12 AM, Alexander Shiyan wrote: > Вторник, 4 марта 2014, 11:01 +02:00 от Peter Ujfalusi <peter.ujfalusi@xxxxxx>: >> We can then remove the iounmap() calls from probe and remove. >> Since the driver requests the resources via index we can do the mem resource >> request within a for loop. >> >> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@xxxxxx> >> Reviewed-by: Santosh Shilimkar <santosh.shilimkar@ti> >> --- > >> + /* Get mem resources */ >> + for (i = 0; i < 3; i++) { >> + struct resource *res = platform_get_resource(pdev, >> + IORESOURCE_MEM, i); >> + if (!res) { >> + dev_err(&pdev->dev, "couldn't find resource %d\n", i); >> + return -ENODEV; >> + } > > No need to check "res". devm_request_and_ioremap() do all for us. > >> - l3->l3_base[2] = ioremap(res->start, resource_size(res)); >> - if (!l3->l3_base[2]) { >> - dev_err(&pdev->dev, "ioremap failed\n"); >> - ret = -ENOMEM; >> - goto err2; >> + l3->l3_base[i] = devm_request_and_ioremap(&pdev->dev, res); >> + if (!l3->l3_base[i]) { > > if (IS_ERR(l3->l3_base[i])) > >> + dev_err(&pdev->dev, "ioremap %d failed\n", i); > > Unnecessary. I'm going to keep this since it tells which resource of the three has failed. > >> + return -ENOMEM; > > return PTR_ERR(l3->l3_base[i]); > > --- > -- Péter -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html