[coreboot-gerrit] Change in coreboot[master]: mb/google/zork: de-assert eMMC RST for Morphius

[Keith Tzeng (Code Review)]

Keith Tzeng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45756 )

Change subject: mb/google/zork: de-assert eMMC RST for Morphius
......................................................................

Patch Set 1:

(2 comments)

https://review.coreboot.org/c/coreboot/+/45756/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/45756/1//COMMIT_MSG@11 
PS1, Line 11: 169211959
> Did you mean: 169222156?

Should be b:169222156

https://review.coreboot.org/c/coreboot/+/45756/1//COMMIT_MSG@14 
PS1, Line 14: check GPIO86 state is PU
> What is the difference from a functional standpoint? Are you seeing some error go away because of th […]

as on tracker: https://partnerissuetracker.corp.google.com/issues/169222156#comment7
the reset pin keep low on your measurement, eMMC vendor feedback that it need to be high.