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

dts: msm8916-samsung: add j5xnltegn #444

Closed

Conversation

V6lhost
Copy link

@V6lhost V6lhost commented Nov 14, 2024

Added support for J510GN. Almost identical to the J510F.

J5 2016 (J510GN)
@V6lhost V6lhost changed the title J510GN Support dts: msm8916-samsung: add j5xnltegn Nov 14, 2024
lk2nd,dtb-files = "msm8916-samsung-j5x";

qcom,msm-id = <QCOM_ID_MSM8916 0>;
qcom,board-id = <0xCE08FF01 4>;
Copy link
Member

Choose a reason for hiding this comment

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

This file is sorted by revision, I'd appreciate if you move this node into "rev 4"

Also I'd appreciate if you change the commit message similarly to others changing this file

Copy link
Author

Choose a reason for hiding this comment

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

Okay. Sorry, It's my first time.

@V6lhost V6lhost closed this Nov 15, 2024
@V6lhost V6lhost deleted the V6lhost-patch-1-J510GN-support branch November 15, 2024 15:22
@V6lhost V6lhost restored the V6lhost-patch-1-J510GN-support branch November 15, 2024 15:28
Copy link
Contributor

Choose a reason for hiding this comment

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

reword the commit to dts: msm8916-samsung: add j5xnltedx

@@ -850,4 +850,22 @@
i2c-scl-gpios = <&tlmm 3 I2C_GPIO_FLAGS>;
};
};

j5xnltegn {

Choose a reason for hiding this comment

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

Suggested change
j5xnltegn {
j5xnltedx {

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.

3 participants