[PATCH] rockchip: Pinebook Pro: Do not initialize i2c before relocation

Kever Yang kever.yang at rock-chips.com
Sun Dec 18 11:17:00 CET 2022


On 2022/12/3 20:31, Michal Suchanek wrote:
> The i2c locks up when initialized before relocation, and it stays broken
> in Linux as well breaking the ability to boot Linux.
>
> The i2c bus and pmic was not actually used in pre-reloc before
> commit ad607512f575 ("power: pmic: rk8xx: Support sysreset shutdown method")
>
> The cause is not known.
>
> This is board-specific, other boards that do not add the option to
> include the i2c bus in pre-reloc DT are not affected.
>
> Signed-off-by: Michal Suchanek <msuchanek at suse.de>
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
>
> This is not tested, my board does not currentl;y boot at all, YMMV
> ---
>   arch/arm/dts/rk3399-pinebook-pro-u-boot.dtsi | 8 --------
>   1 file changed, 8 deletions(-)
>
> diff --git a/arch/arm/dts/rk3399-pinebook-pro-u-boot.dtsi b/arch/arm/dts/rk3399-pinebook-pro-u-boot.dtsi
> index 2d87bea933..fd87102c0b 100644
> --- a/arch/arm/dts/rk3399-pinebook-pro-u-boot.dtsi
> +++ b/arch/arm/dts/rk3399-pinebook-pro-u-boot.dtsi
> @@ -20,14 +20,6 @@
>   	rockchip,panel = <&edp_panel>;
>   };
>   
> -&i2c0 {
> -	u-boot,dm-pre-reloc;
> -};
> -
> -&rk808 {
> -	u-boot,dm-pre-reloc;
> -};
> -
>   &sdhci {
>   	max-frequency = <25000000>;
>   	u-boot,dm-pre-reloc;


More information about the U-Boot mailing list