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

Thierry Reding thierry.reding at gmail.com
Thu May 4 02:11:36 PDT 2023


On Thu, May 04, 2023 at 08:47:30AM +0200, Uwe Kleine-König wrote:
> Hello Heiner,
> 
> 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.

For things like this it's easier to just send them out as a series,
which implies the dependency. Obviously sometimes you get situations
where you send out a patch and only then realize there's cleanup you
can do, so mentioning this is usually enough. Most bots will end up
building this after I apply as well, so we'll get the coverage either
way.

Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-amlogic/attachments/20230504/e705c6ce/attachment.sig>


More information about the linux-amlogic mailing list