Re: [PATCH 3/7] scsi: ufs: core: Simplify ufshcd_try_to_abort_task()

[Bart Van Assche <bvanassche@xxxxxxx>]

On 10/16/24 11:31 PM, Avri Altman wrote:
> > -                       /* Single Doorbell Mode */
> > -                       reg = ufshcd_readl(hba,
> > REG_UTP_TRANSFER_REQ_DOOR_BELL);
> > -                       if (reg & (1 << tag)) {
> > -                               /* sleep for max. 200us to stabilize */
> > -                               usleep_range(100, 200);
> > -                               continue;
> > -                       }
>
> If we no longer use the doorbell to determine the inflight requests,
> I think this should be your patch title, or at least a clear indication in your commit log.

Hmm ... I see this as an implementation detail. To me, the key
aspect of this patch is that the legacy and MCQ mode code paths
are combined into a single code path.

Thanks,

Bart.