On Wed, Aug 05, 2015 at 11:22:32AM +0800, Axel Lin wrote: > 2015-08-05 11:20 GMT+08:00 Vinod Koul <vinod.koul@xxxxxxxxx>: > > On Fri, Jul 10, 2015 at 10:13:19PM +0800, Axel Lin wrote: > >> devm_ioremap_resource returns ERR_PTR on failure. > >> > >> Signed-off-by: Axel Lin <axel.lin@xxxxxxxxxx> > >> --- > >> drivers/dma/ti-dma-crossbar.c | 7 ++----- > >> 1 file changed, 2 insertions(+), 5 deletions(-) > >> > >> diff --git a/drivers/dma/ti-dma-crossbar.c b/drivers/dma/ti-dma-crossbar.c > >> index 24f5ca2..5e261d1 100644 > >> --- a/drivers/dma/ti-dma-crossbar.c > >> +++ b/drivers/dma/ti-dma-crossbar.c > >> @@ -139,12 +139,9 @@ static int ti_dma_xbar_probe(struct platform_device *pdev) > >> xbar->safe_val = (u16)safe_val; > >> > >> res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > >> - if (!res) > >> - return -ENODEV; > > and why remove this check? > > > > and is is not documented why you removed this > > The checking is done in devm_ioremap_resource(), that is the usually way > to use devm_ioremap_resource(). > I think it's very common in kernel code, so I didn't document it in commit log. Yes devm_ioremap_resource() does check for this, so this is fine. I will apply this with Peter's ack -- ~Vinod -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html