Re: [PATCH 24/27] imx: imx8ulp_evk: enlarge CONFIG_NR_DRAM_BANKS

2022-03-03 Thread Peng Fan (OSS)




On 2022/2/26 21:28, Fabio Estevam wrote:

Hi Peng,

On Mon, Feb 14, 2022 at 9:09 AM Peng Fan (OSS)  wrote:


From: Peng Fan 

When TEE is present, the DRAM maybe split to two parts,
so enlarge CONFIG_NR_DRAM_BANKS


I think this description is confusing.

CONFIG_NR_DRAM_BANKS should be set according to how many DDR chips are
connected.

The usage of TEE is not relevant to CONFIG_NR_DRAM_BANKS.


TEE locates in the middle of DRAM memory space, it will split
the DRAM. I not have other idea how to support this without
enlarge CONFIG_NR_DRAM_BANKS.

Thanks,
Peng.





Re: [PATCH 24/27] imx: imx8ulp_evk: enlarge CONFIG_NR_DRAM_BANKS

2022-02-26 Thread Fabio Estevam
Hi Peng,

On Mon, Feb 14, 2022 at 9:09 AM Peng Fan (OSS)  wrote:
>
> From: Peng Fan 
>
> When TEE is present, the DRAM maybe split to two parts,
> so enlarge CONFIG_NR_DRAM_BANKS

I think this description is confusing.

CONFIG_NR_DRAM_BANKS should be set according to how many DDR chips are
connected.

The usage of TEE is not relevant to CONFIG_NR_DRAM_BANKS.