Re: [PATCH] arm: rockchip: switch to mainline dts for rk3568-evb1

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jun 08, 2022 at 10:17:47PM +0200, Michael Riesch wrote:
> The RK3568 EVB1 device tree in mainline Linux has matured since
> the initial RK3568 support entered barebox. Therefore, we can now
> switch to the mainline device tree.
> 
> Signed-off-by: Michael Riesch <michael.riesch@xxxxxxxxxxxxxx>
> ---
>  arch/arm/boards/rockchip-rk3568-evb/board.c |   5 +-
>  arch/arm/dts/rk3568-evb1-v10.dts            | 519 +-------------------
>  2 files changed, 5 insertions(+), 519 deletions(-)

Applied, thanks

Sascha

> 
> diff --git a/arch/arm/boards/rockchip-rk3568-evb/board.c b/arch/arm/boards/rockchip-rk3568-evb/board.c
> index 09385bea29..a466d385a2 100644
> --- a/arch/arm/boards/rockchip-rk3568-evb/board.c
> +++ b/arch/arm/boards/rockchip-rk3568-evb/board.c
> @@ -28,8 +28,9 @@ static int rk3568_evb_probe(struct device_d *dev)
>  	else
>  		of_device_enable_path("/chosen/environment-emmc");
>  
> -	rk3568_bbu_mmc_register("emmc", BBU_HANDLER_FLAG_DEFAULT, "/dev/emmc");
> -	rk3568_bbu_mmc_register("sd", 0, "/dev/sd");
> +	rk3568_bbu_mmc_register("sd", 0, "/dev/mmc0");
> +	rk3568_bbu_mmc_register("emmc", BBU_HANDLER_FLAG_DEFAULT,
> +				"/dev/mmc1");
>  
>  	return 0;
>  }
> diff --git a/arch/arm/dts/rk3568-evb1-v10.dts b/arch/arm/dts/rk3568-evb1-v10.dts
> index df5633978d..d2c1fc89a8 100644
> --- a/arch/arm/dts/rk3568-evb1-v10.dts
> +++ b/arch/arm/dts/rk3568-evb1-v10.dts
> @@ -5,22 +5,11 @@
>   */
>  
>  /dts-v1/;
> -#include <dt-bindings/gpio/gpio.h>
> -#include <dt-bindings/pinctrl/rockchip.h>
> -#include "rk3568.dtsi"
>  
> -/ {
> -	model = "Rockchip RK3568 EVB1 DDR4 V10 Board";
> -	compatible = "rockchip,rk3568-evb1-v10", "rockchip,rk3568";
> -
> -	aliases {
> -		emmc = &sdhci;
> -		sd = &sdmmc0;
> -	};
> +#include <arm64/rockchip/rk3568-evb1-v10.dts>
>  
> +/ {
>  	chosen: chosen {
> -		stdout-path = "serial2:1500000n8";
> -
>  		environment-sd {
>  			compatible = "barebox,environment";
>  			device-path = &environment_sd;
> @@ -38,437 +27,10 @@
>  		device_type = "memory";
>  		reg = <0x0 0x00a00000 0x0 0x7f600000>;
>  	};
> -
> -	dc_12v: dc-12v {
> -		compatible = "regulator-fixed";
> -		regulator-name = "dc_12v";
> -		regulator-always-on;
> -		regulator-boot-on;
> -		regulator-min-microvolt = <12000000>;
> -		regulator-max-microvolt = <12000000>;
> -	};
> -
> -	vcc3v3_sys: vcc3v3-sys {
> -		compatible = "regulator-fixed";
> -		regulator-name = "vcc3v3_sys";
> -		regulator-always-on;
> -		regulator-boot-on;
> -		regulator-min-microvolt = <3300000>;
> -		regulator-max-microvolt = <3300000>;
> -		vin-supply = <&dc_12v>;
> -	};
> -
> -	vcc5v0_sys: vcc5v0-sys {
> -		compatible = "regulator-fixed";
> -		regulator-name = "vcc5v0_sys";
> -		regulator-always-on;
> -		regulator-boot-on;
> -		regulator-min-microvolt = <5000000>;
> -		regulator-max-microvolt = <5000000>;
> -		vin-supply = <&dc_12v>;
> -	};
> -
> -	vcc3v3_lcd0_n: vcc3v3-lcd0-n {
> -		compatible = "regulator-fixed";
> -		regulator-name = "vcc3v3_lcd0_n";
> -		regulator-boot-on;
> -
> -		regulator-state-mem {
> -			regulator-off-in-suspend;
> -		};
> -	};
> -
> -	vcc3v3_lcd1_n: vcc3v3-lcd1-n {
> -		compatible = "regulator-fixed";
> -		regulator-name = "vcc3v3_lcd1_n";
> -		regulator-boot-on;
> -
> -		regulator-state-mem {
> -			regulator-off-in-suspend;
> -		};
> -	};
> -
> -	vcc5v0_otg: vcc5v0-otg-regulator {
> -		compatible = "regulator-fixed";
> -		enable-active-high;
> -		gpio = <&gpio0 RK_PA5 GPIO_ACTIVE_HIGH>;
> -		pinctrl-names = "default";
> -		pinctrl-0 = <&vcc5v0_otg_en>;
> -		regulator-name = "vcc5v0_otg";
> -	};
> -
> -	vcc5v0_host: vcc5v0-host-regulator {
> -		compatible = "regulator-fixed";
> -		enable-active-high;
> -		gpio = <&gpio0 RK_PA6 GPIO_ACTIVE_HIGH>;
> -		pinctrl-names = "default";
> -		pinctrl-0 = <&vcc5v0_host_en>;
> -		regulator-name = "vcc5v0_host";
> -		regulator-always-on;
> -	};
> -};
> -
> -&gmac0 {
> -	phy-mode = "rgmii";
> -	clock_in_out = "output";
> -
> -	assigned-clocks = <&cru SCLK_GMAC0_RX_TX>, <&cru SCLK_GMAC0>;
> -	assigned-clock-parents = <&cru SCLK_GMAC0_RGMII_SPEED>;
> -	assigned-clock-rates = <0>, <125000000>;
> -
> -	pinctrl-names = "default";
> -	pinctrl-0 = <&gmac0_miim
> -		     &gmac0_tx_bus2
> -		     &gmac0_rx_bus2
> -		     &gmac0_rgmii_clk
> -		     &gmac0_rgmii_bus>;
> -
> -	tx_delay = <0x3c>;
> -	rx_delay = <0x2f>;
> -
> -	phy-handle = <&rgmii_phy0>;
> -	status = "okay";
> -};
> -
> -&gmac1 {
> -	phy-mode = "rgmii";
> -	clock_in_out = "output";
> -
> -	assigned-clocks = <&cru SCLK_GMAC1_RX_TX>, <&cru SCLK_GMAC1>;
> -	assigned-clock-parents = <&cru SCLK_GMAC1_RGMII_SPEED>;
> -	assigned-clock-rates = <0>, <125000000>;
> -
> -	pinctrl-names = "default";
> -	pinctrl-0 = <&gmac1m1_miim
> -		     &gmac1m1_tx_bus2
> -		     &gmac1m1_rx_bus2
> -		     &gmac1m1_rgmii_clk
> -		     &gmac1m1_rgmii_bus>;
> -
> -	tx_delay = <0x4f>;
> -	rx_delay = <0x26>;
> -
> -	phy-handle = <&rgmii_phy1>;
> -	status = "okay";
> -};
> -
> -&i2c0 {
> -	status = "okay";
> -
> -	rk809: pmic@20 {
> -		compatible = "rockchip,rk809";
> -		reg = <0x20>;
> -		interrupt-parent = <&gpio0>;
> -		interrupts = <3 IRQ_TYPE_LEVEL_LOW>;
> -
> -		pinctrl-names = "default", "pmic-sleep",
> -				"pmic-power-off", "pmic-reset";
> -		pinctrl-0 = <&pmic_int>;
> -		pinctrl-1 = <&soc_slppin_slp>, <&rk817_slppin_slp>;
> -		pinctrl-2 = <&soc_slppin_gpio>, <&rk817_slppin_pwrdn>;
> -		pinctrl-3 = <&soc_slppin_gpio>, <&rk817_slppin_rst>;
> -
> -		rockchip,system-power-controller;
> -		wakeup-source;
> -		#clock-cells = <1>;
> -		clock-output-names = "rk808-clkout1", "rk808-clkout2";
> -		//fb-inner-reg-idxs = <2>;
> -		/* 1: rst regs (default in codes), 0: rst the pmic */
> -		pmic-reset-func = <0>;
> -
> -		vcc1-supply = <&vcc3v3_sys>;
> -		vcc2-supply = <&vcc3v3_sys>;
> -		vcc3-supply = <&vcc3v3_sys>;
> -		vcc4-supply = <&vcc3v3_sys>;
> -		vcc5-supply = <&vcc3v3_sys>;
> -		vcc6-supply = <&vcc3v3_sys>;
> -		vcc7-supply = <&vcc3v3_sys>;
> -		vcc8-supply = <&vcc3v3_sys>;
> -		vcc9-supply = <&vcc3v3_sys>;
> -
> -		pwrkey {
> -			status = "okay";
> -		};
> -
> -		pinctrl_rk8xx: pinctrl_rk8xx {
> -			gpio-controller;
> -			#gpio-cells = <2>;
> -
> -			rk817_slppin_null: rk817_slppin_null {
> -				pins = "gpio_slp";
> -				function = "pin_fun0";
> -			};
> -
> -			rk817_slppin_slp: rk817_slppin_slp {
> -				pins = "gpio_slp";
> -				function = "pin_fun1";
> -			};
> -
> -			rk817_slppin_pwrdn: rk817_slppin_pwrdn {
> -				pins = "gpio_slp";
> -				function = "pin_fun2";
> -			};
> -
> -			rk817_slppin_rst: rk817_slppin_rst {
> -				pins = "gpio_slp";
> -				function = "pin_fun3";
> -			};
> -		};
> -
> -		regulators {
> -			vdd_logic: DCDC_REG1 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <500000>;
> -				regulator-max-microvolt = <1350000>;
> -				regulator-init-microvolt = <900000>;
> -				regulator-ramp-delay = <6001>;
> -				regulator-initial-mode = <0x2>;
> -				regulator-name = "vdd_logic";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vdd_gpu: DCDC_REG2 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <500000>;
> -				regulator-max-microvolt = <1350000>;
> -				regulator-init-microvolt = <900000>;
> -				regulator-ramp-delay = <6001>;
> -				regulator-initial-mode = <0x2>;
> -				regulator-name = "vdd_gpu";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vcc_ddr: DCDC_REG3 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-initial-mode = <0x2>;
> -				regulator-name = "vcc_ddr";
> -				regulator-state-mem {
> -					regulator-on-in-suspend;
> -				};
> -			};
> -
> -			vdd_npu: DCDC_REG4 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <500000>;
> -				regulator-max-microvolt = <1350000>;
> -				regulator-init-microvolt = <900000>;
> -				regulator-ramp-delay = <6001>;
> -				regulator-initial-mode = <0x2>;
> -				regulator-name = "vdd_npu";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vdda0v9_image: LDO_REG1 {
> -				regulator-boot-on;
> -				regulator-always-on;
> -				regulator-min-microvolt = <900000>;
> -				regulator-max-microvolt = <900000>;
> -				regulator-name = "vdda0v9_image";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vdda_0v9: LDO_REG2 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <900000>;
> -				regulator-max-microvolt = <900000>;
> -				regulator-name = "vdda_0v9";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vdda0v9_pmu: LDO_REG3 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <900000>;
> -				regulator-max-microvolt = <900000>;
> -				regulator-name = "vdda0v9_pmu";
> -				regulator-state-mem {
> -					regulator-on-in-suspend;
> -					regulator-suspend-microvolt = <900000>;
> -				};
> -			};
> -
> -			vccio_acodec: LDO_REG4 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <3300000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-name = "vccio_acodec";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vccio_sd: LDO_REG5 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-name = "vccio_sd";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vcc3v3_pmu: LDO_REG6 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <3300000>;
> -				regulator-max-microvolt = <3300000>;
> -				regulator-name = "vcc3v3_pmu";
> -				regulator-state-mem {
> -					regulator-on-in-suspend;
> -					regulator-suspend-microvolt = <3300000>;
> -				};
> -			};
> -
> -			vcca_1v8: LDO_REG7 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <1800000>;
> -				regulator-name = "vcca_1v8";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vcca1v8_pmu: LDO_REG8 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <1800000>;
> -				regulator-name = "vcca1v8_pmu";
> -				regulator-state-mem {
> -					regulator-on-in-suspend;
> -					regulator-suspend-microvolt = <1800000>;
> -				};
> -			};
> -
> -			vcca1v8_image: LDO_REG9 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <1800000>;
> -				regulator-name = "vcca1v8_image";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vcc_1v8: DCDC_REG5 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-min-microvolt = <1800000>;
> -				regulator-max-microvolt = <1800000>;
> -				regulator-name = "vcc_1v8";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vcc_3v3: SWITCH_REG1 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-name = "vcc_3v3";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -
> -			vcc3v3_sd: SWITCH_REG2 {
> -				regulator-always-on;
> -				regulator-boot-on;
> -				regulator-name = "vcc3v3_sd";
> -				regulator-state-mem {
> -					regulator-off-in-suspend;
> -				};
> -			};
> -		};
> -	};
> -};
> -
> -&mdio0 {
> -	rgmii_phy0: phy@0 {
> -		compatible = "ethernet-phy-ieee802.3-c22";
> -		reset-gpios = <&gpio2 RK_PD3 GPIO_ACTIVE_LOW>;
> -		reset-assert-us = <20000>;
> -		reset-deassert-us = <100000>;
> -		reg = <0x0>;
> -	};
> -};
> -
> -&mdio1 {
> -	rgmii_phy1: phy@0 {
> -		compatible = "ethernet-phy-ieee802.3-c22";
> -		reset-gpios = <&gpio2 RK_PD1 GPIO_ACTIVE_LOW>;
> -		reset-assert-us = <20000>;
> -		reset-deassert-us = <100000>;
> -		reg = <0x0>;
> -	};
> -};
> -
> -&pinctrl {
> -	pmic {
> -		pmic_int: pmic_int {
> -			rockchip,pins =
> -				<0 RK_PA3 RK_FUNC_GPIO &pcfg_pull_up>;
> -		};
> -
> -		soc_slppin_gpio: soc_slppin_gpio {
> -			rockchip,pins =
> -				<0 RK_PA2 RK_FUNC_GPIO &pcfg_output_low>;
> -		};
> -
> -		soc_slppin_slp: soc_slppin_slp {
> -			rockchip,pins =
> -				<0 RK_PA2 1 &pcfg_pull_none>;
> -		};
> -
> -		soc_slppin_rst: soc_slppin_rst {
> -			rockchip,pins =
> -				<0 RK_PA2 2 &pcfg_pull_none>;
> -		};
> -	};
> -
> -	usb {
> -		vcc5v0_host_en: vcc5v0-host-en {
> -			rockchip,pins = <0 RK_PA6 RK_FUNC_GPIO &pcfg_pull_none>;
> -		};
> -
> -		vcc5v0_otg_en: vcc5v0-otg-en {
> -			rockchip,pins = <0 RK_PA5 RK_FUNC_GPIO &pcfg_pull_none>;
> -		};
> -	};
> -};
> -
> -&saradc {
> -	vref-supply = <&vcca_1v8>;
> -	status = "okay";
>  };
>  
>  &sdhci {
> -	bus-width = <8>;
> -	max-frequency = <200000000>;
> -	non-removable;
>  	no-sd;
> -	status = "okay";
> -	pinctrl-names = "default";
> -	pinctrl-0 = <&emmc_bus8 &emmc_clk &emmc_cmd &emmc_datastrobe>;
>  
>  	partitions {
>  		compatible = "fixed-partitions";
> @@ -483,18 +45,6 @@
>  };
>  
>  &sdmmc0 {
> -	max-frequency = <150000000>;
> -	bus-width = <4>;
> -	cap-mmc-highspeed;
> -	cap-sd-highspeed;
> -	disable-wp;
> -	sd-uhs-sdr104;
> -	vmmc-supply = <&vcc3v3_sd>;
> -	vqmmc-supply = <&vccio_sd>;
> -	pinctrl-names = "default";
> -	pinctrl-0 = <&sdmmc0_bus4 &sdmmc0_clk &sdmmc0_cmd &sdmmc0_det>;
> -	status = "okay";
> -
>  	partitions {
>  		compatible = "fixed-partitions";
>  		#address-cells = <2>;
> @@ -506,68 +56,3 @@
>  		};
>  	};
>  };
> -
> -&uart2 {
> -	status = "okay";
> -};
> -
> -&u2phy0_host {
> -	phy-supply = <&vcc5v0_host>;
> -	status = "okay";
> -};
> -
> -&u2phy0_otg {
> -	vbus-supply = <&vcc5v0_otg>;
> -	status = "okay";
> -};
> -
> -&u2phy1_host {
> -	phy-supply = <&vcc5v0_host>;
> -	status = "okay";
> -};
> -
> -&u2phy1_otg {
> -	phy-supply = <&vcc5v0_host>;
> -	status = "okay";
> -};
> -
> -&usb2phy0 {
> -	status = "okay";
> -};
> -
> -&usb2phy1 {
> -	status = "okay";
> -};
> -
> -&usb_host0_ehci {
> -	status = "okay";
> -};
> -
> -&usb_host0_ohci {
> -	status = "okay";
> -};
> -
> -&usb_host0_xhci {
> -	extcon = <&usb2phy0>;
> -	status = "okay";
> -};
> -
> -&usb_host1_ehci {
> -	status = "okay";
> -};
> -
> -&usb_host1_ohci {
> -	status = "okay";
> -};
> -
> -&usb_host1_xhci {
> -	status = "okay";
> -};
> -
> -&combphy0_us {
> -	status = "okay";
> -};
> -
> -&combphy1_usq {
> -	status = "okay";
> -};
> -- 
> 2.30.2
> 
> 
> _______________________________________________
> barebox mailing list
> barebox@xxxxxxxxxxxxxxxxxxx
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux