Thanks for fix.

Lindar.

-----Original Message-----
From: James Bottomley [mailto:[email protected]] 
Sent: Wednesday, May 08, 2013 5:44 AM
To: linux-scsi
Cc: Lukasz Dorau; Maciej Patelczyk; Dave Jiang; [email protected];
[email protected]; Xiangliang Yu
Subject: [PATCH] sas: unify the pointlessly separated enums sas_dev_type and
sas_device_type

These enums have been separate since the dawn of SAS, mainly because the
latter is a procotol only enum and the former includes additional state for
libsas.  The dichotomy causes endless confusion about which one you should
use where and leads to pointless warnings like this:

drivers/scsi/mvsas/mv_sas.c: In function 'mvs_update_phyinfo':

drivers/scsi/mvsas/mv_sas.c:1162:34: warning: comparison between 'enum
sas_device_type' and 'enum sas_dev_type' [-Wenum-compare]

Fix by eliminating one of them.  The one kept is effectively the sas.h one,
but call it sas_device_type and make sure the enums are all properly
namespaced with the SAS_ prefix.

Signed-off-by: James Bottomley <[email protected]>

---


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to