Commit 945f390f authored by Tobias Klauser's avatar Tobias Klauser Committed by Linus Torvalds

[PATCH] drivers/block: Use ARRAY_SIZE macro

Use ARRAY_SIZE macro instead of sizeof(x)/sizeof(x[0]) and remove a
duplicate of ARRAY_SIZE. Some trailing whitespaces are also removed.

drivers/block/acsi* has been left out as it's marked BROKEN.
Signed-off-by: default avatarTobias Klauser <tklauser@nuerscht.ch>
Cc: Jens Axboe <axboe@suse.de>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent ef9ceab2
...@@ -3760,7 +3760,7 @@ static void DAC960_V1_ProcessCompletedCommand(DAC960_Command_T *Command) ...@@ -3760,7 +3760,7 @@ static void DAC960_V1_ProcessCompletedCommand(DAC960_Command_T *Command)
if (SenseKey == DAC960_SenseKey_VendorSpecific && if (SenseKey == DAC960_SenseKey_VendorSpecific &&
AdditionalSenseCode == 0x80 && AdditionalSenseCode == 0x80 &&
AdditionalSenseCodeQualifier < AdditionalSenseCodeQualifier <
sizeof(DAC960_EventMessages) / sizeof(char *)) ARRAY_SIZE(DAC960_EventMessages))
DAC960_Critical("Physical Device %d:%d %s\n", Controller, DAC960_Critical("Physical Device %d:%d %s\n", Controller,
EventLogEntry->Channel, EventLogEntry->Channel,
EventLogEntry->TargetID, EventLogEntry->TargetID,
......
...@@ -131,7 +131,7 @@ static struct fd_drive_type drive_types[] = { ...@@ -131,7 +131,7 @@ static struct fd_drive_type drive_types[] = {
{ FD_DD_5, "DD 5.25", 40, 2, 14716, 13630, 1, 40, 81, 6, 30, 2}, { FD_DD_5, "DD 5.25", 40, 2, 14716, 13630, 1, 40, 81, 6, 30, 2},
{ FD_NODRIVE, "No Drive", 0, 0, 0, 0, 0, 0, 0, 0, 0, 0} { FD_NODRIVE, "No Drive", 0, 0, 0, 0, 0, 0, 0, 0, 0, 0}
}; };
static int num_dr_types = sizeof(drive_types) / sizeof(drive_types[0]); static int num_dr_types = ARRAY_SIZE(drive_types);
static int amiga_read(int), dos_read(int); static int amiga_read(int), dos_read(int);
static void amiga_write(int), dos_write(int); static void amiga_write(int), dos_write(int);
......
...@@ -181,7 +181,7 @@ static struct { ...@@ -181,7 +181,7 @@ static struct {
{ 6, TYPE_HD }, /* 31: H1640 <- was H1600 == h1600 for PC */ { 6, TYPE_HD }, /* 31: H1640 <- was H1600 == h1600 for PC */
}; };
#define NUM_DISK_MINORS (sizeof(minor2disktype)/sizeof(*minor2disktype)) #define NUM_DISK_MINORS ARRAY_SIZE(minor2disktype)
/* /*
* Maximum disk size (in kilobytes). This default is used whenever the * Maximum disk size (in kilobytes). This default is used whenever the
......
...@@ -103,7 +103,7 @@ static const struct pci_device_id cciss_pci_device_id[] = { ...@@ -103,7 +103,7 @@ static const struct pci_device_id cciss_pci_device_id[] = {
}; };
MODULE_DEVICE_TABLE(pci, cciss_pci_device_id); MODULE_DEVICE_TABLE(pci, cciss_pci_device_id);
#define NR_PRODUCTS (sizeof(products)/sizeof(struct board_type)) #define NR_PRODUCTS ARRAY_SIZE(products)
/* board_id = Subsystem Device ID & Vendor ID /* board_id = Subsystem Device ID & Vendor ID
* product = Marketing Name for the board * product = Marketing Name for the board
...@@ -2833,7 +2833,7 @@ static int cciss_pci_init(ctlr_info_t *c, struct pci_dev *pdev) ...@@ -2833,7 +2833,7 @@ static int cciss_pci_init(ctlr_info_t *c, struct pci_dev *pdev)
c->board_id = board_id; c->board_id = board_id;
#ifdef CCISS_DEBUG #ifdef CCISS_DEBUG
print_cfg_table(c->cfgtable); print_cfg_table(c->cfgtable);
#endif /* CCISS_DEBUG */ #endif /* CCISS_DEBUG */
for(i=0; i<NR_PRODUCTS; i++) { for(i=0; i<NR_PRODUCTS; i++) {
......
...@@ -72,11 +72,11 @@ static ctlr_info_t *hba[MAX_CTLR]; ...@@ -72,11 +72,11 @@ static ctlr_info_t *hba[MAX_CTLR];
static int eisa[8]; static int eisa[8];
#define NR_PRODUCTS (sizeof(products)/sizeof(struct board_type)) #define NR_PRODUCTS ARRAY_SIZE(products)
/* board_id = Subsystem Device ID & Vendor ID /* board_id = Subsystem Device ID & Vendor ID
* product = Marketing Name for the board * product = Marketing Name for the board
* access = Address of the struct of function pointers * access = Address of the struct of function pointers
*/ */
static struct board_type products[] = { static struct board_type products[] = {
{ 0x0040110E, "IDA", &smart1_access }, { 0x0040110E, "IDA", &smart1_access },
......
...@@ -479,7 +479,6 @@ static struct floppy_struct floppy_type[32] = { ...@@ -479,7 +479,6 @@ static struct floppy_struct floppy_type[32] = {
{ 3200,20,2,80,0,0x1C,0x00,0xCF,0x2C,"H1600" }, /* 31 1.6MB 3.5" */ { 3200,20,2,80,0,0x1C,0x00,0xCF,0x2C,"H1600" }, /* 31 1.6MB 3.5" */
}; };
#define NUMBER(x) (sizeof(x) / sizeof(*(x)))
#define SECTSIZE (_FD_SECTSIZE(*floppy)) #define SECTSIZE (_FD_SECTSIZE(*floppy))
/* Auto-detection: Disk type used until the next media change occurs. */ /* Auto-detection: Disk type used until the next media change occurs. */
...@@ -3645,7 +3644,7 @@ static void __init config_types(void) ...@@ -3645,7 +3644,7 @@ static void __init config_types(void)
const char *name = NULL; const char *name = NULL;
static char temparea[32]; static char temparea[32];
if (type < NUMBER(default_drive_params)) { if (type < ARRAY_SIZE(default_drive_params)) {
params = &default_drive_params[type].params; params = &default_drive_params[type].params;
if (type) { if (type) {
name = default_drive_params[type].name; name = default_drive_params[type].name;
...@@ -3961,7 +3960,7 @@ static void __init register_devfs_entries(int drive) ...@@ -3961,7 +3960,7 @@ static void __init register_devfs_entries(int drive)
{ {
int base_minor = (drive < 4) ? drive : (124 + drive); int base_minor = (drive < 4) ? drive : (124 + drive);
if (UDP->cmos < NUMBER(default_drive_params)) { if (UDP->cmos < ARRAY_SIZE(default_drive_params)) {
int i = 0; int i = 0;
do { do {
int minor = base_minor + (table_sup[UDP->cmos][i] << 2); int minor = base_minor + (table_sup[UDP->cmos][i] << 2);
...@@ -4219,7 +4218,7 @@ static struct kobject *floppy_find(dev_t dev, int *part, void *data) ...@@ -4219,7 +4218,7 @@ static struct kobject *floppy_find(dev_t dev, int *part, void *data)
!(allowed_drive_mask & (1 << drive)) || !(allowed_drive_mask & (1 << drive)) ||
fdc_state[FDC(drive)].version == FDC_NONE) fdc_state[FDC(drive)].version == FDC_NONE)
return NULL; return NULL;
if (((*part >> 2) & 0x1f) >= NUMBER(floppy_type)) if (((*part >> 2) & 0x1f) >= ARRAY_SIZE(floppy_type))
return NULL; return NULL;
*part = 0; *part = 0;
return get_disk(disks[drive]); return get_disk(disks[drive]);
...@@ -4571,7 +4570,7 @@ static void unregister_devfs_entries(int drive) ...@@ -4571,7 +4570,7 @@ static void unregister_devfs_entries(int drive)
{ {
int i; int i;
if (UDP->cmos < NUMBER(default_drive_params)) { if (UDP->cmos < ARRAY_SIZE(default_drive_params)) {
i = 0; i = 0;
do { do {
devfs_remove("floppy/%d%s", drive, devfs_remove("floppy/%d%s", drive,
......
...@@ -279,11 +279,11 @@ static u_char __init xd_detect (u_char *controller, unsigned int *address) ...@@ -279,11 +279,11 @@ static u_char __init xd_detect (u_char *controller, unsigned int *address)
return(1); return(1);
} }
for (i = 0; i < (sizeof(xd_bases) / sizeof(xd_bases[0])); i++) { for (i = 0; i < ARRAY_SIZE(xd_bases); i++) {
void __iomem *p = ioremap(xd_bases[i], 0x2000); void __iomem *p = ioremap(xd_bases[i], 0x2000);
if (!p) if (!p)
continue; continue;
for (j = 1; j < (sizeof(xd_sigs) / sizeof(xd_sigs[0])); j++) { for (j = 1; j < ARRAY_SIZE(xd_sigs); j++) {
const char *s = xd_sigs[j].string; const char *s = xd_sigs[j].string;
if (check_signature(p + xd_sigs[j].offset, s, strlen(s))) { if (check_signature(p + xd_sigs[j].offset, s, strlen(s))) {
*controller = j; *controller = j;
...@@ -1018,7 +1018,7 @@ static void __init do_xd_setup (int *integers) ...@@ -1018,7 +1018,7 @@ static void __init do_xd_setup (int *integers)
case 2: if ((integers[2] > 0) && (integers[2] < 16)) case 2: if ((integers[2] > 0) && (integers[2] < 16))
xd_irq = integers[2]; xd_irq = integers[2];
case 1: xd_override = 1; case 1: xd_override = 1;
if ((integers[1] >= 0) && (integers[1] < (sizeof(xd_sigs) / sizeof(xd_sigs[0])))) if ((integers[1] >= 0) && (integers[1] < ARRAY_SIZE(xd_sigs)))
xd_type = integers[1]; xd_type = integers[1];
case 0: break; case 0: break;
default:printk("xd: too many parameters for xd\n"); default:printk("xd: too many parameters for xd\n");
......
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