[PATCH v7 3/6] rockchip: px30: add the serial flash controller

Jagan Teki jagan at amarulasolutions.com
Thu Aug 5 13:30:44 CEST 2021


On Thu, Aug 5, 2021 at 1:57 PM Jon Lin <jon.lin at rock-chips.com> wrote:
>
> From: Chris Morgan <macromorgan at hotmail.com>
>
> Add the serial flash controller to the devicetree for the PX30.
>
> Signed-off-by: Chris Morgan <macromorgan at hotmail.com>
> Signed-off-by: Jon Lin <jon.lin at rock-chips.com>
> ---
>
> (no changes since v5)
>
> Changes in v5:
> - px30 use "rockchip, sfc" as compatible id
>
>  arch/arm/dts/px30.dtsi | 38 ++++++++++++++++++++++++++++++++++++++
>  1 file changed, 38 insertions(+)
>
> diff --git a/arch/arm/dts/px30.dtsi b/arch/arm/dts/px30.dtsi
> index b6c79e7ed3..aaa8ae2235 100644
> --- a/arch/arm/dts/px30.dtsi
> +++ b/arch/arm/dts/px30.dtsi
> @@ -960,6 +960,18 @@
>                 status = "disabled";
>         };
>
> +       sfc: sfc at ff3a0000 {
> +               compatible = "rockchip,sfc";
> +               reg = <0x0 0xff3a0000 0x0 0x4000>;
> +               interrupts = <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>;
> +               clocks = <&cru SCLK_SFC>, <&cru HCLK_SFC>;
> +               clock-names = "clk_sfc", "hclk_sfc";
> +               pinctrl-names = "default";
> +               pinctrl-0 = <&sfc_clk &sfc_cs &sfc_bus4>;
> +               power-domains = <&power PX30_PD_MMC_NAND>;
> +               status = "disabled";
> +       };
> +
>         gpu: gpu at ff400000 {
>                 compatible = "rockchip,px30-mali", "arm,mali-bifrost";
>                 reg = <0x0 0xff400000 0x0 0x4000>;
> @@ -1926,6 +1938,32 @@
>                         };
>                 };
>
> +               serial_flash {
> +                       sfc_bus4: sfc-bus4 {
> +                               rockchip,pins =
> +                                       <1 RK_PA0 3 &pcfg_pull_none>,
> +                                       <1 RK_PA1 3 &pcfg_pull_none>,
> +                                       <1 RK_PA2 3 &pcfg_pull_none>,
> +                                       <1 RK_PA3 3 &pcfg_pull_none>;
> +                       };
> +
> +                       sfc_bus2: sfc-bus2 {
> +                               rockchip,pins =
> +                                       <1 RK_PA0 3 &pcfg_pull_none>,
> +                                       <1 RK_PA1 3 &pcfg_pull_none>;
> +                       };
> +
> +                       sfc_cs: sfc-cs {
> +                               rockchip,pins =
> +                                       <1 RK_PA4 3 &pcfg_pull_none>;
> +                       };
> +
> +                       sfc_clk: sfc-clk {
> +                               rockchip,pins =
> +                                       <1 RK_PB1 3 &pcfg_pull_none>;
> +                       };
> +               };
> +

Better to sync the Linux, instead of intermediate add-ons. Add it in
-u-boot.dtsi so-that it can remove while syncing rc tags, may be Kever
can comment. Whole idea is to keep px30.dtsi as same as Linux.

Thanks,
Jagan.


More information about the U-Boot mailing list