Re: [PATCH v2 1/2] scsi: ufs: Refector ufshcd_setup_clocks() to remove skip_ref_clk

From: Can Guo
Date: Wed Nov 25 2020 - 07:28:37 EST


On 2020-11-25 19:54, Bean Huo wrote:
On Wed, 2020-11-25 at 08:53 +0800, Can Guo wrote:
> > + bool always_on_while_link_active;
>
> Can,
> using a sentence as a parameter name looks a little bit clumsy to
> me.
> The meaning has been explained in the comments section. How about
> simplify it and in line with other parameters in the structure?
>

Do you have a better name in mind?

no specail input in mind, maybe just "bool eternal_on"

It is like plain "always_on", but it cannot tell the whole story.
If it is not something crutial, let's just let it go first so long
as it does not break the original functionality. What do you say?

Thanks,

Can Guo.


Thanks,

Can Guo.