The return value test of msr_cmd (fd, MSR_CMD_FWREV) in msr_fwrev is incorrect.
The function aborts on a on a non-zero return (error value should be -1) and
does not read the firmware revision response. Subsequent commands will read
data from this response first, making them unable to read their own responses.
This seems to have been introduced in r52.
Original issue reported on code.google.com by
aili...@gmail.comon 25 Feb 2012 at 2:19