From mboxrd@z Thu Jan  1 00:00:00 1970
From: Matthias Fuchs <matthias.fuchs@esd-electronics.com>
Date: Mon, 21 Apr 2008 09:23:17 +0200
Subject: [U-Boot-Users] [PATCH V2] cfi-flash: Add
	CFG_FLASH_AUTOPROTECT_LIST
In-Reply-To: <20080420234539.708B6248A6@gemini.denx.de>
References: <20080420234539.708B6248A6@gemini.denx.de>
Message-ID: <200804210923.18006.matthias.fuchs@esd-electronics.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi,

let me summarize: let's take my initial patch that I posted on Friday.
At least two people will be satisfied :-)

Matthias

On Monday 21 April 2008 01:45, Wolfgang Denk wrote:
> In message <200804201937.32557.matthias.fuchs@esd-electronics.com> you wrote:
> > This patch adds a configurable flash auto protection list that can be used
> > to make U-Boot protect flash regions in flash_init().
> > 
> > The idea has been discussed on the u-boot mailing list starting
> > on Nov 18th, 2007.
> > 
> > Even this patch brings a new feature it is used as a bugfix for 4xx
> > platforms where flash_init() does not completely protect the
> > monitor's flash range in all situations.
> > 
> > U-Boot protects the flash range from CFG_MONITOR_BASE to
> > (CFG_MONITOR_BASE + monitor_flash_len  - 1) by default. This does not
> > include the reset vector at 0xfffffffc.
> > 
> > Example:
> > #define CFG_FLASH_AUTOPROTECT_LIST {{0xfff80000, 0x80000}}
> > 
> > This config option will auto protect the last 512k of flash that
> > contains the bootloader on board like APC405 and PMC405.
> > 
> > Signed-off-by: Matthias Fuchs <matthias.fuchs@esd-electronics.com>
> > ---
> >  drivers/mtd/cfi_flash.c |   21 +++++++++++++++++++++
> >  1 files changed, 21 insertions(+), 0 deletions(-)
> > 
> > diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
> > index e3cfb8a..fad9e8b 100644
> > --- a/drivers/mtd/cfi_flash.c
> > +++ b/drivers/mtd/cfi_flash.c
> > @@ -169,6 +169,17 @@ flash_info_t flash_info[CFG_MAX_FLASH_BANKS];		/* FLASH chips info */
> >  #define CFG_FLASH_CFI_WIDTH	FLASH_CFI_8BIT
> >  #endif
> >  
> > +struct apl_s {
> > +	ulong start;
> > +	ulong size;
> > +};
> > +
> > +#if !defined(CFG_FLASH_AUTOPROTECT_LIST)
> > +struct apl_s apl[] = {};
> > +#else
> > +struct apl_s apl[] = CFG_FLASH_AUTOPROTECT_LIST;
> > +#endif
> 
> No. Please don't add any code nor data when CFG_FLASH_AUTOPROTECT_LIST
> is undefined.
> 
> >  typedef unsigned long flash_sect_t;
> >  
> >  /* CFI standard query structure */
> > @@ -1966,6 +1977,16 @@ unsigned long flash_init (void)
> >  		       CFG_ENV_ADDR_REDUND + CFG_ENV_SIZE_REDUND - 1,
> >  		       flash_get_info(CFG_ENV_ADDR_REDUND));
> >  #endif
> > +
> > +	for (i = 0; i < ARRAY_SIZE(apl); 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));
> > +	}
> > +
> 
> No. Please don't add any code nor data when CFG_FLASH_AUTOPROTECT_LIST
> is undefined.
> 
> NAK.
> 
> Best regards,
> 
> Wolfgang Denk
>