Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-25 Thread David Woodhouse
On Mon, 2010-10-18 at 17:46 +0800, tiejun.chen 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 'e

RE: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-20 Thread Zang Roy-R61911
bs.org; a...@linux-foundation.org; > dw...@infradead.org; Gala Kumar-B11780 > Subject: Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to > elbc devices > > > On Oct 20, 2010, at 12:12 AM, Zang Roy-R61911 wrote: > > > > > > >> -Original Message-

Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-19 Thread Kumar Gala
t-B07421; > dedeki...@gmail.com; Lan >> Chunhe-B25806; linuxppc-...@ozlabs.org; a...@linux-foundation.org; >> dw...@infradead.org; Gala Kumar-B11780 >> Subject: Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt > common to >> elbc devices >> >>

RE: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-19 Thread Zang Roy-R61911
nux-foundation.org; > dw...@infradead.org; Gala Kumar-B11780 > Subject: Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to > elbc devices > > > On Oct 18, 2010, at 2:22 AM, Roy Zang wrote: > > > Move Freescale elbc interrupt from nand dirver to elbc

Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-19 Thread Kumar Gala
On Oct 18, 2010, at 2:22 AM, Roy Zang wrote: > Move Freescale elbc interrupt from nand dirver to elbc driver. > Then all elbc devices can use the interrupt instead of ONLY nand. > > For former nand driver, it had the two functions: > > 1. detecting nand flash partitions; > 2. registering elbc i

RE: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread Zang Roy-R61911
t; foundation.org; dw...@infradead.org; 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" wrote: > > > Looks you always iounmap(fsl_lbc_ctrl_dev->regs) o

Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread tiejun.chen
Scott Wood wrote: > On Mon, 18 Oct 2010 16:55:49 +0800 > "tiejun.chen" 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

Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread Scott Wood
On Mon, 18 Oct 2010 16:55:49 +0800 "tiejun.chen" 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' > i

Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread tiejun.chen
B07421; dedeki...@gmail.com; >>> Lan >>> Chunhe-B25806; linuxppc-...@ozlabs.org; a...@linux-foundation.org; >>> dw...@infradead.org; Gala Kumar-B11780 >>> Subject: Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to >>> elbc devic

Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread tiejun.chen
>> Lan >> Chunhe-B25806; linuxppc-...@ozlabs.org; a...@linux-foundation.org; >> dw...@infradead.org; Gala Kumar-B11780 >> Subject: Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to >> elbc devices >> >> Roy Zang wrote: >>> Move Freescale e

RE: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread Zang Roy-R61911
nux-foundation.org; > dw...@infradead.org; Gala Kumar-B11780 > Subject: Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to > elbc devices > > Roy Zang wrote: > > Move Freescale elbc interrupt from nand dirver to elbc driver. > > Then all elbc devic

Re: [PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread tiejun.chen
Roy Zang wrote: > Move Freescale elbc interrupt from nand dirver to elbc driver. > Then all elbc devices can use the interrupt instead of ONLY nand. > > For former nand driver, it had the two functions: > > 1. detecting nand flash partitions; > 2. registering elbc interrupt. > > Now, second func

[PATCH 1/2] P4080/eLBC: Make Freescale elbc interrupt common to elbc devices

2010-10-18 Thread Roy Zang
Move Freescale elbc interrupt from nand dirver to elbc driver. Then all elbc devices can use the interrupt instead of ONLY nand. For former nand driver, it had the two functions: 1. detecting nand flash partitions; 2. registering elbc interrupt. Now, second function is removed to fsl_lbc.c. Sig