Re: [PATCH v2 3/3] net/ncsi: add dummy response handler for Intel boards
From: Joel Stanley <joel@jms.id.au>
Date: 2021-07-12 10:04:05
Also in:
lkml, openbmc
On Thu, 8 Jul 2021 at 12:28, Ivan Mikhaylov [off-list ref] wrote:
Add the dummy response handler for Intel boards to prevent incorrect handling of OEM commands.
What do you mean? Is this to handle the response to the link up OEM command? If so, include it in the same patch. Can you check that the response is to the link up command and print a warning if not?
quoted hunk ↗ jump to hunk
Signed-off-by: Ivan Mikhaylov <redacted> --- net/ncsi/ncsi-rsp.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-)diff --git a/net/ncsi/ncsi-rsp.c b/net/ncsi/ncsi-rsp.c index 04bc50be5c01..d48374894817 100644 --- a/net/ncsi/ncsi-rsp.c +++ b/net/ncsi/ncsi-rsp.c@@ -699,12 +699,19 @@ static int ncsi_rsp_handler_oem_bcm(struct ncsi_request *nr) return 0; } +/* Response handler for Intel card */ +static int ncsi_rsp_handler_oem_intel(struct ncsi_request *nr) +{ + return 0; +} + static struct ncsi_rsp_oem_handler { unsigned int mfr_id; int (*handler)(struct ncsi_request *nr); } ncsi_rsp_oem_handlers[] = { { NCSI_OEM_MFR_MLX_ID, ncsi_rsp_handler_oem_mlx }, - { NCSI_OEM_MFR_BCM_ID, ncsi_rsp_handler_oem_bcm } + { NCSI_OEM_MFR_BCM_ID, ncsi_rsp_handler_oem_bcm }, + { NCSI_OEM_MFR_INTEL_ID, ncsi_rsp_handler_oem_intel } }; /* Response handler for OEM command */ --2.31.1