Re: [PATCH v4 1/5] scsi: Adjust DBD setting in mode sense for caching mode page per LLD

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

To: Can Guo <cang@xxxxxxxxxxxxxx>
Subject: Re: [PATCH v4 1/5] scsi: Adjust DBD setting in mode sense for caching mode page per LLD
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Date: Tue, 19 Nov 2019 00:50:11 -0500
Cc: asutoshd@xxxxxxxxxxxxxx, nguyenb@xxxxxxxxxxxxxx, rnayak@xxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, kernel-team@xxxxxxxxxxx, saravanak@xxxxxxxxxx, salyzyn@xxxxxxxxxx, "James E.J. Bottomley" <jejb@xxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx (open list)
In-reply-to: <1573624824-671-2-git-send-email-cang@codeaurora.org> (Can Guo's message of "Tue, 12 Nov 2019 22:00:19 -0800")
Organization: Oracle Corporation
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1.92 (gnu/linux)

Can,

> UFS JEDEC standards require DBD field to be set to 1 in mode sense
> command.  This patch allows LLD to define the setting of DBD if
> required.

These changes look OK to me, btw. Please make sure each patch has a
changelog which reflects how the review feedback was addressed.

-- 
Martin K. Petersen	Oracle Linux Engineering

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

References:
- [PATCH v4 0/5] UFS driver general fixes bundle 1
  - From: Can Guo
- [PATCH v4 1/5] scsi: Adjust DBD setting in mode sense for caching mode page per LLD
  - From: Can Guo

Prev by Date: Re: [PATCH] scsi: bnx2i: fix potential use after free
Next by Date: Re: [EXT] Re: [PATCH v5 4/4] ufs: Simplify the clock scaling mechanism implementation
Previous by thread: [PATCH v4 1/5] scsi: Adjust DBD setting in mode sense for caching mode page per LLD
Next by thread: [PATCH v4 2/5] scsi: ufs: Use DBD setting in mode sense
Index(es):
- Date
- Thread

[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]