Commit 96fafe7c authored by James Smart's avatar James Smart Committed by Martin K. Petersen

scsi: elx: efct: Fix void-pointer-to-enum-cast warning for efc_nport_topology

The kernel test robot flagged an warning for ".../efc_device.c:932:6:
warning: cast to smaller integer type 'enum efc_nport_topology' from 'void
*'"

For the topology events, the "arg" field is generically defined as a void *
and is used to pass different arguments. Most of the arguments are pointers
to data structures. But for the EFC_EVT_NPORT_TOPOLOGY_NOTIFY event, the
argument is an enum value, and the code is typecasting the void * to an
enum generating the warning.

Fix by converting the EFC_EVT_NPORT_TOPOLOGY_NOTIFY event to pass a pointer
to the enum, thus it's a straight-forward pointer dereference in the event
handler.

Link: https://lore.kernel.org/r/20210830231050.5951-1-jsmart2021@gmail.com
Fixes: 202bfdff ("scsi: elx: libefc: FC node ELS and state handling")
Reported-by: default avatarkernel test robot <lkp@intel.com>
Co-developed-by: default avatarRam Vegesna <ram.vegesna@broadcom.com>
Signed-off-by: default avatarRam Vegesna <ram.vegesna@broadcom.com>
Signed-off-by: default avatarJames Smart <jsmart2021@gmail.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 6a2ea0d3
...@@ -928,22 +928,21 @@ __efc_d_wait_topology_notify(struct efc_sm_ctx *ctx, ...@@ -928,22 +928,21 @@ __efc_d_wait_topology_notify(struct efc_sm_ctx *ctx,
break; break;
case EFC_EVT_NPORT_TOPOLOGY_NOTIFY: { case EFC_EVT_NPORT_TOPOLOGY_NOTIFY: {
enum efc_nport_topology topology = enum efc_nport_topology *topology = arg;
(enum efc_nport_topology)arg;
WARN_ON(node->nport->domain->attached); WARN_ON(node->nport->domain->attached);
WARN_ON(node->send_ls_acc != EFC_NODE_SEND_LS_ACC_PLOGI); WARN_ON(node->send_ls_acc != EFC_NODE_SEND_LS_ACC_PLOGI);
node_printf(node, "topology notification, topology=%d\n", node_printf(node, "topology notification, topology=%d\n",
topology); *topology);
/* At the time the PLOGI was received, the topology was unknown, /* At the time the PLOGI was received, the topology was unknown,
* so we didn't know which node would perform the domain attach: * so we didn't know which node would perform the domain attach:
* 1. The node from which the PLOGI was sent (p2p) or * 1. The node from which the PLOGI was sent (p2p) or
* 2. The node to which the FLOGI was sent (fabric). * 2. The node to which the FLOGI was sent (fabric).
*/ */
if (topology == EFC_NPORT_TOPO_P2P) { if (*topology == EFC_NPORT_TOPO_P2P) {
/* if this is p2p, need to attach to the domain using /* if this is p2p, need to attach to the domain using
* the d_id from the PLOGI received * the d_id from the PLOGI received
*/ */
......
...@@ -107,7 +107,6 @@ void ...@@ -107,7 +107,6 @@ void
efc_fabric_notify_topology(struct efc_node *node) efc_fabric_notify_topology(struct efc_node *node)
{ {
struct efc_node *tmp_node; struct efc_node *tmp_node;
enum efc_nport_topology topology = node->nport->topology;
unsigned long index; unsigned long index;
/* /*
...@@ -118,7 +117,7 @@ efc_fabric_notify_topology(struct efc_node *node) ...@@ -118,7 +117,7 @@ efc_fabric_notify_topology(struct efc_node *node)
if (tmp_node != node) { if (tmp_node != node) {
efc_node_post_event(tmp_node, efc_node_post_event(tmp_node,
EFC_EVT_NPORT_TOPOLOGY_NOTIFY, EFC_EVT_NPORT_TOPOLOGY_NOTIFY,
(void *)topology); &node->nport->topology);
} }
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment