Re: [PATCH V1 2/2] scsi: ufs: ufs-qcom: check host controller state

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

To: Nitin Rawat <quic_nitirawa@xxxxxxxxxxx>, powen.kao@xxxxxxxxxxxx, alim.akhtar@xxxxxxxxxxx, adrian.hunter@xxxxxxxxx, jejb@xxxxxxxxxxxxx, stanley.chu@xxxxxxxxxxxx, asutoshd@xxxxxxxxxxxxxx, quic_cang@xxxxxxxxxxx, mani@xxxxxxxxxx, martin.petersen@xxxxxxxxxx, beanhuo@xxxxxxxxxx, ebiggers@xxxxxxxxxx, agross@xxxxxxxxxx, Arthur.Simchaev@xxxxxxx, konrad.dybcio@xxxxxxxxxx
Subject: Re: [PATCH V1 2/2] scsi: ufs: ufs-qcom: check host controller state
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Tue, 25 Jul 2023 12:56:27 -0700
Cc: quic_ziqichen@xxxxxxxxxxx, quic_nguyenb@xxxxxxxxxxx, quic_narepall@xxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx, Manish Pandey <quic_mapa@xxxxxxxxxxx>
In-reply-to: <20230725192710.26698-3-quic_nitirawa@quicinc.com>
References: <20230725192710.26698-1-quic_nitirawa@quicinc.com> <20230725192710.26698-3-quic_nitirawa@quicinc.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0

On 7/25/23 12:27, Nitin Rawat wrote:

Check host controller state before sending hibern8 command.

A patch description should mention what has been changed and also why.The above explains what has been changed but not why. Please explain inthe commit message why this change is necessary.


Thanks,

Bart.

References:
- [PATCH V1 0/2] Fix hibern8 enter failure during host reset
  - From: Nitin Rawat
- [PATCH V1 2/2] scsi: ufs: ufs-qcom: check host controller state
  - From: Nitin Rawat

Prev by Date: Re: [PATCH v7 08/13] coresight-tpdm: Add node to set dsb programming mode
Next by Date: Re: [PATCH v2] net: Explicitly include correct DT includes
Previous by thread: [PATCH V1 2/2] scsi: ufs: ufs-qcom: check host controller state
Next by thread: Re: [PATCH V1 2/2] scsi: ufs: ufs-qcom: check host controller state
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]