RE: [PATCH v2 00/12] add support for host controller v3.00

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Chris,


> -----Original Message-----
> From: Chris Ball [mailto:cjb@xxxxxxxxxx]
> Sent: Friday, March 04, 2011 8:47 PM
> To: Nath, Arindam
> Cc: zhangfei.gao@xxxxxxxxx; prakity@xxxxxxxxxxx;
> subhashj@xxxxxxxxxxxxxx; linux-mmc@xxxxxxxxxxxxxxx; Su, Henry; Lu,
> Aaron; anath.amd@xxxxxxxxx
> Subject: Re: [PATCH v2 00/12] add support for host controller v3.00
> 
> Hi Arindam,
> 
> On Fri, Mar 04 2011, Arindam Nath wrote:
> > V2
> 
> Thanks very much for sending this!  Since you mentioned you aren't
> seeing compiler warnings, here's the set I see here:
> 
> drivers/mmc/core/sd.c: In function âmmc_sd_init_cardâ:
> drivers/mmc/core/sd.c:584:21: warning: âcurrent_limitâ may be used
> uninitialized in this function [-Wuninitialized]
> drivers/mmc/core/sd.c:558:32: note: âcurrent_limitâ was declared here
> 
> drivers/mmc/core/sd.c:547:18: warning: âtimingâ may be used
> uninitialized in this function [-Wuninitialized]
> drivers/mmc/core/sd.c:471:26: note: âtimingâ was declared here
> 
> drivers/mmc/core/sd.c:538:21: warning: âbus_speedâ may be used
> uninitialized in this function [-Wuninitialized]
> drivers/mmc/core/sd.c:471:15: note: âbus_speedâ was declared here
> 
> drivers/mmc/core/sd.c:456:21: warning: âcard_set_drv_typeâ may be used
> uninitialized in this function [-Wuninitialized]
> drivers/mmc/core/sd.c:433:25: note: âcard_set_drv_typeâ was declared
> here
> 
> drivers/mmc/core/sd.c:464:22: warning: âhost_set_drv_typeâ may be used
> uninitialized in this function [-Wuninitialized]
> drivers/mmc/core/sd.c:433:6: note: âhost_set_drv_typeâ was declared
> here
> 
> drivers/mmc/host/sdhci.c: In function âsdhci_add_hostâ:
> drivers/mmc/host/sdhci.c:2366:27: warning: âcaps[1]â may be used
> uninitialized in this function [-Wuninitialized]

Thanks for pointing these out. I will make a note of it, and fix it in the next version.

Regards,
Arindam

> 
> --
> Chris Ball   <cjb@xxxxxxxxxx>   <http://printf.net/>
> One Laptop Per Child

ÿô.nlj·Ÿ®‰­†+%ŠË±é¥Šwÿº{.nlj·¥Š{±þiœþ)íèjg¬±¨¶‰šŽŠÝjÿ¾«þG«é¸¢·¦j:+v‰¨Šwèm¶Ÿÿþø®w¥þŠà£¢·hšâÿ†Ù



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux