Skip to content

Conversation

@Han-Chen-BC
Copy link

work done:
Familiarized with StarFive2 DW-AXI-DMAC driver: register configs, descriptor management, channel scheduling, interrupts, and dmaengine ↔ virt-dma call paths.

Updated synchronization APIs to replace blocking locks with EVL-safe mechanisms, ensuring OOB and IB context compatibility without deadlocks.

Extended dmaengine to support cyclic transfers and OOB pulse mode for low-latency real-time tasks.

Introduced desc_submitted_oob queue for OOB descriptors, prioritizing them over IB descriptors.

Modified virt-dma framework to allow OOB descriptor “insertion” during IB transfers and ensure context-aware completion callbacks.

…lock to vchan_lock,the inband operation success
…ace all lock api in driver to do ib dma operation successfully
…ock to vchan_lock,the inband dma operation by spi is successful
…y use dma to send data to dac device.The spi path is still not working for now.
Copy link

@ruiqurm ruiqurm left a comment

Choose a reason for hiding this comment

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

Look good for me. Please remove some unused comments and files.

Copy link

Choose a reason for hiding this comment

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

What is the ib_test used for? Is it useless?

Copy link
Author

Choose a reason for hiding this comment

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

sorry,This is a test script for triggering in-band DMA transfers. Its functionality has already been integrated into test_tool. I will remove it soon.

config Outdated
@@ -0,0 +1,8599 @@
#
Copy link

Choose a reason for hiding this comment

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

Remove auto generated file

read_cpsr = readw(SSP_CPSR(spi_base_addr_virt));
read_imsc = readw(SSP_IMSC(spi_base_addr_virt));

pr_info("SPI CR0=0x%x\n",read_cr0);
Copy link

Choose a reason for hiding this comment

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

Cleanup the logging here.

Copy link
Author

Choose a reason for hiding this comment

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

okay,the useless stuff in this driver will be removed

@@ -0,0 +1,52 @@
-----BEGIN PGP PUBLIC KEY BLOCK-----
Copy link

Choose a reason for hiding this comment

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

Remove the gpg key.

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.

2 participants