[U-Boot] [U-Boot, v2, 3/8] board: sama5d4_xplained: clean up code

Andreas Bießmann andreas at biessmann.org
Sat Feb 4 22:47:26 UTC 2017


On Fri, Oct 28, 2016 at 03:16:44PM +0800, Wenyou Yang wrote:
> Since the introduction of the pinctrl and clk driver and
> the dts file, remove unneeded hard coded related code from
> the board file.
> 
> Signed-off-by: Wenyou Yang <wenyou.yang at atmel.com>

Reviewed-by: Andreas Bießmann <andreas at biessmann.org>

> ---
> 
> Changes in v2:
>  - Restore the wrong removal of USB related code.
>  - Update the commit log.
> 
>  board/atmel/sama5d4_xplained/sama5d4_xplained.c | 87 +------------------------
>  1 file changed, 1 insertion(+), 86 deletions(-)
> 
> diff --git a/board/atmel/sama5d4_xplained/sama5d4_xplained.c b/board/atmel/sama5d4_xplained/sama5d4_xplained.c
> index 6684276..2b51e4c 100644
> --- a/board/atmel/sama5d4_xplained/sama5d4_xplained.c
> +++ b/board/atmel/sama5d4_xplained/sama5d4_xplained.c
> @@ -16,48 +16,14 @@
>  #include <asm/arch/sama5d3_smc.h>
>  #include <asm/arch/sama5d4.h>
>  #include <atmel_hlcdc.h>
> -#include <atmel_mci.h>
>  #include <lcd.h>
> -#include <mmc.h>
>  #include <net.h>
>  #include <netdev.h>
>  #include <nand.h>
> -#include <spi.h>
>  #include <version.h>
>  
>  DECLARE_GLOBAL_DATA_PTR;
>  
> -#ifdef CONFIG_ATMEL_SPI
> -#ifndef CONFIG_DM_SPI
> -int spi_cs_is_valid(unsigned int bus, unsigned int cs)
> -{
> -	return bus == 0 && cs == 0;
> -}
> -#endif
> -
> -void spi_cs_activate(struct spi_slave *slave)
> -{
> -	at91_set_pio_output(AT91_PIO_PORTC, 3, 0);
> -}
> -
> -void spi_cs_deactivate(struct spi_slave *slave)
> -{
> -	at91_set_pio_output(AT91_PIO_PORTC, 3, 1);
> -}
> -
> -static void sama5d4_xplained_spi0_hw_init(void)
> -{
> -	at91_pio3_set_a_periph(AT91_PIO_PORTC, 0, 0);	/* SPI0_MISO */
> -	at91_pio3_set_a_periph(AT91_PIO_PORTC, 1, 0);	/* SPI0_MOSI */
> -	at91_pio3_set_a_periph(AT91_PIO_PORTC, 2, 0);	/* SPI0_SPCK */
> -
> -	at91_set_pio_output(AT91_PIO_PORTC, 3, 1);	/* SPI0_CS0 */
> -
> -	/* Enable clock */
> -	at91_periph_clk_enable(ATMEL_ID_SPI0);
> -}
> -#endif /* CONFIG_ATMEL_SPI */
> -
>  #ifdef CONFIG_NAND_ATMEL
>  static void sama5d4_xplained_nand_hw_init(void)
>  {
> @@ -202,41 +168,6 @@ void lcd_show_board_info(void)
>  
>  #endif /* CONFIG_LCD */
>  
> -#ifdef CONFIG_GENERIC_ATMEL_MCI
> -void sama5d4_xplained_mci1_hw_init(void)
> -{
> -	at91_pio3_set_c_periph(AT91_PIO_PORTE, 19, 1);	/* MCI1 CDA */
> -	at91_pio3_set_c_periph(AT91_PIO_PORTE, 20, 1);	/* MCI1 DA0 */
> -	at91_pio3_set_c_periph(AT91_PIO_PORTE, 21, 1);	/* MCI1 DA1 */
> -	at91_pio3_set_c_periph(AT91_PIO_PORTE, 22, 1);	/* MCI1 DA2 */
> -	at91_pio3_set_c_periph(AT91_PIO_PORTE, 23, 1);	/* MCI1 DA3 */
> -	at91_pio3_set_c_periph(AT91_PIO_PORTE, 18, 0);	/* MCI1 CLK */
> -
> -	/*
> -	 * As the mci io internal pull down is too strong, so if the io needs
> -	 * external pull up, the pull up resistor will be very small, if so
> -	 * the power consumption will increase, so disable the interanl pull
> -	 * down to save the power.
> -	 */
> -	at91_pio3_set_pio_pulldown(AT91_PIO_PORTE, 18, 0);
> -	at91_pio3_set_pio_pulldown(AT91_PIO_PORTE, 19, 0);
> -	at91_pio3_set_pio_pulldown(AT91_PIO_PORTE, 20, 0);
> -	at91_pio3_set_pio_pulldown(AT91_PIO_PORTE, 21, 0);
> -	at91_pio3_set_pio_pulldown(AT91_PIO_PORTE, 22, 0);
> -	at91_pio3_set_pio_pulldown(AT91_PIO_PORTE, 23, 0);
> -
> -	/* Enable clock */
> -	at91_periph_clk_enable(ATMEL_ID_MCI1);
> -}
> -
> -int board_mmc_init(bd_t *bis)
> -{
> -	/* Enable the power supply */
> -	at91_set_pio_output(AT91_PIO_PORTE, 4, 0);
> -
> -	return atmel_mci_init((void *)ATMEL_BASE_MCI1);
> -}
> -#endif /* CONFIG_GENERIC_ATMEL_MCI */
>  
>  #ifdef CONFIG_MACB
>  void sama5d4_xplained_macb0_hw_init(void)
> @@ -268,12 +199,6 @@ static void sama5d4_xplained_serial3_hw_init(void)
>  
>  int board_early_init_f(void)
>  {
> -	at91_periph_clk_enable(ATMEL_ID_PIOA);
> -	at91_periph_clk_enable(ATMEL_ID_PIOB);
> -	at91_periph_clk_enable(ATMEL_ID_PIOC);
> -	at91_periph_clk_enable(ATMEL_ID_PIOD);
> -	at91_periph_clk_enable(ATMEL_ID_PIOE);
> -
>  	sama5d4_xplained_serial3_hw_init();
>  
>  	return 0;
> @@ -284,15 +209,9 @@ int board_init(void)
>  	/* adress of boot parameters */
>  	gd->bd->bi_boot_params = CONFIG_SYS_SDRAM_BASE + 0x100;
>  
> -#ifdef CONFIG_ATMEL_SPI
> -	sama5d4_xplained_spi0_hw_init();
> -#endif
>  #ifdef CONFIG_NAND_ATMEL
>  	sama5d4_xplained_nand_hw_init();
>  #endif
> -#ifdef CONFIG_GENERIC_ATMEL_MCI
> -	sama5d4_xplained_mci1_hw_init();
> -#endif
>  #ifdef CONFIG_MACB
>  	sama5d4_xplained_macb0_hw_init();
>  #endif
> @@ -338,12 +257,8 @@ int board_eth_init(bd_t *bis)
>  #ifdef CONFIG_SPL_BUILD
>  void spl_board_init(void)
>  {
> -#ifdef CONFIG_SYS_USE_MMC
> -	sama5d4_xplained_mci1_hw_init();
> -#elif CONFIG_SYS_USE_NANDFLASH
> +#if CONFIG_SYS_USE_NANDFLASH
>  	sama5d4_xplained_nand_hw_init();
> -#elif CONFIG_SYS_USE_SERIALFLASH
> -	sama5d4_xplained_spi0_hw_init();
>  #endif
>  }
>  


More information about the U-Boot mailing list