diff options
| author | Andrew Jeffery <andrew@aj.id.au> | 2019-02-21 16:58:12 +1030 |
|---|---|---|
| committer | Vasant Hegde <hegdevasant@linux.vnet.ibm.com> | 2019-03-04 23:03:26 +0530 |
| commit | 5117b2c56d65c3b0142a083a306b4cd1c678a953 (patch) | |
| tree | c1d3ddf36b38889f1e1101653ee8875e84581ae9 | |
| parent | 0576b743bf18be8572ac9f05f7e03dcb8857dfa1 (diff) | |
| download | skiboot-5117b2c56d65c3b0142a083a306b4cd1c678a953.zip skiboot-5117b2c56d65c3b0142a083a306b4cd1c678a953.tar.gz skiboot-5117b2c56d65c3b0142a083a306b4cd1c678a953.tar.bz2 | |
libflash/ipmi-hiomap: Remove unused close handling
[ Upstream commit 6d8bd2dd9eab3792d3aafde33d5573c0276b72b8 ]
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>
Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.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: |