diff options
author | Arthur Heymans <arthur@aheymans.xyz> | 2017-08-04 14:28:50 +0200 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-08-20 20:45:00 +0000 |
commit | 1b04aa25917e4bad63f1befa378660f01af1b811 (patch) | |
tree | 8948c7862fa1aa0b8047d2382b53f7318272eb64 /payloads/external | |
parent | 24798a1544a5fa46baca2f7d207fdfbf60517c31 (diff) | |
download | coreboot-1b04aa25917e4bad63f1befa378660f01af1b811.tar.xz |
sb/intel/common: Fix SMBus block commands
Clear LAST_BYTE flag at beginning of block commands.
For reads, slave device determines in the message format how
many bytes it has to transfer out, host firmware only dictates
the maximum buffer length. Return SMBUS_ERROR if only
partial message was received.
For writes, return SMBUS_ERROR if length > 32.
For writes, fix off-by-one error reading memory one
byte past the buffer end.
Change-Id: If9e5d24255a0a03039b52d2863bc278d0eefc311
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/20879
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'payloads/external')
0 files changed, 0 insertions, 0 deletions