Attention is currently required from: Patrick Rudolph.

Patch set 1:Code-Review +2

View Change

1 comment:

• File linux_mtd.c:

  • Patch Set #1, Line 464: extract_programmer_param("name");

    I thought it's already mutually exclusive as extract_programmer_param("name"); is in the else branch.
    I did not reuse the existing dev as a partition label might contain digits only. In that case you'd not know if the number is the MTD device or the label.

    I see, thanks for explaining.

    It's mutually exclusive at runtime but it won't complain if you provide both "name" and "dev". We don't check for other meaningless programmer params so that's fine.

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