8 comments:

Commit Message:
- Patch Set #10, Line 7: Adds
  Add
- Patch Set #10, Line 9: Adds
  Add
- Patch Set #10, Line 9: documentation of it
  its documentation
File raiden_debug_spi.c:
- Patch Set #10, Line 450:
```
#define PAYLOAD_SIZE_V2_START           (58)

#define PAYLOAD_SIZE_V2_RESPONSE        (60)

#define PAYLOAD_SIZE_V2_CONTINUE        (60)

#define PAYLOAD_SIZE_V2_ERROR           (60)
```
  This is V2, maybe put it next to the other V2 definitions?
- Patch Set #10, Line 875: " protocol = %u\n"
  This isn't aligned with the other definitions
- Patch Set #10, Line 930:
  Double empty line
- Patch Set #10, Line 950: unsigned int
  I'd rather declare loop variables outside of the loop body for compatibility purposes. See CB:38034 for the reasons.
- Patch Set #10, Line 1112:
  Two spaces

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