Re: [PATCH v3 2/2] i2c: dwsignware: determine HS tHIGH and tLOW based on HW parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Oct 02, 2024 at 09:31:00AM +0000, Michael Wu wrote:
> > On Tue, Oct 01, 2024 at 04:29:34PM +0800, Michael Wu wrote:

...

> > > + * @bus_capacitance_pf: bus capacitance in picofarads
> > 
> > Since it seems a new version of the series is warranted, and looking into
> > the current kernel source (no other users of this unit were observed),
> > I think we may do correct capitalisation here for the sake of physics
> > and unit system, i.e.
> > 
> >  * @bus_capacitance_pF: bus capacitance in picofarads
> 
> Are you serious?

Are we in the circus?

> I have never seen capital letters used to declare a
> variable name. Although checkpatch.pl does not consider this as an issue,
> is this against everyone's custom or unspoken rules?

I do not really care about checkpatch or any rules as any good rule will have
a few exceptions anyway. The rationale here is to follow the science with much
longer history than anything related to the computer programming.

Now, since you haven't looked for the existing examples, try this

	git grep -n _u[AV]\;

-- 
With Best Regards,
Andy Shevchenko






[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux