On 7/29/22 5:09 AM, Bart Van Assche wrote:
On 7/28/22 00:16, peter.wang@xxxxxxxxxxxx wrote:
Mediatek ufs do not want to toggle write booster when clock scaling.
This patch set allow vendor disable wb toggle in clock scaling.
I don't like this approach. Whether or not to toggle the write booster
when scaling the clock is not dependent on the host controller and
hence should not depend on the host controller driver.
Has it been considered to add a sysfs attribute in the UFS driver core
to control this behavior?
Thanks,
Bart.
Hi Bart,
Write booster binding with clock scaling is not make sense.
Clock scaling should always do clock scaling related things, and write
bootster is not related to clock, right?
So Mediatek don't want to toggle wb with clock scaling.
Consider legacy design is binding, so we provide a flag to decouple them
instead remove ufshcd_wb_toggle directly.
Or, do you think we can direct remove ufshcd_wb_toggle in clock scaling
and only let sysfs to control wb behavior?
Thanks.
Peter