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

Add Pixel 3a (sargo) #38

Merged
merged 7 commits into from
Nov 8, 2020
Merged

Add Pixel 3a (sargo) #38

merged 7 commits into from
Nov 8, 2020

Conversation

fredldotme
Copy link
Contributor

This PR includes new device support for the Google Pixel 3a.

Full device support is dependent on changes in the rootfs, so keep this PR as a draft until those issues have been solved.

@fredldotme fredldotme marked this pull request as ready for review October 27, 2020 08:51
@fredldotme
Copy link
Contributor Author

As all the necessary rootfs changes are in now I consider this ready for review!

This requires a UBports installer with fastboot:set_active functionality, so make sure to only merge it when such a version is released.

@NeoTheThird
Copy link
Member

fastboot:set_active will be available beginning in UBports Installer version 0.5.7-beta

Sargo exposes weird behavior when flashing onto the 'a' slot.
After installation it automatically reboots into the recovery again,
causing the boot counter to decrement with every reboot due to the
phone not rebooting into a working rootfs anymore. So, the test device
wins now, set the active slot to 'b' as that's the only slot it
successfully boots from.
Copy link
Member

@NeoTheThird NeoTheThird left a comment

Choose a reason for hiding this comment

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

LGTM! With UBports Installer 0.6.0-beta released, i'm ok if you are @fredldotme. Good job!

@NeoTheThird NeoTheThird merged commit 9b91ac8 into master Nov 8, 2020
@NeoTheThird NeoTheThird deleted the sargo branch November 8, 2020 13:59
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.

3 participants