[U-Boot-Users] [PATCH] IXP: add dynamic microcode addr

[Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>]

On 14:23 Wed 09 Jan     , Stefan Roese wrote:
> On Wednesday 09 January 2008, Jean-Christophe PLAGNIOL-VILLARD wrote:
> > allow to load the microde from flash or ram by download it through
> > the serial or other.
> >
> > Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>
> >
> > diff --git a/cpu/ixp/npe/IxNpeDlImageMgr.c b/cpu/ixp/npe/IxNpeDlImageMgr.c
> > index c0e7349..1c47661 100644
> > --- a/cpu/ixp/npe/IxNpeDlImageMgr.c
> > +++ b/cpu/ixp/npe/IxNpeDlImageMgr.c
> > @@ -140,6 +140,13 @@ static UINT32 *IxNpeMicroCodeImageLibrary = (UINT32
> > *)CONFIG_IXP4XX_NPE_EXT_UCOD static UINT32 *IxNpeMicroCodeImageLibrary =
> > (UINT32 *)IxNpeMicrocode_array; #endif
> >
> > +static UNIT32* getIxNpeMicroCodeImageLibrary()
> > +{
> > +	if ((s = getenv("microcodeaddr")) != NULL) {
> > +		load_addr = simple_strtoul(s, NULL, 16);
> > +	}
> > +	else return IxNpeMicroCodeImageLibrary;
> 
> 	if ((s = getenv("microcodeaddr")) != NULL)
> 		load_addr = simple_strtoul(s, NULL, 16);
> 	else
> 		return IxNpeMicroCodeImageLibrary;
> 
> please.
Done.
> 
> I'm not so sure about the naming of the env variable too. How 
> about "npe_ucode"?

Why not

Best regards,
J.