On Mon, Apr 05, 2021 at 12:26:37PM +0000, Salil Mehta wrote:
Hi Leon, Thanks for the review.
From: Leon Romanovsky [mailto:leon@kernel.org] Sent: Sunday, April 4, 2021 7:26 AM To: Salil Mehta salil.mehta@huawei.com Cc: davem@davemloft.net; kuba@kernel.org; netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Linuxarm linuxarm@huawei.com; linuxarm@openeuler.org Subject: Re: [PATCH net 1/2] net: hns3: Remove the left over redundant check & assignment
On Sat, Apr 03, 2021 at 02:35:19AM +0100, Salil Mehta wrote:
This removes the left over check and assignment which is no longer used anywhere in the function and should have been removed as part of the below mentioned patch.
Fixes: 012fcb52f67c ("net: hns3: activate reset timer when calling
reset_event")
Signed-off-by: Salil Mehta salil.mehta@huawei.com
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 2 -- 1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index e3f81c7e0ce7..7ad0722383f5 100644 --- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c +++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c @@ -3976,8 +3976,6 @@ static void hclge_reset_event(struct pci_dev *pdev,
struct hnae3_handle *handle)
* want to make sure we throttle the reset request. Therefore, we will * not allow it again before 3*HZ times. */
- if (!handle)
handle = &hdev->vport[0].nic;The comment above should be updated too, and probably the signature of hclge_reset_event() worth to be changed.
Yes, true. Both the comment and the prototype will be updated in near future. I can assure you this did not go un-noticed during the change. There are some internal subtleties which I am trying to sort out. Those might come as part of different patch-set which deals with other related changes as well.
I can buy such explanation for the change in function signature, but have hard time to believe that extra commit is needed to change comment above.
Thanks
The current change(and some other) will pave the way for necessary refactoring Of the code being done.
Thanks
if (time_before(jiffies, (hdev->last_reset_time + HCLGE_RESET_INTERVAL))) { -- 2.17.1