[U-Boot] [PATCH v2 01/11] arm: mvebu: maxbcm: Fix compilation warning and add Spansion SPI NOR support

Jagan Teki jagannadh.teki at gmail.com
Tue Feb 17 10:41:43 CET 2015


On 19 January 2015 at 16:03, Stefan Roese <sr at denx.de> wrote:
> This patch fixes the following compilation warning for maxbcm:
>
> Building maxbcm board...
>    text    data     bss     dec     hex filename
>  160075    6596   38240  204911   3206f ./u-boot
> board/maxbcm/maxbcm.c: In function 'reset_phy':
> board/maxbcm/maxbcm.c:68:6: warning: unused variable 'reg' [-Wunused-variable]
>   u16 reg;
>       ^
> board/maxbcm/maxbcm.c:66:6: warning: unused variable 'devadr' [-Wunused-variable]
>   u16 devadr = CONFIG_PHY_BASE_ADDR;
>       ^
>
> Additionally support Spansion SPI NOR flash is added. With larger SPI device
> support via the CONFIG_SPI_FLASH_BAR define.
>
> Signed-off-by: Stefan Roese <sr at denx.de>

Reviewed-by: Jagannadha Sutradharudu Teki <jagannadh.teki at gmail.com>

> ---
>
> Changes in v2: None
>
>  board/maxbcm/maxbcm.c    | 2 --
>  include/configs/maxbcm.h | 2 ++
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/board/maxbcm/maxbcm.c b/board/maxbcm/maxbcm.c
> index 7fc83ee..94c688f 100644
> --- a/board/maxbcm/maxbcm.c
> +++ b/board/maxbcm/maxbcm.c
> @@ -63,9 +63,7 @@ int checkboard(void)
>  /* Configure and enable MV88E6185 switch */
>  void reset_phy(void)
>  {
> -       u16 devadr = CONFIG_PHY_BASE_ADDR;
>         char *name = "neta0";
> -       u16 reg;
>
>         if (miiphy_set_current_dev(name))
>                 return;
> diff --git a/include/configs/maxbcm.h b/include/configs/maxbcm.h
> index 72217bd..005e136 100644
> --- a/include/configs/maxbcm.h
> +++ b/include/configs/maxbcm.h
> @@ -43,6 +43,8 @@
>  #define CONFIG_SF_DEFAULT_SPEED                1000000
>  #define CONFIG_SF_DEFAULT_MODE         SPI_MODE_3
>  #define CONFIG_SPI_FLASH_STMICRO
> +#define CONFIG_SPI_FLASH_SPANSION
> +#define CONFIG_SPI_FLASH_BAR
>
>  /* Environment in SPI NOR flash */
>  #define CONFIG_ENV_IS_IN_SPI_FLASH
> --
> 2.2.2

thanks!
-- 
Jagan.


More information about the U-Boot mailing list