• Jacob Keller's avatar
    ice: fix pin assignment for E810-T without SMA control · 5a7cee1c
    Jacob Keller authored
    Since commit 43c4958a ("ice: Merge pin initialization of E810 and E810T
    adapters"), the ice_ptp_setup_pins_e810() function has been used for both
    E810 and E810-T devices. The new implementation only distinguishes between
    whether the device has SMA control or not. It was assumed this is always
    true for E810-T devices. In addition, it does not set the n_per_out value
    appropriately when SMA control is enabled.
    
    In some cases, the E810-T device may not have access to SMA control. In
    that case, the E810-T device actually has access to fewer pins than a
    standard E810 device.
    
    Fix the implementation to correctly assign the appropriate pin counts for
    E810-T devices both with and without SMA control. The mentioned commit
    already includes the appropriate macro values for these pin counts but they
    were unused.
    
    Instead of assigning the default E810 values and then overwriting them,
    handle the cases separately in order of E810-T with SMA, E810-T without
    SMA, and then standard E810. This flow makes following the logic easier.
    
    Fixes: 43c4958a ("ice: Merge pin initialization of E810 and E810T adapters")
    Signed-off-by: default avatarJacob Keller <jacob.e.keller@intel.com>
    Tested-by: Sunitha Mekala <sunithax.d.mekala@intel.com> (A Contingent worker at Intel)
    Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
    5a7cee1c
ice_ptp.c 76.2 KB