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] [day] [month] [year] [list]

Message-ID: <dmtb46zsvj4hnn455n4gjhabl3sgpglcb4meoe3glwlzcpvqm7@3u7ds2rdmsve>
Date: Fri, 26 Sep 2025 19:41:29 +0200
From: Uwe Kleine-König <ukleinek@...nel.org>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
Cc: Christian Marangi <ansuelsmth@...il.com>, 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 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/

Looking through that, that matches my expectations, apart from:

	Use big K for the multiplier (1024) common in computing

and

	When applied to raw bits, bytes or pixels:

	    M (mega) denotes (1024 K); and
	    G (giga) denotes . 

. In my bubble at least Ki, Mi, Gi, Ti etc. is usual for base-2
prefixes, see https://en.wikipedia.org/wiki/Binary_prefix for a
reference which also has:

	Prior to the 1999 IEC standard, some industry organizations,
	such as the Joint Electron Device Engineering Council (JEDEC),
	noted the common use of the terms kilobyte, megabyte, and
	gigabyte, and the corresponding symbols KB, MB, and GB in the
	binary sense, for use in storage capacity measurements. However,
	other computer industry sectors (such as magnetic storage)
	continued using those same terms and symbols with the decimal
	meaning. Since then, the major standards organizations have
	expressly disapproved the use of SI prefixes to denote binary
	multiples, and recommended or mandated the use of the IEC
	prefixes for that purpose, but the use of SI prefixes in this
	sense has persisted in some fields. 

Best regards
Uwe

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)