This patchset adresses checkpatch errors in a few of the files in usb
storage. More to follow.
Bas Peters (3):
drivers: usb: storage: alauda.c: properly place braces after function
declarations
drivers: usb: storage: cypress_atacb.c: trivial checkpatch fixes
drivers: usb: storage: datafab.c: clean up a variety of checkpatch
errors.
drivers/usb/storage/alauda.c | 15 ++-
drivers/usb/storage/cypress_atacb.c | 17 ++--
drivers/usb/storage/datafab.c | 183 ++++++++++++++++++------------------
3 files changed, 111 insertions(+), 104 deletions(-)
--
2.1.0
This patch places braces on a new line following function declarations.
Signed-off-by: Bas Peters <[email protected]>
---
drivers/usb/storage/alauda.c | 15 ++++++++++-----
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/drivers/usb/storage/alauda.c b/drivers/usb/storage/alauda.c
index 62c2d9d..4b55ab6 100644
--- a/drivers/usb/storage/alauda.c
+++ b/drivers/usb/storage/alauda.c
@@ -207,7 +207,8 @@ static struct alauda_card_info alauda_card_ids[] = {
{ 0,}
};
-static struct alauda_card_info *alauda_card_find_id(unsigned char id) {
+static struct alauda_card_info *alauda_card_find_id(unsigned char id)
+{
int i;
for (i = 0; alauda_card_ids[i].id != 0; i++)
@@ -223,7 +224,8 @@ static struct alauda_card_info *alauda_card_find_id(unsigned char id) {
static unsigned char parity[256];
static unsigned char ecc2[256];
-static void nand_init_ecc(void) {
+static void nand_init_ecc(void)
+{
int i, j, a;
parity[0] = 0;
@@ -247,7 +249,8 @@ static void nand_init_ecc(void) {
}
/* compute 3-byte ecc on 256 bytes */
-static void nand_compute_ecc(unsigned char *data, unsigned char *ecc) {
+static void nand_compute_ecc(unsigned char *data, unsigned char *ecc)
+{
int i, j, a;
unsigned char par = 0, bit, bits[8] = {0};
@@ -270,11 +273,13 @@ static void nand_compute_ecc(unsigned char *data, unsigned char *ecc) {
ecc[2] = ecc2[par];
}
-static int nand_compare_ecc(unsigned char *data, unsigned char *ecc) {
+static int nand_compare_ecc(unsigned char *data, unsigned char *ecc)
+{
return (data[0] == ecc[0] && data[1] == ecc[1] && data[2] == ecc[2]);
}
-static void nand_store_ecc(unsigned char *data, unsigned char *ecc) {
+static void nand_store_ecc(unsigned char *data, unsigned char *ecc)
+{
memcpy(data, ecc, 3);
}
--
2.1.0
Fixes errors thrown by checkpatch over a space issue and the
incorrect indentation of a switch statement.
Signed-off-by: Bas Peters <[email protected]>
---
drivers/usb/storage/cypress_atacb.c | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/drivers/usb/storage/cypress_atacb.c b/drivers/usb/storage/cypress_atacb.c
index 8514a2d..b3466d1 100644
--- a/drivers/usb/storage/cypress_atacb.c
+++ b/drivers/usb/storage/cypress_atacb.c
@@ -96,13 +96,13 @@ static void cypress_atacb_passthrough(struct scsi_cmnd *srb, struct us_data *us)
if (save_cmnd[1] >> 5) /* MULTIPLE_COUNT */
goto invalid_fld;
/* check protocol */
- switch((save_cmnd[1] >> 1) & 0xf) {
- case 3: /*no DATA */
- case 4: /* PIO in */
- case 5: /* PIO out */
- break;
- default:
- goto invalid_fld;
+ switch ((save_cmnd[1] >> 1) & 0xf) {
+ case 3: /*no DATA */
+ case 4: /* PIO in */
+ case 5: /* PIO out */
+ break;
+ default:
+ goto invalid_fld;
}
/* first build the ATACB command */
@@ -132,8 +132,7 @@ static void cypress_atacb_passthrough(struct scsi_cmnd *srb, struct us_data *us)
|| save_cmnd[11])
goto invalid_fld;
}
- }
- else { /* ATA12 */
+ } else { /* ATA12 */
srb->cmnd[ 6] = save_cmnd[3]; /* features */
srb->cmnd[ 7] = save_cmnd[4]; /* sector count */
srb->cmnd[ 8] = save_cmnd[5]; /* lba low */
--
2.1.0
This patch cleans up a variety of checkpatch errors:
Bunch of space issues.
C99 comments converted to /* */ format.
Some switch statement indentations.
"foo * bar" -> "foo *bar"
Signed-off-by: Bas Peters <[email protected]>
---
drivers/usb/storage/datafab.c | 183 +++++++++++++++++++++---------------------
1 file changed, 93 insertions(+), 90 deletions(-)
diff --git a/drivers/usb/storage/datafab.c b/drivers/usb/storage/datafab.c
index 7b17c21..78f867d 100644
--- a/drivers/usb/storage/datafab.c
+++ b/drivers/usb/storage/datafab.c
@@ -10,7 +10,7 @@
* Many thanks to Robert Baruch for the SanDisk SmartMedia reader driver
* which I used as a template for this driver.
*
- * Some bugfixes and scatter-gather code by Gregory P. Smith
+ * Some bugfixes and scatter-gather code by Gregory P. Smith
* ([email protected])
*
* Fix for media change by Joerg Schneider ([email protected])
@@ -35,8 +35,8 @@
/*
* This driver attempts to support USB CompactFlash reader/writer devices
- * based on Datafab USB-to-ATA chips. It was specifically developed for the
- * Datafab MDCFE-B USB CompactFlash reader but has since been found to work
+ * based on Datafab USB-to-ATA chips. It was specifically developed for the
+ * Datafab MDCFE-B USB CompactFlash reader but has since been found to work
* with a variety of Datafab-based devices from a number of manufacturers.
* I've received a report of this driver working with a Datafab-based
* SmartMedia device though please be aware that I'm personally unable to
@@ -153,11 +153,12 @@ static int datafab_read_data(struct us_data *us,
unsigned int sg_offset = 0;
struct scatterlist *sg = NULL;
- // we're working in LBA mode. according to the ATA spec,
- // we can support up to 28-bit addressing. I don't know if Datafab
- // supports beyond 24-bit addressing. It's kind of hard to test
- // since it requires > 8GB CF card.
- //
+ /* we're working in LBA mode. according to the ATA spec,
+ * we can support up to 28-bit addressing. I don't know if Datafab
+ * supports beyond 24-bit addressing. It's kind of hard to test
+ * since it requires > 8GB CF card.
+ */
+
if (sectors > 0x0FFFFFFF)
return USB_STOR_TRANSPORT_ERROR;
@@ -169,9 +170,10 @@ static int datafab_read_data(struct us_data *us,
totallen = sectors * info->ssize;
- // Since we don't read more than 64 KB at a time, we have to create
- // a bounce buffer and move the data a piece at a time between the
- // bounce buffer and the actual transfer buffer.
+ /* Since we don't read more than 64 KB at a time, we have to create
+ * a bounce buffer and move the data a piece at a time between the
+ * bounce buffer and the actual transfer buffer.
+ */
alloclen = min(totallen, 65536u);
buffer = kmalloc(alloclen, GFP_NOIO);
@@ -179,8 +181,9 @@ static int datafab_read_data(struct us_data *us,
return USB_STOR_TRANSPORT_ERROR;
do {
- // loop, never allocate or transfer more than 64k at once
- // (min(128k, 255*info->ssize) is the real limit)
+ /* loop, never allocate or transfer more than 64k at once
+ * (min(128k, 255*info->ssize) is the real limit)
+ */
len = min(totallen, alloclen);
thistime = (len / info->ssize) & 0xff;
@@ -196,17 +199,17 @@ static int datafab_read_data(struct us_data *us,
command[6] = 0x20;
command[7] = 0x01;
- // send the read command
+ /* send the read command */
result = datafab_bulk_write(us, command, 8);
if (result != USB_STOR_XFER_GOOD)
goto leave;
- // read the result
+ /* read the result */
result = datafab_bulk_read(us, buffer, len);
if (result != USB_STOR_XFER_GOOD)
goto leave;
- // Store the data in the transfer buffer
+ /* Store the data in the transfer buffer */
usb_stor_access_xfer_buf(buffer, len, us->srb,
&sg, &sg_offset, TO_XFER_BUF);
@@ -237,11 +240,11 @@ static int datafab_write_data(struct us_data *us,
unsigned int sg_offset = 0;
struct scatterlist *sg = NULL;
- // we're working in LBA mode. according to the ATA spec,
- // we can support up to 28-bit addressing. I don't know if Datafab
- // supports beyond 24-bit addressing. It's kind of hard to test
- // since it requires > 8GB CF card.
- //
+ /* we're working in LBA mode. according to the ATA spec,
+ * we can support up to 28-bit addressing. I don't know if Datafab
+ * supports beyond 24-bit addressing. It's kind of hard to test
+ * since it requires > 8GB CF card.
+ */
if (sectors > 0x0FFFFFFF)
return USB_STOR_TRANSPORT_ERROR;
@@ -253,9 +256,10 @@ static int datafab_write_data(struct us_data *us,
totallen = sectors * info->ssize;
- // Since we don't write more than 64 KB at a time, we have to create
- // a bounce buffer and move the data a piece at a time between the
- // bounce buffer and the actual transfer buffer.
+ /* Since we don't write more than 64 KB at a time, we have to create
+ * a bounce buffer and move the data a piece at a time between the
+ * bounce buffer and the actual transfer buffer.
+ */
alloclen = min(totallen, 65536u);
buffer = kmalloc(alloclen, GFP_NOIO);
@@ -263,13 +267,14 @@ static int datafab_write_data(struct us_data *us,
return USB_STOR_TRANSPORT_ERROR;
do {
- // loop, never allocate or transfer more than 64k at once
- // (min(128k, 255*info->ssize) is the real limit)
+ /* loop, never allocate or transfer more than 64k at once
+ * (min(128k, 255*info->ssize) is the real limit)
+ */
len = min(totallen, alloclen);
thistime = (len / info->ssize) & 0xff;
- // Get the data from the transfer buffer
+ /* Get the data from the transfer buffer */
usb_stor_access_xfer_buf(buffer, len, us->srb,
&sg, &sg_offset, FROM_XFER_BUF);
@@ -284,17 +289,17 @@ static int datafab_write_data(struct us_data *us,
command[6] = 0x30;
command[7] = 0x02;
- // send the command
+ /* send the command */
result = datafab_bulk_write(us, command, 8);
if (result != USB_STOR_XFER_GOOD)
goto leave;
- // send the data
+ /* send the data */
result = datafab_bulk_write(us, buffer, len);
if (result != USB_STOR_XFER_GOOD)
goto leave;
- // read the result
+ /* read the result */
result = datafab_bulk_read(us, reply, 2);
if (result != USB_STOR_XFER_GOOD)
goto leave;
@@ -322,11 +327,12 @@ static int datafab_write_data(struct us_data *us,
static int datafab_determine_lun(struct us_data *us,
struct datafab_info *info)
{
- // Dual-slot readers can be thought of as dual-LUN devices.
- // We need to determine which card slot is being used.
- // We'll send an IDENTIFY DEVICE command and see which LUN responds...
- //
- // There might be a better way of doing this?
+ /* Dual-slot readers can be thought of as dual-LUN devices.
+ * We need to determine which card slot is being used.
+ * We'll send an IDENTIFY DEVICE command and see which LUN responds...
+ *
+ * There might be a better way of doing this?
+ */
static unsigned char scommand[8] = { 0, 1, 0, 0, 0, 0xa0, 0xec, 1 };
unsigned char *command = us->iobuf;
@@ -343,8 +349,7 @@ static int datafab_determine_lun(struct us_data *us,
usb_stor_dbg(us, "locating...\n");
- // we'll try 3 times before giving up...
- //
+ /* we'll try 3 times before giving up... */
while (count++ < 3) {
command[5] = 0xa0;
@@ -389,10 +394,10 @@ static int datafab_determine_lun(struct us_data *us,
static int datafab_id_device(struct us_data *us,
struct datafab_info *info)
{
- // this is a variation of the ATA "IDENTIFY DEVICE" command...according
- // to the ATA spec, 'Sector Count' isn't used but the Windows driver
- // sets this bit so we do too...
- //
+ /* this is a variation of the ATA "IDENTIFY DEVICE" command...according
+ * to the ATA spec, 'Sector Count' isn't used but the Windows driver
+ * sets this bit so we do too...
+ */
static unsigned char scommand[8] = { 0, 1, 0, 0, 0, 0xa0, 0xec, 1 };
unsigned char *command = us->iobuf;
unsigned char *reply;
@@ -420,15 +425,15 @@ static int datafab_id_device(struct us_data *us,
goto leave;
}
- // we'll go ahead and extract the media capacity while we're here...
- //
+ /* we'll go ahead and extract the media capacity while we're here... */
+
rc = datafab_bulk_read(us, reply, 512);
if (rc == USB_STOR_XFER_GOOD) {
- // capacity is at word offset 57-58
- //
- info->sectors = ((u32)(reply[117]) << 24) |
+ /* capacity is at word offset 57-58 */
+
+ info->sectors = ((u32)(reply[117]) << 24) |
((u32)(reply[116]) << 16) |
- ((u32)(reply[115]) << 8) |
+ ((u32)(reply[115]) << 8) |
((u32)(reply[114]) );
rc = USB_STOR_TRANSPORT_GOOD;
goto leave;
@@ -443,7 +448,7 @@ static int datafab_id_device(struct us_data *us,
static int datafab_handle_mode_sense(struct us_data *us,
- struct scsi_cmnd * srb,
+ struct scsi_cmnd *srb,
int sense_6)
{
static unsigned char rw_err_page[12] = {
@@ -463,67 +468,67 @@ static int datafab_handle_mode_sense(struct us_data *us,
struct datafab_info *info = (struct datafab_info *) (us->extra);
unsigned char *ptr = us->iobuf;
- // most of this stuff is just a hack to get things working. the
- // datafab reader doesn't present a SCSI interface so we
- // fudge the SCSI commands...
- //
+ /* most of this stuff is just a hack to get things working. the
+ * datafab reader doesn't present a SCSI interface so we
+ * fudge the SCSI commands...
+ */
pc = srb->cmnd[2] >> 6;
page_code = srb->cmnd[2] & 0x3F;
switch (pc) {
- case 0x0:
- usb_stor_dbg(us, "Current values\n");
+ case 0x0:
+ usb_stor_dbg(us, "Current values\n");
break;
- case 0x1:
- usb_stor_dbg(us, "Changeable values\n");
+ case 0x1:
+ usb_stor_dbg(us, "Changeable values\n");
break;
- case 0x2:
- usb_stor_dbg(us, "Default values\n");
+ case 0x2:
+ usb_stor_dbg(us, "Default values\n");
break;
- case 0x3:
- usb_stor_dbg(us, "Saves values\n");
+ case 0x3:
+ usb_stor_dbg(us, "Saves values\n");
break;
}
memset(ptr, 0, 8);
if (sense_6) {
- ptr[2] = 0x00; // WP enable: 0x80
+ ptr[2] = 0x00; /* WP enable: 0x80 */
i = 4;
} else {
- ptr[3] = 0x00; // WP enable: 0x80
+ ptr[3] = 0x00; /* WP enable: 0x80 */
i = 8;
}
switch (page_code) {
- default:
- // vendor-specific mode
+ default:
+ /* vendor-specific mode */
info->sense_key = 0x05;
info->sense_asc = 0x24;
info->sense_ascq = 0x00;
return USB_STOR_TRANSPORT_FAILED;
- case 0x1:
+ case 0x1:
memcpy(ptr + i, rw_err_page, sizeof(rw_err_page));
i += sizeof(rw_err_page);
break;
- case 0x8:
+ case 0x8:
memcpy(ptr + i, cache_page, sizeof(cache_page));
i += sizeof(cache_page);
break;
- case 0x1B:
+ case 0x1B:
memcpy(ptr + i, rbac_page, sizeof(rbac_page));
i += sizeof(rbac_page);
break;
- case 0x1C:
+ case 0x1C:
memcpy(ptr + i, timer_page, sizeof(timer_page));
i += sizeof(timer_page);
break;
- case 0x3F: // retrieve all pages
+ case 0x3F: /* retrieve all pages */
memcpy(ptr + i, timer_page, sizeof(timer_page));
i += sizeof(timer_page);
memcpy(ptr + i, rbac_page, sizeof(rbac_page));
@@ -546,13 +551,13 @@ static int datafab_handle_mode_sense(struct us_data *us,
static void datafab_info_destructor(void *extra)
{
- // this routine is a placeholder...
- // currently, we don't allocate any extra memory so we're okay
+ /* this routine is a placeholder...
+ * currently, we don't allocate any extra memory so we're okay
+ */
}
-// Transport for the Datafab MDCFE-B
-//
+/* Transport for the Datafab MDCFE-B */
static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
{
struct datafab_info *info;
@@ -569,7 +574,7 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
return USB_STOR_TRANSPORT_ERROR;
us->extra_destructor = datafab_info_destructor;
- ((struct datafab_info *)us->extra)->lun = -1;
+ ((struct datafab_info *)us->extra)->lun = -1;
}
info = (struct datafab_info *) (us->extra);
@@ -582,7 +587,7 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
}
if (srb->cmnd[0] == READ_CAPACITY) {
- info->ssize = 0x200; // hard coded 512 byte sectors as per ATA spec
+ info->ssize = 0x200; /* hard coded 512 byte sectors as per ATA spec */
rc = datafab_id_device(us, info);
if (rc != USB_STOR_TRANSPORT_GOOD)
return rc;
@@ -590,8 +595,9 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
usb_stor_dbg(us, "READ_CAPACITY: %ld sectors, %ld bytes per sector\n",
info->sectors, info->ssize);
- // build the reply
- // we need the last sector, not the number of sectors
+ /* build the reply
+ * we need the last sector, not the number of sectors
+ */
((__be32 *) ptr)[0] = cpu_to_be32(info->sectors - 1);
((__be32 *) ptr)[1] = cpu_to_be32(info->ssize);
usb_stor_set_xfer_buf(ptr, 8, srb);
@@ -604,8 +610,7 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
return USB_STOR_TRANSPORT_ERROR;
}
- // don't bother implementing READ_6 or WRITE_6.
- //
+ /* don't bother implementing READ_6 or WRITE_6. */
if (srb->cmnd[0] == READ_10) {
block = ((u32)(srb->cmnd[2]) << 24) | ((u32)(srb->cmnd[3]) << 16) |
((u32)(srb->cmnd[4]) << 8) | ((u32)(srb->cmnd[5]));
@@ -618,8 +623,7 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
}
if (srb->cmnd[0] == READ_12) {
- // we'll probably never see a READ_12 but we'll do it anyway...
- //
+ /* we'll probably never see a READ_12 but we'll do it anyway... */
block = ((u32)(srb->cmnd[2]) << 24) | ((u32)(srb->cmnd[3]) << 16) |
((u32)(srb->cmnd[4]) << 8) | ((u32)(srb->cmnd[5]));
@@ -643,8 +647,7 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
}
if (srb->cmnd[0] == WRITE_12) {
- // we'll probably never see a WRITE_12 but we'll do it anyway...
- //
+ /* we'll probably never see a WRITE_12 but we'll do it anyway... */
block = ((u32)(srb->cmnd[2]) << 24) | ((u32)(srb->cmnd[3]) << 16) |
((u32)(srb->cmnd[4]) << 8) | ((u32)(srb->cmnd[5]));
@@ -664,10 +667,10 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
if (srb->cmnd[0] == REQUEST_SENSE) {
usb_stor_dbg(us, "REQUEST_SENSE - Returning faked response\n");
- // this response is pretty bogus right now. eventually if necessary
- // we can set the correct sense data. so far though it hasn't been
- // necessary
- //
+ /* this response is pretty bogus right now. eventually if necessary
+ * we can set the correct sense data. so far though it hasn't been
+ * necessary
+ */
memset(ptr, 0, 18);
ptr[0] = 0xF0;
ptr[2] = info->sense_key;
@@ -690,9 +693,9 @@ static int datafab_transport(struct scsi_cmnd *srb, struct us_data *us)
}
if (srb->cmnd[0] == ALLOW_MEDIUM_REMOVAL) {
- // sure. whatever. not like we can stop the user from
- // popping the media out of the device (no locking doors, etc)
- //
+ /* sure. whatever. not like we can stop the user from
+ * popping the media out of the device (no locking doors, etc)
+ */
return USB_STOR_TRANSPORT_GOOD;
}
--
2.1.0
On 2/8/2015 1:42 AM, Bas Peters wrote:
> Fixes errors thrown by checkpatch over a space issue and the
> incorrect indentation of a switch statement.
> Signed-off-by: Bas Peters <[email protected]>
> ---
> drivers/usb/storage/cypress_atacb.c | 17 ++++++++---------
> 1 file changed, 8 insertions(+), 9 deletions(-)
> diff --git a/drivers/usb/storage/cypress_atacb.c b/drivers/usb/storage/cypress_atacb.c
> index 8514a2d..b3466d1 100644
> --- a/drivers/usb/storage/cypress_atacb.c
> +++ b/drivers/usb/storage/cypress_atacb.c
> @@ -96,13 +96,13 @@ static void cypress_atacb_passthrough(struct scsi_cmnd *srb, struct us_data *us)
> if (save_cmnd[1] >> 5) /* MULTIPLE_COUNT */
> goto invalid_fld;
> /* check protocol */
> - switch((save_cmnd[1] >> 1) & 0xf) {
> - case 3: /*no DATA */
> - case 4: /* PIO in */
> - case 5: /* PIO out */
> - break;
> - default:
> - goto invalid_fld;
> + switch ((save_cmnd[1] >> 1) & 0xf) {
> + case 3: /*no DATA */
Could also add space after /*, while at it.
> + case 4: /* PIO in */
> + case 5: /* PIO out */
> + break;
> + default:
> + goto invalid_fld;
> }
>
> /* first build the ATACB command */
[...]
WBR, Sergei