Skip to content

drivers: net: oa_tc6: Fix RTSA/RTSP handling#3065

Merged
CiprianRegus merged 2 commits intoanalogdevicesinc:mainfrom
BrentK-ADI:oaspi_rtsa_fix
Apr 20, 2026
Merged

drivers: net: oa_tc6: Fix RTSA/RTSP handling#3065
CiprianRegus merged 2 commits intoanalogdevicesinc:mainfrom
BrentK-ADI:oaspi_rtsa_fix

Conversation

@BrentK-ADI
Copy link
Copy Markdown
Collaborator

@BrentK-ADI BrentK-ADI commented Apr 9, 2026

Pull Request Description

Corrects the handling of the RTSA and RTSP flags. They are valid when SV=1. The implementation was setting them based on the EV=1 bit instead. This only has an impact if RX timestamping is enabled on the OASPI interface.

Adds support for the TSC flags in the transmit header to enable Tx timestamping. Also corrects a minor constant usage issue which has no functional impact since the values were the same.

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the Coding style guidelines
  • I have complied with the Submission Checklist
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

Corrects the handling of the RTSA and RTSP flags. They are valid when
SV=1.  The implementation was setting them based on the EV=1 bit instead.

Signed-off-by: Brent Kowal <brent.kowal@analog.com>
Adds the option for the driver/user to specify the TSC (Transmit Frame
Timestamp Capture) flags when submitting a Tx frame to the oa_tc6.

Corrects a minor issue using the wrong constant for the Header SV flag.
This has no functional impact since the constants are the same value.

Signed-off-by: Brent Kowal <brent.kowal@analog.com>
@BrentK-ADI BrentK-ADI marked this pull request as ready for review April 20, 2026 12:00
@CiprianRegus CiprianRegus merged commit 1cc397d into analogdevicesinc:main Apr 20, 2026
12 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants