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

Huawei Mediapad T3 10: new device #454

Merged
merged 1 commit into from
Nov 24, 2024
Merged

Conversation

NekoCWD
Copy link
Contributor

@NekoCWD NekoCWD commented Nov 24, 2024

No description provided.

lk2nd/device/dts/msm8952/msm8917-huawei-agassi.dts Outdated Show resolved Hide resolved
Comment on lines 9 to 17
/*
* This device is quirky. It excluded from generic build.
* To run lk2nd on this device you need to flash custom bootloader and unlock it. Read more https://wiki.postmarketos.org/wiki/Huawei_Mediapad_T3_10_(huawei-agassi)#Bootloader_unlocking.
*
* Compile lk2nd with the following flag to run on this device
* LK2ND_ADTBS="msm8917-huawei-agassi.dtb"
*/
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 the comment to be folded at 80-100 where possible so it's a bit easier to read

model = "Huawei MediaPad T3 10 (agassi)";
compatible = "huawei,agassi";
lk2nd,dtb-files = "msm8917-huawei-agassi";
lk2nd,match-cmdline = "*androidboot.bootloader.version=P2200_DEFAULT_BOOTLOADER_170523*";
Copy link
Member

Choose a reason for hiding this comment

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

I'm actually not sure if this is useful, perhaps could drop this and use LK2ND_COMPATIBLE="huawei,agassi" at build time additionally? This would force lk2nd to always find this node

@@ -16,3 +16,6 @@ ADTBS += \
$(LOCAL_DIR)/msm8940-xiaomi-santoni.dtb \
$(LOCAL_DIR)/msm8956-mtp.dtb \
$(LOCAL_DIR)/msm8976-qrd.dtb \

DTBS += \
$(LOCAL_DIR)/msm8917-huawei-agassi.dtb
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
$(LOCAL_DIR)/msm8917-huawei-agassi.dtb
$(LOCAL_DIR)/msm8917-huawei-agassi.dtb \

let's always escape newlines so if we need to add anything it would not change old lines

@TravMurav TravMurav merged commit 7da4fad into msm8916-mainline:main Nov 24, 2024
14 checks passed
@TravMurav
Copy link
Member

Thanks!


/ {
qcom,msm-id = <QCOM_ID_MSM8917 0>;
qcom,board-id = <QCOM_BOARD_ID(QRD, 1, 0) 0>;
Copy link
Member

Choose a reason for hiding this comment

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

I think here the file name doesn't really matter since this is a dtb that will only be manually included for that one device. We can rename it later if we have more here


/ {
qcom,msm-id = <QCOM_ID_MSM8917 0>;
qcom,board-id = <QCOM_BOARD_ID(QRD, 1, 0) 0>;

Choose a reason for hiding this comment

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

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