[U-Boot] [PATCH] fsl_i2c.c: Fix warning on gcc-6.x
Heiko Schocher
hs at denx.de
Fri Feb 10 07:02:31 UTC 2017
Hello Tom,
Am 09.02.2017 um 21:40 schrieb Tom Rini:
> With gcc-6.x we see:
> drivers/i2c/fsl_i2c.c:86:3: warning: ‘fsl_i2c_speed_map’ defined but not
> used [-Wunused-const-variable=]
>
> The easy way to fix this is that since we only use fsl_i2c_speed_map at
> all on __M68K__ move the existing guards around slightly.
>
> Reported-by: Thomas Schaefer <Thomas.Schaefer at kontron.com>
> Signed-off-by: Tom Rini <trini at konsulko.com>
> ---
> drivers/i2c/fsl_i2c.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Thanks!
Acked-by: Heiko Schocher <hs at denx.de>
Do you want to pick up this patch directly, or should I send an i2c pull
request?
bye,
Heiko
>
> diff --git a/drivers/i2c/fsl_i2c.c b/drivers/i2c/fsl_i2c.c
> index e9fbf828f34b..a1406baa87b2 100644
> --- a/drivers/i2c/fsl_i2c.c
> +++ b/drivers/i2c/fsl_i2c.c
> @@ -53,6 +53,7 @@ static const struct fsl_i2c_base *i2c_base[4] = {
>
> /* I2C speed map for a DFSR value of 1 */
>
> +#ifdef __M68K__
> /*
> * Map I2C frequency dividers to FDR and DFSR values
> *
> @@ -84,7 +85,6 @@ static const struct {
> unsigned short divider;
> u8 fdr;
> } fsl_i2c_speed_map[] = {
> -#ifdef __M68K__
> {20, 32}, {22, 33}, {24, 34}, {26, 35},
> {28, 0}, {28, 36}, {30, 1}, {32, 37},
> {34, 2}, {36, 38}, {40, 3}, {40, 39},
> @@ -102,8 +102,8 @@ static const struct {
> {1536, 61}, {1792, 62}, {1920, 27}, {2048, 63},
> {2304, 28}, {2560, 29}, {3072, 30}, {3840, 31},
> {-1, 31}
> -#endif
> };
> +#endif
>
> /**
> * Set the I2C bus speed for a given I2C device
>
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list