Dear Kumar Gala,
In message <[email protected]> you
wrote:
> From: Poonam Aggrwal <[email protected]>
>
> Issue: Address masking doesn't work properly.
> When sum of the base address, defined by BA, and memory bank size,
> defined by AM, exceeds 4GB (0xffff_ffff) then AMASKn[AM] doesn't mask
> CSPRn[BA] bits.
>
> Impact:
> This will impact booting when we are reprogramming CSPR0(BA) and
> AMASK0(AMASK) while executing from NOR Flash.
>
> Workaround:
> Re-programming of CSPR(BA) and AMASK is done while not executing from NOR
> Flash. The code which programs the BA and AMASK is executed from L2-SRAM.
>
> Signed-off-by: Poonam Aggrwal <[email protected]>
> Signed-off-by: Kumar Gala <[email protected]>
This commit introdces new build warnings for the following boards:
P1010RDB_36BIT_NOR P1010RDB_NOR
P1010RDB_36BIT_NOR_SECBOOT P1010RDB_NOR_SECBOOT
For example:
Configuring for P1010RDB_NOR - Board: P1010RDB, Options: P1010RDB
cpu_init_early.c: In function 'cpu_init_early_f':
cpu_init_early.c:74: warning: 'l2srbar' may be used uninitialized in this
function
Please fix!
Kumar, Poonam - I'm really p*ssed off. Both of you have more than
enough of experience to know that you should not submit
untested patches. especially here, where I already had to reject this
patch because it did not even pass checkpatch:
I wrote in message <[email protected]>:
| Dear Kumar Gala,
|
| In message <[email protected]>
| you wrote:
| >
| > > Kumar, could you __please__ get used to running your patches
| > > throuch
| > > checkpatch __before__ submitting? Thanks.
| >
| > I try to, but not all of them are by me ;)
|
| I know. But you submitted them, so you are responsible.
This level of neglect is really disappointing.
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected]
1000 pains = 1 Megahertz
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot
