Re: [U-Boot] [PATCH 2/3] mmc: add boundary check for mmc operation

2010-09-12 Thread Lei Wen
Fix it in new version of patch set. Thanks, Lei On Thu, Sep 9, 2010 at 10:51 PM, Wolfgang Denk wrote: > Dear Lei Wen, > > In message you > wrote: >> >> >> >> +       if (start > mmc->block_dev.lba || (start + blkcnt) >> >> mmc->block_dev.lba) { >> >> +                   printf("\noperation ex

Re: [U-Boot] [PATCH 2/3] mmc: add boundary check for mmc operation

2010-09-09 Thread Wolfgang Denk
Dear Lei Wen, In message you wrote: > >> > >> + if (start > mmc->block_dev.lba || (start + blkcnt) > >> mmc->block_dev.lba) { > >> + printf("\noperation excceed mmc boudary..\n"); > > > > No initial newline, please. > > You mean add additional line or just reduce a line

Re: [U-Boot] [PATCH 2/3] mmc: add boundary check for mmc operation

2010-09-09 Thread Lei Wen
On Tue, Sep 7, 2010 at 9:49 PM, Wolfgang Denk wrote: > Dear Lei Wen, > > In message <1283862729-17045-2-git-send-email-lei...@marvell.com> you wrote: >> Signed-off-by: Lei Wen >> --- >>  drivers/mmc/mmc.c |    8 >>  1 files changed, 8 insertions(+), 0 deletions(-) >> >> diff --git a/driv

Re: [U-Boot] [PATCH 2/3] mmc: add boundary check for mmc operation

2010-09-07 Thread Wolfgang Denk
Dear Lei Wen, In message <1283862729-17045-2-git-send-email-lei...@marvell.com> you wrote: > Signed-off-by: Lei Wen > --- > drivers/mmc/mmc.c |8 > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c > index 5cc1904..9a50b2f 10064

[U-Boot] [PATCH 2/3] mmc: add boundary check for mmc operation

2010-09-07 Thread Lei Wen
Signed-off-by: Lei Wen --- drivers/mmc/mmc.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index 5cc1904..9a50b2f 100644 --- a/drivers/mmc/mmc.c +++ b/drivers/mmc/mmc.c @@ -134,6 +134,10 @@ mmc_bwrite(int dev_num, ulong start