mbox series

[v1,0/5] ufs: core: mcq: Add ufshcd_abort() and error handler support in MCQ mode

Message ID cover.1680083571.git.quic_nguyenb@quicinc.com (mailing list archive)
Headers show
Series ufs: core: mcq: Add ufshcd_abort() and error handler support in MCQ mode | expand

Message

Bao D. Nguyen March 29, 2023, 10:01 a.m. UTC 
This patch series enable support for ufshcd_abort() and error handler in MCQ mode.
The first 3 patches are for supporting ufshcd_abort().
The 4th and 5th patches are for supporting error handler.

Bao D. Nguyen (5):
  ufs: mcq: Add supporting functions for mcq abort
  ufs: mcq: Add support for clean up mcq resources
  ufs: mcq: Added ufshcd_mcq_abort()
  ufs: mcq: Use ufshcd_mcq_poll_cqe_lock() in mcq mode
  ufs: core: Add error handling for MCQ mode

 drivers/ufs/core/ufs-mcq.c     | 242 ++++++++++++++++++++++++++++++++++++++++-
 drivers/ufs/core/ufshcd-priv.h |  15 ++-
 drivers/ufs/core/ufshcd.c      | 140 ++++++++++++++++++++----
 drivers/ufs/host/ufs-qcom.c    |   2 +-
 include/ufs/ufshcd.h           |   2 +-
 include/ufs/ufshci.h           |  17 +++
 6 files changed, 388 insertions(+), 30 deletions(-)

Comments

Bart Van Assche March 29, 2023, 8:47 p.m. UTC | #1 
On 3/29/23 03:01, Bao D. Nguyen wrote:
> This patch series enable support for ufshcd_abort() and error handler in MCQ mode.
> The first 3 patches are for supporting ufshcd_abort().
> The 4th and 5th patches are for supporting error handler.

Is this perhaps a resend of v1? Last time this series was posted it had 
version number 3.

Thanks,

Bart.
Bao D. Nguyen March 29, 2023, 8:50 p.m. UTC | #2 
On 3/29/2023 1:47 PM, Bart Van Assche wrote:
> On 3/29/23 03:01, Bao D. Nguyen wrote:
>> This patch series enable support for ufshcd_abort() and error handler 
>> in MCQ mode.
>> The first 3 patches are for supporting ufshcd_abort().
>> The 4th and 5th patches are for supporting error handler.
>
> Is this perhaps a resend of v1? Last time this series was posted it 
> had version number 3.
>
Hi Bart, I am not sure if I did it correctly or not. The last time I 
posted, it was with RFC tag in the subject. I have made some minor 
changes to address the comments and some bug fixes. I am posting the 
patches again without "RFC" tag now.

Thanks,

Bao

> Thanks,
>
> Bart.
Bart Van Assche March 29, 2023, 8:51 p.m. UTC | #3 
On 3/29/23 13:50, Bao D. Nguyen wrote:
> Hi Bart, I am not sure if I did it correctly or not. The last time I 
> posted, it was with RFC tag in the subject. I have made some minor 
> changes to address the comments and some bug fixes. I am posting the 
> patches again without "RFC" tag now.

The change history is missing. This could have been made clear in the 
change history. Anyway, I will take a closer look at these patches.

Bart.
Bao D. Nguyen March 29, 2023, 8:53 p.m. UTC | #4 
On 3/29/2023 1:51 PM, Bart Van Assche wrote:
> The change history is missing. This could have been made clear in the 
> change history. Anyway, I will take a closer look at these patches.
>
> Bart. 

Got it. I will add the change history back in the next revision.

Thanks,
Bao