Am Freitag, den 18.02.2011, 15:12 +0100 schrieb Steffen Sledz:
> From: Mario Schuknecht <m.schukne...@dresearch.de>
> 
> * AMIC flashes are AMD clones and need the same fixups.

… otherwise the uBoot command `saveenv` does not work.

Please add in the commit message or in the patch file that this patch
has been sent upstream [1].

[1] http://patchwork.ozlabs.org/patch/82763/

> Signed-off-by: Mario Schuknecht <m.schukne...@dresearch.de>
> Signed-off-by: Steffen Sledz <sl...@dresearch.de>
> ---
>  recipes/u-boot/u-boot-2009.03/cfi-amic-fixup.patch |   11 +++++++++++
>  recipes/u-boot/u-boot-2009.03/hipox/00-hipox.patch |    3 ++-
>  recipes/u-boot/u-boot_2009.03.bb                   |    7 +++++--
>  3 files changed, 18 insertions(+), 3 deletions(-)
>  create mode 100644 recipes/u-boot/u-boot-2009.03/cfi-amic-fixup.patch
> 
> diff --git a/recipes/u-boot/u-boot-2009.03/cfi-amic-fixup.patch 
> b/recipes/u-boot/u-boot-2009.03/cfi-amic-fixup.patch
> new file mode 100644
> index 0000000..9d8c7ab
> --- /dev/null
> +++ b/recipes/u-boot/u-boot-2009.03/cfi-amic-fixup.patch
> @@ -0,0 +1,11 @@
> +diff -Nurd u-boot-2009.03.orig//drivers/mtd/cfi_flash.c 
> u-boot-2009.03/drivers/mtd/cfi_flash.c
> +--- u-boot-2009.03.orig//drivers/mtd/cfi_flash.c     2011-02-18 
> 14:11:24.870432001 +0100
> ++++ u-boot-2009.03/drivers/mtd/cfi_flash.c   2011-02-18 14:11:45.440432001 
> +0100
> +@@ -1883,6 +1883,7 @@
> +             /* Do manufacturer-specific fixups */
> +             switch (info->manufacturer_id) {
> +             case 0x0001:
> ++            case 0x0037: // AMIC
> +                     flash_fixup_amd(info, &qry);
> +                     break;
> +             case 0x001f:
> diff --git a/recipes/u-boot/u-boot-2009.03/hipox/00-hipox.patch 
> b/recipes/u-boot/u-boot-2009.03/hipox/00-hipox.patch
> index 5b91d28..20a47da 100644
> --- a/recipes/u-boot/u-boot-2009.03/hipox/00-hipox.patch
> +++ b/recipes/u-boot/u-boot-2009.03/hipox/00-hipox.patch
> @@ -6190,10 +6190,11 @@ diff -Nurd u-boot-2009.03.orig/cpu/arm926ejs/start.S 
> u-boot-2009.03/cpu/arm926ej
>  diff -Nurd u-boot-2009.03.orig/drivers/mtd/cfi_flash.c 
> u-boot-2009.03/drivers/mtd/cfi_flash.c
>  --- u-boot-2009.03.orig/drivers/mtd/cfi_flash.c      2009-04-15 
> 11:05:24.000000000 +0200
>  +++ u-boot-2009.03/drivers/mtd/cfi_flash.c   2009-04-15 11:08:18.000000000 
> +0200
> -@@ -1883,6 +1883,7 @@
> +@@ -1883,7 +1883,8 @@
>               /* Do manufacturer-specific fixups */
>               switch (info->manufacturer_id) {
>               case 0x0001:
> +             case 0x0037: // AMIC
>  +            case 0x00da: // Winbond
>                       flash_fixup_amd(info, &qry);
>                       break;
> diff --git a/recipes/u-boot/u-boot_2009.03.bb 
> b/recipes/u-boot/u-boot_2009.03.bb
> index 2cbded9..4b822f1 100644
> --- a/recipes/u-boot/u-boot_2009.03.bb
> +++ b/recipes/u-boot/u-boot_2009.03.bb
> @@ -6,9 +6,12 @@ DEFAULT_PREFERENCE_hipox = "1"
>  
>  DEPENDS_append_hipox = " oxnas-boot-tools oxnas-boot-tools-native "
>  
> -SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2 "
> +SRC_URI = "ftp://ftp.denx.de/pub/u-boot/u-boot-${PV}.tar.bz2 \
> +     file://cfi-amic-fixup.patch \
> +"
>  
> -SRC_URI_append_hipox = "file://00-hipox.patch \
> +SRC_URI_append_hipox = " \
> +     file://00-hipox.patch \
>       file://01-hipox-fix-gmac-reset.patch \
>       file://02-hipox-enable-mmu.patch \
>       file://03-hipox-direct-switch.patch \

If you build for `hipox` is not your fix applied twice? Or am I
mistaken?

If I am and with the changes requested in the beginning,

Acked-by: Paul Menzel <paulepan...@users.sourceforge.net>


Thanks,

Paul

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel

Reply via email to