Message ID | 1567698828-26825-1-git-send-email-zhongjiang@huawei.com |
---|---|
State | Accepted |
Delegated to: | David Miller |
Headers | show |
Series | ethernet: micrel: Use DIV_ROUND_CLOSEST directly to make it readable | expand |
On Thu, Sep 05, 2019 at 11:53:48PM +0800, zhong jiang wrote: > The kernel.h macro DIV_ROUND_CLOSEST performs the computation (x + d/2)/d > but is perhaps more readable. Hi Zhong Did you find this by hand, or did you use a tool. If a tool is used, it is normal to give some credit to the tool. Thanks Andrew
On 2019/9/7 3:40, Andrew Lunn wrote: > On Thu, Sep 05, 2019 at 11:53:48PM +0800, zhong jiang wrote: >> The kernel.h macro DIV_ROUND_CLOSEST performs the computation (x + d/2)/d >> but is perhaps more readable. > Hi Zhong > > Did you find this by hand, or did you use a tool. If a tool is used, > it is normal to give some credit to the tool. With the following help of Coccinelle. -(((x) + ((__divisor) / 2)) / (__divisor)) + DIV_ROUND_CLOSEST(x,__divisor) Sometimes, I will add the information in the description. Sometimes, I desn't do that. I will certainly add the description when I send an series of patches to modify the case. Thanks, zhong jiang > Thanks > Andrew > > . >
From: zhong jiang <zhongjiang@huawei.com> Date: Thu, 5 Sep 2019 23:53:48 +0800 > The kernel.h macro DIV_ROUND_CLOSEST performs the computation (x + d/2)/d > but is perhaps more readable. > > Signed-off-by: zhong jiang <zhongjiang@huawei.com> Applied to net-next.
On Sat, Sep 07, 2019 at 11:14:00AM +0800, zhong jiang wrote: > On 2019/9/7 3:40, Andrew Lunn wrote: > > On Thu, Sep 05, 2019 at 11:53:48PM +0800, zhong jiang wrote: > >> The kernel.h macro DIV_ROUND_CLOSEST performs the computation (x + d/2)/d > >> but is perhaps more readable. > > Hi Zhong > > > > Did you find this by hand, or did you use a tool. If a tool is used, > > it is normal to give some credit to the tool. > With the following help of Coccinelle. It is good to mention Coccinelle or other such tools. They often exist because of university research work, and funding for such tools does depend on publicity of the tools, getting the credit they deserve. Andrew
diff --git a/drivers/net/ethernet/micrel/ksz884x.c b/drivers/net/ethernet/micrel/ksz884x.c index 3103446..e102e15 100644 --- a/drivers/net/ethernet/micrel/ksz884x.c +++ b/drivers/net/ethernet/micrel/ksz884x.c @@ -2166,7 +2166,7 @@ static void sw_get_broad_storm(struct ksz_hw *hw, u8 *percent) num = (data & BROADCAST_STORM_RATE_HI); num <<= 8; num |= (data & BROADCAST_STORM_RATE_LO) >> 8; - num = (num * 100 + BROADCAST_STORM_VALUE / 2) / BROADCAST_STORM_VALUE; + num = DIV_ROUND_CLOSEST(num * 100, BROADCAST_STORM_VALUE); *percent = (u8) num; }
The kernel.h macro DIV_ROUND_CLOSEST performs the computation (x + d/2)/d but is perhaps more readable. Signed-off-by: zhong jiang <zhongjiang@huawei.com> --- drivers/net/ethernet/micrel/ksz884x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)