[U-Boot] [PATCH 09/12] drivers/i2c/davinci_i2c.c: Fix GCC 4.6 warning

[Anatolij Gustschin <agust@denx.de>]

On Wed, 16 Nov 2011 08:42:07 -0700
Tom Rini <tom.rini@gmail.com> wrote:
...
> > --- a/drivers/i2c/davinci_i2c.c
> > +++ b/drivers/i2c/davinci_i2c.c
> > @@ -78,13 +78,11 @@ static int poll_i2c_irq(int mask)
> >
> > ?void flush_rx(void)
> > ?{
> > - ? ? ? int ? ? dummy;
> > -
> > ? ? ? ?while (1) {
> > ? ? ? ? ? ? ? ?if (!(REG(I2C_STAT) & I2C_STAT_RRDY))
> > ? ? ? ? ? ? ? ? ? ? ? ?break;
> >
> > - ? ? ? ? ? ? ? dummy = REG(I2C_DRR);
> > + ? ? ? ? ? ? ? (void)REG(I2C_DRR);
> 
> This becomes:
> (void)(*(volatile unsigned int *)(I2C_DDR))
>
> So we don't need that cast.  Also, can we at get a comment about what
> / why we're doing this?

Ah, thanks for the pointer. Fixed in patch v2. I think that the
code is self-explanatory and we don't need a comment. As long
there is something shifted in and placed in data reception register
we read and throw it away since this is a flush_rx().

Anatolij