lkml.org 
[lkml]   [2018]   [Mar]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] arm64: defconfig: enable Rockchip io-domain driver
From
Date
On 08/03/18 21:35, Enric Balletbo i Serra wrote:
> Heiko Stübner justified pretty well the change in commit e330eb86ba0
> 'ARM: multi_v7_defconfig: enable Rockchip io-domain driver'. This change
> is also needed for arm64 rockchip boards, so, do the same for arm64.
>
> The io-domain driver is necessary to notify the soc about voltages
> changes happening on supplying regulators. Probably the most important
> user right now is the mmc tuning code, where the soc needs to get
> notified when the voltage is dropped to the 1.8V point.
>
> As this option is necessary to successfully tune UHS cards etc, it
> should get built in. Otherwise, tunning will fail with,
>
> dwmmc_rockchip fe320000.dwmmc: All phases bad!
> mmc0: tuning execution failed: -5

It also causes ethernet to fail in a rather odd manner if your board
needs RGMII at <3.3v but you're using firmware which leaves the relevant
IO domain set inappropriately (I guess the parts expecting to be driven
by the external phy clock just lock up). Given the several hours I spent
scratching my head over that one,

Acked-by: Robin Murphy <robin.murphy@arm.com>

> Cc: Heiko Stübner <heiko@sntech.de>
> Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
> ---
> arch/arm64/configs/defconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index ba859709c2d4..d65d53b1ff77 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -313,6 +313,8 @@ CONFIG_GPIO_XGENE_SB=y
> CONFIG_GPIO_PCA953X=y
> CONFIG_GPIO_PCA953X_IRQ=y
> CONFIG_GPIO_MAX77620=y
> +CONFIG_POWER_AVS=y
> +CONFIG_ROCKCHIP_IODOMAIN=y
> CONFIG_POWER_RESET_MSM=y
> CONFIG_POWER_RESET_XGENE=y
> CONFIG_POWER_RESET_SYSCON=y
>

\
 
 \ /
  Last update: 2018-03-09 13:50    [W:0.030 / U:0.880 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site