Hi, Anton

This patch was for public C file. That was for different Host controller.
I think let them standalone will be better.

The u32 conversion will be removed.
Thanks. 


Regards
Haijun.


> -----Original Message-----
> From: Anton Vorontsov [mailto:cbouatmai...@gmail.com]
> Sent: Monday, November 26, 2012 5:12 PM
> To: Zhang Haijun-B42677
> Cc: linux-mmc@vger.kernel.org; Huang Changming-R66093; Chris Ball
> Subject: Re: [PATCH] Powerpc eSDHC: Convert the cmd_timeou_ms to u64 to
> avoid overflow
> 
> On Mon, Nov 26, 2012 at 04:35:58PM +0800, Haijun Zhang wrote:
> > Convert the cmd_timeou_ms to u64 to avoid overflow
> >
> > Signed-off-by: Jerry Huang <chang-ming.hu...@freescale.com>
> > Signed-off-by: Haijun Zhang <haijun.zh...@freescale.com>
> 
> Reviewed-by: Anton Vorontsov <cbouatmai...@gmail.com>
> 
> But I think you should merge the two patches...
> 
> Thanks,
> 
> > CC: Chris Ball <c...@laptop.org>
> > ---
> >  drivers/mmc/card/block.c |    3 ++-
> >  1 files changed, 2 insertions(+), 1 deletions(-)
> >
> > diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c index
> > 21056b9..51903de 100644
> > --- a/drivers/mmc/card/block.c
> > +++ b/drivers/mmc/card/block.c
> > @@ -469,7 +469,8 @@ static int mmc_blk_ioctl_cmd(struct block_device
> *bdev,
> >                      *     mrq.data = NULL;
> >                      *     cmd.cmd_timeout = idata->ic.cmd_timeout_ms;
> >                      */
> > -                   data.timeout_ns = idata->ic.cmd_timeout_ms * 1000000;
> > +                   data.timeout_ns = (u64)idata->ic.cmd_timeout_ms *
> > +                                           1000000;
> >             }
> >
> >             mrq.data = &data;
> > --
> > 1.7.0.4

N�����r��y����b�X��ǧv�^�)޺{.n�+����{��g"��^n�r���z���h�����&���G���h�(�階�ݢj"���m������z�ޖ���f���h���~�m�

Reply via email to