Am Sonntag, den 21.07.2013, 22:19 -0300 schrieb Fabio Estevam: > From: Fabio Estevam <fabio.este...@freescale.com> > > As devm_ioremap_resource() is used on probe, there is no need to explicitly > check the return value from platform_get_resource(), as this is something that > devm_ioremap_resource() takes care by itself. > > Signed-off-by: Fabio Estevam <fabio.este...@freescale.com> > --- > drivers/staging/imx-drm/imx-tve.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/staging/imx-drm/imx-tve.c > b/drivers/staging/imx-drm/imx-tve.c > index a56797d..8232cea 100644 > --- a/drivers/staging/imx-drm/imx-tve.c > +++ b/drivers/staging/imx-drm/imx-tve.c > @@ -623,11 +623,6 @@ static int imx_tve_probe(struct platform_device *pdev) > } > > res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - if (!res) { > - dev_err(&pdev->dev, "failed to get memory region\n"); > - return -ENOENT; > - } > - > base = devm_ioremap_resource(&pdev->dev, res); > if (IS_ERR(base)) > return PTR_ERR(base);
Acked-by: Philipp Zabel <p.za...@pengutronix.de> regards Philipp -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/