* [PATCH 0/4] arm64: dts: qcom: Support Retroid Pocket QCS8550 Devices
@ 2026-08-08 17:37 Aaron Kling via B4 Relay
2026-08-08 17:37 ` [PATCH 1/4] dt-bindings: vendor-prefixes: Add Retroid Pocket Aaron Kling via B4 Relay
` (3 more replies)
0 siblings, 4 replies; 14+ messages in thread
From: Aaron Kling via B4 Relay @ 2026-08-08 17:37 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio
Cc: devicetree, linux-kernel, linux-arm-msm, Aaron Kling
This specifically includes:
* Nova
* RP6
Retroid Pocket is effectively a sister company/brand to AYN. They are
both under Moorechip and all hardware designs are handled by Moorechip.
Thus the electrical designs are largely shared within a SoC generation.
Hence these include the AYN QCS8550 common dtsi. As such, this series
depends on the AYN QCS8550 series [0].
These two devices are almost electrically identical. The only notable
difference is the panels.
Definitions for parts of the hardware that are not yet supported by
mainline are not included. Such as:
* Panels
* All rgb leds
* The built-in uart gamepad
[0] https://lore.kernel.org/linux-arm-msm/20260727-ayn-qcs8550-v9-0-e3db456e10e5@gmail.com/
Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
---
Aaron Kling (4):
dt-bindings: vendor-prefixes: Add Retroid Pocket
dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices
arm64: dts: qcom: Add Retroid Pocket 6
arm64: dts: qcom: Add Retroid Pocket Nova
Documentation/devicetree/bindings/arm/qcom.yaml | 7 ++
.../devicetree/bindings/vendor-prefixes.yaml | 2 +
arch/arm64/boot/dts/qcom/Makefile | 2 +
.../boot/dts/qcom/qcs8550-retroidpocket-nova.dts | 118 ++++++++++++++++++++
.../boot/dts/qcom/qcs8550-retroidpocket-rp6.dts | 122 +++++++++++++++++++++
5 files changed, 251 insertions(+)
---
base-commit: 6b8c8af514d739d0335f5579b585e02babe8a727
change-id: 20260808-rp-qcs8550-85c84760f6f3
prerequisite-change-id: 20260217-ayn-qcs8550-16c07b63de26:v9
prerequisite-patch-id: 17ba304f7a8a08cc78f98e198db9c51f4786a22c
prerequisite-patch-id: 21873631f5dcdb9bbbdb5c20c3f5e9816cc405f9
prerequisite-patch-id: bf7b5dea0097cbbdf04929a4246d1fc65c607ea8
prerequisite-patch-id: d7f546eddc96a4f31a1e681abddbc96bd68abb32
prerequisite-patch-id: d40d784ad8efbb1204c024408be2067725a06cf2
prerequisite-patch-id: 25bbd55199a8250a912ef542b31f43dd5ccdbc90
Best regards,
--
Aaron Kling <webgeek1234@gmail.com>
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH 1/4] dt-bindings: vendor-prefixes: Add Retroid Pocket
2026-08-08 17:37 [PATCH 0/4] arm64: dts: qcom: Support Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
@ 2026-08-08 17:37 ` Aaron Kling via B4 Relay
2026-08-13 9:20 ` Krzysztof Kozlowski
2026-08-08 17:37 ` [PATCH 2/4] dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
` (2 subsequent siblings)
3 siblings, 1 reply; 14+ messages in thread
From: Aaron Kling via B4 Relay @ 2026-08-08 17:37 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio
Cc: devicetree, linux-kernel, linux-arm-msm, Aaron Kling
From: Aaron Kling <webgeek1234@gmail.com>
Add an entry for Retroid Pocket (https://www.goretroid.com/)
Signed-off-by: Aaron Kling <webgeek1234@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 420cf4c41354f..9719110762be5 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
@@ -1419,6 +1419,8 @@ patternProperties:
description: Renesas Electronics Corporation
"^rervision,.*":
description: Shenzhen Rervision Technology Co., Ltd.
+ "^retroidpocket,.*":
+ description: Retroid Pocket
"^retronix,.*":
description: Retronix Technology Inc.
"^revotics,.*":
--
2.54.0
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 2/4] dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices
2026-08-08 17:37 [PATCH 0/4] arm64: dts: qcom: Support Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
2026-08-08 17:37 ` [PATCH 1/4] dt-bindings: vendor-prefixes: Add Retroid Pocket Aaron Kling via B4 Relay
@ 2026-08-08 17:37 ` Aaron Kling via B4 Relay
2026-08-13 9:23 ` Krzysztof Kozlowski
2026-08-08 17:37 ` [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6 Aaron Kling via B4 Relay
2026-08-08 17:37 ` [PATCH 4/4] arm64: dts: qcom: Add Retroid Pocket Nova Aaron Kling via B4 Relay
3 siblings, 1 reply; 14+ messages in thread
From: Aaron Kling via B4 Relay @ 2026-08-08 17:37 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio
Cc: devicetree, linux-kernel, linux-arm-msm, Aaron Kling
From: Aaron Kling <webgeek1234@gmail.com>
Namely:
* Nova
* RP6
Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
---
Documentation/devicetree/bindings/arm/qcom.yaml | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
index 954be28203017..52ef669d6fe9b 100644
--- a/Documentation/devicetree/bindings/arm/qcom.yaml
+++ b/Documentation/devicetree/bindings/arm/qcom.yaml
@@ -1161,6 +1161,13 @@ properties:
- const: qcom,qcs8550
- const: qcom,sm8550
+ - items:
+ - enum:
+ - retroidpocket,nova
+ - retroidpocket,rp6
+ - const: qcom,qcs8550
+ - const: qcom,sm8550
+
- items:
- enum:
- qcom,qcs8550-rb5gen2
--
2.54.0
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6
2026-08-08 17:37 [PATCH 0/4] arm64: dts: qcom: Support Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
2026-08-08 17:37 ` [PATCH 1/4] dt-bindings: vendor-prefixes: Add Retroid Pocket Aaron Kling via B4 Relay
2026-08-08 17:37 ` [PATCH 2/4] dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
@ 2026-08-08 17:37 ` Aaron Kling via B4 Relay
2026-08-13 19:05 ` Dmitry Baryshkov
2026-08-17 16:11 ` Konrad Dybcio
2026-08-08 17:37 ` [PATCH 4/4] arm64: dts: qcom: Add Retroid Pocket Nova Aaron Kling via B4 Relay
3 siblings, 2 replies; 14+ messages in thread
From: Aaron Kling via B4 Relay @ 2026-08-08 17:37 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio
Cc: devicetree, linux-kernel, linux-arm-msm, Aaron Kling
From: Aaron Kling <webgeek1234@gmail.com>
The Retroid Pocket 6 is a high-performance Android-based handheld gaming
console powered by the Qualcomm Snapdragon 8 Gen 2 processor featuring a
5.5-inch AMOLED touchscreen.
This is based on the support by ROCKNIX, however their history does not
show who the original author was, so direct attributation cannot be
given.
Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
---
arch/arm64/boot/dts/qcom/Makefile | 1 +
.../boot/dts/qcom/qcs8550-retroidpocket-rp6.dts | 122 +++++++++++++++++++++
2 files changed, 123 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index a7b0fb759099e..09b3c5b2d9dab 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -196,6 +196,7 @@ dtb-$(CONFIG_ARCH_QCOM) += qcs8550-aim300-aiot.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2mini.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2portal.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-thor.dtb
+dtb-$(CONFIG_ARCH_QCOM) += qcs8550-retroidpocket-rp6.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-rb5gen2.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs9100-ride.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs9100-ride-r3.dtb
diff --git a/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-rp6.dts b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-rp6.dts
new file mode 100644
index 0000000000000..eaffa2bd12671
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-rp6.dts
@@ -0,0 +1,122 @@
+// SPDX-License-Identifier: BSD-3-Clause
+/*
+ * Copyright (C) 2026-present ROCKNIX (https://github.com/ROCKNIX).
+ */
+
+/dts-v1/;
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+#include "qcs8550-ayntec-common.dtsi"
+
+&{/} {
+ model = "Retroid Pocket 6";
+ compatible = "retroidpocket,rp6", "qcom,qcs8550", "qcom,sm8550";
+
+ backlight: backlight {
+ compatible = "pwm-backlight";
+ pwms = <&pmk8550_pwm 0 860000>;
+ brightness-levels = <1023 0>;
+ num-interpolated-steps = <1023>;
+ default-brightness-level = <600>;
+ power-supply = <&vph_pwr>;
+ enable-gpios = <&pmk8550_gpios 5 GPIO_ACTIVE_HIGH>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pwm_backlight_default>;
+
+ status = "okay";
+ };
+
+ vdd_disp_1v8: vdd-disp-1v8-regulator {
+ compatible = "regulator-fixed";
+ regulator-name = "vdd_disp_1v8";
+
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+
+ gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+
+ vdd_disp_2v8: vdd-disp-2v8-regulator {
+ compatible = "regulator-fixed";
+ regulator-name = "vdd_disp_2v8";
+
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+
+ gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+
+ ts_avdd_3v0: regulator-ts-avdd-3v0 {
+ compatible = "regulator-fixed";
+ regulator-name = "ts_avdd_3v0";
+
+ regulator-min-microvolt = <3000000>;
+ regulator-max-microvolt = <3000000>;
+
+ gpio = <&tlmm 144 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+
+ ts_vddio_1v8: regulator-ts-vddio-1v8 {
+ compatible = "regulator-fixed";
+ regulator-name = "ts_vddio_1v8";
+
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+
+ gpio = <&tlmm 102 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+};
+
+&mdss_dsi1 {
+ status = "disabled";
+};
+
+&i2c_hub_3 {
+ clock-frequency = <100000>;
+
+ status = "okay";
+
+ touchscreen@38 {
+ compatible = "focaltech,ft5426";
+ reg = <0x38>;
+
+ interrupt-parent = <&tlmm>;
+ interrupts = <15 IRQ_TYPE_EDGE_FALLING>;
+ reset-gpios = <&tlmm 14 GPIO_ACTIVE_LOW>;
+
+ vcc-supply = <&ts_avdd_3v0>;
+ iovcc-supply = <&ts_vddio_1v8>;
+
+ pinctrl-0 = <&ts_p_rst_default &ts_p_int_default>;
+ pinctrl-1 = <&ts_p_rst_sleep &ts_p_int_sleep>;
+ pinctrl-names = "default", "sleep";
+
+ touchscreen-size-x = <1080>;
+ touchscreen-size-y = <1920>;
+ touchscreen-swapped-x-y;
+ };
+};
+
+&pmk8550_pwm {
+ status = "okay";
+};
+
+&remoteproc_adsp {
+ firmware-name = "qcom/sm8550/retroidpocket/rp6/adsp.mbn",
+ "qcom/sm8550/retroidpocket/rp6/adsp_dtb.mbn";
+
+ status = "okay";
+};
+
+&spk_amp_l {
+ firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
+};
+
+&spk_amp_r {
+ firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
+};
--
2.54.0
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH 4/4] arm64: dts: qcom: Add Retroid Pocket Nova
2026-08-08 17:37 [PATCH 0/4] arm64: dts: qcom: Support Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
` (2 preceding siblings ...)
2026-08-08 17:37 ` [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6 Aaron Kling via B4 Relay
@ 2026-08-08 17:37 ` Aaron Kling via B4 Relay
2026-08-13 9:24 ` Krzysztof Kozlowski
3 siblings, 1 reply; 14+ messages in thread
From: Aaron Kling via B4 Relay @ 2026-08-08 17:37 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio
Cc: devicetree, linux-kernel, linux-arm-msm, Aaron Kling
From: Aaron Kling <webgeek1234@gmail.com>
The Retroid Pocket Nova is a high-performance Android-based handheld
gaming console powered by the Qualcomm Snapdragon 8 Gen 2 processor
featuring a 4.5-inch 4:3 AMOLED touchscreen.
Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
---
arch/arm64/boot/dts/qcom/Makefile | 1 +
.../boot/dts/qcom/qcs8550-retroidpocket-nova.dts | 118 +++++++++++++++++++++
2 files changed, 119 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
index 09b3c5b2d9dab..582805e89f4cb 100644
--- a/arch/arm64/boot/dts/qcom/Makefile
+++ b/arch/arm64/boot/dts/qcom/Makefile
@@ -196,6 +196,7 @@ dtb-$(CONFIG_ARCH_QCOM) += qcs8550-aim300-aiot.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2mini.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2portal.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-thor.dtb
+dtb-$(CONFIG_ARCH_QCOM) += qcs8550-retroidpocket-nova.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-retroidpocket-rp6.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs8550-rb5gen2.dtb
dtb-$(CONFIG_ARCH_QCOM) += qcs9100-ride.dtb
diff --git a/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts
new file mode 100644
index 0000000000000..10dabb1d51f5a
--- /dev/null
+++ b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts
@@ -0,0 +1,118 @@
+// SPDX-License-Identifier: BSD-3-Clause
+
+/dts-v1/;
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+#include "qcs8550-ayntec-common.dtsi"
+
+&{/} {
+ model = "Retroid Pocket Nova";
+ compatible = "retroidpocket,nova", "qcom,qcs8550", "qcom,sm8550";
+
+ backlight: backlight {
+ compatible = "pwm-backlight";
+ pwms = <&pmk8550_pwm 0 860000>;
+ brightness-levels = <1023 0>;
+ num-interpolated-steps = <1023>;
+ default-brightness-level = <600>;
+ power-supply = <&vph_pwr>;
+ enable-gpios = <&pmk8550_gpios 5 GPIO_ACTIVE_HIGH>;
+ pinctrl-names = "default";
+ pinctrl-0 = <&pwm_backlight_default>;
+
+ status = "okay";
+ };
+
+ vdd_disp_1v8: vdd-disp-1v8-regulator {
+ compatible = "regulator-fixed";
+ regulator-name = "vdd_disp_1v8";
+
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+
+ gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+
+ vdd_disp_2v8: vdd-disp-2v8-regulator {
+ compatible = "regulator-fixed";
+ regulator-name = "vdd_disp_2v8";
+
+ regulator-min-microvolt = <2800000>;
+ regulator-max-microvolt = <2800000>;
+
+ gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+
+ ts_avdd_3v0: regulator-ts-avdd-3v0 {
+ compatible = "regulator-fixed";
+ regulator-name = "ts_avdd_3v0";
+
+ regulator-min-microvolt = <3000000>;
+ regulator-max-microvolt = <3000000>;
+
+ gpio = <&tlmm 144 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+
+ ts_vddio_1v8: regulator-ts-vddio-1v8 {
+ compatible = "regulator-fixed";
+ regulator-name = "ts_vddio_1v8";
+
+ regulator-min-microvolt = <1800000>;
+ regulator-max-microvolt = <1800000>;
+
+ gpio = <&tlmm 102 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+ };
+};
+
+&mdss_dsi1 {
+ status = "disabled";
+};
+
+&i2c_hub_3 {
+ clock-frequency = <100000>;
+
+ status = "okay";
+
+ touchscreen@38 {
+ compatible = "focaltech,ft5426";
+ reg = <0x38>;
+
+ interrupt-parent = <&tlmm>;
+ interrupts = <15 IRQ_TYPE_EDGE_FALLING>;
+ reset-gpios = <&tlmm 14 GPIO_ACTIVE_LOW>;
+
+ vcc-supply = <&ts_avdd_3v0>;
+ iovcc-supply = <&ts_vddio_1v8>;
+
+ pinctrl-0 = <&ts_p_rst_default &ts_p_int_default>;
+ pinctrl-1 = <&ts_p_rst_sleep &ts_p_int_sleep>;
+ pinctrl-names = "default", "sleep";
+
+ touchscreen-size-x = <1280>;
+ touchscreen-size-y = <960>;
+ };
+};
+
+&pmk8550_pwm {
+ status = "okay";
+};
+
+&remoteproc_adsp {
+ firmware-name = "qcom/sm8550/retroidpocket/nova/adsp.mbn",
+ "qcom/sm8550/retroidpocket/nova/adsp_dtb.mbn";
+
+ status = "okay";
+};
+
+&spk_amp_l {
+ firmware-name = "qcom/sm8550/retroidpocket/nova/aw883xx_acf.bin";
+};
+
+&spk_amp_r {
+ firmware-name = "qcom/sm8550/retroidpocket/nova/aw883xx_acf.bin";
+};
--
2.54.0
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH 1/4] dt-bindings: vendor-prefixes: Add Retroid Pocket
2026-08-08 17:37 ` [PATCH 1/4] dt-bindings: vendor-prefixes: Add Retroid Pocket Aaron Kling via B4 Relay
@ 2026-08-13 9:20 ` Krzysztof Kozlowski
0 siblings, 0 replies; 14+ messages in thread
From: Krzysztof Kozlowski @ 2026-08-13 9:20 UTC (permalink / raw)
To: Aaron Kling
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Sat, Aug 08, 2026 at 12:37:55PM -0500, Aaron Kling wrote:
> Add an entry for Retroid Pocket (https://www.goretroid.com/)
>
> Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> ---
> Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++
> 1 file changed, 2 insertions(+)
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/4] dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices
2026-08-08 17:37 ` [PATCH 2/4] dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
@ 2026-08-13 9:23 ` Krzysztof Kozlowski
2026-08-14 18:40 ` Aaron Kling
0 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2026-08-13 9:23 UTC (permalink / raw)
To: Aaron Kling
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Sat, Aug 08, 2026 at 12:37:56PM -0500, Aaron Kling wrote:
> Namely:
> * Nova
> * RP6
Nova what? This is some cut commit msg.
>
> Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> ---
> Documentation/devicetree/bindings/arm/qcom.yaml | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> index 954be28203017..52ef669d6fe9b 100644
> --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> @@ -1161,6 +1161,13 @@ properties:
> - const: qcom,qcs8550
> - const: qcom,sm8550
>
> + - items:
> + - enum:
> + - retroidpocket,nova
> + - retroidpocket,rp6
This goes to the existing enum.
> + - const: qcom,qcs8550
> + - const: qcom,sm8550
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 4/4] arm64: dts: qcom: Add Retroid Pocket Nova
2026-08-08 17:37 ` [PATCH 4/4] arm64: dts: qcom: Add Retroid Pocket Nova Aaron Kling via B4 Relay
@ 2026-08-13 9:24 ` Krzysztof Kozlowski
2026-08-14 19:03 ` Aaron Kling
0 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2026-08-13 9:24 UTC (permalink / raw)
To: Aaron Kling
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Sat, Aug 08, 2026 at 12:37:58PM -0500, Aaron Kling wrote:
> The Retroid Pocket Nova is a high-performance Android-based handheld
> gaming console powered by the Qualcomm Snapdragon 8 Gen 2 processor
> featuring a 4.5-inch 4:3 AMOLED touchscreen.
>
> Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> ---
> arch/arm64/boot/dts/qcom/Makefile | 1 +
> .../boot/dts/qcom/qcs8550-retroidpocket-nova.dts | 118 +++++++++++++++++++++
> 2 files changed, 119 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> index 09b3c5b2d9dab..582805e89f4cb 100644
> --- a/arch/arm64/boot/dts/qcom/Makefile
> +++ b/arch/arm64/boot/dts/qcom/Makefile
> @@ -196,6 +196,7 @@ dtb-$(CONFIG_ARCH_QCOM) += qcs8550-aim300-aiot.dtb
> dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2mini.dtb
> dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2portal.dtb
> dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-thor.dtb
> +dtb-$(CONFIG_ARCH_QCOM) += qcs8550-retroidpocket-nova.dtb
> dtb-$(CONFIG_ARCH_QCOM) += qcs8550-retroidpocket-rp6.dtb
> dtb-$(CONFIG_ARCH_QCOM) += qcs8550-rb5gen2.dtb
> dtb-$(CONFIG_ARCH_QCOM) += qcs9100-ride.dtb
> diff --git a/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts
> new file mode 100644
> index 0000000000000..10dabb1d51f5a
> --- /dev/null
> +++ b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts
> @@ -0,0 +1,118 @@
> +// SPDX-License-Identifier: BSD-3-Clause
> +
> +/dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/interrupt-controller/irq.h>
> +#include "qcs8550-ayntec-common.dtsi"
> +
> +&{/} {
> + model = "Retroid Pocket Nova";
> + compatible = "retroidpocket,nova", "qcom,qcs8550", "qcom,sm8550";
> +
> + backlight: backlight {
> + compatible = "pwm-backlight";
> + pwms = <&pmk8550_pwm 0 860000>;
> + brightness-levels = <1023 0>;
> + num-interpolated-steps = <1023>;
> + default-brightness-level = <600>;
> + power-supply = <&vph_pwr>;
> + enable-gpios = <&pmk8550_gpios 5 GPIO_ACTIVE_HIGH>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pwm_backlight_default>;
> +
> + status = "okay";
> + };
> +
> + vdd_disp_1v8: vdd-disp-1v8-regulator {
So foo-regulator?
> + compatible = "regulator-fixed";
> + regulator-name = "vdd_disp_1v8";
> +
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> +
> + gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> + };
> +
> + vdd_disp_2v8: vdd-disp-2v8-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "vdd_disp_2v8";
> +
> + regulator-min-microvolt = <2800000>;
> + regulator-max-microvolt = <2800000>;
> +
> + gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> + };
> +
> + ts_avdd_3v0: regulator-ts-avdd-3v0 {
Or regulator-foo?
Decide and also read:
https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6
2026-08-08 17:37 ` [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6 Aaron Kling via B4 Relay
@ 2026-08-13 19:05 ` Dmitry Baryshkov
2026-08-14 18:59 ` Aaron Kling
2026-08-17 16:11 ` Konrad Dybcio
1 sibling, 1 reply; 14+ messages in thread
From: Dmitry Baryshkov @ 2026-08-13 19:05 UTC (permalink / raw)
To: webgeek1234
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Sat, Aug 08, 2026 at 12:37:57PM -0500, Aaron Kling via B4 Relay wrote:
> From: Aaron Kling <webgeek1234@gmail.com>
>
> The Retroid Pocket 6 is a high-performance Android-based handheld gaming
> console powered by the Qualcomm Snapdragon 8 Gen 2 processor featuring a
> 5.5-inch AMOLED touchscreen.
>
> This is based on the support by ROCKNIX, however their history does not
> show who the original author was, so direct attributation cannot be
> given.
>
> Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> ---
> arch/arm64/boot/dts/qcom/Makefile | 1 +
> .../boot/dts/qcom/qcs8550-retroidpocket-rp6.dts | 122 +++++++++++++++++++++
> 2 files changed, 123 insertions(+)
>
> +
> + vdd_disp_1v8: vdd-disp-1v8-regulator {
regulator-foo
> + compatible = "regulator-fixed";
> + regulator-name = "vdd_disp_1v8";
> +
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> +
> + gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> + };
> +
> + vdd_disp_2v8: vdd-disp-2v8-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "vdd_disp_2v8";
> +
> + regulator-min-microvolt = <2800000>;
> + regulator-max-microvolt = <2800000>;
> +
> + gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> + };
> +
> + ts_avdd_3v0: regulator-ts-avdd-3v0 {
> + compatible = "regulator-fixed";
> + regulator-name = "ts_avdd_3v0";
> +
> + regulator-min-microvolt = <3000000>;
> + regulator-max-microvolt = <3000000>;
> +
> + gpio = <&tlmm 144 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> + };
> +
> + ts_vddio_1v8: regulator-ts-vddio-1v8 {
> + compatible = "regulator-fixed";
> + regulator-name = "ts_vddio_1v8";
> +
> + regulator-min-microvolt = <1800000>;
> + regulator-max-microvolt = <1800000>;
> +
> + gpio = <&tlmm 102 GPIO_ACTIVE_HIGH>;
> + enable-active-high;
> + };
> +};
> +
> +&mdss_dsi1 {
Out of place.
> + status = "disabled";
Isn't this a default? If so, no need to repeat it.
> +};
> +
> +&i2c_hub_3 {
> + clock-frequency = <100000>;
> +
> + status = "okay";
> +
> + touchscreen@38 {
> + compatible = "focaltech,ft5426";
> + reg = <0x38>;
> +
> + interrupt-parent = <&tlmm>;
> + interrupts = <15 IRQ_TYPE_EDGE_FALLING>;
> + reset-gpios = <&tlmm 14 GPIO_ACTIVE_LOW>;
> +
> + vcc-supply = <&ts_avdd_3v0>;
> + iovcc-supply = <&ts_vddio_1v8>;
> +
> + pinctrl-0 = <&ts_p_rst_default &ts_p_int_default>;
> + pinctrl-1 = <&ts_p_rst_sleep &ts_p_int_sleep>;
> + pinctrl-names = "default", "sleep";
> +
> + touchscreen-size-x = <1080>;
> + touchscreen-size-y = <1920>;
> + touchscreen-swapped-x-y;
> + };
> +};
> +
> +&pmk8550_pwm {
> + status = "okay";
> +};
> +
> +&remoteproc_adsp {
> + firmware-name = "qcom/sm8550/retroidpocket/rp6/adsp.mbn",
> + "qcom/sm8550/retroidpocket/rp6/adsp_dtb.mbn";
> +
> + status = "okay";
> +};
> +
> +&spk_amp_l {
> + firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
> +};
> +
> +&spk_amp_r {
> + firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
> +};
>
> --
> 2.54.0
>
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 2/4] dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices
2026-08-13 9:23 ` Krzysztof Kozlowski
@ 2026-08-14 18:40 ` Aaron Kling
0 siblings, 0 replies; 14+ messages in thread
From: Aaron Kling @ 2026-08-14 18:40 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Thu, Aug 13, 2026 at 4:23 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On Sat, Aug 08, 2026 at 12:37:56PM -0500, Aaron Kling wrote:
> > Namely:
> > * Nova
> > * RP6
>
> Nova what? This is some cut commit msg.
This is the same structure I used for the matching AYN commit. The
vendor is Retroid Pocket and the product is Nova. It's marketed as the
Retroid Pocket Nova. The vendor is listed in the subject line and I'm
listing the product names in the bullet list.
> >
> > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > ---
> > Documentation/devicetree/bindings/arm/qcom.yaml | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/qcom.yaml b/Documentation/devicetree/bindings/arm/qcom.yaml
> > index 954be28203017..52ef669d6fe9b 100644
> > --- a/Documentation/devicetree/bindings/arm/qcom.yaml
> > +++ b/Documentation/devicetree/bindings/arm/qcom.yaml
> > @@ -1161,6 +1161,13 @@ properties:
> > - const: qcom,qcs8550
> > - const: qcom,sm8550
> >
> > + - items:
> > + - enum:
> > + - retroidpocket,nova
> > + - retroidpocket,rp6
>
> This goes to the existing enum.
Ack.
> > + - const: qcom,qcs8550
> > + - const: qcom,sm8550
>
> Best regards,
> Krzysztof
>
Aaron
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6
2026-08-13 19:05 ` Dmitry Baryshkov
@ 2026-08-14 18:59 ` Aaron Kling
2026-08-15 14:44 ` Dmitry Baryshkov
0 siblings, 1 reply; 14+ messages in thread
From: Aaron Kling @ 2026-08-14 18:59 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Thu, Aug 13, 2026 at 2:06 PM Dmitry Baryshkov
<dmitry.baryshkov@oss.qualcomm.com> wrote:
>
> On Sat, Aug 08, 2026 at 12:37:57PM -0500, Aaron Kling via B4 Relay wrote:
> > From: Aaron Kling <webgeek1234@gmail.com>
> >
> > The Retroid Pocket 6 is a high-performance Android-based handheld gaming
> > console powered by the Qualcomm Snapdragon 8 Gen 2 processor featuring a
> > 5.5-inch AMOLED touchscreen.
> >
> > This is based on the support by ROCKNIX, however their history does not
> > show who the original author was, so direct attributation cannot be
> > given.
> >
> > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > ---
> > arch/arm64/boot/dts/qcom/Makefile | 1 +
> > .../boot/dts/qcom/qcs8550-retroidpocket-rp6.dts | 122 +++++++++++++++++++++
> > 2 files changed, 123 insertions(+)
> >
> > +
> > + vdd_disp_1v8: vdd-disp-1v8-regulator {
>
> regulator-foo
I see these are inconsistent, I missed that when pulling the support
in. However, looking at the associated AYN series, all regulators are
named foo-regulator. Using regulator-foo would make that inconsistent,
is that really desired?
> > + compatible = "regulator-fixed";
> > + regulator-name = "vdd_disp_1v8";
> > +
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1800000>;
> > +
> > + gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + };
> > +
> > + vdd_disp_2v8: vdd-disp-2v8-regulator {
> > + compatible = "regulator-fixed";
> > + regulator-name = "vdd_disp_2v8";
> > +
> > + regulator-min-microvolt = <2800000>;
> > + regulator-max-microvolt = <2800000>;
> > +
> > + gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + };
> > +
> > + ts_avdd_3v0: regulator-ts-avdd-3v0 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "ts_avdd_3v0";
> > +
> > + regulator-min-microvolt = <3000000>;
> > + regulator-max-microvolt = <3000000>;
> > +
> > + gpio = <&tlmm 144 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + };
> > +
> > + ts_vddio_1v8: regulator-ts-vddio-1v8 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "ts_vddio_1v8";
> > +
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1800000>;
> > +
> > + gpio = <&tlmm 102 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + };
> > +};
> > +
> > +&mdss_dsi1 {
>
> Out of place.
Oops.
> > + status = "disabled";
>
> Isn't this a default? If so, no need to repeat it.
qcs8550-ayntec-common.dtsi enables this because most of the variants
have a panel on this connector. So as one of the odd ones out, this
needs turned back off.
> > +};
> > +
> > +&i2c_hub_3 {
> > + clock-frequency = <100000>;
> > +
> > + status = "okay";
> > +
> > + touchscreen@38 {
> > + compatible = "focaltech,ft5426";
> > + reg = <0x38>;
> > +
> > + interrupt-parent = <&tlmm>;
> > + interrupts = <15 IRQ_TYPE_EDGE_FALLING>;
> > + reset-gpios = <&tlmm 14 GPIO_ACTIVE_LOW>;
> > +
> > + vcc-supply = <&ts_avdd_3v0>;
> > + iovcc-supply = <&ts_vddio_1v8>;
> > +
> > + pinctrl-0 = <&ts_p_rst_default &ts_p_int_default>;
> > + pinctrl-1 = <&ts_p_rst_sleep &ts_p_int_sleep>;
> > + pinctrl-names = "default", "sleep";
> > +
> > + touchscreen-size-x = <1080>;
> > + touchscreen-size-y = <1920>;
> > + touchscreen-swapped-x-y;
> > + };
> > +};
> > +
> > +&pmk8550_pwm {
> > + status = "okay";
> > +};
> > +
> > +&remoteproc_adsp {
> > + firmware-name = "qcom/sm8550/retroidpocket/rp6/adsp.mbn",
> > + "qcom/sm8550/retroidpocket/rp6/adsp_dtb.mbn";
> > +
> > + status = "okay";
> > +};
> > +
> > +&spk_amp_l {
> > + firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
> > +};
> > +
> > +&spk_amp_r {
> > + firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
> > +};
> >
> > --
> > 2.54.0
> >
> >
>
> --
> With best wishes
> Dmitry
Aaron
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 4/4] arm64: dts: qcom: Add Retroid Pocket Nova
2026-08-13 9:24 ` Krzysztof Kozlowski
@ 2026-08-14 19:03 ` Aaron Kling
0 siblings, 0 replies; 14+ messages in thread
From: Aaron Kling @ 2026-08-14 19:03 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Thu, Aug 13, 2026 at 4:24 AM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>
> On Sat, Aug 08, 2026 at 12:37:58PM -0500, Aaron Kling wrote:
> > The Retroid Pocket Nova is a high-performance Android-based handheld
> > gaming console powered by the Qualcomm Snapdragon 8 Gen 2 processor
> > featuring a 4.5-inch 4:3 AMOLED touchscreen.
> >
> > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > ---
> > arch/arm64/boot/dts/qcom/Makefile | 1 +
> > .../boot/dts/qcom/qcs8550-retroidpocket-nova.dts | 118 +++++++++++++++++++++
> > 2 files changed, 119 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/Makefile b/arch/arm64/boot/dts/qcom/Makefile
> > index 09b3c5b2d9dab..582805e89f4cb 100644
> > --- a/arch/arm64/boot/dts/qcom/Makefile
> > +++ b/arch/arm64/boot/dts/qcom/Makefile
> > @@ -196,6 +196,7 @@ dtb-$(CONFIG_ARCH_QCOM) += qcs8550-aim300-aiot.dtb
> > dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2mini.dtb
> > dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-odin2portal.dtb
> > dtb-$(CONFIG_ARCH_QCOM) += qcs8550-ayntec-thor.dtb
> > +dtb-$(CONFIG_ARCH_QCOM) += qcs8550-retroidpocket-nova.dtb
> > dtb-$(CONFIG_ARCH_QCOM) += qcs8550-retroidpocket-rp6.dtb
> > dtb-$(CONFIG_ARCH_QCOM) += qcs8550-rb5gen2.dtb
> > dtb-$(CONFIG_ARCH_QCOM) += qcs9100-ride.dtb
> > diff --git a/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts
> > new file mode 100644
> > index 0000000000000..10dabb1d51f5a
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/qcom/qcs8550-retroidpocket-nova.dts
> > @@ -0,0 +1,118 @@
> > +// SPDX-License-Identifier: BSD-3-Clause
> > +
> > +/dts-v1/;
> > +
> > +#include <dt-bindings/gpio/gpio.h>
> > +#include <dt-bindings/interrupt-controller/irq.h>
> > +#include "qcs8550-ayntec-common.dtsi"
> > +
> > +&{/} {
> > + model = "Retroid Pocket Nova";
> > + compatible = "retroidpocket,nova", "qcom,qcs8550", "qcom,sm8550";
> > +
> > + backlight: backlight {
> > + compatible = "pwm-backlight";
> > + pwms = <&pmk8550_pwm 0 860000>;
> > + brightness-levels = <1023 0>;
> > + num-interpolated-steps = <1023>;
> > + default-brightness-level = <600>;
> > + power-supply = <&vph_pwr>;
> > + enable-gpios = <&pmk8550_gpios 5 GPIO_ACTIVE_HIGH>;
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pwm_backlight_default>;
> > +
> > + status = "okay";
> > + };
> > +
> > + vdd_disp_1v8: vdd-disp-1v8-regulator {
>
> So foo-regulator?
>
> > + compatible = "regulator-fixed";
> > + regulator-name = "vdd_disp_1v8";
> > +
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <1800000>;
> > +
> > + gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + };
> > +
> > + vdd_disp_2v8: vdd-disp-2v8-regulator {
> > + compatible = "regulator-fixed";
> > + regulator-name = "vdd_disp_2v8";
> > +
> > + regulator-min-microvolt = <2800000>;
> > + regulator-max-microvolt = <2800000>;
> > +
> > + gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + };
> > +
> > + ts_avdd_3v0: regulator-ts-avdd-3v0 {
>
> Or regulator-foo?
Given that the associated AYN series uses foo-regulator, it makes more
sense to me to remain consistent with that. The question has been
raised on the RP6 patch in this series, so I will wait for a response
there before sending a new revision.
> Decide and also read:
> https://web.git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/regulator/fixed-regulator.yaml
Ack.
> Best regards,
> Krzysztof
Aaron
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6
2026-08-14 18:59 ` Aaron Kling
@ 2026-08-15 14:44 ` Dmitry Baryshkov
0 siblings, 0 replies; 14+ messages in thread
From: Dmitry Baryshkov @ 2026-08-15 14:44 UTC (permalink / raw)
To: Aaron Kling
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Bjorn Andersson,
Konrad Dybcio, devicetree, linux-kernel, linux-arm-msm
On Fri, Aug 14, 2026 at 01:59:55PM -0500, Aaron Kling wrote:
> On Thu, Aug 13, 2026 at 2:06 PM Dmitry Baryshkov
> <dmitry.baryshkov@oss.qualcomm.com> wrote:
> >
> > On Sat, Aug 08, 2026 at 12:37:57PM -0500, Aaron Kling via B4 Relay wrote:
> > > From: Aaron Kling <webgeek1234@gmail.com>
> > >
> > > The Retroid Pocket 6 is a high-performance Android-based handheld gaming
> > > console powered by the Qualcomm Snapdragon 8 Gen 2 processor featuring a
> > > 5.5-inch AMOLED touchscreen.
> > >
> > > This is based on the support by ROCKNIX, however their history does not
> > > show who the original author was, so direct attributation cannot be
> > > given.
> > >
> > > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > > ---
> > > arch/arm64/boot/dts/qcom/Makefile | 1 +
> > > .../boot/dts/qcom/qcs8550-retroidpocket-rp6.dts | 122 +++++++++++++++++++++
> > > 2 files changed, 123 insertions(+)
> > >
> > > +
> > > + vdd_disp_1v8: vdd-disp-1v8-regulator {
> >
> > regulator-foo
>
> I see these are inconsistent, I missed that when pulling the support
> in. However, looking at the associated AYN series, all regulators are
> named foo-regulator. Using regulator-foo would make that inconsistent,
> is that really desired?
I think, it's generally a better ide. Feel free to rename the regulator
nodes in the existing DTs that you import in a spearate patch (although
that might get rejected by other reviewers as "noise").
>
> > > + compatible = "regulator-fixed";
> > > + regulator-name = "vdd_disp_1v8";
> > > +
> > > + regulator-min-microvolt = <1800000>;
> > > + regulator-max-microvolt = <1800000>;
> > > +
> > > + gpio = <&tlmm 70 GPIO_ACTIVE_HIGH>;
> > > + enable-active-high;
> > > + };
> > > +
> > > + vdd_disp_2v8: vdd-disp-2v8-regulator {
> > > + compatible = "regulator-fixed";
> > > + regulator-name = "vdd_disp_2v8";
> > > +
> > > + regulator-min-microvolt = <2800000>;
> > > + regulator-max-microvolt = <2800000>;
> > > +
> > > + gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
> > > + enable-active-high;
> > > + };
> > > +
> > > + ts_avdd_3v0: regulator-ts-avdd-3v0 {
> > > + compatible = "regulator-fixed";
> > > + regulator-name = "ts_avdd_3v0";
> > > +
> > > + regulator-min-microvolt = <3000000>;
> > > + regulator-max-microvolt = <3000000>;
> > > +
> > > + gpio = <&tlmm 144 GPIO_ACTIVE_HIGH>;
> > > + enable-active-high;
> > > + };
> > > +
> > > + ts_vddio_1v8: regulator-ts-vddio-1v8 {
> > > + compatible = "regulator-fixed";
> > > + regulator-name = "ts_vddio_1v8";
> > > +
> > > + regulator-min-microvolt = <1800000>;
> > > + regulator-max-microvolt = <1800000>;
> > > +
> > > + gpio = <&tlmm 102 GPIO_ACTIVE_HIGH>;
> > > + enable-active-high;
> > > + };
> > > +};
> > > +
> > > +&mdss_dsi1 {
> >
> > Out of place.
>
> Oops.
>
> > > + status = "disabled";
> >
> > Isn't this a default? If so, no need to repeat it.
>
> qcs8550-ayntec-common.dtsi enables this because most of the variants
> have a panel on this connector. So as one of the odd ones out, this
> needs turned back off.
Yep, I missed it. THanks for the pointer.
>
> > > +};
> > > +
> > > +&i2c_hub_3 {
> > > + clock-frequency = <100000>;
> > > +
> > > + status = "okay";
> > > +
> > > + touchscreen@38 {
> > > + compatible = "focaltech,ft5426";
> > > + reg = <0x38>;
> > > +
> > > + interrupt-parent = <&tlmm>;
> > > + interrupts = <15 IRQ_TYPE_EDGE_FALLING>;
> > > + reset-gpios = <&tlmm 14 GPIO_ACTIVE_LOW>;
> > > +
> > > + vcc-supply = <&ts_avdd_3v0>;
> > > + iovcc-supply = <&ts_vddio_1v8>;
> > > +
> > > + pinctrl-0 = <&ts_p_rst_default &ts_p_int_default>;
> > > + pinctrl-1 = <&ts_p_rst_sleep &ts_p_int_sleep>;
> > > + pinctrl-names = "default", "sleep";
> > > +
> > > + touchscreen-size-x = <1080>;
> > > + touchscreen-size-y = <1920>;
> > > + touchscreen-swapped-x-y;
> > > + };
> > > +};
> > > +
> > > +&pmk8550_pwm {
> > > + status = "okay";
> > > +};
> > > +
> > > +&remoteproc_adsp {
> > > + firmware-name = "qcom/sm8550/retroidpocket/rp6/adsp.mbn",
> > > + "qcom/sm8550/retroidpocket/rp6/adsp_dtb.mbn";
> > > +
> > > + status = "okay";
> > > +};
> > > +
> > > +&spk_amp_l {
> > > + firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
> > > +};
> > > +
> > > +&spk_amp_r {
> > > + firmware-name = "qcom/sm8550/retroidpocket/rp6/aw883xx_acf.bin";
> > > +};
> > >
> > > --
> > > 2.54.0
> > >
> > >
> >
> > --
> > With best wishes
> > Dmitry
>
> Aaron
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6
2026-08-08 17:37 ` [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6 Aaron Kling via B4 Relay
2026-08-13 19:05 ` Dmitry Baryshkov
@ 2026-08-17 16:11 ` Konrad Dybcio
1 sibling, 0 replies; 14+ messages in thread
From: Konrad Dybcio @ 2026-08-17 16:11 UTC (permalink / raw)
To: webgeek1234, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Bjorn Andersson, Konrad Dybcio
Cc: devicetree, linux-kernel, linux-arm-msm
On 8/8/26 7:37 PM, Aaron Kling via B4 Relay wrote:
> From: Aaron Kling <webgeek1234@gmail.com>
>
> The Retroid Pocket 6 is a high-performance Android-based handheld gaming
> console powered by the Qualcomm Snapdragon 8 Gen 2 processor featuring a
> 5.5-inch AMOLED touchscreen.
[...]
> +&{/} {
> + model = "Retroid Pocket 6";
> + compatible = "retroidpocket,rp6", "qcom,qcs8550", "qcom,sm8550";
> +
> + backlight: backlight {
> + compatible = "pwm-backlight";
> + pwms = <&pmk8550_pwm 0 860000>;
> + brightness-levels = <1023 0>;
> + num-interpolated-steps = <1023>;
> + default-brightness-level = <600>;
> + power-supply = <&vph_pwr>;
> + enable-gpios = <&pmk8550_gpios 5 GPIO_ACTIVE_HIGH>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pwm_backlight_default>;
property-n
property-names
in this order please
> +&i2c_hub_3 {
> + clock-frequency = <100000>;
> +
> + status = "okay";
> +
> + touchscreen@38 {
> + compatible = "focaltech,ft5426";
> + reg = <0x38>;
> +
> + interrupt-parent = <&tlmm>;
> + interrupts = <15 IRQ_TYPE_EDGE_FALLING>;
> + reset-gpios = <&tlmm 14 GPIO_ACTIVE_LOW>;
> +
> + vcc-supply = <&ts_avdd_3v0>;
> + iovcc-supply = <&ts_vddio_1v8>;
> +
> + pinctrl-0 = <&ts_p_rst_default &ts_p_int_default>;
<&foo>, <&bar> preferably (dtc treats it all the same unforunately..)
otherwise largely lgtm
Konrad
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2026-08-17 16:11 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-08 17:37 [PATCH 0/4] arm64: dts: qcom: Support Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
2026-08-08 17:37 ` [PATCH 1/4] dt-bindings: vendor-prefixes: Add Retroid Pocket Aaron Kling via B4 Relay
2026-08-13 9:20 ` Krzysztof Kozlowski
2026-08-08 17:37 ` [PATCH 2/4] dt-bindings: arm: qcom: Add Retroid Pocket QCS8550 Devices Aaron Kling via B4 Relay
2026-08-13 9:23 ` Krzysztof Kozlowski
2026-08-14 18:40 ` Aaron Kling
2026-08-08 17:37 ` [PATCH 3/4] arm64: dts: qcom: Add Retroid Pocket 6 Aaron Kling via B4 Relay
2026-08-13 19:05 ` Dmitry Baryshkov
2026-08-14 18:59 ` Aaron Kling
2026-08-15 14:44 ` Dmitry Baryshkov
2026-08-17 16:11 ` Konrad Dybcio
2026-08-08 17:37 ` [PATCH 4/4] arm64: dts: qcom: Add Retroid Pocket Nova Aaron Kling via B4 Relay
2026-08-13 9:24 ` Krzysztof Kozlowski
2026-08-14 19:03 ` Aaron Kling
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox