Skip to content

Conversation

@PopPaul2021
Copy link
Contributor

PR Description

In the updated HDL project the CDC was moved from the DO to the DMA and the cache coherency was disabled in order to have the HP ports working at maximum frequency.

When the data offload is configured on the RX side the oneshot is enabled by default, if the user needs the offload to be bypassed the control register will be 0x3 instead of 0x1.

bit 0- controls the bypass
bit 1- controls the oneshot

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have compiled my changes, including the documentation
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly
  • I have provided links for the relevant upstream lore

When the data offload is configured on the RX side the oneshot
is enabled by default, if the user needs the offload to be bypassed
the control register will be 0x3 instead of 0x1.
bit 0- controls the bypass
bit 1- controls the oneshot

Signed-off-by: PopPaul2021 <[email protected]>
In the updated HDL project the CDC was moved from the
DO to the DMA and the cache coherency was disabled
in order to have the HP ports working at maximum frequency.

Signed-off-by: PopPaul2021 <[email protected]>
Copy link
Collaborator

@nunojsa nunojsa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems there various style complains. Please address them...

Also follow guidelines for commit messages in the kernel. Example, run git log --oneline arch/arm64/boot/dts/xilinx/ and see the style used the most (In our tree you won't likely see an unified style but I'm trying to change that :)). Same goes for the driver patch

@cseci
Copy link
Collaborator

cseci commented Dec 15, 2025

Linux Test Harness results now available here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants