Skip to content

Add Samsung Galaxy Tab S2 8.0 (2016) support#709

Open
jdkjkjj wants to merge 5 commits into
msm8916-mainline:mainfrom
jdkjkjj:main
Open

Add Samsung Galaxy Tab S2 8.0 (2016) support#709
jdkjkjj wants to merge 5 commits into
msm8916-mainline:mainfrom
jdkjkjj:main

Conversation

@jdkjkjj

@jdkjkjj jdkjkjj commented Jun 7, 2026

Copy link
Copy Markdown

I added the dts

Comment thread Documentation/devices.md
- Samsung Galaxy Tab A 8.0 (2015) - SM-T350, SM-T355, SM-T355Y, SM-T357W
- Samsung Galaxy Tab A 9.7 (2015) - SM-T550, SM-T555
- Samsung Galaxy Tab E 9.6 WiFi (2015) - SM-T560NU
- Samsung Galaxy Tab S2 8.0 WiFi (2016) - SM-T713

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not here. Move this to lk2nd-msm8952.

Comment on lines +7 to +8
qcom,msm-id = <QCOM_ID_MSM8976 0x10001>;
qcom,board-id = <QCOM_BOARD_ID_QRD 0x0>;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Currently duplicated with msm8976-qrd. You might want to edit msm8976-qrd instead.

compatible = "samsung,gts28vewifi";

lk2nd,match-bootloader = "*T713*";
lk2nd,match-panel;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You don't need this when there is lk2nd,match-bootloader.

Comment on lines +51 to +57
panel {
compatible = "samsung,gts28vewifi-panel", "lk2nd,panel";

ss_dsi_panel_S6E3HA2_AMS801HK01_QXGA {
compatible = "samsung,gts28vewifi-s6e3ha2";
};
};

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ditto. Or are there more panel variants?

$(LOCAL_DIR)/qm215-mtp.dtb \

QCDTBS += \
$(LOCAL_DIR)/msm8976-gts28vewifi.dtb \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Strange that this DTB is being added to ADTBS as well as QCDTBS. I think it's worth documenting why this is necessary as a comment here.

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