Messages in this thread | | | Subject | Re: [PATCH v2 1/2] scsi: ufs: Refector ufshcd_setup_clocks() to remove skip_ref_clk | From | Bean Huo <> | Date | Wed, 25 Nov 2020 12:54:19 +0100 |
| |
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"
> Thanks, > > Can Guo.
| |