Re: [PATCH 3/3] scsi: ufs: ufs-qcom: Apply DELAY_AFTER_LPM quirk for Toshiba devices

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

To: Manish Pandey <quic_mapa@xxxxxxxxxxx>, Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>, "James E.J. Bottomley" <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Subject: Re: [PATCH 3/3] scsi: ufs: ufs-qcom: Apply DELAY_AFTER_LPM quirk for Toshiba devices
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Tue, 20 Aug 2024 14:35:22 -0700
Cc: linux-arm-msm@xxxxxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, quic_cang@xxxxxxxxxxx, quic_nguyenb@xxxxxxxxxxx, quic_nitirawa@xxxxxxxxxxx, quic_bhaskarv@xxxxxxxxxxx, quic_narepall@xxxxxxxxxxx, quic_rampraka@xxxxxxxxxxx
In-reply-to: <20240820123756.24590-4-quic_mapa@quicinc.com>
References: <20240820123756.24590-1-quic_mapa@quicinc.com> <20240820123756.24590-4-quic_mapa@quicinc.com>
User-agent: Mozilla Thunderbird

On 8/20/24 5:37 AM, Manish Pandey wrote:

+	{ .wmanufacturerid = UFS_VENDOR_TOSHIBA,
+	  .model = UFS_ANY_MODEL,
+	  .quirk = UFS_DEVICE_QUIRK_DELAY_AFTER_LPM },


Isn't three patches a bit much for these changes? I think all three
patches can be combined into a single patch without making it harder for
reviewers to understand what is going on.

Thanks,

Bart.

Follow-Ups:
- Re: [PATCH 3/3] scsi: ufs: ufs-qcom: Apply DELAY_AFTER_LPM quirk for Toshiba devices
  - From: MANISH PANDEY

References:
- [PATCH 0/3] add fixup_dev_quirks vops for ufs-qcom
  - From: Manish Pandey
- [PATCH 3/3] scsi: ufs: ufs-qcom: Apply DELAY_AFTER_LPM quirk for Toshiba devices
  - From: Manish Pandey

Prev by Date: Re: [PATCH 2/3] scsi: ufs: ufs-qcom: Add DELAY_BEFORE_LPM quirk for Micron and Skhynix
Next by Date: Re: (subset) [PATCH 1/2] dt-bindings: misc: qcom,fastrpc: document new domain ID
Previous by thread: [PATCH 3/3] scsi: ufs: ufs-qcom: Apply DELAY_AFTER_LPM quirk for Toshiba devices
Next by thread: Re: [PATCH 3/3] scsi: ufs: ufs-qcom: Apply DELAY_AFTER_LPM quirk for Toshiba devices
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [Linux for Sparc] [IETF Annouce] [Security] [Bugtraq] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]