linux-kernel - Re: [PATCH v23] pwm: airoha: Add support for EN7581 SoC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aMAoxyG__5HoEcyv@smile.fi.intel.com>
Date: Tue, 9 Sep 2025 16:16:55 +0300
From: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
To: Christian Marangi <ansuelsmth@...il.com>
Cc: Uwe Kleine-König <ukleinek@...nel.org>,
	linux-kernel@...r.kernel.org, linux-pwm@...r.kernel.org,
	Benjamin Larsson <benjamin.larsson@...exis.eu>,
	AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
	Lorenzo Bianconi <lorenzo@...nel.org>
Subject: Re: [PATCH v23] pwm: airoha: Add support for EN7581 SoC

On Tue, Sep 09, 2025 at 01:51:11PM +0200, Christian Marangi wrote:
> On Tue, Sep 09, 2025 at 02:20:43PM +0300, Andy Shevchenko wrote:
> > On Mon, Sep 08, 2025 at 08:48:38PM +0200, Christian Marangi wrote:
> > > On Fri, Aug 01, 2025 at 11:15:41AM +0200, Uwe Kleine-König wrote:
> > > > On Tue, Jul 08, 2025 at 04:50:52PM +0200, Christian Marangi wrote:

> > > > > +	 * Period goes at 4ns step, normalize it to check if we can
> > > > 
> > > > 4 ms or 4 ns?
> > > 
> > > 4ms you are right
> > 
> > One small but important side note (to Uwe as well, however he seems
> > follows what I am about to say). Recently I discovered a nice wrap-up [1]
> > on the writing values with units. And I think we should try to follow it
> > (at bare minimum to be consistent with chosen — Journalism vs. Scientific —
> >  style).
> > 
> > [1]: https://poynton.ca/notes/units/
> 
> Just to be more precise, on comments we should use NUMBER SPACE UNIT? I
> think for variables it's problematic.

Of course the note only about documentation and comments, not about the code :-)
And yes, $NUMBER space $UNIT (personally I prefer the Scientific way).

-- 
With Best Regards,
Andy Shevchenko