Messages in this thread | | | Date | Tue, 21 May 2013 01:38:23 +0900 | From | Kukjin Kim <> | Subject | Re: [PATCH 30/33] arch/arm/plat-samsung: don't check resource with devm_ioremap_resource |
| |
On 05/16/13 20:15, Wolfram Sang wrote: > devm_ioremap_resource does sanity checks on the given resource. No need to > duplicate this in the driver. > > Signed-off-by: Wolfram Sang<wsa@the-dreams.de>
Looks good to me, applied into Samsung tree.
Wolfram, if you want to take this into your tree, let me know.
Thanks.
- Kukjin
> --- > arch/arm/plat-samsung/adc.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/arch/arm/plat-samsung/adc.c b/arch/arm/plat-samsung/adc.c > index ca07cb1..79690f2 100644 > --- a/arch/arm/plat-samsung/adc.c > +++ b/arch/arm/plat-samsung/adc.c > @@ -381,11 +381,6 @@ static int s3c_adc_probe(struct platform_device *pdev) > } > > regs = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - if (!regs) { > - dev_err(dev, "failed to find registers\n"); > - return -ENXIO; > - } > - > adc->regs = devm_ioremap_resource(dev, regs); > if (IS_ERR(adc->regs)) > return PTR_ERR(adc->regs);
| |