diff options
| author |   Joe Eykholt <jeykholt@cisco.com> | 2009-08-25 14:00:45 -0700 |
|---|---|---|
| committer |   James Bottomley <James.Bottomley@suse.de> | 2009-09-10 12:07:40 -0500 |
| commit | 922aa210bcad4b34a7bb98ec9d318b7e59e7a5ca (patch) | |
| tree | 480d1304252f34cdb8286ec104850648b852e598 /include/scsi/libfc.h | |
| parent | [SCSI] libfc: change interface for rport_create (diff) | |
| download | linux-dev-922aa210bcad4b34a7bb98ec9d318b7e59e7a5ca.tar.xz linux-dev-922aa210bcad4b34a7bb98ec9d318b7e59e7a5ca.zip | |
[SCSI] libfc: fix RPORT_TO_PRIV and PRIV_TO_RPORT() macros.
These macros introduce extra undesirable semicolons that keep
them from being used in expressions, and they don't protect
against being passed an expression.
Add parens and remove the semicolons.
Signed-off-by: Joe Eykholt <jeykholt@cisco.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'include/scsi/libfc.h')
| -rw-r--r-- | include/scsi/libfc.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/scsi/libfc.h b/include/scsi/libfc.h index d888cbecd72e..2fdd8ac12b89 100644 --- a/include/scsi/libfc.h +++ b/include/scsi/libfc.h @@ -223,9 +223,9 @@ struct fc_rport_libfc_priv { }; #define PRIV_TO_RPORT(x) \ - (struct fc_rport *)((void *)x - sizeof(struct fc_rport)); + ((struct fc_rport *)((void *)(x) - sizeof(struct fc_rport))) #define RPORT_TO_PRIV(x) \ - (struct fc_rport_libfc_priv *)((void *)x + sizeof(struct fc_rport)); + ((struct fc_rport_libfc_priv *)((void *)(x) + sizeof(struct fc_rport))) static inline void fc_rport_set_name(struct fc_rport *rport, u64 wwpn, u64 wwnn) { |