-
Notifications
You must be signed in to change notification settings - Fork 336
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: msm8926: add Support for Samsung Galaxy S5 Mini Duos (kmini3g) #279
Conversation
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.
Thanks for this PR, here are some comments.
@@ -119,6 +119,7 @@ page on the EFIDroid wiki for an exact mapping of LK targets to SoCs. | |||
- Motorola Moto G 2013 - falcon | |||
- Samsung Galaxy Grand 2 - SM-G7102 | |||
- Samsung Galaxy Tab 4 10.1 (2014) - SM-T530, SM-T535 | |||
- Samsung Galaxy S5 Mini Duos - SM-G800H |
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 believe S goes before T in the alphabet :)
buttons working fine |
i am having error (MODEL- unknown (FIXME!)) and does not recognize in my computer |
dts/msm8226/rules.mk
Outdated
@@ -10,4 +10,5 @@ DTBS += \ | |||
$(LOCAL_DIR)/msm8226-samsung-ms013g.dtb \ | |||
$(LOCAL_DIR)/msm8926-huawei-g6-l11-vb.dtb \ | |||
$(LOCAL_DIR)/msm8926-samsung-r02.dtb \ | |||
$(LOCAL_DIR)/msm8926-v2-720p-mtp.dtb | |||
$(LOCAL_DIR)/msm8926-v2-720p-mtp.dtb \ | |||
$(LOCAL_DIR)/msm8926-samsung-r04.dtb |
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.
How about msm8228
instead?
https://desktop.firmware.mobi/device:346
dts/msm8226/msm8926-samsung-r04.dts
Outdated
|
||
/ { | ||
// This is used by the bootloader to find the correct DTB | ||
qcom,msm-id = <0x6E08FF01 4 0x20000>; |
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.
Where do you know about 0x6E
(110)? It's neither msm8226, 8926, 8228 or 8928
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 renamed dts
dts/msm8226/msm8228-samsung-r03.dts
Outdated
qcom,msm-id = <221 0>; | ||
qcom,board-id = <0xdf08ff01 3>; |
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 guess older SoC/bootloader doesn't seem to support qcom,board-id
@@ -10,4 +10,5 @@ DTBS += \ | |||
$(LOCAL_DIR)/msm8226-samsung-ms013g.dtb \ | |||
$(LOCAL_DIR)/msm8926-huawei-g6-l11-vb.dtb \ | |||
$(LOCAL_DIR)/msm8926-samsung-r02.dtb \ | |||
$(LOCAL_DIR)/msm8926-v2-720p-mtp.dtb | |||
$(LOCAL_DIR)/msm8926-v2-720p-mtp.dtb \ | |||
$(LOCAL_DIR)/msm8228-samsung-r03.dtb |
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.
...
$(LOCAL_DIR)/msm8226-*.dtb \
$(LOCAL_DIR)/msm8228-*.dtb \
$(LOCAL_DIR)/msm8926-*.dtb \
...
this removes the home button support from dts
this adds support for Samsung Galaxy S5 Mini Duos (kmini3g)