public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] da850_am18xxevm: Add CONFIG_SPI_FLASH_STMICRO to the board config
@ 2011-12-15  7:31 Christian Riesch
  2011-12-15 14:45 ` Tom Rini
  0 siblings, 1 reply; 3+ messages in thread
From: Christian Riesch @ 2011-12-15  7:31 UTC (permalink / raw)
  To: u-boot

The board config for the da850_am18xxevm board is missing
the CONFIG_SPI_FLASH_STMICRO option and therefore the SPI flash
on the AM1808 experimenter's kit could not be detected.

Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
Cc: Tom Rini <trini@ti.com>
---
 include/configs/da850_am18xxevm.h |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/configs/da850_am18xxevm.h b/include/configs/da850_am18xxevm.h
index 04961c6..354f045 100644
--- a/include/configs/da850_am18xxevm.h
+++ b/include/configs/da850_am18xxevm.h
@@ -146,6 +146,7 @@
 
 #define CONFIG_SPI
 #define CONFIG_SPI_FLASH
+#define CONFIG_SPI_FLASH_STMICRO
 #define CONFIG_SPI_FLASH_WINBOND
 #define CONFIG_DAVINCI_SPI
 #define CONFIG_SYS_SPI_BASE		DAVINCI_SPI1_BASE
-- 
1.7.0.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] da850_am18xxevm: Add CONFIG_SPI_FLASH_STMICRO to the board config
  2011-12-15  7:31 [U-Boot] [PATCH] da850_am18xxevm: Add CONFIG_SPI_FLASH_STMICRO to the board config Christian Riesch
@ 2011-12-15 14:45 ` Tom Rini
  2011-12-15 16:20   ` Christian Riesch
  0 siblings, 1 reply; 3+ messages in thread
From: Tom Rini @ 2011-12-15 14:45 UTC (permalink / raw)
  To: u-boot

On 12/15/2011 12:31 AM, Christian Riesch wrote:
> The board config for the da850_am18xxevm board is missing
> the CONFIG_SPI_FLASH_STMICRO option and therefore the SPI flash
> on the AM1808 experimenter's kit could not be detected.
> 
> Signed-off-by: Christian Riesch <christian.riesch@omicron.at>
> Cc: Tom Rini <trini@ti.com>
> ---
>  include/configs/da850_am18xxevm.h |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/include/configs/da850_am18xxevm.h b/include/configs/da850_am18xxevm.h
> index 04961c6..354f045 100644
> --- a/include/configs/da850_am18xxevm.h
> +++ b/include/configs/da850_am18xxevm.h
> @@ -146,6 +146,7 @@
>  
>  #define CONFIG_SPI
>  #define CONFIG_SPI_FLASH
> +#define CONFIG_SPI_FLASH_STMICRO
>  #define CONFIG_SPI_FLASH_WINBOND
>  #define CONFIG_DAVINCI_SPI
>  #define CONFIG_SYS_SPI_BASE		DAVINCI_SPI1_BASE

After some internal digging to confirm, da850_am18xxevm.h should go away
and we should do da850_am18xxevm as just a boards.cfg option that uses
the da850evm.h file and sets CONFIG_DA850_AM18X_EVM.  Would you mind
doing the respin?  Thanks!

-- 
Tom

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [U-Boot] [PATCH] da850_am18xxevm: Add CONFIG_SPI_FLASH_STMICRO to the board config
  2011-12-15 14:45 ` Tom Rini
@ 2011-12-15 16:20   ` Christian Riesch
  0 siblings, 0 replies; 3+ messages in thread
From: Christian Riesch @ 2011-12-15 16:20 UTC (permalink / raw)
  To: u-boot

Hi Tom,

On Thu, Dec 15, 2011 at 3:45 PM, Tom Rini <trini@ti.com> wrote:
> On 12/15/2011 12:31 AM, Christian Riesch wrote:
>> The board config for the da850_am18xxevm board is missing
>> the CONFIG_SPI_FLASH_STMICRO option and therefore the SPI flash
>> on the AM1808 experimenter's kit could not be detected.
[...]
>
> After some internal digging to confirm, da850_am18xxevm.h should go away
> and we should do da850_am18xxevm as just a boards.cfg option that uses
> the da850evm.h file and sets CONFIG_DA850_AM18X_EVM. ?Would you mind
> doing the respin? ?Thanks!

Ok, I will submit a patch.
Christian

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-12-15 16:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-12-15  7:31 [U-Boot] [PATCH] da850_am18xxevm: Add CONFIG_SPI_FLASH_STMICRO to the board config Christian Riesch
2011-12-15 14:45 ` Tom Rini
2011-12-15 16:20   ` Christian Riesch

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox