[PATCH] pwm: pwm-meson: remove not needed check in meson_pwm_calc

[Uwe Kleine-König]

Hello Thierry,

On Thu, May 04, 2023 at 11:11:36AM +0200, Thierry Reding wrote:
> On Thu, May 04, 2023 at 08:47:30AM +0200, Uwe Kleine-König wrote:
> > On Thu, May 04, 2023 at 07:35:55AM +0200, Heiner Kallweit wrote:
> > > duty <= period implies that duty_cnt <= cnt. We verified before
> > > that cnt <= 0xffff, therefore we can omit the check here.
> > > 
> > > Suggested-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
> > > Signed-off-by: Heiner Kallweit <hkallweit1 at gmail.com>
> > > ---
> > > Note: Applies only after "pwm: pwm-meson: fix handling of period/duty if greater than UINT_MAX"
> > 
> > Do you know there is git magic to express this that (at least some of)
> > the build bots understand? Look at git format-patch's --base option.
> 
> I think --base is really only applicable if the base patch is applied to
> a well-known tree, otherwise the commit is unknown.

Yes, "base" should be something like v6.3. On a tree that has the patch
under discussion applied on top of "pwm: pwm-meson: fix handling of
period/duty if greater than UINT_MAX" on top of v6.3, 

	git format-patch --base v6.3 -1

results in the footers:

	base-commit: 457391b0380335d5e9a5babdec90ac53928b23b4
	prerequisite-patch-id: 76d61171becf4a80716637610b6f33d9617eb9b9

Where 457391b0380335d5e9a5babdec90ac53928b23b4 is v6.3 and
76d61171becf4a80716637610b6f33d9617eb9b9 is the patch-id[1] of the
UINT_MAX patch. While this doesn't replace some human readable hints,
this is the machine-parsable information that this patch is supposed to
be applied on top of that UINT patch. This doesn't help finding that
patch, but at least it stops reports about broken stuff in v6.3 + this
patch without the prerequisite patch. So if used in the right way, this
is helpful also in Heiner's case here.

Best regards
Uwe

[1] git show @^{/"pwm: pwm-meson: fix handling of period/duty if greater than UINT_MAX"} | git patch-id

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-amlogic/attachments/20230504/3d4f5eb3/attachment.sig>