Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

arm64: dts: qcom: msm8939-huawei-rio: Add initial device tree #385

Open
wants to merge 1 commit into
base: msm8916/6.12.1
Choose a base branch
from

Conversation

bzy-080408
Copy link

@bzy-080408 bzy-080408 commented Dec 27, 2024

Not fully working yet.
Tested peripherals:

  • USB network(need kernel support)
  • USB console(need kernel support)
  • WIFI
  • display
  • touchscreen
  • button
  • GPU
    Untested peripherals:
  • USB OTG
    TODO:
  • most of sensor
  • camera
  • USB charging
  • Audio

@bzy-080408 bzy-080408 force-pushed the msm8916/6.12.1 branch 3 times, most recently from c632d86 to 18cb6d5 Compare December 27, 2024 11:20
@@ -0,0 +1,328 @@
// SPDX-License-Identifier: GPL-2.0-only
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit message: the prefix is wrong, I'd also appreciate if you could fill in the message like it's done for other device introduction commits.

// SPDX-License-Identifier: GPL-2.0-only

/*
*Copyright (c) 2024 Zhiyuan Bai<[email protected]>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit

Suggested change
*Copyright (c) 2024 Zhiyuan Bai<[email protected]>
* Copyright (c) 2024 Zhiyuan Bai <[email protected]>

Comment on lines +132 to +133
};
&blsp_i2c5 {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit

Suggested change
};
&blsp_i2c5 {
};
&blsp_i2c5 {

Comment on lines +137 to +139
compatible = "cypress,tt21000";

reg = <0x1c>;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
compatible = "cypress,tt21000";
reg = <0x1c>;
compatible = "cypress,tt21000";
reg = <0x1c>;

Comment on lines +168 to +202
&mdss {
status = "okay";
};

&mdss_dsi0 {
pinctrl-0 = <&mdss_default>;
pinctrl-1 = <&mdss_sleep>;
pinctrl-names = "default", "sleep";
status = "okay";

panel@0 {
compatible = "huawei,rio-panel";
reg = <0>;

vsp-supply = <&reg_lcd_pos>;
vsn-supply = <&reg_lcd_neg>;
reset-gpios = <&tlmm 25 GPIO_ACTIVE_LOW>;

port {
panel_in: endpoint {
remote-endpoint = <&mdss_dsi0_out>;
};
};
};
};

&mdss_dsi0_out {
data-lanes = <0 1 2 3>;
remote-endpoint = <&panel_in>;
};

&mdss_dsi0_phy {
qcom,dsi-phy-regulator-ldo-mode;
status = "okay";
};
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is optional but if you split up display into a separate commit (i.e. like 3e3d375) then you can upstream most of the other changes already, which I'd greatly appreciate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants