[coreboot-gerrit] [S] Change in coreboot[master]: device/mdio: Provide helper functions for read and write

[Arthur Heymans (Code Review)]

Attention is currently required from: Mario Scheithauer, Angel Pons, Werner Zeh.

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69611 )

Change subject: device/mdio: Provide helper functions for read and write
......................................................................

Patch Set 5: Code-Review+2

(1 comment)

File src/device/mdio.c:

https://review.coreboot.org/c/coreboot/+/69611/comment/53ffa39f_a5d6b70d 
PS4, Line 27: 0
> Yes, the error log was the one I had in mind for debug purposes. Is it OK to fully rely on that?

Yes. C is sadly not like golang where you can easily return tuples of error and values :-). The writes should fail for the same reason so things are then just a noop with error messages