r8169: remove not needed check in rtl_fw_write_firmware
authorHeiner Kallweit <hkallweit1@gmail.com>
Thu, 23 Nov 2023 09:53:26 +0000 (10:53 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 24 Nov 2023 15:27:22 +0000 (15:27 +0000)
This check can never be true for a firmware file with a correct format.
Existing checks in rtl_fw_data_ok() are sufficient, no problems with
invalid firmware files are known.

Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/r8169_firmware.c

index cbc6b84..ed6e721 100644 (file)
@@ -151,9 +151,6 @@ void rtl_fw_write_firmware(struct rtl8169_private *tp, struct rtl_fw *rtl_fw)
                u32 regno = (action & 0x0fff0000) >> 16;
                enum rtl_fw_opcode opcode = action >> 28;
 
-               if (!action)
-                       break;
-
                switch (opcode) {
                case PHY_READ:
                        predata = fw_read(tp, regno);