mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-12-26 20:44:32 +08:00
48d83282db
A regression fix introduced a harmless type mismatch warning:
drivers/scsi/bfa/bfad_bsg.c: In function 'bfad_im_bsg_vendor_request':
drivers/scsi/bfa/bfad_bsg.c:3137:35: error: initialization of 'struct bfad_im_port_s *' from 'long unsigned int' makes pointer from integer without a cast [-Werror=int-conversion]
struct bfad_im_port_s *im_port = shost->hostdata[0];
^~~~~
drivers/scsi/bfa/bfad_bsg.c: In function 'bfad_im_bsg_els_ct_request':
drivers/scsi/bfa/bfad_bsg.c:3353:35: error: initialization of 'struct bfad_im_port_s *' from 'long unsigned int' makes pointer from integer without a cast [-Werror=int-conversion]
struct bfad_im_port_s *im_port = shost->hostdata[0];
This changes the code back to shost_priv() once more, but encapsulates
it in an inline function to document the rather unusual way of
using the private data only as a pointer to the previously allocated
structure.
I did not try to get rid of the extra indirection level entirely,
which would have been rather invasive and required reworking the entire
initialization sequence.
Fixes:
|
||
|---|---|---|
| .. | ||
| bfa_core.c | ||
| bfa_cs.h | ||
| bfa_defs_fcs.h | ||
| bfa_defs_svc.h | ||
| bfa_defs.h | ||
| bfa_fc.h | ||
| bfa_fcbuild.c | ||
| bfa_fcbuild.h | ||
| bfa_fcpim.c | ||
| bfa_fcpim.h | ||
| bfa_fcs_fcpim.c | ||
| bfa_fcs_lport.c | ||
| bfa_fcs_rport.c | ||
| bfa_fcs.c | ||
| bfa_fcs.h | ||
| bfa_hw_cb.c | ||
| bfa_hw_ct.c | ||
| bfa_ioc_cb.c | ||
| bfa_ioc_ct.c | ||
| bfa_ioc.c | ||
| bfa_ioc.h | ||
| bfa_modules.h | ||
| bfa_plog.h | ||
| bfa_port.c | ||
| bfa_port.h | ||
| bfa_svc.c | ||
| bfa_svc.h | ||
| bfa.h | ||
| bfad_attr.c | ||
| bfad_bsg.c | ||
| bfad_bsg.h | ||
| bfad_debugfs.c | ||
| bfad_drv.h | ||
| bfad_im.c | ||
| bfad_im.h | ||
| bfad.c | ||
| bfi_ms.h | ||
| bfi_reg.h | ||
| bfi.h | ||
| Makefile | ||