Re: [PATCH v2 2/2] arm64: dts: imx8mn-ddr4-evk: correct ldo1/ldo2 voltage range

[Fabio Estevam]

Hi Robin,

On Thu, May 21, 2020 at 3:17 AM Robin Gong <yibin.gong@xxxxxxx> wrote:
>
> Correct ldo1 voltage range from wrong high group(3.0v~3.3v) to low group

Please use capital letter V for Volt.

> (1.6v~1.9v) because the ldo1 should be 1.8v. Actually, two voltage groups
> have been supported at bd718x7-regulator driver, hence, just corrrect the
> voltage range to 1.6v~3.3v. For ldo2@xxxx, correct voltage range too.
> Otherwise, ldo1 would be kept @3.0v and ldo2@xxxx which violate i.mx8mn
> datasheet as the below warning log in kernel:
>
> [    0.995524] LDO1: Bringing 1800000uV into 3000000-3000000uV
> [    0.999196] LDO2: Bringing 800000uV into 900000-900000uV
>
> Signed-off-by: Robin Gong <yibin.gong@xxxxxxx>

You should add a Fixes tag and Cc stable on this one.

Thanks