Re: [PATCH] ram: stm32mp1: Conditionally enable ASR

2022-04-26 Thread Patrick DELAUNAY
On 4/26/22 12:49, Patrick DELAUNAY wrote: Hi Marek, On 4/14/22 19:34, Marek Vasut wrote: On 4/14/22 18:48, Marek Vasut wrote: On 4/14/22 18:37, Patrick DELAUNAY wrote: Hi Marek, Hi, on ST platform the ASR/SSR/HSR request are already provided by the DDR settings with pwrctl register

Re: [PATCH] ram: stm32mp1: Conditionally enable ASR

2022-04-26 Thread Patrick DELAUNAY
Hi Marek, On 4/14/22 19:34, Marek Vasut wrote: On 4/14/22 18:48, Marek Vasut wrote: On 4/14/22 18:37, Patrick DELAUNAY wrote: Hi Marek, Hi, on ST platform the ASR/SSR/HSR request are already provided by the DDR settings with pwrctl register value it is managed in TF-A by

Re: [PATCH] ram: stm32mp1: Conditionally enable ASR

2022-04-22 Thread Patrice CHOTARD
Hi Marek On 4/13/22 04:49, Marek Vasut wrote: > Enable DRAM ASR, auto self-refresh, conditionally, based on DT property > "st,mem-enable-asr" . While ASR does save considerable amount of power > at runtime automatically, it also causes LTDC underruns on large panels. > Let user select whether or

Re: [PATCH] ram: stm32mp1: Conditionally enable ASR

2022-04-14 Thread Marek Vasut
On 4/14/22 18:48, Marek Vasut wrote: On 4/14/22 18:37, Patrick DELAUNAY wrote: Hi Marek, Hi, on ST platform the ASR/SSR/HSR request are already provided by the DDR settings with pwrctl register value it is managed in TF-A by arm-trusted-firmware/drivers/st/ddr/stm32mp1_ddr_helpers.c

Re: [PATCH] ram: stm32mp1: Conditionally enable ASR

2022-04-14 Thread Marek Vasut
On 4/14/22 18:37, Patrick DELAUNAY wrote: Hi Marek, Hi, on ST platform the ASR/SSR/HSR request are already provided by the DDR settings with pwrctl register value it is managed in TF-A by arm-trusted-firmware/drivers/st/ddr/stm32mp1_ddr_helpers.c Sure, I don't use ATF and I have no

Re: [PATCH] ram: stm32mp1: Conditionally enable ASR

2022-04-14 Thread Patrick DELAUNAY
Hi Marek, on ST platform the ASR/SSR/HSR request are already provided by the DDR settings with pwrctl register value it is managed in TF-A by arm-trusted-firmware/drivers/st/ddr/stm32mp1_ddr_helpers.c enumstm32mp1_ddr_sr_mode ddr_read_sr_mode(void) { uint32_tpwrctl =

[PATCH] ram: stm32mp1: Conditionally enable ASR

2022-04-12 Thread Marek Vasut
Enable DRAM ASR, auto self-refresh, conditionally, based on DT property "st,mem-enable-asr" . While ASR does save considerable amount of power at runtime automatically, it also causes LTDC underruns on large panels. Let user select whether or not ASR is required or not, generally ASR should be