obsidiansystems / ledger-app-nervos

MIT License
10 stars 10 forks source link

Use flow UI for nano S. #125

Closed jonored closed 4 years ago

jonored commented 4 years ago

This does not work as we thoroughly run out of ram when signing, due to the 232 byte difference in global structs for flow UI vs. og legacy UI.

jhartzell42 commented 4 years ago

This works on tests on hardware and Speculos. Concerns remain about this line: https://github.com/obsidiansystems/ledger-app-nervos/blob/develop/src/molecule_reader_streaming.h#L225