Re: [PATCH] clk: qcom: clk-alpha-pll: Replace divide operator with comparison

From: Dan Carpenter
Date: Thu Aug 29 2024 - 05:15:01 EST


On Wed, Aug 28, 2024 at 11:44:20PM +0300, Dmitry Baryshkov wrote:
> > > > Reported-by: kernel test robot <lkp@xxxxxxxxx>
> > > > Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> > > > Closes: https://lore.kernel.org/r/202408110724.8pqbpDiD-lkp@xxxxxxxxx/
>
> this Closes tag must come after lkp's Reported-by. Please also add
> Closes with the link to Dan's report.
>

No, this one is okay. What happens is with some Smatch warnings, the bot
sends the email to me, I look it over and either discard or forward it on so
we get two Reported-bys for one email.

> >
> > There is also the above smatch warning that was reported.
>
> And the Smatch warning too should be a part of the commit message.
>
> Last, but not least, as it is a fix, there should be a Fixes: tag and
> optionally a cc:stable.
>

To be fair, at the time no one thought this was a Fix, just a cleanup.

regards,
dan carpenter