Re: [PATCH] ethernet: micrel: Use DIV_ROUND_CLOSEST directly to make it readable
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
Re: [PATCH] ethernet: micrel: Use DIV_ROUND_CLOSEST directly to make it readable
From: zhong jiang 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 Applied to net-next.
Re: [PATCH] ethernet: micrel: Use DIV_ROUND_CLOSEST directly to make it readable
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 > > . >
Re: [PATCH] ethernet: micrel: Use DIV_ROUND_CLOSEST directly to make it readable
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
[PATCH] ethernet: micrel: Use DIV_ROUND_CLOSEST directly to make it readable
The kernel.h macro DIV_ROUND_CLOSEST performs the computation (x + d/2)/d but is perhaps more readable. Signed-off-by: zhong jiang --- drivers/net/ethernet/micrel/ksz884x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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; } -- 1.7.12.4