* [PATCH v2 0/3] rk3318 A95X Z2 board @ 2020-07-18 10:53 Johan Jonker 2020-07-18 10:53 ` [PATCH v2 1/3] dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd Johan Jonker ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Johan Jonker @ 2020-07-18 10:53 UTC (permalink / raw) To: heiko; +Cc: devicetree, robh+dt, linux-kernel, linux-arm-kernel, linux-rockchip Add rk3318 A95X Z2 board. What works: adc key emmc ethernet hdmi gpu ir mmc sd card vop usb2 uart2 Not tested: av uart0 / bt spdif wifi No support in mainline: usb3 (works with manufacturer driver) front display (works with custom gpio driver AD1628/HT1628) Problems: U-boot only starts on emmc, then loads kernel from sd card or emmc. Hdmi hotplug detects DVI modes, but marks them all BAD. The dw-hdmi driver needs more pre_pll_cfg_table entries. Slow boot behavior. Screen available way after boot. (no penguins) GPU support for Debian is not up to date. (old version) Use Github version instead. Test with glmark2-es2-drm gives glmark2 Score: 29 Kernel log: lima ff300000.gpu: pp pp0 reset time out lima ff300000.gpu: pp pp1 reset time out No mainline support for GPU thermal and voltage to frequency. PWM1 supply connected to DMC, GPU and vcodec. The box only has a model name. Vendor prefix not verified. Etc. Changed V2: add vendor prefix changed compatible string restyle removed usb3 regulator node changed led active high to active low Johan Jonker (3): dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd. dt-bindings: arm: rockchip: add Zkmagic A95X Z2 description arm64: dts: rockchip: add rk3318 A95X Z2 board .../devicetree/bindings/arm/rockchip.yaml | 5 + .../devicetree/bindings/vendor-prefixes.yaml | 2 + arch/arm64/boot/dts/rockchip/Makefile | 1 + arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts | 393 +++++++++++++++++++++ 4 files changed, 401 insertions(+) create mode 100644 arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts -- 2.11.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 1/3] dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd. 2020-07-18 10:53 [PATCH v2 0/3] rk3318 A95X Z2 board Johan Jonker @ 2020-07-18 10:53 ` Johan Jonker 2020-07-23 17:36 ` Rob Herring 2020-07-18 10:53 ` [PATCH v2 2/3] dt-bindings: arm: rockchip: add Zkmagic A95X Z2 description Johan Jonker 2020-07-18 10:53 ` [PATCH v2 3/3] arm64: dts: rockchip: add rk3318 A95X Z2 board Johan Jonker 2 siblings, 1 reply; 8+ messages in thread From: Johan Jonker @ 2020-07-18 10:53 UTC (permalink / raw) To: heiko; +Cc: devicetree, robh+dt, linux-kernel, linux-arm-kernel, linux-rockchip Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd. Signed-off-by: Johan Jonker <jbx6244@gmail.com> --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index 9aeab66be..5ab3d7c93 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -1181,6 +1181,8 @@ patternProperties: description: Shenzhen Zidoo Technology Co., Ltd. "^zii,.*": description: Zodiac Inflight Innovations + "^zkmagic,.*": + description: Shenzhen Zkmagic Technology Co., Ltd. "^zte,.*": description: ZTE Corp. "^zyxel,.*": -- 2.11.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2 1/3] dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd. 2020-07-18 10:53 ` [PATCH v2 1/3] dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd Johan Jonker @ 2020-07-23 17:36 ` Rob Herring 0 siblings, 0 replies; 8+ messages in thread From: Rob Herring @ 2020-07-23 17:36 UTC (permalink / raw) To: Johan Jonker Cc: devicetree, heiko, linux-kernel, linux-rockchip, robh+dt, linux-arm-kernel On Sat, 18 Jul 2020 12:53:41 +0200, Johan Jonker wrote: > Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd. > > Signed-off-by: Johan Jonker <jbx6244@gmail.com> > --- > Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ > 1 file changed, 2 insertions(+) > Acked-by: Rob Herring <robh@kernel.org> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 2/3] dt-bindings: arm: rockchip: add Zkmagic A95X Z2 description 2020-07-18 10:53 [PATCH v2 0/3] rk3318 A95X Z2 board Johan Jonker 2020-07-18 10:53 ` [PATCH v2 1/3] dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd Johan Jonker @ 2020-07-18 10:53 ` Johan Jonker 2020-07-23 17:36 ` Rob Herring 2020-07-18 10:53 ` [PATCH v2 3/3] arm64: dts: rockchip: add rk3318 A95X Z2 board Johan Jonker 2 siblings, 1 reply; 8+ messages in thread From: Johan Jonker @ 2020-07-18 10:53 UTC (permalink / raw) To: heiko; +Cc: devicetree, robh+dt, linux-kernel, linux-arm-kernel, linux-rockchip Add Zkmagic A95X Z2 description for a board with rk3318 processor. Signed-off-by: Johan Jonker <jbx6244@gmail.com> --- Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml index d4a404509..441fe82d2 100644 --- a/Documentation/devicetree/bindings/arm/rockchip.yaml +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml @@ -549,4 +549,9 @@ properties: items: - const: tronsmart,orion-r68-meta - const: rockchip,rk3368 + + - description: Zkmagic A95X Z2 + items: + - const: zkmagic,a95x-z2 + - const: rockchip,rk3318 ... -- 2.11.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2 2/3] dt-bindings: arm: rockchip: add Zkmagic A95X Z2 description 2020-07-18 10:53 ` [PATCH v2 2/3] dt-bindings: arm: rockchip: add Zkmagic A95X Z2 description Johan Jonker @ 2020-07-23 17:36 ` Rob Herring 0 siblings, 0 replies; 8+ messages in thread From: Rob Herring @ 2020-07-23 17:36 UTC (permalink / raw) To: Johan Jonker Cc: devicetree, heiko, linux-kernel, linux-rockchip, robh+dt, linux-arm-kernel On Sat, 18 Jul 2020 12:53:42 +0200, Johan Jonker wrote: > Add Zkmagic A95X Z2 description for a board with rk3318 processor. > > Signed-off-by: Johan Jonker <jbx6244@gmail.com> > --- > Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++ > 1 file changed, 5 insertions(+) > Acked-by: Rob Herring <robh@kernel.org> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 3/3] arm64: dts: rockchip: add rk3318 A95X Z2 board 2020-07-18 10:53 [PATCH v2 0/3] rk3318 A95X Z2 board Johan Jonker 2020-07-18 10:53 ` [PATCH v2 1/3] dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd Johan Jonker 2020-07-18 10:53 ` [PATCH v2 2/3] dt-bindings: arm: rockchip: add Zkmagic A95X Z2 description Johan Jonker @ 2020-07-18 10:53 ` Johan Jonker 2020-08-03 21:41 ` Robin Murphy 2 siblings, 1 reply; 8+ messages in thread From: Johan Jonker @ 2020-07-18 10:53 UTC (permalink / raw) To: heiko; +Cc: devicetree, robh+dt, linux-kernel, linux-arm-kernel, linux-rockchip The rk3318 A95X Z2 boards are sold as TV box. No further documentation is given, but from the dts files extracted it seems that the rk3318 processor is simulair to the rk3328. This dts file contains only the basic nodes that have support in the mainline kernel. Features: CPU: RK3318 Quad-Core Cortex-A53 GPU: Mali-450 RAM: 2/4GB DDR3 ROM: EMMC 16/32/64GB HDMI: HDMI 2.0a for 4k@60Hz Ethernet: 10/100M standard RJ-45 WiFi: 2.4G+5G WIFI, 802.11 b/g/n Bluetooth: 4.0 1 x USB 3.0 1 x USB 2.0 1 x Micro SD card slot 1 x SPDIF 1 x AV 1 x DC IN Signed-off-by: Johan Jonker <jbx6244@gmail.com> --- arch/arm64/boot/dts/rockchip/Makefile | 1 + arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts | 393 ++++++++++++++++++++++++ 2 files changed, 394 insertions(+) create mode 100644 arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile index b87b1f773..aa508bc4a 100644 --- a/arch/arm64/boot/dts/rockchip/Makefile +++ b/arch/arm64/boot/dts/rockchip/Makefile @@ -2,6 +2,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += px30-evb.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3308-evb.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3308-roc-cc.dtb +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3318-a95x-z2.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3326-odroid-go2.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3328-a1.dtb dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3328-evb.dtb diff --git a/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts b/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts new file mode 100644 index 000000000..413418ea1 --- /dev/null +++ b/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts @@ -0,0 +1,393 @@ +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) + +/dts-v1/; +#include <dt-bindings/input/input.h> +#include "rk3328.dtsi" + +/ { + model = "A95X Z2"; + compatible = "zkmagic,a95x-z2", "rockchip,rk3318"; + + chosen { + stdout-path = "serial2:1500000n8"; + }; + + adc-keys { + compatible = "adc-keys"; + io-channels = <&saradc 0>; + io-channel-names = "buttons"; + keyup-threshold-microvolt = <1800000>; + poll-interval = <100>; + + vol-up-key { + label = "volume up"; + linux,code = <KEY_VOLUMEUP>; + press-threshold-microvolt = <17000>; + }; + }; + + ir-receiver { + compatible = "gpio-ir-receiver"; + gpios = <&gpio2 RK_PA2 GPIO_ACTIVE_LOW>; + pinctrl-0 = <&ir_int>; + pinctrl-names = "default"; + }; + + leds { + compatible = "gpio-leds"; + pinctrl-0 = <&cyx_led_pin>; + pinctrl-names = "default"; + + cyx_led: led-0 { + default-state = "on"; + gpios = <&gpio2 RK_PC7 GPIO_ACTIVE_LOW>; + label = "CYX_LED"; + }; + }; + + sdio_pwrseq: sdio-pwrseq { + compatible = "mmc-pwrseq-simple"; + pinctrl-0 = <&wifi_enable_h>; + pinctrl-names = "default"; + reset-gpios = <&gpio1 RK_PC2 GPIO_ACTIVE_LOW>; + }; + + spdif-sound { + compatible = "simple-audio-card"; + simple-audio-card,name = "spdif"; + + simple-audio-card,cpu { + sound-dai = <&spdif>; + }; + + simple-audio-card,codec { + sound-dai = <&spdif_out>; + }; + }; + + spdif_out: spdif-out { + compatible = "linux,spdif-dit"; + #sound-dai-cells = <0>; + }; + + /* Power tree */ + regulators { + compatible = "simple-mfd"; + #address-cells = <1>; + #size-cells = <0>; + + vccio_1v8_reg: regulator@0 { + compatible = "regulator-fixed"; + reg = <0>; + regulator-name = "vccio_1v8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; + }; + + vccio_3v3_reg: regulator@1 { + compatible = "regulator-fixed"; + reg = <1>; + regulator-name = "vccio_3v3"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-always-on; + }; + }; + + vcc_otg_vbus: otg-vbus-regulator { + compatible = "regulator-fixed"; + gpio = <&gpio0 RK_PA2 GPIO_ACTIVE_HIGH>; + pinctrl-0 = <&otg_vbus_drv>; + pinctrl-names = "default"; + regulator-name = "vcc_otg_vbus"; + regulator-min-microvolt = <5000000>; + regulator-max-microvolt = <5000000>; + enable-active-high; + }; + + vcc_phy: vcc-phy-regulator { + compatible = "regulator-fixed"; + regulator-name = "vcc_phy"; + regulator-always-on; + regulator-boot-on; + }; + + vcc_sd: sdmmc-regulator { + compatible = "regulator-fixed"; + gpio = <&gpio0 RK_PD6 GPIO_ACTIVE_LOW>; + pinctrl-0 = <&sdmmc0m1_gpio>; + pinctrl-names = "default"; + regulator-name = "vcc_sd"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + vin-supply = <&vccio_3v3_reg>; + }; + + vdd_arm: vdd-arm { + compatible = "pwm-regulator"; + pwms = <&pwm0 0 5000 1>; + regulator-name = "vdd_arm"; + regulator-min-microvolt = <950000>; + regulator-max-microvolt = <1400000>; + regulator-settling-time-up-us = <250>; + regulator-always-on; + regulator-boot-on; + }; + + vdd_log: vdd-log { + compatible = "pwm-regulator"; + pwms = <&pwm1 0 5000 1>; + regulator-name = "vdd_log"; + regulator-min-microvolt = <900000>; + regulator-max-microvolt = <1300000>; + regulator-settling-time-up-us = <250>; + regulator-always-on; + regulator-boot-on; + }; +}; + +&analog_sound { + status = "okay"; +}; + +&codec { + status = "okay"; +}; + +&cpu0 { + cpu-supply = <&vdd_arm>; +}; + +&cpu1 { + cpu-supply = <&vdd_arm>; +}; + +&cpu2 { + cpu-supply = <&vdd_arm>; +}; + +&cpu3 { + cpu-supply = <&vdd_arm>; +}; + +&cpu0_opp_table { + opp-1200000000 { + status = "disabled"; + }; + + opp-1296000000 { + status = "disabled"; + }; +}; + +&emmc { + bus-width = <8>; + cap-mmc-highspeed; + non-removable; + pinctrl-0 = <&emmc_clk &emmc_cmd &emmc_bus8>; + pinctrl-names = "default"; + status = "okay"; +}; + +&gmac2phy { + assigned-clock-parents = <&cru SCLK_MAC2PHY_SRC>; + assigned-clock-rate = <50000000>; + assigned-clocks = <&cru SCLK_MAC2PHY>; + clock_in_out = "output"; + phy-supply = <&vcc_phy>; + status = "okay"; +}; + +&gpu { + mali-supply = <&vdd_log>; +}; + +&hdmi { + ddc-i2c-scl-high-time-ns = <9625>; + ddc-i2c-scl-low-time-ns = <10000>; + status = "okay"; +}; + +&hdmiphy { + status = "okay"; +}; + +&hdmi_sound { + status = "okay"; +}; + +&i2s0 { + status = "okay"; +}; + +&i2s1 { + status = "okay"; +}; + +&io_domains { + pmuio-supply = <&vccio_3v3_reg>; + vccio1-supply = <&vccio_3v3_reg>; + vccio2-supply = <&vccio_1v8_reg>; + vccio3-supply = <&vccio_3v3_reg>; + vccio4-supply = <&vccio_1v8_reg>; + vccio5-supply = <&vccio_3v3_reg>; + vccio6-supply = <&vccio_3v3_reg>; + status = "okay"; +}; + +&pinctrl { + ir { + ir_int: ir-int { + rockchip,pins = <2 RK_PA2 RK_FUNC_GPIO &pcfg_pull_none>; + }; + }; + + leds { + cyx_led_pin: cyx-led-pin { + rockchip,pins = <2 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>; + }; + }; + + pwm0 { + pwm0_pin_pull_up: pwm0-pin-pull-up { + rockchip,pins = <2 RK_PA4 1 &pcfg_pull_up>; + }; + }; + + pwm1 { + pwm1_pin_pull_up: pwm1-pin-pull-up { + rockchip,pins = <2 RK_PA5 1 &pcfg_pull_up>; + }; + }; + + sdio-pwrseq { + wifi_enable_h: wifi-enable-h { + rockchip,pins = <1 RK_PC2 RK_FUNC_GPIO &pcfg_pull_none>; + }; + }; + + sdmmc1 { + clk_32k_out: clk-32k-out { + rockchip,pins = <1 RK_PD4 1 &pcfg_pull_none>; + }; + }; + + usb { + host_vbus_drv: host-vbus-drv { + rockchip,pins = <0 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>; + }; + + otg_vbus_drv: otg-vbus-drv { + rockchip,pins = <0 RK_PA2 RK_FUNC_GPIO &pcfg_pull_none>; + }; + }; +}; + +&pwm0 { + pinctrl-0 = <&pwm0_pin_pull_up>; + pinctrl-names = "active"; + status = "okay"; +}; + +&pwm1 { + pinctrl-0 = <&pwm1_pin_pull_up>; + pinctrl-names = "active"; + status = "okay"; +}; + +&saradc { + vref-supply = <&vccio_1v8_reg>; + status = "okay"; +}; + +&sdio { + bus-width = <4>; + cap-sd-highspeed; + cap-sdio-irq; + keep-power-in-suspend; + max-frequency = <125000000>; + mmc-pwrseq = <&sdio_pwrseq>; + non-removable; + pinctrl-0 = <&sdmmc1_bus4 &sdmmc1_cmd &sdmmc1_clk &clk_32k_out>; + pinctrl-names = "default"; + sd-uhs-sdr104; + status = "okay"; +}; + +&sdmmc { + bus-width = <4>; + cap-sd-highspeed; + disable-wp; + pinctrl-0 = <&sdmmc0_clk &sdmmc0_cmd &sdmmc0_dectn &sdmmc0_bus4>; + pinctrl-names = "default"; + vmmc-supply = <&vcc_sd>; + status = "okay"; +}; + +&spdif { + pinctrl-0 = <&spdifm0_tx>; + pinctrl-names = "default"; + status = "okay"; +}; + +&soc_crit { + temperature = <115000>; /* millicelsius */ +}; + +&target { + temperature = <105000>; /* millicelsius */ +}; + +&threshold { + temperature = <90000>; /* millicelsius */ +}; + +&tsadc { + rockchip,hw-tshut-temp = <120000>; + status = "okay"; +}; + +&u2phy { + status = "okay"; + + u2phy_host: host-port { + status = "okay"; + }; + + u2phy_otg: otg-port { + phy-supply = <&vcc_otg_vbus>; + status = "okay"; + }; +}; + +&uart0 { + pinctrl-0 = <&uart0_xfer &uart0_cts>; + pinctrl-names = "default"; + status = "okay"; +}; + +&uart2 { + status = "okay"; +}; + +&usb20_otg { + dr_mode = "host"; + status = "okay"; +}; + +&usb_host0_ehci { + status = "okay"; +}; + +&usb_host0_ohci { + status = "okay"; +}; + +&vop { + status = "okay"; +}; + +&vop_mmu { + status = "okay"; +}; -- 2.11.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v2 3/3] arm64: dts: rockchip: add rk3318 A95X Z2 board 2020-07-18 10:53 ` [PATCH v2 3/3] arm64: dts: rockchip: add rk3318 A95X Z2 board Johan Jonker @ 2020-08-03 21:41 ` Robin Murphy 2020-08-04 15:54 ` Johan Jonker 0 siblings, 1 reply; 8+ messages in thread From: Robin Murphy @ 2020-08-03 21:41 UTC (permalink / raw) To: Johan Jonker, heiko Cc: devicetree, robh+dt, linux-kernel, linux-arm-kernel, linux-rockchip On 2020-07-18 11:53, Johan Jonker wrote: > The rk3318 A95X Z2 boards are sold as TV box. > No further documentation is given, but from the dts files > extracted it seems that the rk3318 processor is simulair > to the rk3328. This dts file contains only the basic nodes > that have support in the mainline kernel. > > Features: > > CPU: RK3318 Quad-Core Cortex-A53 > GPU: Mali-450 > RAM: 2/4GB DDR3 > ROM: EMMC 16/32/64GB > HDMI: HDMI 2.0a for 4k@60Hz > Ethernet: 10/100M standard RJ-45 > WiFi: 2.4G+5G WIFI, 802.11 b/g/n > Bluetooth: 4.0 > 1 x USB 3.0 > 1 x USB 2.0 > 1 x Micro SD card slot > 1 x SPDIF > 1 x AV > 1 x DC IN > > Signed-off-by: Johan Jonker <jbx6244@gmail.com> > --- > arch/arm64/boot/dts/rockchip/Makefile | 1 + > arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts | 393 ++++++++++++++++++++++++ > 2 files changed, 394 insertions(+) > create mode 100644 arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts > > diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile > index b87b1f773..aa508bc4a 100644 > --- a/arch/arm64/boot/dts/rockchip/Makefile > +++ b/arch/arm64/boot/dts/rockchip/Makefile > @@ -2,6 +2,7 @@ > dtb-$(CONFIG_ARCH_ROCKCHIP) += px30-evb.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3308-evb.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3308-roc-cc.dtb > +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3318-a95x-z2.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3326-odroid-go2.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3328-a1.dtb > dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3328-evb.dtb > diff --git a/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts b/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts > new file mode 100644 > index 000000000..413418ea1 > --- /dev/null > +++ b/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts > @@ -0,0 +1,393 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > + > +/dts-v1/; > +#include <dt-bindings/input/input.h> > +#include "rk3328.dtsi" > + > +/ { > + model = "A95X Z2"; > + compatible = "zkmagic,a95x-z2", "rockchip,rk3318"; > + > + chosen { > + stdout-path = "serial2:1500000n8"; > + }; > + > + adc-keys { > + compatible = "adc-keys"; > + io-channels = <&saradc 0>; > + io-channel-names = "buttons"; > + keyup-threshold-microvolt = <1800000>; > + poll-interval = <100>; > + > + vol-up-key { > + label = "volume up"; > + linux,code = <KEY_VOLUMEUP>; Really? If as usual this is actually the recovery button hidden inside the box, it's somewhat questionable whether it's worth describing in DT at all. At the very least it should probably be presented as something more realistic than half a volume control. > + press-threshold-microvolt = <17000>; > + }; > + }; > + > + ir-receiver { > + compatible = "gpio-ir-receiver"; > + gpios = <&gpio2 RK_PA2 GPIO_ACTIVE_LOW>; > + pinctrl-0 = <&ir_int>; > + pinctrl-names = "default"; > + }; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-0 = <&cyx_led_pin>; > + pinctrl-names = "default"; > + > + cyx_led: led-0 { What needs a phandle to this node to warrant a label? > + default-state = "on"; > + gpios = <&gpio2 RK_PC7 GPIO_ACTIVE_LOW>; > + label = "CYX_LED"; > + }; > + }; > + > + sdio_pwrseq: sdio-pwrseq { > + compatible = "mmc-pwrseq-simple"; > + pinctrl-0 = <&wifi_enable_h>; > + pinctrl-names = "default"; > + reset-gpios = <&gpio1 RK_PC2 GPIO_ACTIVE_LOW>; > + }; > + > + spdif-sound { > + compatible = "simple-audio-card"; > + simple-audio-card,name = "spdif"; > + > + simple-audio-card,cpu { > + sound-dai = <&spdif>; > + }; > + > + simple-audio-card,codec { > + sound-dai = <&spdif_out>; > + }; > + }; > + > + spdif_out: spdif-out { > + compatible = "linux,spdif-dit"; > + #sound-dai-cells = <0>; > + }; > + > + /* Power tree */ > + regulators { Don't create fake "buses" for regulators, especially if they don't even contain most of the regulators anyway. > + compatible = "simple-mfd"; > + #address-cells = <1>; > + #size-cells = <0>; > + > + vccio_1v8_reg: regulator@0 { > + compatible = "regulator-fixed"; > + reg = <0>; > + regulator-name = "vccio_1v8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > + }; > + > + vccio_3v3_reg: regulator@1 { > + compatible = "regulator-fixed"; > + reg = <1>; > + regulator-name = "vccio_3v3"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + }; > + }; > + > + vcc_otg_vbus: otg-vbus-regulator { > + compatible = "regulator-fixed"; > + gpio = <&gpio0 RK_PA2 GPIO_ACTIVE_HIGH>; > + pinctrl-0 = <&otg_vbus_drv>; > + pinctrl-names = "default"; > + regulator-name = "vcc_otg_vbus"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + enable-active-high; > + }; > + > + vcc_phy: vcc-phy-regulator { > + compatible = "regulator-fixed"; > + regulator-name = "vcc_phy"; > + regulator-always-on; > + regulator-boot-on; > + }; Is this actually a distinct regulator? The reference design has the internal FEPHY block powered pretty much directly from the general 1V8 and 1V0 supplies that also power USB/HDMI/etc., so I'd rather expect low-end boxes to follow suit. Either way there doesn't seem to be a great deal of value in describing something with no controls, no voltage information, and no connection to the power tree. > + > + vcc_sd: sdmmc-regulator { > + compatible = "regulator-fixed"; > + gpio = <&gpio0 RK_PD6 GPIO_ACTIVE_LOW>; > + pinctrl-0 = <&sdmmc0m1_gpio>; > + pinctrl-names = "default"; > + regulator-name = "vcc_sd"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + vin-supply = <&vccio_3v3_reg>; > + }; > + > + vdd_arm: vdd-arm { > + compatible = "pwm-regulator"; > + pwms = <&pwm0 0 5000 1>; > + regulator-name = "vdd_arm"; > + regulator-min-microvolt = <950000>; > + regulator-max-microvolt = <1400000>; > + regulator-settling-time-up-us = <250>; > + regulator-always-on; > + regulator-boot-on; > + }; > + > + vdd_log: vdd-log { > + compatible = "pwm-regulator"; > + pwms = <&pwm1 0 5000 1>; > + regulator-name = "vdd_log"; > + regulator-min-microvolt = <900000>; > + regulator-max-microvolt = <1300000>; > + regulator-settling-time-up-us = <250>; > + regulator-always-on; > + regulator-boot-on; > + }; > +}; > + > +&analog_sound { > + status = "okay"; > +}; > + > +&codec { > + status = "okay"; > +}; > + > +&cpu0 { > + cpu-supply = <&vdd_arm>; > +}; > + > +&cpu1 { > + cpu-supply = <&vdd_arm>; > +}; > + > +&cpu2 { > + cpu-supply = <&vdd_arm>; > +}; > + > +&cpu3 { > + cpu-supply = <&vdd_arm>; > +}; > + > +&cpu0_opp_table { > + opp-1200000000 { > + status = "disabled"; > + }; > + > + opp-1296000000 { > + status = "disabled"; > + }; > +}; > + > +&emmc { > + bus-width = <8>; > + cap-mmc-highspeed; > + non-removable; > + pinctrl-0 = <&emmc_clk &emmc_cmd &emmc_bus8>; > + pinctrl-names = "default"; > + status = "okay"; > +}; > + > +&gmac2phy { > + assigned-clock-parents = <&cru SCLK_MAC2PHY_SRC>; > + assigned-clock-rate = <50000000>; > + assigned-clocks = <&cru SCLK_MAC2PHY>; > + clock_in_out = "output"; > + phy-supply = <&vcc_phy>; > + status = "okay"; > +}; > + > +&gpu { > + mali-supply = <&vdd_log>; > +}; > + > +&hdmi { > + ddc-i2c-scl-high-time-ns = <9625>; > + ddc-i2c-scl-low-time-ns = <10000>; > + status = "okay"; > +}; > + > +&hdmiphy { > + status = "okay"; > +}; > + > +&hdmi_sound { > + status = "okay"; > +}; > + > +&i2s0 { > + status = "okay"; > +}; > + > +&i2s1 { > + status = "okay"; > +}; > + > +&io_domains { > + pmuio-supply = <&vccio_3v3_reg>; > + vccio1-supply = <&vccio_3v3_reg>; > + vccio2-supply = <&vccio_1v8_reg>; > + vccio3-supply = <&vccio_3v3_reg>; > + vccio4-supply = <&vccio_1v8_reg>; > + vccio5-supply = <&vccio_3v3_reg>; > + vccio6-supply = <&vccio_3v3_reg>; > + status = "okay"; > +}; > + > +&pinctrl { > + ir { > + ir_int: ir-int { > + rockchip,pins = <2 RK_PA2 RK_FUNC_GPIO &pcfg_pull_none>; > + }; > + }; > + > + leds { > + cyx_led_pin: cyx-led-pin { > + rockchip,pins = <2 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>; > + }; > + }; > + > + pwm0 { > + pwm0_pin_pull_up: pwm0-pin-pull-up { > + rockchip,pins = <2 RK_PA4 1 &pcfg_pull_up>; > + }; > + }; > + > + pwm1 { > + pwm1_pin_pull_up: pwm1-pin-pull-up { > + rockchip,pins = <2 RK_PA5 1 &pcfg_pull_up>; > + }; > + }; > + > + sdio-pwrseq { > + wifi_enable_h: wifi-enable-h { > + rockchip,pins = <1 RK_PC2 RK_FUNC_GPIO &pcfg_pull_none>; > + }; > + }; > + > + sdmmc1 { > + clk_32k_out: clk-32k-out { > + rockchip,pins = <1 RK_PD4 1 &pcfg_pull_none>; > + }; > + }; > + > + usb { > + host_vbus_drv: host-vbus-drv { > + rockchip,pins = <0 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>; > + }; > + > + otg_vbus_drv: otg-vbus-drv { > + rockchip,pins = <0 RK_PA2 RK_FUNC_GPIO &pcfg_pull_none>; > + }; > + }; > +}; > + > +&pwm0 { > + pinctrl-0 = <&pwm0_pin_pull_up>; > + pinctrl-names = "active"; > + status = "okay"; > +}; > + > +&pwm1 { > + pinctrl-0 = <&pwm1_pin_pull_up>; > + pinctrl-names = "active"; > + status = "okay"; > +}; > + > +&saradc { > + vref-supply = <&vccio_1v8_reg>; > + status = "okay"; > +}; > + > +&sdio { > + bus-width = <4>; > + cap-sd-highspeed; > + cap-sdio-irq; > + keep-power-in-suspend; > + max-frequency = <125000000>; > + mmc-pwrseq = <&sdio_pwrseq>; > + non-removable; > + pinctrl-0 = <&sdmmc1_bus4 &sdmmc1_cmd &sdmmc1_clk &clk_32k_out>; > + pinctrl-names = "default"; > + sd-uhs-sdr104; > + status = "okay"; > +}; > + > +&sdmmc { > + bus-width = <4>; > + cap-sd-highspeed; > + disable-wp; > + pinctrl-0 = <&sdmmc0_clk &sdmmc0_cmd &sdmmc0_dectn &sdmmc0_bus4>; > + pinctrl-names = "default"; > + vmmc-supply = <&vcc_sd>; > + status = "okay"; > +}; > + > +&spdif { > + pinctrl-0 = <&spdifm0_tx>; > + pinctrl-names = "default"; > + status = "okay"; > +}; > + > +&soc_crit { > + temperature = <115000>; /* millicelsius */ > +}; > + > +&target { > + temperature = <105000>; /* millicelsius */ > +}; > + > +&threshold { > + temperature = <90000>; /* millicelsius */ > +}; > + > +&tsadc { > + rockchip,hw-tshut-temp = <120000>; > + status = "okay"; > +}; > + > +&u2phy { > + status = "okay"; > + > + u2phy_host: host-port { Just refer to the ports normally by label rather than duplicate a random snippet of DTSI hierarchy - I'm not sure why this particular pattern seems to get copied around so much. Robin. > + status = "okay"; > + }; > + > + u2phy_otg: otg-port { > + phy-supply = <&vcc_otg_vbus>; > + status = "okay"; > + }; > +}; > + > +&uart0 { > + pinctrl-0 = <&uart0_xfer &uart0_cts>; > + pinctrl-names = "default"; > + status = "okay"; > +}; > + > +&uart2 { > + status = "okay"; > +}; > + > +&usb20_otg { > + dr_mode = "host"; > + status = "okay"; > +}; > + > +&usb_host0_ehci { > + status = "okay"; > +}; > + > +&usb_host0_ohci { > + status = "okay"; > +}; > + > +&vop { > + status = "okay"; > +}; > + > +&vop_mmu { > + status = "okay"; > +}; > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v2 3/3] arm64: dts: rockchip: add rk3318 A95X Z2 board 2020-08-03 21:41 ` Robin Murphy @ 2020-08-04 15:54 ` Johan Jonker 0 siblings, 0 replies; 8+ messages in thread From: Johan Jonker @ 2020-08-04 15:54 UTC (permalink / raw) To: Robin Murphy, heiko Cc: devicetree, robh+dt, linux-kernel, linux-arm-kernel, linux-rockchip Hi Robin, Thank you for your review. Added some questions in return. This dts file seems to be some sort of variation of rk3328-box-liantong.dtsi in the manufacturer tree. Depending on the design the vendor then changes the wifi chip or removes the spdif connector to save some cost. On 8/3/20 11:41 PM, Robin Murphy wrote: > On 2020-07-18 11:53, Johan Jonker wrote: >> The rk3318 A95X Z2 boards are sold as TV box. >> No further documentation is given, but from the dts files >> extracted it seems that the rk3318 processor is simulair >> to the rk3328. This dts file contains only the basic nodes >> that have support in the mainline kernel. >> >> Features: >> >> CPU: RK3318 Quad-Core Cortex-A53 >> GPU: Mali-450 >> RAM: 2/4GB DDR3 >> ROM: EMMC 16/32/64GB >> HDMI: HDMI 2.0a for 4k@60Hz >> Ethernet: 10/100M standard RJ-45 >> WiFi: 2.4G+5G WIFI, 802.11 b/g/n >> Bluetooth: 4.0 >> 1 x USB 3.0 >> 1 x USB 2.0 >> 1 x Micro SD card slot >> 1 x SPDIF >> 1 x AV >> 1 x DC IN >> >> Signed-off-by: Johan Jonker <jbx6244@gmail.com> >> --- >> arch/arm64/boot/dts/rockchip/Makefile | 1 + >> arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts | 393 >> ++++++++++++++++++++++++ >> 2 files changed, 394 insertions(+) >> create mode 100644 arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts >> >> diff --git a/arch/arm64/boot/dts/rockchip/Makefile >> b/arch/arm64/boot/dts/rockchip/Makefile >> index b87b1f773..aa508bc4a 100644 >> --- a/arch/arm64/boot/dts/rockchip/Makefile >> +++ b/arch/arm64/boot/dts/rockchip/Makefile >> @@ -2,6 +2,7 @@ >> dtb-$(CONFIG_ARCH_ROCKCHIP) += px30-evb.dtb >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3308-evb.dtb >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3308-roc-cc.dtb >> +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3318-a95x-z2.dtb >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3326-odroid-go2.dtb >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3328-a1.dtb >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3328-evb.dtb >> diff --git a/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts >> b/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts >> new file mode 100644 >> index 000000000..413418ea1 >> --- /dev/null >> +++ b/arch/arm64/boot/dts/rockchip/rk3318-a95x-z2.dts >> @@ -0,0 +1,393 @@ >> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) >> + >> +/dts-v1/; >> +#include <dt-bindings/input/input.h> >> +#include "rk3328.dtsi" >> + >> +/ { >> + model = "A95X Z2"; >> + compatible = "zkmagic,a95x-z2", "rockchip,rk3318"; >> + >> + chosen { >> + stdout-path = "serial2:1500000n8"; >> + }; >> + >> + adc-keys { >> + compatible = "adc-keys"; >> + io-channels = <&saradc 0>; >> + io-channel-names = "buttons"; >> + keyup-threshold-microvolt = <1800000>; >> + poll-interval = <100>; >> + >> + vol-up-key { >> + label = "volume up"; >> + linux,code = <KEY_VOLUMEUP>; > > Really? If as usual this is actually the recovery button hidden inside > the box, it's somewhat questionable whether it's worth describing in DT > at all. At the very least it should probably be presented as something > more realistic than half a volume control. Dts use is not limited to a mainline kernel setting. In Android that <KEY_VOLUMEUP> seem to be a standard way needed to enter a menu. Then make it as complete as possible to make it work. What would be your realistic key proposal? > >> + press-threshold-microvolt = <17000>; >> + }; >> + }; >> + >> + ir-receiver { >> + compatible = "gpio-ir-receiver"; >> + gpios = <&gpio2 RK_PA2 GPIO_ACTIVE_LOW>; >> + pinctrl-0 = <&ir_int>; >> + pinctrl-names = "default"; >> + }; >> + >> + leds { >> + compatible = "gpio-leds"; >> + pinctrl-0 = <&cyx_led_pin>; >> + pinctrl-names = "default"; >> + >> + cyx_led: led-0 { > > What needs a phandle to this node to warrant a label? Without repeating previous discussions, I made a proposal that was approved. So we just stick to that set of rules consequently. A little explanation: With the conversion to yaml a preferred nodename was introduced. Legacy Rockchip node names were sort of free style and causing the dts-checker to produce a flood of warnings. The led properties are described in 2 places, under leds and under pinctrl. To improve the readability given the legacy and to prevent the disconnect between schematics, leds and pinctrl we add a label with a consistent name. Fix with help of the following rules: 1: Add nodename in the preferred form. 2: Always add a label that ends with '_led' to prevent conflicts with other labels such as 'power' and 'mmc' 3: If leds need pinctrl add a label that ends with '_led_pin' also to prevent conflicts with other labels. > >> + default-state = "on"; >> + gpios = <&gpio2 RK_PC7 GPIO_ACTIVE_LOW>; >> + label = "CYX_LED"; This label with capital letters is used by an Android app, so keep it. >> + }; >> + }; >> + >> + sdio_pwrseq: sdio-pwrseq { >> + compatible = "mmc-pwrseq-simple"; >> + pinctrl-0 = <&wifi_enable_h>; >> + pinctrl-names = "default"; >> + reset-gpios = <&gpio1 RK_PC2 GPIO_ACTIVE_LOW>; >> + }; >> + >> + spdif-sound { >> + compatible = "simple-audio-card"; >> + simple-audio-card,name = "spdif"; Some programs don't like long names here. Recently I see others also use "SPDIF". Should I change? Currently for rk3328 we have: "Analog" and "HDMI" Is there a standard? Please advise. >> + >> + simple-audio-card,cpu { >> + sound-dai = <&spdif>; >> + }; >> + >> + simple-audio-card,codec { >> + sound-dai = <&spdif_out>; >> + }; >> + }; >> + >> + spdif_out: spdif-out { >> + compatible = "linux,spdif-dit"; >> + #sound-dai-cells = <0>; >> + }; >> + >> + /* Power tree */ >> + regulators { > > Don't create fake "buses" for regulators, especially if they don't even > contain most of the regulators anyway. Agree. The original reason was not given. vccio_1v8_reg and vccio_3v3_reg are used mostly in &io_domains. Is there a dependency requirement that both must be probed/enabled before being used? Without these regulators in a bus should we then add a vcc-sys-regulator for a complete supply chain? vcc_sys: vcc-sys-regulator { compatible = "regulator-fixed"; regulator-name = "vcc_sys"; regulator-min-microvolt = <5000000>; regulator-max-microvolt = <5000000>; regulator-always-on; regulator-boot-on; }; > >> + compatible = "simple-mfd"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + vccio_1v8_reg: regulator@0 { >> + compatible = "regulator-fixed"; >> + reg = <0>; >> + regulator-name = "vccio_1v8"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + regulator-always-on; Just an example: vin-supply = <&vcc_sys>; >> + }; >> + >> + vccio_3v3_reg: regulator@1 { >> + compatible = "regulator-fixed"; >> + reg = <1>; >> + regulator-name = "vccio_3v3"; >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-always-on; Just an example: vin-supply = <&vcc_sys>; >> + }; >> + }; >> + >> + vcc_otg_vbus: otg-vbus-regulator { >> + compatible = "regulator-fixed"; >> + gpio = <&gpio0 RK_PA2 GPIO_ACTIVE_HIGH>; >> + pinctrl-0 = <&otg_vbus_drv>; >> + pinctrl-names = "default"; >> + regulator-name = "vcc_otg_vbus"; >> + regulator-min-microvolt = <5000000>; >> + regulator-max-microvolt = <5000000>; >> + enable-active-high; >> + }; >> + >> + vcc_phy: vcc-phy-regulator { >> + compatible = "regulator-fixed"; >> + regulator-name = "vcc_phy"; >> + regulator-always-on; >> + regulator-boot-on; >> + }; > > Is this actually a distinct regulator? The reference design has the > internal FEPHY block powered pretty much directly from the general 1V8 > and 1V0 supplies that also power USB/HDMI/etc., so I'd rather expect > low-end boxes to follow suit. Either way there doesn't seem to be a > great deal of value in describing something with no controls, no voltage > information, and no connection to the power tree. Agree. phy-supply appears to be an optional property, so removing it is possible. It shows up in rk3328-roc-cc.dts and rk3328-evb.dts. What does the maintainer prefer? Remove or not? > >> + >> + vcc_sd: sdmmc-regulator { >> + compatible = "regulator-fixed"; >> + gpio = <&gpio0 RK_PD6 GPIO_ACTIVE_LOW>; >> + pinctrl-0 = <&sdmmc0m1_gpio>; >> + pinctrl-names = "default"; >> + regulator-name = "vcc_sd"; >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + vin-supply = <&vccio_3v3_reg>; >> + }; >> + >> + vdd_arm: vdd-arm { >> + compatible = "pwm-regulator"; >> + pwms = <&pwm0 0 5000 1>; >> + regulator-name = "vdd_arm"; >> + regulator-min-microvolt = <950000>; >> + regulator-max-microvolt = <1400000>; >> + regulator-settling-time-up-us = <250>; >> + regulator-always-on; >> + regulator-boot-on; >> + }; >> + >> + vdd_log: vdd-log { >> + compatible = "pwm-regulator"; >> + pwms = <&pwm1 0 5000 1>; >> + regulator-name = "vdd_log"; >> + regulator-min-microvolt = <900000>; >> + regulator-max-microvolt = <1300000>; >> + regulator-settling-time-up-us = <250>; >> + regulator-always-on; >> + regulator-boot-on; >> + }; >> +}; >> + >> +&analog_sound { >> + status = "okay"; >> +}; >> + >> +&codec { >> + status = "okay"; >> +}; >> + >> +&cpu0 { >> + cpu-supply = <&vdd_arm>; >> +}; >> + >> +&cpu1 { >> + cpu-supply = <&vdd_arm>; >> +}; >> + >> +&cpu2 { >> + cpu-supply = <&vdd_arm>; >> +}; >> + >> +&cpu3 { >> + cpu-supply = <&vdd_arm>; >> +}; >> + >> +&cpu0_opp_table { >> + opp-1200000000 { >> + status = "disabled"; >> + }; >> + >> + opp-1296000000 { >> + status = "disabled"; >> + }; >> +}; >> + >> +&emmc { >> + bus-width = <8>; >> + cap-mmc-highspeed; >> + non-removable; >> + pinctrl-0 = <&emmc_clk &emmc_cmd &emmc_bus8>; >> + pinctrl-names = "default"; >> + status = "okay"; >> +}; >> + >> +&gmac2phy { >> + assigned-clock-parents = <&cru SCLK_MAC2PHY_SRC>; >> + assigned-clock-rate = <50000000>; >> + assigned-clocks = <&cru SCLK_MAC2PHY>; >> + clock_in_out = "output"; >> + phy-supply = <&vcc_phy>; Remove or not? >> + status = "okay"; >> +}; >> + >> +&gpu { >> + mali-supply = <&vdd_log>; >> +}; >> + >> +&hdmi { >> + ddc-i2c-scl-high-time-ns = <9625>; >> + ddc-i2c-scl-low-time-ns = <10000>; >> + status = "okay"; >> +}; >> + >> +&hdmiphy { >> + status = "okay"; >> +}; >> + >> +&hdmi_sound { >> + status = "okay"; >> +}; >> + >> +&i2s0 { >> + status = "okay"; >> +}; >> + >> +&i2s1 { >> + status = "okay"; >> +}; >> + >> +&io_domains { >> + pmuio-supply = <&vccio_3v3_reg>; >> + vccio1-supply = <&vccio_3v3_reg>; >> + vccio2-supply = <&vccio_1v8_reg>; >> + vccio3-supply = <&vccio_3v3_reg>; >> + vccio4-supply = <&vccio_1v8_reg>; >> + vccio5-supply = <&vccio_3v3_reg>; >> + vccio6-supply = <&vccio_3v3_reg>; >> + status = "okay"; >> +}; >> + >> +&pinctrl { >> + ir { >> + ir_int: ir-int { >> + rockchip,pins = <2 RK_PA2 RK_FUNC_GPIO &pcfg_pull_none>; >> + }; >> + }; >> + >> + leds { >> + cyx_led_pin: cyx-led-pin { >> + rockchip,pins = <2 RK_PC7 RK_FUNC_GPIO &pcfg_pull_none>; >> + }; >> + }; >> + >> + pwm0 { >> + pwm0_pin_pull_up: pwm0-pin-pull-up { >> + rockchip,pins = <2 RK_PA4 1 &pcfg_pull_up>; >> + }; >> + }; >> + >> + pwm1 { >> + pwm1_pin_pull_up: pwm1-pin-pull-up { >> + rockchip,pins = <2 RK_PA5 1 &pcfg_pull_up>; >> + }; >> + }; >> + >> + sdio-pwrseq { >> + wifi_enable_h: wifi-enable-h { >> + rockchip,pins = <1 RK_PC2 RK_FUNC_GPIO &pcfg_pull_none>; >> + }; >> + }; >> + >> + sdmmc1 { >> + clk_32k_out: clk-32k-out { >> + rockchip,pins = <1 RK_PD4 1 &pcfg_pull_none>; >> + }; >> + }; >> + >> + usb { >> + host_vbus_drv: host-vbus-drv { >> + rockchip,pins = <0 RK_PA0 RK_FUNC_GPIO &pcfg_pull_none>; >> + }; >> + >> + otg_vbus_drv: otg-vbus-drv { >> + rockchip,pins = <0 RK_PA2 RK_FUNC_GPIO &pcfg_pull_none>; >> + }; >> + }; >> +}; >> + >> +&pwm0 { >> + pinctrl-0 = <&pwm0_pin_pull_up>; >> + pinctrl-names = "active"; >> + status = "okay"; >> +}; >> + >> +&pwm1 { >> + pinctrl-0 = <&pwm1_pin_pull_up>; >> + pinctrl-names = "active"; >> + status = "okay"; >> +}; >> + >> +&saradc { >> + vref-supply = <&vccio_1v8_reg>; >> + status = "okay"; >> +}; >> + >> +&sdio { >> + bus-width = <4>; >> + cap-sd-highspeed; >> + cap-sdio-irq; >> + keep-power-in-suspend; >> + max-frequency = <125000000>; >> + mmc-pwrseq = <&sdio_pwrseq>; >> + non-removable; >> + pinctrl-0 = <&sdmmc1_bus4 &sdmmc1_cmd &sdmmc1_clk &clk_32k_out>; >> + pinctrl-names = "default"; >> + sd-uhs-sdr104; >> + status = "okay"; >> +}; >> + >> +&sdmmc { >> + bus-width = <4>; >> + cap-sd-highspeed; >> + disable-wp; This is from the manufacturer tree for a micro SD card. Remove or not? Please advise. >> + pinctrl-0 = <&sdmmc0_clk &sdmmc0_cmd &sdmmc0_dectn &sdmmc0_bus4>; >> + pinctrl-names = "default"; >> + vmmc-supply = <&vcc_sd>; >> + status = "okay"; >> +}; >> + >> +&spdif { >> + pinctrl-0 = <&spdifm0_tx>; >> + pinctrl-names = "default"; Remove. Already in dtsi. >> + status = "okay"; >> +}; >> + >> +&soc_crit { >> + temperature = <115000>; /* millicelsius */ >> +}; >> + >> +&target { >> + temperature = <105000>; /* millicelsius */ >> +}; >> + >> +&threshold { >> + temperature = <90000>; /* millicelsius */ >> +}; >> + >> +&tsadc { >> + rockchip,hw-tshut-temp = <120000>; >> + status = "okay"; >> +}; >> + >> +&u2phy { >> + status = "okay"; >> + >> + u2phy_host: host-port { > > Just refer to the ports normally by label rather than duplicate a random > snippet of DTSI hierarchy - I'm not sure why this particular pattern > seems to get copied around so much. Here the node labels start with the same letters and the result will stay together. In other cases it will scatter all over the dts file and affects the readability. I just followed the manufacturer's example. If the maintainer agrees then I'll change. > > Robin. > >> + status = "okay"; >> + }; >> + >> + u2phy_otg: otg-port { >> + phy-supply = <&vcc_otg_vbus>; >> + status = "okay"; >> + }; >> +}; >> + >> +&uart0 { >> + pinctrl-0 = <&uart0_xfer &uart0_cts>; >> + pinctrl-names = "default"; Remove. Already in dtsi. >> + status = "okay"; >> +}; >> + >> +&uart2 { >> + status = "okay"; >> +}; >> + >> +&usb20_otg { >> + dr_mode = "host"; >> + status = "okay"; >> +}; >> + >> +&usb_host0_ehci { >> + status = "okay"; >> +}; >> + >> +&usb_host0_ohci { >> + status = "okay"; >> +}; >> + >> +&vop { >> + status = "okay"; >> +}; >> + >> +&vop_mmu { >> + status = "okay"; >> +}; >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2020-08-04 15:56 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-07-18 10:53 [PATCH v2 0/3] rk3318 A95X Z2 board Johan Jonker 2020-07-18 10:53 ` [PATCH v2 1/3] dt-bindings: Add vendor prefix for Shenzhen Zkmagic Technology Co., Ltd Johan Jonker 2020-07-23 17:36 ` Rob Herring 2020-07-18 10:53 ` [PATCH v2 2/3] dt-bindings: arm: rockchip: add Zkmagic A95X Z2 description Johan Jonker 2020-07-23 17:36 ` Rob Herring 2020-07-18 10:53 ` [PATCH v2 3/3] arm64: dts: rockchip: add rk3318 A95X Z2 board Johan Jonker 2020-08-03 21:41 ` Robin Murphy 2020-08-04 15:54 ` Johan Jonker
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).