Attention is currently required from: Edward O'Callaghan, Angel Pons, Nikolai Artemiev, Sergii Dmytruk.

View Change

1 comment:

• Patchset:

  • Patch Set #22:

    I first thought the API is missing a function to create a new `config` […]

    Thinking further about this, there are more reasons to tie
    flash context and wp config together. The current internal
    representation is actually only valid for a specific chip
    model (it's storing the exact bits found on the chip; same
    bit values could have a different meaning for another chip).

    Completely different idea: we could pass an abstract (decoded)
    config around instead of the current bit representation. Just
    like one would deserialize data from a byte stream and use
    a better, program-specific representation and only serialize
    it again for writing.

To view, visit change 58479. To unsubscribe, or for help writing mail filters, visit settings.