[flashrom-gerrit] Change in flashrom[master]: raiden_debug: Upstream ChromiumOS servo debug board prog

[Stefan Reinauer (Code Review)]

Stefan Reinauer has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/38209 )

Change subject: raiden_debug: Upstream ChromiumOS servo debug board prog
......................................................................

Patch Set 3:

(2 comments)

https://review.coreboot.org/c/flashrom/+/38209/3/raiden_debug_spi.c 
File raiden_debug_spi.c:

https://review.coreboot.org/c/flashrom/+/38209/3/raiden_debug_spi.c@238 
PS3, Line 238: )
(void)

https://review.coreboot.org/c/flashrom/+/38209/3/usb_device.c 
File usb_device.c:

https://review.coreboot.org/c/flashrom/+/38209/3/usb_device.c@86 
PS3, Line 86: void usb_match_value_default
Why are all externally visible functions undocumented (here, not in the .h file), but all static functions have comments?