[U-Boot] [PATCH 05/11] Blackfin: bf60x: support big cplb page

Wolfgang Denk wd at denx.de
Thu Feb 7 11:33:42 CET 2013


Dear Sonic Zhang,

In message <1360223258-6945-6-git-send-email-sonic.adi at gmail.com> you wrote:
> From: Bob Liu <lliubbo at gmail.com>
> 
> Bf60x support 16K, 64K, 16M and 64M cplb pages, this patch add support for them.
> So that bf609-ezkit can use it's 128M memory.

Checkpatch issues, please fix.


> -#define CPLB_PAGE_SIZE (4 * 1024 * 1024)
> -#define CPLB_PAGE_MASK (~(CPLB_PAGE_SIZE - 1))
...
> +	uint32_t cplb_page_size;
> +	uint32_t cplb_page_mask;
...
> +	cplb_page_size = (4 * 1024 * 1024);
> +	cplb_page_mask = (~(cplb_page_size - 1));
...
> -		mbase &= CPLB_PAGE_MASK;
> -		mend &= CPLB_PAGE_MASK;
> +		mbase &= cplb_page_mask;
> +		mend &= cplb_page_mask;

What exsactly is the reason for replacing a compile-time calculation
with a run-time one?

Best regards,

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: wd at denx.de
Whom the gods would destroy, they first teach BASIC.


More information about the U-Boot mailing list