[PATCH v2 1/2] mtd: nand: orion: fix clk handling

Boris Brezillon boris.brezillon at free-electrons.com
Wed Mar 29 12:40:20 PDT 2017


On Wed, 29 Mar 2017 21:36:08 +0200
Simon Baatz <gmbnomis at gmail.com> wrote:

> Hello Uwe,
> 
> On Mon, Mar 27, 2017 at 08:19:49PM +0200, Uwe Kleine-K?nig wrote:
> > Hello,
> >   
> > > @@ -145,15 +151,13 @@ static int __init orion_nand_probe(struct platform_device *pdev)
> > >  	if (board->dev_ready)
> > >  		nc->dev_ready = board->dev_ready;
> > >  
> > > -	platform_set_drvdata(pdev, mtd);
> > > +	platform_set_drvdata(pdev, info);
> > >  
> > >  	/* Not all platforms can gate the clock, so it is not
> > >  	   an error if the clock does not exists. */
> > > -	clk = clk_get(&pdev->dev, NULL);
> > > -	if (!IS_ERR(clk)) {
> > > -		clk_prepare_enable(clk);
> > > -		clk_put(clk);
> > > -	}
> > > +	info->clk = devm_clk_get(&pdev->dev, NULL);
> > > +	if (!IS_ERR(info->clk))
> > > +		clk_prepare_enable(info->clk);  
> > 
> > you could to the following here instead:
> > 
> > 	info->clk = devm_clk_get(&pdev->dev, NULL);
> > 	if (IS_ERR(info->clk))
> > 		/*
> > 		 * We ignore all errors here, that's wrong but only for
> > 		 * one commit.
> > 		 */
> > 		info->clk = NULL;
> > 
> > 	ret = clk_prepare_enable(info->clk);
> > 	if (ret) ...
> >   
> 
> Makes sense.  I don't think we should have such a comment, though. 
> The patch description states what it fixes and the patch fixes
> exactly that.  Moreover, I tagged this patch for stable (because of
> the double free it fixes) and there might be no other commit.  (I
> think the error handling problem is not severe enough for stable.  It
> has been this way a couple of years and nobody seems to have
> noticed.)

No need to resend a new version, I'm fine with this one (actually I
already applied it and am about to push it on nand/next).



More information about the linux-arm-kernel mailing list