From: Xilin Wu <wuxilin123@gmail.com>
To: "Krzysztof Kozlowski" <krzk@kernel.org>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Junhao Xie" <bigfoot@classfun.cn>,
"Neil Armstrong" <neil.armstrong@linaro.org>,
"Jessica Zhang" <quic_jesszhan@quicinc.com>,
"Sam Ravnborg" <sam@ravnborg.org>,
"David Airlie" <airlied@gmail.com>,
"Daniel Vetter" <daniel@ffwll.ch>,
"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
"Bjorn Andersson" <andersson@kernel.org>,
"Konrad Dybcio" <konrad.dybcio@linaro.org>,
"Tengfei Fan" <quic_tengfan@quicinc.com>,
"Molly Sophia" <mollysophia379@gmail.com>
Cc: linux-pwm@vger.kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org,
linux-arm-msm@vger.kernel.org
Subject: Re: [PATCH 10/10] arm64: dts: qcom: Add AYN Odin 2
Date: Sun, 28 Apr 2024 11:54:07 +0800 [thread overview]
Message-ID: <f80384eb-d7eb-4a68-952d-529230ec51a6@gmail.com> (raw)
In-Reply-To: <0d1c6ed9-0c20-4b0e-876d-4b0adf5aa75d@kernel.org>
On 2024/4/25 14:28, Krzysztof Kozlowski wrote:
> On 24/04/2024 17:29, Xilin Wu via B4 Relay wrote:
>> From: Xilin Wu <wuxilin123@gmail.com>
>>
>> AYN Odin 2 is a gaming handheld based on QCS8550, which is derived
>> from SM8550 but without modem RF system.
>>
>
>
>
>> +
>> +/ {
>> + model = "AYN Odin 2";
>> + compatible = "ayn,odin2", "qcom,qcs8550", "qcom,sm8550";
>> + chassis-type = "handset";
>> +
>> + qcom,msm-id = <QCOM_ID_QCS8550 0x20000>;
>> + qcom,board-id = <0x1001f 0>;
>
> No, these are not allowed. You did not test your dts.
>
> It does not look like you tested the DTS against bindings. Please run
> `make dtbs_check W=1` (see
> Documentation/devicetree/bindings/writing-schema.rst or
> https://www.linaro.org/blog/tips-and-tricks-for-validating-devicetree-sources-with-the-devicetree-schema/
> for instructions).
>
>> +
>> + aliases {
>> + serial0 = &uart7;
>> + serial1 = &uart14;
>> + serial2 = &uart15;
>> + };
>> +
>> + 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";
>
> Drop, why do you need it? Do you see it anywhere else in the backlight
> nodes in DTS?
>
>
>> + };
>> +
>> + fan_pwr: fan-pwr-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "fan_pwr";
>> +
>> + regulator-min-microvolt = <5000000>;
>> + regulator-max-microvolt = <5000000>;
>> +
>> + gpio = <&tlmm 109 GPIO_ACTIVE_HIGH>;
>> + enable-active-high;
>> +
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&fan_pwr_en>;
>> +
>> + regulator-state-mem {
>> + regulator-off-in-suspend;
>> + };
>> + };
>> +
>> + gpio-keys {
>> + compatible = "gpio-keys";
>> +
>> + pinctrl-0 = <&volume_up_n>, <&m1_m2_keys_default>;
>> + pinctrl-names = "default";
>> +
>> + key-volume-up {
>> + label = "Volume Up";
>> + linux,code = <KEY_VOLUMEUP>;
>> + gpios = <&pm8550_gpios 6 GPIO_ACTIVE_LOW>;
>> + debounce-interval = <15>;
>> + linux,can-disable;
>> + wakeup-source;
>> + };
>> +
>> + m1-button {
>> + label = "M1";
>> + linux,code = <BTN_TRIGGER_HAPPY1>;
>> + gpios = <&tlmm 57 GPIO_ACTIVE_LOW>;
>> + };
>> +
>> + m2-button {
>> + label = "M2";
>> + linux,code = <BTN_TRIGGER_HAPPY2>;
>> + gpios = <&tlmm 58 GPIO_ACTIVE_LOW>;
>> + };
>> + };
>> +
>> + hdmi-out {
>> + compatible = "hdmi-connector";
>> + type = "d";
>> + hpd-gpios = <&tlmm 9 GPIO_ACTIVE_HIGH>;
>> +
>> + port {
>> + hdmi_con: endpoint {
>> + remote-endpoint = <<8912_out>;
>> + };
>> + };
>> + };
>> +
>> + hdmi_pwr: hdmi-pwr-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "hdmi_pwr";
>> +
>> + regulator-min-microvolt = <1800000>;
>> + regulator-max-microvolt = <1800000>;
>> +
>> + gpio = <&tlmm 10 GPIO_ACTIVE_HIGH>;
>> + enable-active-high;
>> + };
>> +
>> + vdd_lcm_2p8: vdd-lcm-2p8-regulator {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vdd_lcm_2p8";
>> +
>> + regulator-min-microvolt = <2800000>;
>> + regulator-max-microvolt = <2800000>;
>> +
>> + gpio = <&tlmm 142 GPIO_ACTIVE_HIGH>;
>> + enable-active-high;
>> + };
>> +
>> + led_left_side: led-controller-1 {
>> + compatible = "pwm-leds-multicolor";
>> +
>> + multi-led {
>> + label = "left-side";
>> + color = <LED_COLOR_ID_RGB>;
>> + max-brightness = <255>;
>> +
>> + led-red {
>> + color = <LED_COLOR_ID_RED>;
>> + pwms = <&pwm_rgb_left 0>;
>> + };
>> +
>> + led-green {
>> + color = <LED_COLOR_ID_GREEN>;
>> + pwms = <&pwm_rgb_left 1>;
>> + };
>> +
>> + led-blue {
>> + color = <LED_COLOR_ID_BLUE>;
>> + pwms = <&pwm_rgb_left 2>;
>> + };
>> + };
>> + };
>> +
>> + led_left_joystick: led-controller-2 {
>> + compatible = "pwm-leds-multicolor";
>> +
>> + multi-led {
>> + label = "left-joystick";
>> + color = <LED_COLOR_ID_RGB>;
>> + max-brightness = <255>;
>> +
>> + led-red {
>> + color = <LED_COLOR_ID_RED>;
>> + pwms = <&pwm_rgb_left 6>;
>> + };
>> +
>> + led-green {
>> + color = <LED_COLOR_ID_GREEN>;
>> + pwms = <&pwm_rgb_left 7>;
>> + };
>> +
>> + led-blue {
>> + color = <LED_COLOR_ID_BLUE>;
>> + pwms = <&pwm_rgb_left 8>;
>> + };
>> + };
>> + };
>> +
>> + led_right_side: led-controller-3 {
>> + compatible = "pwm-leds-multicolor";
>> +
>> + multi-led {
>> + label = "right-side";
>> + color = <LED_COLOR_ID_RGB>;
>> + max-brightness = <255>;
>> +
>> + led-red {
>> + color = <LED_COLOR_ID_RED>;
>> + pwms = <&pwm_rgb_right 0>;
>> + };
>> +
>> + led-green {
>> + color = <LED_COLOR_ID_GREEN>;
>> + pwms = <&pwm_rgb_right 1>;
>> + };
>> +
>> + led-blue {
>> + color = <LED_COLOR_ID_BLUE>;
>> + pwms = <&pwm_rgb_right 2>;
>> + };
>> + };
>> + };
>> +
>> + led_right_joystick: led-controller-4 {
>> + compatible = "pwm-leds-multicolor";
>> +
>> + multi-led {
>> + label = "right-joystick";
>> + color = <LED_COLOR_ID_RGB>;
>> + max-brightness = <255>;
>> +
>> + led-red {
>> + color = <LED_COLOR_ID_RED>;
>> + pwms = <&pwm_rgb_right 6>;
>> + };
>> +
>> + led-green {
>> + color = <LED_COLOR_ID_GREEN>;
>> + pwms = <&pwm_rgb_right 7>;
>> + };
>> +
>> + led-blue {
>> + color = <LED_COLOR_ID_BLUE>;
>> + pwms = <&pwm_rgb_right 8>;
>> + };
>> + };
>> + };
>> +
>> + mcu_3v3: mcu-3v3-regulator {
>
> Name all regulators regulator-n, where n is decimal number. Then order
> the nodes by name.
>
>
> ...
>
>> +
>> +&i2c4 {
>> + clock-frequency = <400000>;
>> + status = "okay";
>> +
>> + touchscreen@20 {
>> + compatible = "syna,rmi4-i2c";
>> + reg = <0x20>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + interrupts-extended = <&tlmm 25 0x2008>;
>> +
>> + pinctrl-names = "default", "sleep";
>> + pinctrl-0 = <&ts_int_default>;
>> + pinctrl-1 = <&ts_int_sleep>;
>> +
>> + vio-supply = <&vreg_l12b_1p8>;
>> +
>> + syna,startup-delay-ms = <200>;
>> + syna,reset-delay-ms = <200>;
>> +
>> + rmi4-f01@1 {
>> + syna,nosleep-mode = <0x1>;
>> + reg = <0x1>;
>> + };
>> +
>> + rmi4-f12@12 {
>> + reg = <0x12>;
>> + syna,rezero-wait-ms = <20>;
>> + syna,clip-x-low = <0>;
>> + syna,clip-y-low = <0>;
>> + syna,clip-x-high = <1080>;
>> + syna,clip-y-high = <1920>;
>> + syna,sensor-type = <1>;
>> + touchscreen-inverted-x;
>> + };
>> + };
>
> Please confirm the status of dtbs_check for your board. I am pretty sure
> it fails.
>
> Best regards,
> Krzysztof
>
I will correct all the mistakes in v2. Thanks for catching them!
--
Thanks,
Xilin Wu
next prev parent reply other threads:[~2024-04-28 3:54 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-24 15:29 [PATCH 00/10] AYN Odin 2 support Xilin Wu via B4 Relay
2024-04-24 15:29 ` [PATCH 01/10] dt-bindings: pwm: Add SI-EN SN3112 PWM support Xilin Wu via B4 Relay
2024-04-25 6:04 ` Uwe Kleine-König
2024-04-24 15:29 ` [PATCH 02/10] " Xilin Wu via B4 Relay
2024-04-24 15:55 ` Bryan O'Donoghue
2024-04-24 19:09 ` BigfootACA
2024-04-25 6:08 ` Uwe Kleine-König
2024-04-24 19:37 ` Konrad Dybcio
2024-04-25 0:57 ` Junhao Xie
2024-04-25 7:57 ` Neil Armstrong
2024-04-25 6:14 ` Uwe Kleine-König
2024-04-25 6:02 ` Uwe Kleine-König
2024-04-25 7:03 ` Junhao Xie
2024-04-24 15:29 ` [PATCH 03/10] dt-bindings: display: panel: Add Synaptics TD4328 Xilin Wu via B4 Relay
2024-04-25 16:11 ` Rob Herring
2024-04-28 3:25 ` Xilin Wu
2024-04-24 15:29 ` [PATCH 04/10] drm/panel: Add driver for Synaptics TD4328 LCD panel Xilin Wu via B4 Relay
2024-04-24 15:58 ` Bryan O'Donoghue
2024-04-28 3:29 ` Xilin Wu
2024-04-24 15:29 ` [PATCH 05/10] arm64: dts: qcom: pmk8550: Add PWM controller Xilin Wu via B4 Relay
2024-04-24 19:38 ` Konrad Dybcio
2024-04-24 15:29 ` [PATCH 06/10] arm64: dts: qcom: sm8550: Add UART15 Xilin Wu via B4 Relay
2024-04-24 23:39 ` Dmitry Baryshkov
2024-04-24 15:29 ` [PATCH 07/10] arm64: dts: qcom: sm8550: Update EAS properties Xilin Wu via B4 Relay
2024-04-24 22:45 ` Bryan O'Donoghue
2024-04-28 3:43 ` Xilin Wu
2024-04-24 15:29 ` [PATCH 08/10] dt-bindings: vendor-prefixes: Add AYN Technologies Xilin Wu via B4 Relay
2024-04-25 16:13 ` Rob Herring
2024-04-28 3:44 ` Xilin Wu
2024-04-24 15:29 ` [PATCH 09/10] dt-bindings: arm: qcom: Add AYN Odin 2 Xilin Wu via B4 Relay
2024-04-24 15:43 ` Bryan O'Donoghue
2024-04-24 15:29 ` [PATCH 10/10] arm64: dts: " Xilin Wu via B4 Relay
2024-04-25 6:28 ` Krzysztof Kozlowski
2024-04-28 3:54 ` Xilin Wu [this message]
2024-04-30 10:26 ` Konrad Dybcio
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=f80384eb-d7eb-4a68-952d-529230ec51a6@gmail.com \
--to=wuxilin123@gmail.com \
--cc=airlied@gmail.com \
--cc=andersson@kernel.org \
--cc=bigfoot@classfun.cn \
--cc=conor+dt@kernel.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=konrad.dybcio@linaro.org \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pwm@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mollysophia379@gmail.com \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=quic_jesszhan@quicinc.com \
--cc=quic_tengfan@quicinc.com \
--cc=robh@kernel.org \
--cc=sam@ravnborg.org \
--cc=tzimmermann@suse.de \
--cc=u.kleine-koenig@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).