Skip to content

I2C DW and Realtek driver comparison #87832

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

keith-zephyr
Copy link
Collaborator

This PR is for demonstration purposes only. This shows a comparison of the existing i2c_dw.c driver to the proposed Realtek I2C driver on #87106

Remove DMA support just to demonstrate similarity of the i2c_dw.c
driver to the Realtek proposoal on PR#87106

Signed-off-by: Keith Short <keithshort@google.com>
This takes the Realtek I2C driver proposed on PR#87106 and is applied
to the original i2c_dw.c driver.

The following changes from PR#81706 were made to make it easier to
review the material changes.

* Remove CONFIG_I2C_CALLBACK suport
* Rename "i2c_rts5912" to "i2c_dw"
* Rename local variables "bus" and "data" to "dw" to match i2c_dw.c
  convention

Signed-off-by: Keith Short <keithshort@google.com>
@keith-zephyr keith-zephyr added the DNM This PR should not be merged (Do Not Merge) label Mar 28, 2025
@github-actions github-actions bot requested a review from teburd March 28, 2025 16:06
@keith-zephyr keith-zephyr marked this pull request as draft March 28, 2025 16:16
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: I2C DNM This PR should not be merged (Do Not Merge) Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants