diff options
| author | Andrew Jeffery <andrew@aj.id.au> | 2019-02-21 16:58:12 +1030 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.ibm.com> | 2019-02-24 17:43:37 -0600 |
| commit | 6d8bd2dd9eab3792d3aafde33d5573c0276b72b8 (patch) | |
| tree | 16c54d0c110fec6c821370f03b0b555ef7b0b347 | |
| parent | 403d527269bed128a5a114edc2a8e040fc28fc78 (diff) | |
| download | skiboot-6d8bd2dd9eab3792d3aafde33d5573c0276b72b8.zip skiboot-6d8bd2dd9eab3792d3aafde33d5573c0276b72b8.tar.gz skiboot-6d8bd2dd9eab3792d3aafde33d5573c0276b72b8.tar.bz2 | |
libflash/ipmi-hiomap: Remove unused close handling
Issuing a HIOMAP_C_CLOSE is not required by the protocol specification,
rather a close can be implicit in a subsequent
CREATE_{READ,WRITE}_WINDOW request. The implicit close provides an
opportunity to reduce LPC traffic and the implementation takes up that
optimisation, so remove the case from the IPMI callback handler.
Cc: stable
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
| -rw-r--r-- | libflash/ipmi-hiomap.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/libflash/ipmi-hiomap.c b/libflash/ipmi-hiomap.c index fdd12d5..7a36449 100644 --- a/libflash/ipmi-hiomap.c +++ b/libflash/ipmi-hiomap.c @@ -204,11 +204,6 @@ static void ipmi_hiomap_cmd_cb(struct ipmi_msg *msg) break; } - case HIOMAP_C_CLOSE_WINDOW: - lock(&ctx->lock); - ctx->window_state = closed_window; - unlock(&ctx->lock); - break; case HIOMAP_C_MARK_DIRTY: case HIOMAP_C_FLUSH: case HIOMAP_C_ACK: |