On 06/18/2013 03:23 PM, Wei Yongjun wrote: > From: Wei Yongjun <yongjun_...@trendmicro.com.cn> > > Fix missing free_netdev() before return from function xlr_net_probe() > in the devm_ioremap_resource() error handling case. > > Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn> > --- > no compile test. > --- > drivers/staging/netlogic/xlr_net.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/netlogic/xlr_net.c > b/drivers/staging/netlogic/xlr_net.c > index b529d79..2acde54 100644 > --- a/drivers/staging/netlogic/xlr_net.c > +++ b/drivers/staging/netlogic/xlr_net.c > @@ -1023,9 +1023,9 @@ static int xlr_net_probe(struct platform_device *pdev) > ndev->base_addr = (unsigned long) devm_ioremap_resource > (&pdev->dev, res); > if (IS_ERR_VALUE(ndev->base_addr)) { > - dev_err(&pdev->dev, > - "devm_ioremap_resource failed\n"); > - return ndev->base_addr; > + dev_err(&pdev->dev, "devm_ioremap_resource failed\n");
Please remove this error message too. During review of my patchset, it was pointed that this error message is redundant. There is already a similar error message within devm_ioremap_resource call. > + err = ndev->base_addr; > + goto err_gmac; > } > > res = platform_get_resource(pdev, IORESOURCE_IRQ, 0); > -- Tushar Behera -- 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/