On 19:22 Sat 19 Apr     , Matthias Fuchs wrote:
> Hi,
> 
> I tried this in cfi_flash.c:
> 
> struct apl_s   {
>       ulong start;
>       ulong size;
> }
> 
> struct apl_s apl[] __attribute__((weak)) = {};

The problem with weak var is that the first declaration define the
sizeof the var. and the other define the data;

As sugest Wolfgang we could do it like this

in cfi_flash.c


--- a/drivers/mtd/cfi_flash.c
+++ b/drivers/mtd/cfi_flash.c
@@ -1873,6 +1873,12 @@ unsigned long flash_init (void)
 {
        unsigned long size = 0;
        int i;
+if define(CONFIG_FLASH_AUTOPROTECT)
+struct apl_s apl[] = init_flash_autoprotect();
+#endif

#ifdef CFG_FLASH_PROTECTION
        char *s = getenv("unlock");
@@ -1966,6 +1972,17 @@ unsigned long flash_init (void)
                       CFG_ENV_ADDR_REDUND + CFG_ENV_SIZE_REDUND - 1,
                       flash_get_info(CFG_ENV_ADDR_REDUND));
 #endif
+
+#if defined(CONFIG_FLASH_AUTOPROTECT)
+       for (i = 0; i < (sizeof(apl) / sizeof(struct apl_s)); i++) {
+               debug("autoprotecting from %08x to %08x\n",
+                     apl[i].start, apl[i].start + apl[i].size - 1);
+               flash_protect (FLAG_PROTECT_SET,
+                              apl[i].start,
+                              apl[i].start + apl[i].size - 1,
+                              flash_get_info(apl[i].start));
+       }
+#endif

in config header

struct apl_s apl* inline init_flash_autoprotect()
{
        struct apl_s a[] = {
        {1, 3},
        {4, 5},
        };

        return a;
}
Best Regards,
J.

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
U-Boot-Users mailing list
U-Boot-Users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to