while (i--)
                platform_device_unregister(priv->pdev_dpu[i]);
 fail_platform_wd_register:
-       while (--i >= 0)
+       while (i--)
                platform_device_unregister(priv->pdev_wd[i]);
 fail_platform_fan_register:
        if (mlxplat_regs_io)
 
        for (i = MLXPLAT_CPLD_DPU_MAX_DEVS - 1; i >= 0; i--)
                platform_device_unregister(priv->pdev_dpu[i]);
-       for (i = MLXPLAT_CPLD_WD_MAX_DEVS - 1; i >= 0 ; i--)
+       for (i = MLXPLAT_CPLD_WD_MAX_DEVS - 1; i >= 0; i--)
                platform_device_unregister(priv->pdev_wd[i]);
        if (priv->pdev_fan)
                platform_device_unregister(priv->pdev_fan);
        return mlxplat_i2c_mux_complition_notify(priv, NULL, NULL);
 
 fail_platform_mux_register:
-       while (--i >= 0)
+       while (i--)
                platform_device_unregister(priv->pdev_mux[i]);
        return err;
 }
 {
        int i;
 
-       for (i = mlxplat_mux_num - 1; i >= 0 ; i--) {
+       for (i = mlxplat_mux_num - 1; i >= 0; i--) {
                if (priv->pdev_mux[i])
                        platform_device_unregister(priv->pdev_mux[i]);
        }