[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <yvqxc75orn7t4iew3l74yesuclphcx7lfm7omyrid7gm2eq6ks@ulg5tcx75hyo>
Date: Sun, 30 Nov 2025 09:29:05 +0100
From: Uwe Kleine-König <ukleinek@...nel.org>
To: Biju <biju.das.au@...il.com>
Cc: Geert Uytterhoeven <geert+renesas@...der.be>,
Magnus Damm <magnus.damm@...il.com>, Biju Das <biju.das.jz@...renesas.com>, linux-pwm@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-renesas-soc@...r.kernel.org,
Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@...renesas.com>, Tommaso Merciai <tommaso.merciai.xr@...renesas.com>
Subject: Re: [PATCH v3 3/8] pwm: rzg2l-gpt: Add
prescale_pow_of_two_mult_factor variable to struct rzg2l_gpt_info
Hello Biju,
On Tue, Sep 23, 2025 at 03:45:07PM +0100, Biju wrote:
> @@ -91,6 +91,7 @@
>
> struct rzg2l_gpt_info {
> u32 gtcr_tpcs_mask;
> + u8 prescale_pow_of_two_mult_factor;
To reduce the line length a bit I suggest to call this just
"prescale_mult". That this is (up to now :-) a power of two isn't
relevant enough to be mentioned in the name.
Otherwise looks fine.
Best regards
Uwe
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists