View Change

6 comments:

• Commit Message:

  • Patch Set #1, Line 7: fixes on VARIABLE_SIZE chip support

    dummyflasher.c: Fix null par data and size param handling

  • Patch Set #1, Line 10: (https://review.coreboot.org/c/flashrom/+/44879

    Could you use the commit hashes instead of the links like

     `3149822cd45cb2e5841e15d648783748ba1b2ec6` &&
     `3b8fe0f8e907c0ba9f7c7935e950f3e1538d427f`

  • Patch Set #1, Line 13:

    - dummy_init() used to fail for the absence of "size" argument, which broke
    the run on non VARIABLE_SIZE emulation target. Now the run fails
    without "size" only for VARIABLE_SIZE.
    

    Format these with an indent `*`.

     * dummy_init() should not fail in the absence of the size param as this breaks non-VARIABLE_SIZE emulation functionality.

  • Patch Set #1, Line 16:

    - probe_variable_size() used to fail if the bus type is either PARALLEL, LPC or FWH
    because flash->mst->par.data is null. Now dummy_init() sets up par_master_dummy.data
    

    • probe_variable_size() should fail if bus type is 'PARALLEL, LPC or FWH' however 'flash->mst->par.data' is null. Fix init fn to initialise it.

  • Patch Set #1, Line 20:

    - man page is revised to describe how to use VARIABLE_SIZE emulation
    target.
    

    • Revise man page to describe how to use the VARIABLE_SIZE emulation target.

• File flashrom.8.tmpl:

  • Patch Set #1, Line 658: , which is a power of two.

    as a power of two.

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