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

sm8150: Add initial support for Oneplus 7T (hotdogb) #352

Merged
merged 3 commits into from
Oct 3, 2023

Conversation

IonAgorria
Copy link
Contributor

Description

Initial Oneplus 7T support based on existing 7 (guacamoleb) and 7 Pro (guacamole), currently the following has been tested as functional:

  • Simpleinit and EFI shell
  • Volume up/down and power buttons
  • Linux mass storage mode

Also renamed dtb for guacamoleb for consistency since apparently hotdogb one was used

Checklist

  • Have you tested the change you are submitting?
  • Is the commits history nice and clean?

@qaz6750
Copy link
Contributor

qaz6750 commented Oct 1, 2023

It looks good, but I would like to ask if renaming DTB like this would cause some confusion, or should we name it Universal? Xd

@IonAgorria
Copy link
Contributor Author

Mainline seems to use hotdogb as .dts name hence the renaming:
https://gitlab.com/sm8150-mainline/linux/-/blob/sm8150/5.17/arch/arm64/boot/dts/qcom/sm8150-oneplus-hotdogb.dts

And seems to be used for 7, 7T and 7 Pro... maybe we can remove guacamole too and let it use hotdogb.dtb but I don't have it to test

@IonAgorria IonAgorria changed the title Hotdogb sm8150: Add initial support for Oneplus 7T (hotdogb) Oct 2, 2023
Old dtb for old bootloader doesn't work with new version
@BigfootACA BigfootACA merged commit 581cc07 into edk2-porting:master Oct 3, 2023
22 checks passed
@0ffthemap
Copy link

Will there be a release with this commit coming?
@BigfootACA
@IonAgorria

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants