lkml.org 
[lkml]   [2020]   [Jul]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 1/4] scsi: ufs: Add checks before setting clk-gating states
Hi Bart,

On 2020-07-14 11:38, Bart Van Assche wrote:
> On 2020-07-13 19:28, Can Guo wrote:
>> @@ -2538,7 +2540,8 @@ static int ufshcd_queuecommand(struct Scsi_Host
>> *host, struct scsi_cmnd *cmd)
>> err = SCSI_MLQUEUE_HOST_BUSY;
>> goto out;
>> }
>> - WARN_ON(hba->clk_gating.state != CLKS_ON);
>> + if (ufshcd_is_clkgating_allowed(hba))
>> + WARN_ON(hba->clk_gating.state != CLKS_ON);
>
> A minor comment: has it been considered to change this into the
> following?
>
> WARN_ON(ufshcd_is_clkgating_allowed(hba) &&
> hba->clk_gating.state != CLKS_ON);
>
> Thanks,
>
> Bart.

Sure, will do that.

Thanks,

Can Guo.

\
 
 \ /
  Last update: 2020-07-14 06:13    [W:2.258 / U:0.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site