-
Notifications
You must be signed in to change notification settings - Fork 362
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: apq8909-1gb-qrd-skue: add leoven yoga tablet 10 WIFI #491
base: main
Are you sure you want to change the base?
Conversation
missclick i will test it later |
@@ -4,7 +4,12 @@ | |||
#include <lk2nd.dtsi> | |||
|
|||
/ { | |||
qcom,msm-id = <QCOM_ID_MSM8909 0>; | |||
//some variant of lxf_p5100 bootloder is quirky and they only select | |||
//<QCOM_ID_MSM8909 0x00 0x102 0x00 0x109 0x00 0x113 0x00> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As already suggested, please decode those constants to appropriate QCOM_ID and split to two-cell values. Don't think you need to repeat the array twice. I'd also actually appreciate if you can isolate your device to one specific msm-id, not add all of them (i.e. to know if it matches 3g-only 8209 or wifi only 8009)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hell yeah the 8 in one has the same issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i am thinking about making a dedicated dts for them
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No description provided.