[U-Boot] [PATCH] mpc83xx: New board support SIMPC8313

Scott Wood scottwood at freescale.com
Wed Nov 12 22:43:03 CET 2008


Ron Madrid wrote:
>>> +        *((u32 *)(CFG_IMMR + addr)) = data;
>> use the out_be32 i/o memory accessor and friends please (here and
>> elsewhere in the patch).
>>
> 
> I am trying to resolve this last issue with your review and replacing
> all of the i/o memory accesses with in/out_be* has caused my
> code to inflate by about 200 bytes, and now I'm over the 4K limit 
> by about 160 bytes for the 8313 bootstrap.  Without trying to
> squeeze my code is there anything that could be done to shrink
> my code?

Use raw accessors and make sure that explicit barriers are used where 
needed.

-Scott


More information about the U-Boot mailing list