Skip to content

msm8226 is back again#19

Closed
sub77 wants to merge 1 commit into
efidroid:common/sharedfrom
sub77:patch-1
Closed

msm8226 is back again#19
sub77 wants to merge 1 commit into
efidroid:common/sharedfrom
sub77:patch-1

Conversation

@sub77

@sub77 sub77 commented Mar 29, 2017

Copy link
Copy Markdown

No description provided.

@M1cha

M1cha commented Mar 29, 2017

Copy link
Copy Markdown
Member

this commit doesn't make any sense and the arguments are wrong.
Can you explain what you tried to do here?

@sub77

sub77 commented Mar 30, 2017

Copy link
Copy Markdown
Author

i tried to get my falcon working again since the mmap api changes you comitted in january. without this commit my two msm8226 devices (moto g 2013 and galaxy tab4) stopped booting uefi completly.

@M1cha

M1cha commented Mar 30, 2017

Copy link
Copy Markdown
Member

this may be true but this commit only works by accident. I mean look at it. you're completely ignoring the argument order and you only don't see errors because all arguments are of type int.

@sub77

sub77 commented Mar 30, 2017

Copy link
Copy Markdown
Author

for sure, iam a compiler only, no dev. just had a look at the mmap parameters before the api change and now it is working again.

@z3ntu

z3ntu commented Apr 5, 2017

Copy link
Copy Markdown

this commit doesn't make any sense and the arguments are wrong.

Got uefi booting with #19

lol

@sub77

sub77 commented Apr 5, 2017

Copy link
Copy Markdown
Author

#21

@sub77 sub77 closed this Apr 6, 2017
@sub77

sub77 commented Apr 6, 2017

Copy link
Copy Markdown
Author

@joe2k01

joe2k01 commented Apr 6, 2017

Copy link
Copy Markdown

@z3ntu He gave me the file he used and I referenced the wrong commit

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