On Fri, Sep 21, 2012 at 3:19 PM, Wei Yongjun <weiyj...@gmail.com> wrote: > From: Wei Yongjun <yongjun_...@trendmicro.com.cn> > > In case of error, the function clk_get() returns ERR_PTR() > and never returns NULL pointer. The NULL test in the error > handling should be replaced with IS_ERR(). > > dpatch engine is used to auto generated this patch. > (https://github.com/weiyj/dpatch) > > Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn>
Ack > --- > drivers/pcmcia/pxa2xx_base.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/pcmcia/pxa2xx_base.c b/drivers/pcmcia/pxa2xx_base.c > index 490bb82..cfec9dd 100644 > --- a/drivers/pcmcia/pxa2xx_base.c > +++ b/drivers/pcmcia/pxa2xx_base.c > @@ -297,7 +297,7 @@ static int pxa2xx_drv_pcmcia_probe(struct platform_device > *dev) > } > > clk = clk_get(&dev->dev, NULL); > - if (!clk) > + if (IS_ERR(clk)) > return -ENODEV; > > pxa2xx_drv_pcmcia_ops(ops); > > -- 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/
