Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-11 Thread Marek Vasut
On Friday, October 10, 2014 at 11:16:05 PM, Maxime Hadjinlian wrote: > Hi all, Hi! [...] > >> +CC Altera guys, they might help too. > > > > Hmmm quite strange to write for sector aligned. > > Wonder the write related to filesystem access? > > > > Within Cadence QSPI controller, read and wr

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-10 Thread Maxime Hadjinlian
Hi all, On Wed, Oct 8, 2014 at 10:35 AM, Chin Liang See wrote: > On Sun, 2014-10-05 at 22:55 +0200, ma...@denx.de wrote: >> On Sunday, October 05, 2014 at 10:35:06 PM, Maxime Hadjinlian wrote: >> > On Sun, Oct 5, 2014 at 10:28 PM, Marek Vasut wrote: >> > > On Sunday, October 05, 2014 at 09:51:27

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-08 Thread Chin Liang See
On Sun, 2014-10-05 at 22:55 +0200, ma...@denx.de wrote: > On Sunday, October 05, 2014 at 10:35:06 PM, Maxime Hadjinlian wrote: > > On Sun, Oct 5, 2014 at 10:28 PM, Marek Vasut wrote: > > > On Sunday, October 05, 2014 at 09:51:27 PM, Maxime Hadjinlian wrote: > > >> On Sun, Oct 5, 2014 at 9:45 PM, M

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Marek Vasut
On Sunday, October 05, 2014 at 10:35:06 PM, Maxime Hadjinlian wrote: > On Sun, Oct 5, 2014 at 10:28 PM, Marek Vasut wrote: > > On Sunday, October 05, 2014 at 09:51:27 PM, Maxime Hadjinlian wrote: > >> On Sun, Oct 5, 2014 at 9:45 PM, Marek Vasut wrote: > >> > On Sunday, October 05, 2014 at 09:27:0

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Maxime Hadjinlian
On Sun, Oct 5, 2014 at 10:28 PM, Marek Vasut wrote: > On Sunday, October 05, 2014 at 09:51:27 PM, Maxime Hadjinlian wrote: >> On Sun, Oct 5, 2014 at 9:45 PM, Marek Vasut wrote: >> > On Sunday, October 05, 2014 at 09:27:00 PM, Maxime Hadjinlian wrote: >> >> On Sun, Oct 5, 2014 at 9:19 PM, wrote:

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Marek Vasut
On Sunday, October 05, 2014 at 09:51:27 PM, Maxime Hadjinlian wrote: > On Sun, Oct 5, 2014 at 9:45 PM, Marek Vasut wrote: > > On Sunday, October 05, 2014 at 09:27:00 PM, Maxime Hadjinlian wrote: > >> On Sun, Oct 5, 2014 at 9:19 PM, wrote: > >> >> On Sunday, October 05, 2014 at 08:40:26 PM, Maxim

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Maxime Hadjinlian
On Sun, Oct 5, 2014 at 9:45 PM, Marek Vasut wrote: > On Sunday, October 05, 2014 at 09:27:00 PM, Maxime Hadjinlian wrote: >> On Sun, Oct 5, 2014 at 9:19 PM, wrote: >> >> On Sunday, October 05, 2014 at 08:40:26 PM, Maxime Hadjinlian wrote: >> >> > Hi Thomas, all, >> >> > >> >> > On Sun, Oct 5, 20

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Marek Vasut
On Sunday, October 05, 2014 at 09:27:00 PM, Maxime Hadjinlian wrote: > On Sun, Oct 5, 2014 at 9:19 PM, wrote: > >> On Sunday, October 05, 2014 at 08:40:26 PM, Maxime Hadjinlian wrote: > >> > Hi Thomas, all, > >> > > >> > On Sun, Oct 5, 2014 at 7:43 PM, wrote: > >> > > Hello Maxime, > >> > > >

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Maxime Hadjinlian
On Sun, Oct 5, 2014 at 9:19 PM, wrote: >> On Sunday, October 05, 2014 at 08:40:26 PM, Maxime Hadjinlian wrote: >> > Hi Thomas, all, >> > >> > On Sun, Oct 5, 2014 at 7:43 PM, wrote: >> > > Hello Maxime, >> > > >> > > can you explain the usecase? >> > > I think, only the erase is executed per sec

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread thomas.langer
> On Sunday, October 05, 2014 at 08:40:26 PM, Maxime Hadjinlian wrote: > > Hi Thomas, all, > > > > On Sun, Oct 5, 2014 at 7:43 PM, wrote: > > > Hello Maxime, > > > > > > can you explain the usecase? > > > I think, only the erase is executed per sector, all other commands are > > > working fine wi

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Marek Vasut
On Sunday, October 05, 2014 at 08:40:26 PM, Maxime Hadjinlian wrote: > Hi Thomas, all, > > On Sun, Oct 5, 2014 at 7:43 PM, wrote: > > Hello Maxime, > > > > can you explain the usecase? > > I think, only the erase is executed per sector, all other commands are > > working fine with a byte orient

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Maxime Hadjinlian
Hi Thomas, all, On Sun, Oct 5, 2014 at 7:43 PM, wrote: > Hello Maxime, > > can you explain the usecase? > I think, only the erase is executed per sector, all other commands are > working fine with a byte oriented length. I need to write a file that is downloaded through TFTP. So I can get the f

Re: [U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread thomas.langer
Hello Maxime, can you explain the usecase? I think, only the erase is executed per sector, all other commands are working fine with a byte oriented length. Best regards, Thomas --- Sent from my phone. Maxime Hadjinlian hat geschrieben: This patch adds [+]len handler for the all the commands

[U-Boot] [PATCH] cmd_sf: add handler for +len arg for all commands

2014-10-05 Thread Maxime Hadjinlian
This patch adds [+]len handler for the all the commands that will automatically round up the requested erase length to the flash's sector_size. It was previously only available for the erase command. Signed-off-by: Maxime Hadjinlian --- common/cmd_sf.c | 12 ++-- 1 file changed, 6 inser