> -----Original Message----- > From: Wood Scott-B07421 > Sent: Tuesday, October 19, 2010 0:07 AM > To: tiejun.chen > Cc: Zang Roy-R61911; [email protected]; Wood Scott-B07421; > [email protected]; Lan Chunhe-B25806; [email protected]; a...@linux- > foundation.org; [email protected]; Gala Kumar-B11780 > Subject: Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to > elbc devices > > On Mon, 18 Oct 2010 16:55:49 +0800 > "tiejun.chen" <[email protected]> wrote: > > > Looks you always iounmap(fsl_lbc_ctrl_dev->regs) on position 'err' but here > > of_iomap() is already failed you should skip iounmap() fsl_lbc_ctrl_dev- > >regs > > again. So you should improve that as the following on 'err', or layout 'err' > in > > gain. > > ------ > > if(fsl_lbc_ctrl_dev->regs) > > iounmap(fsl_lbc_ctrl_dev->regs); > > > > It looks like iounmap(NULL) is a no-op, just like kfree(NULL). Thanks for the reminder. I will keep original post, if there is no more comment. Roy
_______________________________________________ Linuxppc-dev mailing list [email protected] https://lists.ozlabs.org/listinfo/linuxppc-dev
