Attention is currently required from: Lean Sheng Tan.

View Change

6 comments:

• Commit Message:

  • Patch Set #1, Line 2: Tan, Lean Sheng

    Please use *Lean Sheng Tan*.

  • Patch Set #1, Line 11: low-DMIPS computing

    What does that mean?

  • Patch Set #1, Line 15: add

    adds

  • Patch Set #1, Line 15: This patch add initial PSE loading support for EHL.

    Please describe the coreboot integration in detail. It looks like it’s implemented by loading a blob. What does the blob do? Where will the blob be published? Why is a blob needed at all?

  • Patch Set #1, Line 16:

    1. Tested how?
    2. Please add a reference to the specification.

• File src/soc/intel/elkhartlake/Kconfig:

  • Patch Set #1, Line 200: PSE

    Please spell it out once in the help text.

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