Attention is currently required from: Nico Huber, Rizwan Qureshi, Angel Pons.

View Change

1 comment:

• Patchset:

  • Patch Set #12:

    Sorry I think I didn't write well. We are saying the same thing.

    Saying it's hard to predict the result of future work.
    Exactly this!

    As for the enum, sorry I meant to say enum values. That is to say, a enum value that represents multiple chipsets that are the same and folding up the individual 500 and 600 together as an example.

    Also like you said, I wanted to be consistent with the current approach which is what I meant by not being half-baked that is to say, doing something new but only for subset of chipsets. For example https://review.coreboot.org/c/flashrom/+/62580 could be needed and probably some other consistency cleanups? Subrata and I plan to discuss to have a audit over things to look for incomplete cases.

    Resolved?

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