Quoting Jian Dong (2021-03-15 20:07:25)

From: dongjian <redacted>

for best_rate's initial value is -EINVAL, if get tmp_rate belongs to
[0, req-rate] at first time, then best_rate will be updated to a positive
value, the value check will never run;
if never get the tmp_rate belongs to [0, req-rate], then first cycle will
end when parent get NULL, the best_rate value check also will never run.
So, the value check is redundant.

Signed-off-by: dongjian <redacted>
---

Please resend. It's been months and nobody has reviewed.