diff options
| author | Siddharth Vadapalli <s-vadapalli@ti.com> | 2024-03-06 21:55:58 +0530 |
|---|---|---|
| committer | Praneeth Bajjuri <praneeth@ti.com> | 2024-03-07 11:45:56 -0600 |
| commit | df96f33ceb4890286e8f2bdf9179f9ec62a42b0c (patch) | |
| tree | e4fa25204f4021e83e76bd38baac699587879fe8 | |
| parent | 5e8ede26099d335c192f852b220b7b118c451231 (diff) | |
net: ethernet: ti: cpsw-proxy-client: Drop unused "response_status"
The variable "response_status" in "cpsw_proxy_client_cb()" stores the
status of the response message received from EthFw. However the status
of the response is not checked in the "cpsw_proxy_client_cb()" function.
Rather, it is checked in the "cpsw_proxy_client_send_request()" function
which sends a request to EthFw and processes the response passed to it via
the "cpsw_proxy_client_cb()" callback function. Thus drop this unused
"response_status" variable in the callback function.
Fixes: 9343810f28a1 ("net: ethernet: ti: Add CPSW Proxy Client driver")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
| -rw-r--r-- | drivers/net/ethernet/ti/cpsw-proxy-client.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/ti/cpsw-proxy-client.c b/drivers/net/ethernet/ti/cpsw-proxy-client.c index 11858f955004..090bb61e6d78 100644 --- a/drivers/net/ethernet/ti/cpsw-proxy-client.c +++ b/drivers/net/ethernet/ti/cpsw-proxy-client.c @@ -469,7 +469,7 @@ static int cpsw_proxy_client_cb(struct rpmsg_device *rpdev, void *data, u32 msg_type = msg->msg_hdr.msg_type; u32 notify_type, response_id, token; struct cpsw_virt_port *virt_port; - int response_status, ret = 0; + int ret = 0; switch (msg_type) { case ETHREMOTECFG_MSG_NOTIFY: @@ -519,7 +519,6 @@ static int cpsw_proxy_client_cb(struct rpmsg_device *rpdev, void *data, case ETHREMOTECFG_MSG_RESPONSE: response_msg_hdr = (struct response_message_header *)msg; response_id = response_msg_hdr->response_id; - response_status = response_msg_hdr->response_status; if (response_id == common->request_id - 1) { dev_info(common->dev, "ignoring delayed response for request: %u\n", |