ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

hw/misc/auxbus: Fix MOT/classic I2C mode 

Since its introduction in commit 6fc7f77fd2 i2c_start_transfer()
uses incorrectly the direction of the transfer (the last argument
is called 'is_recv'). Fix by inverting the argument, we now have
is_recv = !is_write.

Fixes: 6fc7f77fd2 ("introduce aux-bus")
Reported-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Acked-by: Corey Minyard <cminyard@mvista.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
This commit is contained in:
Philippe Mathieu-Daudé 2021-06-17 13:53:26 +02:00 committed by Corey Minyard
parent f8ffea755c
commit eb8377386d
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
hw/misc/auxbus.c
View File

@ -139,7 +139,7 @@ AUXReply aux_request(AUXBus *bus, AUXCommand cmd, uint32_t address,
i2c_end_transfer(i2c_bus); i2c_end_transfer(i2c_bus);
} }
if (i2c_start_transfer(i2c_bus, address, is_write)) { if (i2c_start_transfer(i2c_bus, address, !is_write)) {
ret = AUX_I2C_NACK; ret = AUX_I2C_NACK;
break; break;
} }
@ -170,7 +170,7 @@ AUXReply aux_request(AUXBus *bus, AUXCommand cmd, uint32_t address,
/* /*
* No transactions started.. * No transactions started..
*/ */
if (i2c_start_transfer(i2c_bus, address, is_write)) { if (i2c_start_transfer(i2c_bus, address, !is_write)) {
break; break;
} }
} else if ((address != bus->last_i2c_address) || } else if ((address != bus->last_i2c_address) ||
@ -179,7 +179,7 @@ AUXReply aux_request(AUXBus *bus, AUXCommand cmd, uint32_t address,
* Transaction started but we need to restart.. * Transaction started but we need to restart..
*/ */
i2c_end_transfer(i2c_bus); i2c_end_transfer(i2c_bus);
if (i2c_start_transfer(i2c_bus, address, is_write)) { if (i2c_start_transfer(i2c_bus, address, !is_write)) {
break; break;
} }
} }