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

Add Lenovo Yoga Tab 3 10 LTE support #434

Merged
merged 2 commits into from
Oct 17, 2024
Merged

Conversation

exkc
Copy link
Contributor

@exkc exkc commented Oct 17, 2024

aa


&lk2nd {
lxf_p5001_b01 {
model = "Lenovo Yoga Tab 3 10 LTE";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

i proby should name it as lxf_p5001.

apparently there are other Lenovo Yoga Tab 3 10 (wifi/lte) are called lxf_p5001 but different sufixes like b01,j01,etc

};

&lk2nd {
lxf_p5001_b01 {
Copy link
Member

Choose a reason for hiding this comment

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

I'd prefer to conform to linux code style for dts in which underscores are not used in names, you can drop the b01 suffix if you think that's more correct

Suggested change
lxf_p5001_b01 {
lxf-p5001-b01 {

lk2nd/device/dts/msm8909/msm8909-qrd-skub.dts Outdated Show resolved Hide resolved
lk2nd/device/dts/msm8909/rules.mk Outdated Show resolved Hide resolved
lk2nd/device/dts/msm8909/msm8909-qrd-skub.dts Outdated Show resolved Hide resolved
@exkc exkc force-pushed the yt3-x50y branch 2 times, most recently from 16f1ca8 to 2044140 Compare October 17, 2024 16:02
};

&lk2nd {
// The motherboard of it has QR code sticker has a code name printed on it.All Lenovo Yoga Tab 3 10 LTE/WiFi varian code name appear to have lxf-p5100 prefix and a lot of different suffix. this device is particular known for having b1 suffix and yt3-x50y model number.
Copy link
Member

Choose a reason for hiding this comment

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

I'd appreciate if this comment were to be C style (/**/) and folded at 60-80. This would make it easier to read

Copy link
Contributor Author

@exkc exkc Oct 17, 2024

Choose a reason for hiding this comment

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

i removed it bc there are a issue : I don't know how do we gonna match different model/varian.

Copy link
Contributor Author

@exkc exkc Oct 17, 2024

Choose a reason for hiding this comment

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

I mean what if a different model/varian's bootloader select the same tree then how we going to differentiate them. Also Lenovo probably used same panel on Lenovo Yoga Tab 3 10.

the only thing I know so far at least we can differentiate the Wi-Fi vaint and LTE vaint . those vaint proby not goona selecting the same tree

@TravMurav TravMurav merged commit 9b84939 into msm8916-mainline:main Oct 17, 2024
42 checks passed
@TravMurav
Copy link
Member

Thanks!

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.

2 participants