[PATCH v2 12/23] rockchip: rk3588: Update bootph props
Quentin Schulz
quentin.schulz at theobroma-systems.com
Mon Apr 15 10:55:11 CEST 2024
Hi Jonas,
On 4/13/24 20:13, Jonas Karlman wrote:
> After the commit aca95282c1b7 ("Makefile: Use the fdtgrep -u flag")
> bootph props is propagating to parent nodes.
>
> Update bootph props to ensure eMMC, SD-card and SPI flash is available
> in SPL and U-Boot proper pre-reloc phase also remove unneeded bootph
> props that automatically is propagated to parent nodes.
>
> Also adjust pinctrl nodes to only be included in boot phases where they
> are needed and add any missing pinctrl node needed in SPL.
>
> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
> ---
> v2: Add bootph-some-ram props and follow kernel sort order
>
> Following bootph props have been applied:
>
> CRU, GRF and UART nodes:
> - bootph-all - needed at all or most stages
>
> SD-card regulator related nodes:
> - bootph-pre-ram (SPL) - regulator pinctrl may be needed to read FIT
> from SD-card on some boards
>
> eMMC/SD-card/SPI flash related nodes:
> - bootph-pre-ram (SPL)
> - bootph-some-ram (U-Boot proper pre-reloc)
> ---
> .../arm/dts/rk3588-coolpi-cm5-evb-u-boot.dtsi | 8 +--
> arch/arm/dts/rk3588-generic.dts | 1 +
> arch/arm/dts/rk3588-nanopc-t6-u-boot.dtsi | 12 ++---
> .../dts/rk3588-orangepi-5-plus-u-boot.dtsi | 7 ++-
> arch/arm/dts/rk3588-rock-5b-u-boot.dtsi | 8 +--
> arch/arm/dts/rk3588-turing-rk1-u-boot.dtsi | 6 ++-
> arch/arm/dts/rk3588s-coolpi-4b-u-boot.dtsi | 8 +--
> arch/arm/dts/rk3588s-orangepi-5-u-boot.dtsi | 9 ++--
> arch/arm/dts/rk3588s-u-boot.dtsi | 50 ++++++++++++++-----
> configs/evb-rk3588_defconfig | 4 +-
> 10 files changed, 71 insertions(+), 42 deletions(-)
>
> diff --git a/arch/arm/dts/rk3588-coolpi-cm5-evb-u-boot.dtsi b/arch/arm/dts/rk3588-coolpi-cm5-evb-u-boot.dtsi
> index ed15b14ea0ee..f0ef0164664e 100644
> --- a/arch/arm/dts/rk3588-coolpi-cm5-evb-u-boot.dtsi
> +++ b/arch/arm/dts/rk3588-coolpi-cm5-evb-u-boot.dtsi
> @@ -3,7 +3,8 @@
> #include "rk3588-u-boot.dtsi"
>
> &fspim2_pins {
> - bootph-all;
> + bootph-pre-ram;
> + bootph-some-ram;
> };
>
> &sdhci {
> @@ -12,16 +13,15 @@
> };
>
> &sfc {
> - bootph-pre-ram;
> - u-boot,spl-sfc-no-dma;
> pinctrl-names = "default";
> pinctrl-0 = <&fspim2_pins>;
> status = "okay";
>
> flash at 0 {
> - bootph-pre-ram;
> compatible = "jedec,spi-nor";
> reg = <0>;
> + bootph-pre-ram;
> + bootph-some-ram;
> spi-max-frequency = <24000000>;
> spi-rx-bus-width = <4>;
> spi-tx-bus-width = <1>;
> diff --git a/arch/arm/dts/rk3588-generic.dts b/arch/arm/dts/rk3588-generic.dts
> index e4721d97a87d..baafe7463f1b 100644
> --- a/arch/arm/dts/rk3588-generic.dts
> +++ b/arch/arm/dts/rk3588-generic.dts
> @@ -40,5 +40,6 @@
> };
>
> &uart2 {
> + pinctrl-0 = <&uart2m0_xfer>;
I think this should be its own patch.
[...]
> diff --git a/configs/evb-rk3588_defconfig b/configs/evb-rk3588_defconfig
> index 68ecbc54b807..610a8d740fbb 100644
> --- a/configs/evb-rk3588_defconfig
> +++ b/configs/evb-rk3588_defconfig
> @@ -33,7 +33,8 @@ CONFIG_CMD_REGULATOR=y
> # CONFIG_SPL_DOS_PARTITION is not set
> CONFIG_SPL_OF_CONTROL=y
> CONFIG_OF_LIVE=y
> -CONFIG_OF_SPL_REMOVE_PROPS="pinctrl-0 pinctrl-names clock-names interrupt-parent assigned-clocks assigned-clock-rates assigned-clock-parents"
> +CONFIG_OF_SPL_REMOVE_PROPS="clock-names interrupt-parent assigned-clocks assigned-clock-rates assigned-clock-parents"
> +CONFIG_SPL_DM_SEQ_ALIAS=y
> CONFIG_SPL_REGMAP=y
> CONFIG_SPL_SYSCON=y
> CONFIG_SPL_CLK=y
> @@ -52,6 +53,7 @@ CONFIG_DWC_ETH_QOS_ROCKCHIP=y
> CONFIG_PHY_ROCKCHIP_INNO_USB2=y
> CONFIG_PHY_ROCKCHIP_NANENG_COMBOPHY=y
> CONFIG_PHY_ROCKCHIP_USBDP=y
> +CONFIG_SPL_PINCTRL=y
I have a feeling the changes to the defconfig aren't entirely related to
this patch? Specifically, I think at least CONFIG_SPL_DM_SEQ_ALIAS isn't
related?
Otherwise,
Reviewed-by: Quentin Schulz <quentin.schulz at theobroma-systems.com>
Thanks,
Quentin
More information about the U-Boot
mailing list