| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <32f2518d.a66c.190b5fe86a6.Coremail.andyshrk@163.com>
Date: Mon, 15 Jul 2024 18:44:35 +0800 (CST)
From: "Andy Yan" <andyshrk@....com>
To: "Dragan Simic" <dsimic@...jaro.org>
Cc: heiko@...ech.de, krzk+dt@...nel.org, robh@...nel.org,
devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org, linux-rockchip@...ts.infradead.org
Subject: Re:Re: [PATCH 2/2] arm64: dts: rockchip: Add support for rk3588
GenBook
Hello Dragan，
At 2024-07-14 23:27:54, "Dragan Simic" <dsimic@...jaro.org> wrote:
>Hello Andy,
>
>Please see a few comments below.
>
>On 2024-07-14 13:28, Andy Yan wrote:
>> Cool Pi GenBook works as a mother board connect with CM5 [0].
>
>s/mother board/carrier board/
This can be changed to carrier board in next version。
>s/CM5/GenM5/
In fact, the SOM is called CM5 always[0]， even this laptop was called cm5 laptop from the
very beginning of the design。
They just gave it a new name GenBook when it comes to crowdsupply.
The dtsi for CM5 is aready landing upstream，I inclulde it in the current genbook dts。
[0]https://forum.cool-pi.com/topic/98/cool-pi-5-compute-module-minipc-carrier-board-spec
>
>> Specification:
>> - Rockchip RK3588
>> - LPDDR5X 8/32 GB
>> - eMMC 64 GB
>> - HDMI Type A out x 1
>> - USB 3.0 Host x 1
>> - USB-C 3.0 with DisplayPort AltMode
>> - PCIE M.2 E Key for RTL8852BE Wireless connection
>> - PCIE M.2 M Key for NVME connection
>> - eDP panel with 1920x1080
>>
>> This patch add basic support to bringup eMMC/USB HOST/WiFi
>> and can also drive a HDMI output with out of tree hdmi patches.
>>
>> [0]
>> https://www.crowdsupply.com/shenzhen-tianmao-technology-co-ltd/genbook-rk3588
>>
>> Signed-off-by: Andy Yan <andyshrk@....com>
>> ---
>>
>> arch/arm64/boot/dts/rockchip/Makefile | 1 +
>> .../dts/rockchip/rk3588-coolpi-genbook.dts | 296 ++++++++++++++++++
>> 2 files changed, 297 insertions(+)
>> create mode 100644
>> arch/arm64/boot/dts/rockchip/rk3588-coolpi-genbook.dts
>
>It would be better to split this into two files: a dtsi for the GenM5
>system
>on module, and a dts for the GenBook carrier board. That way, the
>dts(i) files
>would describe the actual hardware better.
>
>> diff --git a/arch/arm64/boot/dts/rockchip/Makefile
>> b/arch/arm64/boot/dts/rockchip/Makefile
>> index 8fa469db400c..0fad54939bbd 100644
>> --- a/arch/arm64/boot/dts/rockchip/Makefile
>> +++ b/arch/arm64/boot/dts/rockchip/Makefile
>> @@ -122,6 +122,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) +=
>> rk3568-wolfvision-pf5-display-vz.dtbo
>> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3568-wolfvision-pf5-io-expander.dtbo
>> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-armsom-sige7.dtb
>> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-coolpi-cm5-evb.dtb
>> +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-coolpi-genbook.dtb
>> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-edgeble-neu6a-io.dtb
>> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-edgeble-neu6a-wifi.dtbo
>> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3588-edgeble-neu6b-io.dtb
>> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-coolpi-genbook.dts
>> b/arch/arm64/boot/dts/rockchip/rk3588-coolpi-genbook.dts
>> new file mode 100644
>> index 000000000000..28357c6f05fb
>> --- /dev/null
>> +++ b/arch/arm64/boot/dts/rockchip/rk3588-coolpi-genbook.dts
>> @@ -0,0 +1,296 @@
>> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
>> +/*
>> + * Copyright (c) 2024 Rockchip Electronics Co., Ltd.
>> + *
>> + */
>> +
>> +/dts-v1/;
>> +
>> +#include <dt-bindings/leds/common.h>
>> +#include "rk3588-coolpi-cm5.dtsi"
>> +
>> +/ {
>> + model = "RK3588 CoolPi GenBook";
>> + compatible = "coolpi,genbook", "coolpi,pi-cm5", "rockchip,rk3588";
>> +
>> + backlight: backlight {
>> + compatible = "pwm-backlight";
>> + enable-gpios = <&gpio4 RK_PA3 GPIO_ACTIVE_HIGH>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&bl_en>;
>> + power-supply = <&vcc12v_dcin>;
>> + pwms = <&pwm6 0 25000 0>;
>> + };
>> +
>> + leds: leds {
>> + compatible = "gpio-leds";
>> +
>> + heartbeat_led: led-0 {
>> + color = <LED_COLOR_ID_GREEN>;
>> + function = LED_FUNCTION_STATUS;
>> + gpios = <&gpio1 RK_PB0 GPIO_ACTIVE_HIGH>;
>> + linux,default-trigger = "heartbeat";
>> + };
>> +
>> + wlan_led: led-1 {
>> + color = <LED_COLOR_ID_GREEN>;
>> + function = LED_FUNCTION_WLAN;
>> + gpios = <&gpio0 RK_PC5 GPIO_ACTIVE_HIGH>;
>> + };
>> +
>> + charging_red: led-2 {
>> + function = LED_FUNCTION_CHARGING;
>> + color = <LED_COLOR_ID_RED>;
>> + gpios = <&gpio3 RK_PD2 GPIO_ACTIVE_HIGH>;
>> + };
>> + };
>> +
>> + vcc12v_dcin: vcc12v-dcin-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc12v_dcin";
>> + regulator-always-on;
>> + regulator-boot-on;
>> + regulator-min-microvolt = <12000000>;
>> + regulator-max-microvolt = <12000000>;
>> + };
>> +
>> + vcc_sys: vcc-sys-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc_sys";
>> + regulator-always-on;
>> + regulator-boot-on;
>> + regulator-min-microvolt = <7000000>;
>> + regulator-max-microvolt = <7000000>;
>> + vin-supply = <&vcc12v_dcin>;
>> + };
>> +
>> + vcc5v0_sys: vcc5v0-sys-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc5v0_sys";
>> + regulator-always-on;
>> + regulator-boot-on;
>> + regulator-min-microvolt = <7000000>;
>> + regulator-max-microvolt = <7000000>;
>> + vin-supply = <&vcc_sys>;
>> + };
>> +
>> + vcc3v3_sys: vcc3v3-sys-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc3v3_sys";
>> + regulator-always-on;
>> + regulator-boot-on;
>> + regulator-min-microvolt = <3300000>;
>> + regulator-max-microvolt = <3300000>;
>> + vin-supply = <&vcc5v0_sys>;
>> + };
>> +
>> + vcc3v3_lcd: vcc3v3-lcd-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc3v3_lcd";
>> + enable-active-high;
>> + gpio = <&gpio1 RK_PC4 GPIO_ACTIVE_HIGH>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&lcdpwr_en>;
>> + vin-supply = <&vcc3v3_sys>;
>> + };
>> +
>> + vcc5v0_usb: vcc5v0-usb-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc5v0_usb";
>> + regulator-boot-on;
>> + regulator-always-on;
>> + enable-active-high;
>> + regulator-min-microvolt = <5000000>;
>> + regulator-max-microvolt = <5000000>;
>> + gpio = <&gpio1 RK_PD5 GPIO_ACTIVE_HIGH>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&usb_pwren>;
>> + vin-supply = <&vcc_sys>;
>> + };
>> +
>> + vcc5v0_usb_host0: vcc5v0_usb30_host: vcc5v0-usb-host-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vcc5v0_host";
>> + regulator-boot-on;
>> + regulator-always-on;
>> + enable-active-high;
>> + regulator-min-microvolt = <5000000>;
>> + regulator-max-microvolt = <5000000>;
>> + gpio = <&gpio1 RK_PA7 GPIO_ACTIVE_HIGH>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&usb_host_pwren>;
>> + vin-supply = <&vcc5v0_usb>;
>> + };
>> +};
>> +
>> +&gmac0 {
>> + status = "disabled";
>> +};
>> +
>> +/* M.2 E-Key */
>> +&pcie2x1l0 {
>> + reset-gpios = <&gpio4 RK_PA2 GPIO_ACTIVE_HIGH>;
>> + vpcie3v3-supply = <&vcc3v3_sys>;
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pcie_clkreq &pcie_wake &pcie_rst &wifi_pwron
>> &bt_pwron>;
>> + status = "okay";
>> +};
>> +
>> +&pcie2x1l2 {
>> + status = "disabled";
>> +};
>> +
>> +&pcie30phy {
>> + status = "okay";
>> +};
>> +
>> +/* M.2 M-Key ssd */
>> +&pcie3x4 {
>> + reset-gpios = <&gpio4 RK_PB6 GPIO_ACTIVE_HIGH>;
>> + vpcie3v3-supply = <&vcc3v3_sys>;
>> + status = "okay";
>> +};
>> +
>> +&pinctrl {
>> + lcd {
>> + lcdpwr_en: lcdpwr-en {
>> + rockchip,pins = <1 RK_PC4 RK_FUNC_GPIO &pcfg_pull_down>;
>> + };
>> +
>> + bl_en: bl-en {
>> + rockchip,pins = <4 RK_PA3 RK_FUNC_GPIO &pcfg_pull_none>;
>> + };
>> + };
>> +
>> + usb {
>> + usb_pwren: usb-pwren {
>> + rockchip,pins = <1 RK_PD5 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> +
>> + usb_otg_pwren: usb-otg-pwren {
>> + rockchip,pins = <0 RK_PA0 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> +
>> + usb_host_pwren: usb-host-pwren {
>> + rockchip,pins = <1 RK_PA7 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> + };
>> +
>> + wifi {
>> + bt_pwron: bt-pwron {
>> + rockchip,pins = <3 RK_PA6 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> +
>> + pcie_clkreq: pcie-clkreq {
>> + rockchip,pins = <4 RK_PA0 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> +
>> + pcie_rst: pcie-rst {
>> + rockchip,pins = <4 RK_PA2 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> +
>> + wifi_pwron: wifi-pwron {
>> + rockchip,pins = <3 RK_PB1 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> +
>> + pcie_wake: pcie-wake {
>> + rockchip,pins = <4 RK_PA1 RK_FUNC_GPIO &pcfg_pull_up>;
>> + };
>> + };
>> +};
>> +
>> +&pwm6 {
>> + pinctrl-0 = <&pwm6m1_pins>;
>> + status = "okay";
>> +};
>> +
>> +&sdmmc {
>> + status = "disabled";
>> +};
>> +
>> +&sfc {
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&fspim2_pins>;
>> + status = "okay";
>> +
>> + flash@0 {
>> + compatible = "jedec,spi-nor";
>> + reg = <0x0>;
>> + spi-max-frequency = <100000000>;
>> + spi-rx-bus-width = <4>;
>> + spi-tx-bus-width = <1>;
>> + };
>> +};
>> +
>> +&u2phy0 {
>> + status = "okay";
>> +};
>> +
>> +&u2phy0_otg {
>> + status = "okay";
>> +};
>> +
>> +&usbdp_phy0 {
>> + status = "okay";
>> +};
>> +
>> +&u2phy1 {
>> + status = "okay";
>> +};
>> +
>> +&u2phy1_otg {
>> + status = "okay";
>> +};
>> +
>> +&u2phy2 {
>> + status = "okay";
>> +};
>> +
>> +&u2phy3 {
>> + status = "okay";
>> +};
>> +
>> +&u2phy2_host {
>> + phy-supply = <&vcc5v0_usb_host0>;
>> + status = "okay";
>> +};
>> +
>> +&u2phy3_host {
>> + phy-supply = <&vcc5v0_usb>;
>> + status = "okay";
>> +};
>> +
>> +&usbdp_phy1 {
>> + status = "okay";
>> +};
>> +
>> +/* For Keypad */
>> +&usb_host0_ehci {
>> + status = "okay";
>> +};
>> +
>> +&usb_host0_ohci {
>> + status = "okay";
>> +};
>> +
>> +/* Type C port */
>> +&usb_host0_xhci {
>> + dr_mode = "peripheral";
>> + maximum-speed = "high-speed";
>> + status = "okay";
>> +};
>> +
>> +/* connected to a HUB for camera and BT */
>> +&usb_host1_ehci {
>> + status = "okay";
>> +};
>> +
>> +&usb_host1_ohci {
>> + status = "okay";
>> +};
>> +
>> +/* USB A out */
>> +&usb_host1_xhci {
>> + dr_mode = "host";
>> + status = "okay";
>> +};
Powered by blists - more mailing lists