diff options
| author | Ricardo Quesada <ricardoq@google.com> | 2022-02-15 13:48:26 -0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2022-03-17 20:07:22 +0000 |
| commit | aba3906e91a688ea6ce5eea1fd8ffb694eece1be (patch) | |
| tree | 12f0d52f526d4fee88cc9e8f86f38748cd27d047 /baseboard/cherry/baseboard.c | |
| parent | 940026601876cc7a959ed47c04bb53c879cf4448 (diff) | |
| download | chrome-ec-aba3906e91a688ea6ce5eea1fd8ffb694eece1be.tar.gz | |
i2c: Add addr_flags and cmd to board_allow_i2c_passthru()
This CL adds addr_flags and cmd_id (AKA offset) to
board_allow_i2c_passthru().
This CL is a kind-of-cherrypick of this other CL:
https://crrev.com/c/1588492
BRANCH=none
BUG=b:135642493
TEST=make buildall passes
tested in Madoo where I manually added a board_allow_i2c_passthru()
function that was blocking port 5 (battery) and did:
With EC unlocked:
dut # ectool i2cread 8 0x05 22 1
Read from I2C port 5 at 0x16 offset 0x1 = 0x67
With EC locked:
dut # ectool i2cread 8 0x05 22 1
EC result 4 (ACCESS_DENIED)
Change-Id: Ifba59f21f63f99345b81dd250296214a5cd29eba
Signed-off-by: Ricardo Quesada <ricardoq@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3466422
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Tested-by: Ricardo Quesada <ricardoq@chromium.org>
Auto-Submit: Ricardo Quesada <ricardoq@chromium.org>
Commit-Queue: Ricardo Quesada <ricardoq@chromium.org>
Diffstat (limited to 'baseboard/cherry/baseboard.c')
| -rw-r--r-- | baseboard/cherry/baseboard.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/baseboard/cherry/baseboard.c b/baseboard/cherry/baseboard.c index 4e6b2e324d..d7ab72565a 100644 --- a/baseboard/cherry/baseboard.c +++ b/baseboard/cherry/baseboard.c @@ -346,9 +346,9 @@ const struct i2c_port_t i2c_ports[] = { }; const unsigned int i2c_ports_used = ARRAY_SIZE(i2c_ports); -int board_allow_i2c_passthru(int port) +int board_allow_i2c_passthru(const struct i2c_cmd_desc_t *cmd_desc) { - return (port == I2C_PORT_VIRTUAL_BATTERY); + return (cmd_desc->port == I2C_PORT_VIRTUAL_BATTERY); } /* TCPC */ |