cn7322 / cn2360 loading firmware just like its nothing

gistfile1.txt

[  765.142769] LiquidIO 0000:01:00.0: Initializing device 177d:9702.
[  765.142946] LiquidIO 0000:01:00.0: trs:64 max_vfs:52 rings_per_vf:1 pf_srn:52 num_pf_rings:12
[  776.352906] LiquidIO 0000:01:00.0: Firmware version: 1.7.2
[  776.352911] LiquidIO 0000:01:00.0: octeon_download_firmware: Loading 1 images
[  776.352915] LiquidIO 0000:01:00.0: Loading firmware 1285952 at 21000000
[  776.375532] LiquidIO 0000:01:00.0: Writing boot command: bootoct 0x21000000 numcores=$(numcores) time_sec=1774939247 time_nsec=539216810
[  776.403319] LiquidIO 0000:01:00.1: Initializing device 177d:9702.
[  776.403448] LiquidIO 0000:01:00.1: trs:64 max_vfs:52 rings_per_vf:1 pf_srn:52 num_pf_rings:12
[  779.282224] LiquidIO 0000:01:00.1: Running NIC (1500000000 Hz)
[  779.282237] LiquidIO 0000:01:00.0: Running NIC (1500000000 Hz)
[  779.926064] LiquidIO 0000:01:00.1: eth%d VLAN filter enabled
[  779.938085] LiquidIO 0000:01:00.1 eth0: RX Checksum Offload Enabled
[  779.949063] LiquidIO 0000:01:00.1 eth0: TX Checksum Offload Enabled
[  779.996062] LiquidIO 0000:01:00.0: eth%d VLAN filter enabled
[  780.008063] LiquidIO 0000:01:00.0 eth1: RX Checksum Offload Enabled
[  780.020060] LiquidIO 0000:01:00.0 eth1: TX Checksum Offload Enabled
[  807.766335] LiquidIO 0000:01:00.1 eth0: Link Down
[  807.766342] LiquidIO 0000:01:00.1 eth0: Max MTU changed from 1500 to 16000
[  807.768389] LiquidIO 0000:01:00.1: eth0 interface is opened
[  810.244896] LiquidIO 0000:01:00.0 eth1: Link Down
[  810.244902] LiquidIO 0000:01:00.0 eth1: Max MTU changed from 1500 to 16000
[  810.246497] LiquidIO 0000:01:00.0: eth1 interface is opened

wer'e at no_speed_setting for some reason that stays unclear to me.

static int lio_set_link_ksettings(struct net_device *netdev,
                                  const struct ethtool_link_ksettings *ecmd)
{
        const int speed = ecmd->base.speed;
        struct lio *lio = GET_LIO(netdev);
        struct oct_link_info *linfo;
        struct octeon_device *oct;

        oct = lio->oct_dev;

        linfo = &lio->linfo;

        if (!(oct->subsystem_id == OCTEON_CN2350_25GB_SUBSYS_ID ||
              oct->subsystem_id == OCTEON_CN2360_25GB_SUBSYS_ID))
                return -EOPNOTSUPP;

        if (oct->no_speed_setting) {
                dev_err(&oct->pci_dev->dev, "%s: Changing speed is not supported\n",
                        __func__);
                return -EOPNOTSUPP;
        }

octeon_device.h:#define OCTEON_CN2350_25GB_SUBSYS_ID 0X7177d
01:00.0 0200: 177d:9702 (rev 03)

IDK about that 7 but the rest seems fine?

what the hell ok anyway
I know the card can do 10g...

/** Driver identifies chips by these Ids, created by clubbing together
 *  DeviceId+RevisionId; Where Revision Id is not used to distinguish
 *  between chips, a value of 0 is used for revision id.
 */
#define  OCTEON_CN68XX                0x0091
#define  OCTEON_CN66XX                0x0092
#define  OCTEON_CN23XX_PF_VID         0x9702
#define  OCTEON_CN23XX_VF_VID         0x9712

/**RevisionId for the chips */
#define  OCTEON_CN23XX_REV_1_0        0x00
#define  OCTEON_CN23XX_REV_1_1        0x01
#define  OCTEON_CN23XX_REV_2_0        0x80 

/**SubsystemId for the chips */
#define  OCTEON_CN2350_10GB_SUBSYS_ID_1 0X3177d
#define  OCTEON_CN2350_10GB_SUBSYS_ID_2 0X4177d
#define  OCTEON_CN2360_10GB_SUBSYS_ID   0X5177d
#define  OCTEON_CN2350_25GB_SUBSYS_ID   0X7177d
#define  OCTEON_CN2360_25GB_SUBSYS_ID   0X6177d

So I have a rev03 and that is what? REV_2_0?
I have a 2360 so we should find, for the pf, which ethtool runs against

0x9702 , 0x80 , 0X6177d

lio_core.c is how we get no_speed_setting:

        if (retval == IQ_SEND_FAILED) { <--- we're not here
                dev_info(&oct->pci_dev->dev, "Failed to send soft command\n");
                octeon_free_soft_command(oct, sc);
                retval = -EIO;
        } else { <--- we're here
                retval = wait_for_sc_completion_timeout(oct, sc, 0); <--- we see many timeouts maybe we are here
                if (retval)
                        return retval; <---- but we don't return?

                retval = resp->status;  <--- idk
                if (retval) {  <--- we don't go here so seesms irrelevant
                        dev_err(&oct->pci_dev->dev,
                                "%s failed retval=%d\n", __func__, retval);
                        retval = -EIO;
                } else {  <---- we go here so we have not a retval?
                        u32 var;

                        var = be32_to_cpu((__force __be32)resp->speed); <--- no idea
                        oct->speed_setting = var; <--- 0xffff why
                        if (var == 0xffff) { <--- bad
                                /* unable to access boot variables <--- meanie!
                                 * get the default value based on the NIC type  <---- (see larger note)
                                 */
                                if (oct->subsystem_id ==
                                                OCTEON_CN2350_25GB_SUBSYS_ID ||
                                    oct->subsystem_id ==
                                                OCTEON_CN2360_25GB_SUBSYS_ID) { <--- match
                                        oct->no_speed_setting = 1; <--- set
                                        oct->speed_setting = 25; <--- set
                                } else {
                                        oct->speed_setting = 10;
                                }
                        }

                }
                WRITE_ONCE(sc->caller_is_done, true);
        }

        return retval;

note:
maybe we need to load a newer uboot first, once again, but then we won't be able to easily unload, but nowhere would you see that this wouldn't work just bc porti s down? or, wait? does it happen if the port is down at boot and no speed was set? and whatever? defaults to bad speed, no link, no speed_setting found, no way to set it?
that would support my impression that this is a bug of sorts