Attention is currently required from: Xiang Wang, Stefan Reinauer, Angel Pons.
Patch set 12:Code-Review +1
2 comments:
Commit Message:
Also state the previous mode here that is currently assumed.
File bitbang_spi.c:
if (parse_spi_mode(&cpol, &cpha))
return ERROR_FLASHROM_BUG;
Hmm, the only thing is this could be a little too harsh for this patch in that, could we just assume the previous behavior when no param is passed? So just set cpol, cpha on the error branch here with values that make the patch a nop for other users and previous use-cases.
The case of changing behavior can be its own patch with its own debate, it would definitely need a man page correction as well.
To view, visit change 49255. To unsubscribe, or for help writing mail filters, visit settings.