Re: [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env

2011-01-18 Thread Yoshihiro Shimoda
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
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env

2011-01-18 Thread Wolfgang Denk
Dear Yoshihiro Shimoda,

In message <4d33c078.3090...@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 
> ---
>  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: w...@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 <7...@jpl-devvax.jpl.nasa.gov>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env

2011-01-17 Thread Nobuhiro Iwamatsu
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 
Acked-by: Nobuhiro Iwamatsu 

> ---
>  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@lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] add checking the CONFIG_ENV_IS_IN_SPI_FLASH in Enbedded env

2011-01-16 Thread Yoshihiro Shimoda
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 
---
 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@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot