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

[experimental-tmp2] lk2nd: dts: msm8994: include xiaomi-libra dts #253

Open
wants to merge 3 commits into
base: experimental-tmp2
Choose a base branch
from

Conversation

fekz115
Copy link

@fekz115 fekz115 commented May 9, 2023

No panels for now, because they won't be used for a very long time

stephan-gh and others added 3 commits April 16, 2023 17:20
The msm8916 samsung.dts results in a lot of QCDT entries (records)
because each qcom,msm-id is combined with each qcom,board-id. However,
each board-id is actually only valid for one particular msm-id,
so we can save quite some space by cleaning them up here.

This is a bit of a hack but since this is a custom copy of dtbTool
anyway it's not much of a problem. Also, reverting this (or using the
standard dtbTool) would not result in a functional difference, just
a larger build result.
@fekz115 fekz115 changed the title lk2nd: dts: msm8994: include xiaomi-libra dts [experimental-tmp2] lk2nd: dts: msm8994: include xiaomi-libra dts May 9, 2023
libra {
model = "Xiaomi Mi 4C";
compatible = "xiaomi,libra";

Copy link

@pevik pevik May 26, 2023

Choose a reason for hiding this comment

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

@fekz115 Here is whitespace (2 tabs), please remove it).

};

&lk2nd {
libra {
Copy link

Choose a reason for hiding this comment

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

I also wonder if there needs to be libra nod.

compatible = "xiaomi,libra";

qcom,board-id = <12 0>;
qcom,pmic-id = <65545 65546 0 0>;
Copy link

Choose a reason for hiding this comment

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

I'd move these to the top level / { nod.

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