Re: [net-next PATCH v4] octeontx2-pf: Use PTP HW timestamp counter atomic update feature
From: Leon Romanovsky
Date: Tue Aug 22 2023 - 10:15:58 EST
On Mon, Aug 21, 2023 at 04:06:29PM +0530, Sai Krishna wrote:
> Some of the newer silicon versions in CN10K series supports a feature
> where in the current PTP timestamp in HW can be updated atomically
> without losing any cpu cycles unlike read/modify/write register.
> This patch uses this feature so that PTP accuracy can be improved
> while adjusting the master offset in HW. There is no need for SW
> timecounter when using this feature. So removed references to SW
> timecounter wherever appropriate.
>
> Signed-off-by: Sai Krishna <saikrishnag@xxxxxxxxxxx>
> Signed-off-by: Naveen Mamindlapalli <naveenm@xxxxxxxxxxx>
> Signed-off-by: Sunil Kovvuri Goutham <sgoutham@xxxxxxxxxxx>
> Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@xxxxxxxxxxxx>
> ---
> v4:
> - Addressed review comments given by Leon Romanovsky
> 1. Unlocked mutex in error conditions.
> v3:
> - Addressed review comments given by Jakub Kicinski
> 1. Fixed re-ordering of headers in alphabetical order
> 2. Refactored SoC revision identification logic
> 3. CN10K errata revisions can be different from atomic update
> supported revision devices.
> 4. Removed ptp device check.
> v2:
> - Addressed review comments given by Simon Horman, Kalesh Anakkur Purayil
> 1. Removed inline keyword for function in .c file
> 2. Modified/optimized conditions related boolean
>
> .../net/ethernet/marvell/octeontx2/af/mbox.h | 12 ++
> .../net/ethernet/marvell/octeontx2/af/ptp.c | 155 +++++++++++++--
> .../net/ethernet/marvell/octeontx2/af/ptp.h | 3 +-
> .../net/ethernet/marvell/octeontx2/af/rvu.c | 2 +-
> .../net/ethernet/marvell/octeontx2/af/rvu.h | 12 ++
> .../marvell/octeontx2/nic/otx2_common.h | 1 +
> .../ethernet/marvell/octeontx2/nic/otx2_ptp.c | 177 ++++++++++++++----
> 7 files changed, 307 insertions(+), 55 deletions(-)
>
Thanks,
Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxx>