dts: msm8916: Add support for CMCC A1 (M623C) (Wingtech WT86621) - #725
dts: msm8916: Add support for CMCC A1 (M623C) (Wingtech WT86621)#725affggh wants to merge 1 commit into
Conversation
affggh
commented
Jun 24, 2026
It worked for me!
| $(LOCAL_DIR)/msm8939-qrd-skuk.dtb \ | ||
| $(LOCAL_DIR)/msm8939-qrd-wt82918.dtb \ | ||
| $(LOCAL_DIR)/msm8939-qrd-wt82918hd.dtb \ | ||
| $(LOCAL_DIR)/msm8916-qrd-wt86621.dtb \ |
There was a problem hiding this comment.
This is not sorted alphabetically
There was a problem hiding this comment.
I noticed it named msm8916-qrd-skul in decompiled device tree blob, so I rename it.
There was a problem hiding this comment.
I'm a bit confused since i don't see sku L in the caf msm kernel from a quick look, vendors often do weird mess with the naming (i.e. sometimes they just modify existing file, or sometimes they copy them and leave stuff unchanged. Could you share the lines you've used to figure that out?
In any case, if this is a generic dtb, you'd have to find a match property, like in other generic dtbs then, as it'd imply some other oems could've easily copied it the same way...
There was a problem hiding this comment.
Sorry, It seems only wingtech open source kernel have this dts. And seems only this device use, I will change it back to wt86621.
There was a problem hiding this comment.
Would you mind sharing a link to that kernel?
There was a problem hiding this comment.
It seems from Xiaomi, and another lenovo opened source have fully skul.dts and dtsi.
They are all from wingtech, so I guess I better rename it back to wt86621.dts
There was a problem hiding this comment.
the second source seems to have "skux" use the id=16, so it's probably a mess... Let's just name it as it's model for now but let's try adding device subnode and a match property so it's easier to reconcile multiple devices on this id later, if the need arises
Add document and format dts