ethernet/ti: delete if NULL check befort devm_kfree

devm_kfree check the pointer, there is no need to check before
devm_kfree call.
This change is to cleanup the code a bit.

Signed-off-by: Bernard Zhao <bernard@vivo.com>
Link: https://lore.kernel.org/r/20220516015208.6526-1-bernard@vivo.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>

diff --git a/drivers/net/ethernet/ti/am65-cpsw-qos.c b/drivers/net/ethernet/ti/am65-cpsw-qos.c
index aa32dd905e2bade6e2af78d34c464f63fb0d756a..e162771893af6f1a60e42813e38d776c2f9a3e59 100644 (file)
--- a/drivers/net/ethernet/ti/am65-cpsw-qos.c
+++ b/drivers/net/ethernet/ti/am65-cpsw-qos.c
@@ -166,8 +166,7 @@ static void am65_cpsw_admin_to_oper(struct net_device *ndev)
 {
        struct am65_cpsw_port *port = am65_ndev_to_port(ndev);
 
-       if (port->qos.est_oper)
-               devm_kfree(&ndev->dev, port->qos.est_oper);
+       devm_kfree(&ndev->dev, port->qos.est_oper);
 
        port->qos.est_oper = port->qos.est_admin;
        port->qos.est_admin = NULL;
@@ -434,11 +433,8 @@ static void am65_cpsw_purge_est(struct net_device *ndev)
 
        am65_cpsw_stop_est(ndev);
 
-       if (port->qos.est_admin)
-               devm_kfree(&ndev->dev, port->qos.est_admin);
-
-       if (port->qos.est_oper)
-               devm_kfree(&ndev->dev, port->qos.est_oper);
+       devm_kfree(&ndev->dev, port->qos.est_admin);
+       devm_kfree(&ndev->dev, port->qos.est_oper);
 
        port->qos.est_oper = NULL;
        port->qos.est_admin = NULL;
@@ -524,8 +520,7 @@ static int am65_cpsw_set_taprio(struct net_device *ndev, void *type_data)
        ret = am65_cpsw_configure_taprio(ndev, est_new);
        if (!ret) {
                if (taprio->enable) {
-                       if (port->qos.est_admin)
-                               devm_kfree(&ndev->dev, port->qos.est_admin);
+                       devm_kfree(&ndev->dev, port->qos.est_admin);
 
                        port->qos.est_admin = est_new;
                } else {