Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-07-20 Thread york sun
On 07/19/2016 08:02 PM, Wenbin Song wrote: > Hi: York > > I'm sorry to have misunderstood you mean. > I think it is a good idea that modify/remove the wrong/useless variables in > a new patch. > And I will make it later. > Do you have any suggestion? Yes, please make a cleanup patch first, then

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-07-20 Thread Wenbin Song
Hi: York I'm sorry to have misunderstood you mean. I think it is a good idea that modify/remove the wrong/useless variables in a new patch. And I will make it later. Do you have any suggestion? Best Regards Wenbin Song > -Original Message- > From: york sun > Sent: Wednesday, July 20,

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-07-19 Thread york sun
On 04/06/2016 10:46 PM, Wenbin Song wrote: > Hi: York > > Please see my inline comments. > > Best Regards > Wenbin Song > >> -Original Message- >> From: York Sun [mailto:york@nxp.com] >> Sent: Thursday, April 07, 2016 1:18 AM >> To: Wenbin Song ; Mingkai Hu >>

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-07 Thread Wenbin Song
Hi: York Please see my inline comments. Best Regards Wenbin Song > -Original Message- > From: York Sun [mailto:york@nxp.com] > Sent: Thursday, April 07, 2016 1:18 AM > To: Wenbin Song ; Mingkai Hu > ; u-boot@lists.denx.de > Cc: Qianyu Gong

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-06 Thread York Sun
On 04/06/2016 12:11 AM, Wenbin Song wrote: > Hi: York > > I set bootargs as the following steps: > > => env default mtdparts > => printenv mtdparts >

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-06 Thread Wenbin Song
Hi: York I set bootargs as the following steps: => env default mtdparts => printenv mtdparts

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-06 Thread Wenbin Song
Hi: York 1, There is an issue for sdk2.0 kernel that the nor-flash can't be probed . we can fix it as follow: $make menuconfig x -> Device Drivers

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-06 Thread Wenbin Song
Hi: York Do you mean that I should use $mtdpart to instead of "MTDPARTS_DEFAULT" ? As follows: #define CONFIG_BOOTARGS "console=ttyS0,115200 root=/dev/ram0 " \ "earlycon=uart8250,mmio,0x21c0500 " \

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-05 Thread York Sun
On 04/05/2016 09:20 PM, Wenbin Song wrote: > Hi: York > > Do you mean that I should use $mtdpart to instead of "MTDPARTS_DEFAULT" ? > > As follows: >#define CONFIG_BOOTARGS "console=ttyS0,115200 > root=/dev/ram0 " \ > >

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-05 Thread York Sun
On 04/05/2016 01:32 PM, York Sun wrote: > On 04/01/2016 02:37 AM, Wenbin Song wrote: >> Add and share the the MTD partition scheme with kernel by defualt bootargs. >> And add the "mtdparts" env. >> >> Signed-off-by: Wenbin Song >> --- >> include/configs/ls1043a_common.h | 22

Re: [U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-05 Thread York Sun
On 04/01/2016 02:37 AM, Wenbin Song wrote: > Add and share the the MTD partition scheme with kernel by defualt bootargs. > And add the "mtdparts" env. > > Signed-off-by: Wenbin Song > --- > include/configs/ls1043a_common.h | 22 -- > 1 file changed, 20

[U-Boot] [PATCH] armv8/ls1043a: Add MTD partition scheme

2016-04-02 Thread Wenbin Song
Add and share the the MTD partition scheme with kernel by defualt bootargs. And add the "mtdparts" env. Signed-off-by: Wenbin Song --- include/configs/ls1043a_common.h | 22 -- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git