All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env
@ 2011-01-17  4:07 Yoshihiro Shimoda
  2011-01-17 13:09 ` Nobuhiro Iwamatsu
  2011-01-18  9:29 ` Wolfgang Denk
  0 siblings, 2 replies; 4+ messages in thread
From: Yoshihiro Shimoda @ 2011-01-17  4:07 UTC (permalink / raw)
  To: u-boot

Fix the problem which cannot build the U-boot, if we only set
the CONFIG_ENV_IS_IN_SPI_FLASH.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 include/environment.h |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/environment.h b/include/environment.h
index 082b3e1..8a4f393 100644
--- a/include/environment.h
+++ b/include/environment.h
@@ -106,6 +106,7 @@ extern unsigned long nand_env_oob_offset;
 /* Embedded env is only supported for some flash types */
 #ifdef CONFIG_ENV_IS_EMBEDDED
 # if !defined(CONFIG_ENV_IS_IN_FLASH) && \
+     !defined(CONFIG_ENV_IS_IN_SPI_FLASH) && \
      !defined(CONFIG_ENV_IS_IN_NAND) && \
      !defined(CONFIG_ENV_IS_IN_ONENAND)
 #  error "CONFIG_ENV_IS_EMBEDDED not supported for your flash type"
-- 
1.7.1

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

* [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env
  2011-01-17  4:07 [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env Yoshihiro Shimoda
@ 2011-01-17 13:09 ` Nobuhiro Iwamatsu
  2011-01-18  9:29 ` Wolfgang Denk
  1 sibling, 0 replies; 4+ messages in thread
From: Nobuhiro Iwamatsu @ 2011-01-17 13:09 UTC (permalink / raw)
  To: u-boot

On Mon, Jan 17, 2011 at 01:07:20PM +0900, Yoshihiro Shimoda wrote:
> Fix the problem which cannot build the U-boot, if we only set
> the CONFIG_ENV_IS_IN_SPI_FLASH.
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Acked-by: Nobuhiro Iwamatsu <iwamatsu@nigauri.org>

> ---
>  include/environment.h |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/include/environment.h b/include/environment.h
> index 082b3e1..8a4f393 100644
> --- a/include/environment.h
> +++ b/include/environment.h
> @@ -106,6 +106,7 @@ extern unsigned long nand_env_oob_offset;
>  /* Embedded env is only supported for some flash types */
>  #ifdef CONFIG_ENV_IS_EMBEDDED
>  # if !defined(CONFIG_ENV_IS_IN_FLASH) && \
> +     !defined(CONFIG_ENV_IS_IN_SPI_FLASH) && \
>       !defined(CONFIG_ENV_IS_IN_NAND) && \
>       !defined(CONFIG_ENV_IS_IN_ONENAND)
>  #  error "CONFIG_ENV_IS_EMBEDDED not supported for your flash type"
> -- 
> 1.7.1
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

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

* [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env
  2011-01-17  4:07 [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env Yoshihiro Shimoda
  2011-01-17 13:09 ` Nobuhiro Iwamatsu
@ 2011-01-18  9:29 ` Wolfgang Denk
  2011-01-19  7:45   ` Yoshihiro Shimoda
  1 sibling, 1 reply; 4+ messages in thread
From: Wolfgang Denk @ 2011-01-18  9:29 UTC (permalink / raw)
  To: u-boot

Dear Yoshihiro Shimoda,

In message <4D33C078.3090503@renesas.com> you wrote:
> Fix the problem which cannot build the U-boot, if we only set
> the CONFIG_ENV_IS_IN_SPI_FLASH.
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
> ---
>  include/environment.h |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/include/environment.h b/include/environment.h
> index 082b3e1..8a4f393 100644
> --- a/include/environment.h
> +++ b/include/environment.h
> @@ -106,6 +106,7 @@ extern unsigned long nand_env_oob_offset;
>  /* Embedded env is only supported for some flash types */
>  #ifdef CONFIG_ENV_IS_EMBEDDED
>  # if !defined(CONFIG_ENV_IS_IN_FLASH) && \
> +     !defined(CONFIG_ENV_IS_IN_SPI_FLASH) && \
>       !defined(CONFIG_ENV_IS_IN_NAND) && \
>       !defined(CONFIG_ENV_IS_IN_ONENAND)

Please keep list sorted.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Randal said it would be tough to do in sed. He didn't say  he  didn't
understand  sed.  Randal  understands sed quite well. Which is why he
uses Perl. :-)         - Larry Wall in <7874@jpl-devvax.JPL.NASA.GOV>

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

* [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env
  2011-01-18  9:29 ` Wolfgang Denk
@ 2011-01-19  7:45   ` Yoshihiro Shimoda
  0 siblings, 0 replies; 4+ messages in thread
From: Yoshihiro Shimoda @ 2011-01-19  7:45 UTC (permalink / raw)
  To: u-boot

Dear Wolfgang Denk,

2011/01/18 18:29, Wolfgang Denk wrote:
> Dear Yoshihiro Shimoda,
> 
>>  /* Embedded env is only supported for some flash types */
>>  #ifdef CONFIG_ENV_IS_EMBEDDED
>>  # if !defined(CONFIG_ENV_IS_IN_FLASH) && \
>> +     !defined(CONFIG_ENV_IS_IN_SPI_FLASH) && \
>>       !defined(CONFIG_ENV_IS_IN_NAND) && \
>>       !defined(CONFIG_ENV_IS_IN_ONENAND)
> 
> Please keep list sorted.
Oh, I didn't realize that is sorted by alphabet.
I will fix it.

Best regards,
Yoshihiro Shimoda

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

end of thread, other threads:[~2011-01-19  7:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-17  4:07 [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env Yoshihiro Shimoda
2011-01-17 13:09 ` Nobuhiro Iwamatsu
2011-01-18  9:29 ` Wolfgang Denk
2011-01-19  7:45   ` Yoshihiro Shimoda

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.