Re: [PATCH 2/3] usb: storage: add support write cache quirk on usb hdd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

On 06-06-2012 16:20, Namjae Jeon wrote:

Add support

   You missed "for" here and in the subject.

write cache quirk on usb hdd. scsi driver will be set to wce by detecting write cache quirk in quirk list when plugging usb hdd.

Signed-off-by: Namjae Jeon<namjae.jeon@xxxxxxxxxxx>
Signed-off-by: Pankaj Kumar<pankaj.km@xxxxxxxxxxx>
Signed-off-by: Amit Sahrawat<a.sahrawat@xxxxxxxxxxx>
---
  drivers/usb/storage/scsiglue.c |    6 ++++++
  drivers/usb/storage/usb.c      |    5 ++++-
  include/linux/usb/quirks.h     |    3 +++
  include/linux/usb_usual.h      |    4 +++-
  4 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/drivers/usb/storage/scsiglue.c b/drivers/usb/storage/scsiglue.c
index a324a5d..f7d0ea5 100644
--- a/drivers/usb/storage/scsiglue.c
+++ b/drivers/usb/storage/scsiglue.c
[...]
@@ -230,6 +231,11 @@ static int slave_configure(struct scsi_device *sdev)
  					US_FL_SCM_MULT_TARG))&&
  				us->protocol == USB_PR_BULK)
  			us->use_last_sector_hacks = 1;
+
+		/* Check for the cache quirk present or not */
+		if (us->pusb_dev->quirks & USB_QUIRK_WRITE_CACHE ||
+				us->fflags & US_FL_WRITE_CACHE)
+				sdev->wce_quirk = 1;

You overindented the last statement, so that it looks a bit like a continuation of previous *if*.

WBR, Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux