Hi Chris,

> -----Original Message-----
> From: Chris Ball [mailto:c...@laptop.org]
> Sent: Friday, March 04, 2011 8:47 PM
> To: Nath, Arindam
> Cc: zhangfei....@gmail.com; prak...@marvell.com;
> subha...@codeaurora.org; linux-mmc@vger.kernel.org; Su, Henry; Lu,
> Aaron; anath....@gmail.com
> 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   <c...@laptop.org>   <http://printf.net/>
> One Laptop Per Child

Reply via email to