On Wed, Jun 26, 2024 at 02:11:32PM +0200, Burakov, Anatoly wrote:
> On 6/26/2024 1:42 PM, Anatoly Burakov wrote:
> > The Tx and Rx calibration and timestamp generation blocks are independent.
> > It is
> > not required to wait for both to complete before programming the offsets.
> > Instead, we can program the offset for each independently as soon as they
> > are
> > ready.
> >
> > Modify the ice_phy_cfg_tx_offset_e822 and ice_phy_cfg_rx_offset_e822
> > functions
> > with the following additions:
> >
> > a) Check the relevant P_REG_*_OR bit first and do nothing if the offset
> > has
> > already been programmed
> >
> > b) Check the P_REG_*_OV_STATUS bit to determine if the offset status is
> > ready,
> > meaning that the hardware vernier calibration steps have completed. Return
> > ICE_ERR_NOT_READY if the offset is not yet read.
> >
> > The driver can now simply call ice_phy_cfg_rx_offset_e822 and
> > ice_phy_cfg_tx_offset_e822 periodically until the offsets have been
> > programmed.
> >
> > In this way, the Tx and Rx blocks will be enabled independently without each
> > waiting on the other block to complete. This can enable faster calibration
> > on
> > some setups which have a low rate of either Tx or Rx traffic.
> >
> > Signed-off-by: Siddaraju DH <siddaraju...@intel.com>
> > Signed-off-by: Jacob Keller <jacob.e.kel...@intel.com>
> > Signed-off-by: Anatoly Burakov <anatoly.bura...@intel.com>
> > ---
>
> The patch has accidentally been submitted with incorrect authorship
> information, please amend on apply to being authored by Siddaraju DH (from
> first signoff line).
>
Will fix all such minor issues on apply, no problem.
thanks,
/Bruce
