Commit 9d2a1cea authored by Peng Li's avatar Peng Li Committed by David S. Miller

net: hns3: remove the shaper param magic number

To make the code more readable, this patch adds a definition for
the magic number 126 used for the default shaper param ir_b, and
rename macro DIVISOR_IR_B_126.

No functional change.
Signed-off-by: default avatarPeng Li <lipeng321@huawei.com>
Signed-off-by: default avatarHuazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent ae9e492a
...@@ -41,8 +41,9 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level, ...@@ -41,8 +41,9 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level,
struct hclge_shaper_ir_para *ir_para, struct hclge_shaper_ir_para *ir_para,
u32 max_tm_rate) u32 max_tm_rate)
{ {
#define DEFAULT_SHAPER_IR_B 126
#define DIVISOR_CLK (1000 * 8) #define DIVISOR_CLK (1000 * 8)
#define DIVISOR_IR_B_126 (126 * DIVISOR_CLK) #define DEFAULT_DIVISOR_IR_B (DEFAULT_SHAPER_IR_B * DIVISOR_CLK)
static const u16 tick_array[HCLGE_SHAPER_LVL_CNT] = { static const u16 tick_array[HCLGE_SHAPER_LVL_CNT] = {
6 * 256, /* Prioriy level */ 6 * 256, /* Prioriy level */
...@@ -69,10 +70,10 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level, ...@@ -69,10 +70,10 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level,
* ir_calc = ---------------- * 1000 * ir_calc = ---------------- * 1000
* tick * 1 * tick * 1
*/ */
ir_calc = (DIVISOR_IR_B_126 + (tick >> 1) - 1) / tick; ir_calc = (DEFAULT_DIVISOR_IR_B + (tick >> 1) - 1) / tick;
if (ir_calc == ir) { if (ir_calc == ir) {
ir_para->ir_b = 126; ir_para->ir_b = DEFAULT_SHAPER_IR_B;
ir_para->ir_u = 0; ir_para->ir_u = 0;
ir_para->ir_s = 0; ir_para->ir_s = 0;
...@@ -81,7 +82,8 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level, ...@@ -81,7 +82,8 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level,
/* Increasing the denominator to select ir_s value */ /* Increasing the denominator to select ir_s value */
while (ir_calc >= ir && ir) { while (ir_calc >= ir && ir) {
ir_s_calc++; ir_s_calc++;
ir_calc = DIVISOR_IR_B_126 / (tick * (1 << ir_s_calc)); ir_calc = DEFAULT_DIVISOR_IR_B /
(tick * (1 << ir_s_calc));
} }
ir_para->ir_b = (ir * tick * (1 << ir_s_calc) + ir_para->ir_b = (ir * tick * (1 << ir_s_calc) +
...@@ -92,12 +94,12 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level, ...@@ -92,12 +94,12 @@ static int hclge_shaper_para_calc(u32 ir, u8 shaper_level,
while (ir_calc < ir) { while (ir_calc < ir) {
ir_u_calc++; ir_u_calc++;
numerator = DIVISOR_IR_B_126 * (1 << ir_u_calc); numerator = DEFAULT_DIVISOR_IR_B * (1 << ir_u_calc);
ir_calc = (numerator + (tick >> 1)) / tick; ir_calc = (numerator + (tick >> 1)) / tick;
} }
if (ir_calc == ir) { if (ir_calc == ir) {
ir_para->ir_b = 126; ir_para->ir_b = DEFAULT_SHAPER_IR_B;
} else { } else {
u32 denominator = DIVISOR_CLK * (1 << --ir_u_calc); u32 denominator = DIVISOR_CLK * (1 << --ir_u_calc);
ir_para->ir_b = (ir * tick + (denominator >> 1)) / ir_para->ir_b = (ir * tick + (denominator >> 1)) /
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment