Attention is currently required from: Nico Huber, Edward O'Callaghan, Angel Pons, Anastasia Klimchuk, Sergii Dmytruk.

View Change

2 comments:

• File libflashrom.h:

  • Patch Set #11, Line 138: #define FLASHROM_WP_OUT_STR_SIZE 1024

    Why 1024? Same question for 128 above. […]

    Done

• File writeprotect.c:

  • Patch Set #11, Line 436:

    for (int i = cfg->srp_bit_count - 1; i >= 0; i--) {
    		buf += sprintf(buf, "SRP%d=%u ", i, cfg->srp[i]);
    	}
    
    	if (cfg->cmp_bit_present)
    		buf += sprintf(buf, "CMP=%u ", cfg->cmp);
    
    	if (cfg->sec_bit_present)
    		buf += sprintf(buf, "SEC=%u ", cfg->sec);
    
    	if (cfg->tb_bit_present)
    		buf += sprintf(buf, "TB=%u ",  cfg->tb);
    
    	for (int i = cfg->bp_bit_count - 1; i >= 0; i--) {
    		buf += sprintf(buf, "BP%d=%u ", i, cfg->bp[i]);
    	}
    
    

    I would prefer the same ordering as in wp_chip_struct defined in CB:58478

    This ordering is actually better and more consistent with the way that configs are ordered by the get_available_ranges function. I've changed the ordering in the struct to match.

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