[PATCH 1/3] s390/dasd: Add device type defines for FBA devices

[Volkan Unal <vunal@linux.ibm.com>]

Replace magic numbers in dasd_fba_ids[] with descriptive macros
to improve code readability.

The defines distinguish between:
 - Virtual FBA devices (0x6310/0x9336): Memory backed virtual
   devices.
 - Physical FBA devices (0x3880/0x3370): Physical 3370 disk with
   3880 controller.

Reviewed-by: Stefan Haberland <sth@linux.ibm.com>
Reviewed-by: Jan Höppner <hoeppner@linux.ibm.com>
Reviewed-by: Peter Oberparleiter <oberpar@linux.ibm.com>
Reviewed-by: Mete Durlu <meted@linux.ibm.com>
Signed-off-by: Volkan Unal <vunal@linux.ibm.com>
---
 drivers/s390/block/dasd_fba.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/drivers/s390/block/dasd_fba.c b/drivers/s390/block/dasd_fba.c
index 029acb83ecab..45a488db897f 100644
--- a/drivers/s390/block/dasd_fba.c
+++ b/drivers/s390/block/dasd_fba.c
@@ -31,6 +31,11 @@
 #define DASD_FBA_CCW_LOCATE 0x43
 #define DASD_FBA_CCW_DEFINE_EXTENT 0x63
 
+#define DASD_FBA_CU_TYPE_VIRTUAL	0x6310
+#define DASD_FBA_DEV_TYPE_VIRTUAL	0x9336
+#define DASD_FBA_CU_TYPE_PHYSICAL	0x3880
+#define DASD_FBA_DEV_TYPE_PHYSICAL	0x3370
+
 MODULE_DESCRIPTION("S/390 DASD FBA Disks device driver");
 MODULE_LICENSE("GPL");
 
@@ -42,8 +47,8 @@ struct dasd_fba_private {
 };
 
 static struct ccw_device_id dasd_fba_ids[] = {
-	{ CCW_DEVICE_DEVTYPE (0x6310, 0, 0x9336, 0), .driver_info = 0x1},
-	{ CCW_DEVICE_DEVTYPE (0x3880, 0, 0x3370, 0), .driver_info = 0x2},
+	{ CCW_DEVICE_DEVTYPE (DASD_FBA_CU_TYPE_VIRTUAL, 0, DASD_FBA_DEV_TYPE_VIRTUAL, 0), .driver_info = 0x1},
+	{ CCW_DEVICE_DEVTYPE (DASD_FBA_CU_TYPE_PHYSICAL, 0, DASD_FBA_DEV_TYPE_PHYSICAL, 0), .driver_info = 0x2},
 	{ /* end of list */ },
 };
 
-- 
2.55.0