On 10/10/2024 7:21 AM, Karol Kolacinski wrote:
> E825 products have incorrect initialization procedure, which may lead to
> initialization failures and register values.
>
> Fix E825 products initialization by adding correct sync delay, checking
> the PHY revision only for current PHY and adding proper destination
> device when reading port/quad.
>
> In addition, E825 uses PF ID for indexing per PF registers and as
> a primary PHY lane number, which is incorrect.
>
> Karol Kolacinski (4):
> ice: Fix E825 initialization
> ice: Fix quad registers read on E825
> ice: Fix ETH56G FC-FEC Rx offset value
> ice: Add correct PHY lane assignment
>
I applied these to net-queue, but they don't apply cleanly to next
queue, due to conflicts with the PHY model changes and some other issues.
I also had some questions on one of the patches.
Thanks,
Jake
> drivers/net/ethernet/intel/ice/ice_common.c | 42 ++-
> drivers/net/ethernet/intel/ice/ice_common.h | 1 +
> drivers/net/ethernet/intel/ice/ice_main.c | 6 +-
> drivers/net/ethernet/intel/ice/ice_ptp.c | 23 +-
> drivers/net/ethernet/intel/ice/ice_ptp.h | 4 +-
> .../net/ethernet/intel/ice/ice_ptp_consts.h | 77 +----
> drivers/net/ethernet/intel/ice/ice_ptp_hw.c | 281 ++++++++++--------
> drivers/net/ethernet/intel/ice/ice_ptp_hw.h | 40 ++-
> drivers/net/ethernet/intel/ice/ice_sbq_cmd.h | 7 +-
> drivers/net/ethernet/intel/ice/ice_type.h | 2 -
> 10 files changed, 243 insertions(+), 240 deletions(-)
>
> V1 -> V2: Removed net-next hunks from "ice: Fix E825 initialization",
> whole "ice: Remove unnecessary offset calculation for PF
> scoped registers" patch and fixed kdoc in "ice: Fix quad
> registers read on E825"
>
> base-commit: ab8024f0207eec0f376591c958b5bd30a03ae370
