lkml.org 
[lkml]   [2023]   [Jan]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] scsi: ufs: core: fix devfreq deadlocks
From
On 1/3/23 00:24, Avri Altman wrote:
>
>> On 12/22/22 02:21, Johan Hovold wrote:
>>> + /* Enable Write Booster if we have scaled up else disable it */
>>> + if (ufshcd_enable_wb_if_scaling_up(hba))
>>> + ufshcd_wb_toggle(hba, scale_up);
>>
>> Hi Asutosh,
>>
>> This patch is the second complaint about the mechanism that toggles the
>> WriteBooster during clock scaling. Can this mechanism be removed entirely?
> commit 87bd05016a64 that introduced UFSHCD_CAP_WB_WITH_CLK_SCALING enables
> the platform vendors and OEMs to maintain wb toggling - should they choose so.
> Why remove it in its entirety?

Hi Avri,

I'm in favor of keeping kernel code simple :-)

UFSHCD_CAP_WB_WITH_CLK_SCALING controls whether or not clock scaling
affects the WriteBooster depending on which host controller is in use.
Shouldn't this depend on which UFS device is present instead of on the
type of host controller?

Thanks,

Bart.

\
 
 \ /
  Last update: 2023-03-26 23:27    [W:0.070 / U:0.320 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site