On Wednesday 04 June 2008, Detlev Zundel wrote:
> > On Sunday 01 June 2008, Wolfgang Denk wrote:
> >> In message [EMAIL PROTECTED] you wrote:
> >> > When available, use the on-chip syscontrol() rom function to perform a
> >> > system reset rather than our own local method.
> >> > - asm("jump (%0);" : : "a" (bfin_reset));
> >> > + if (0 /*BOOTROM_CAPS_SYSCONTROL -- current silicon is
> >> > broken*/)
> >> > + syscontrol(SYSCTRL_SOFTRESET, NULL, NULL);
> >> > + else
> >> > + asm("jump (%0);" : : "a" (bfin_reset));
> >> > }
> >>
> >> Your comment suggests you change something, but actually you;re just
> >> adding dead code.
> >
> > it's dead code because the current silicon is broken. i know the code
> > works as ive tested it in scenarios where the anomaly isnt triggered, it
> > just cant be enabled until fixed hardware is released. it's much easier
> > to go back and fix 1 if statement then have to carry this patch somewhere
> > it can got lost and i need to waste time and rewrite it.
>
> Hm. If the current - available - silicon is broken, then even if it is
> fixed in the future, you will have to support both versions, right? So
> in the end you will need to have a discriminating factor on what to use.
>
> So if you introduced some variable or bit in a variable (a define
> doesn't look promising as I guess you want to support both broken and
> non-broken silicon with a single image), you could have correct code now
> and in the future - and no dead code complaints.there is already a Blackfin anomaly framework in place for this. we're just awaiting for the anomaly to be formalized. that is what i meant when i said "just update the if line". > PowerPC Linux btw. does quite a lot of such "cpu_feature" stuff to > dynamically accomodate lots of different silicon. we dont do dynamic stuff like powerpc -mike
signature.asc
Description: This is a digitally signed message part.
------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php
_______________________________________________ U-Boot-Users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/u-boot-users
