[U-Boot] [PATCH v3 2/2] ppc4xx: Add PPC4xx SPI helpers to Sequoia

[Ben Warren <biggerbadderben@gmail.com>]

Hi Steven,

Steven A. Falco wrote:
> This patch adds helper routines needed in support of the PPC4xx SPI driver.
>
> Signed-off-by: Steven A. Falco <sfalco@harris.com>
> ---
>
> Changed CONFIG constant to CONFIG_PPC4xx_SPI.
>
>  board/amcc/sequoia/sequoia.c |   20 ++++++++++++++++++++
>  include/configs/sequoia.h    |    4 ++++
>  2 files changed, 24 insertions(+), 0 deletions(-)
>
> diff --git a/board/amcc/sequoia/sequoia.c b/board/amcc/sequoia/sequoia.c
> index d6668e2..080ee1e 100644
> --- a/board/amcc/sequoia/sequoia.c
> +++ b/board/amcc/sequoia/sequoia.c
> @@ -26,6 +26,7 @@
>  #include <libfdt.h>
>  #include <fdt_support.h>
>  #include <ppc4xx.h>
> +#include <spi.h>
>  #include <asm/gpio.h>
>  #include <asm/processor.h>
>  #include <asm/io.h>
> @@ -513,3 +514,22 @@ int post_hotkeys_pressed(void)
>  	return 0;	/* No hotkeys supported */
>  }
>  #endif /* CONFIG_POST */
> +
> +#if defined(CONFIG_PPC4xx_SPI)
> +int spi_cs_is_valid(unsigned int bus, unsigned int cs)
> +{
> +	return bus == 0 && cs >= 0 && cs < 64;
> +}
> +
> +void spi_cs_activate(struct spi_slave *slave)
> +{
> +	/* Assumes chip-selects are active-low. */
> +	gpio_write_bit(slave->cs, 0);
> +}
> +
> +void spi_cs_deactivate(struct spi_slave *slave)
> +{
> +	gpio_write_bit(slave->cs, 1);
> +}
> +#endif /* CONFIG_PPC4xx_SPI */
> +
> diff --git a/include/configs/sequoia.h b/include/configs/sequoia.h
> index 9321bdc..417fda1 100644
> --- a/include/configs/sequoia.h
> +++ b/include/configs/sequoia.h
> @@ -236,6 +236,10 @@
>  #define CONFIG_SYS_DTT_LOW_TEMP	-30
>  #define CONFIG_SYS_DTT_HYSTERESIS	3
>  
> +/* Define these if you are using the SPI port. */
> +#undef CONFIG_HARD_SPI
> +#undef CONFIG_PPC4xx_SPI
> +
>   
Why not enable this feature on Sequoia?  Wolfgang's argument for keeping 
the patch out then goes away.  IMHO, eval boards should have as many 
options enabled by default as possible, and the user then has the option 
to opt out.
>  /*
>   * Default environment variables
>   */
>   
regards,
Ben