Skip to content

Conversation

@alvsun
Copy link
Contributor

@alvsun alvsun commented Nov 7, 2024

  1. add buffer mode callback api for DMA.
  2. add npcx i3c targt driver.

Copy link
Member

@fabiobaltieri fabiobaltieri left a comment

Choose a reason for hiding this comment

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

hey it looks like second commit has more than just the "add target driver", would it make sense to break it down to smaller changes? it's hard to follow as it is now

@alvsun alvsun force-pushed the npcx_i3c_target_drv branch 2 times, most recently from e162203 to a1baedd Compare November 11, 2024 10:56
@alvsun alvsun force-pushed the npcx_i3c_target_drv branch from a1baedd to 39e1aea Compare November 15, 2024 10:26
@XenuIsWatching XenuIsWatching self-assigned this Nov 17, 2024
@XenuIsWatching XenuIsWatching changed the title npcx i3c target driver drivers: i3c: npcx: i3c target driver Nov 17, 2024
@alvsun alvsun force-pushed the npcx_i3c_target_drv branch from 39e1aea to c77dfb2 Compare November 19, 2024 06:48
@alvsun alvsun force-pushed the npcx_i3c_target_drv branch from c77dfb2 to 26920a9 Compare November 20, 2024 10:33
Copy link
Member

@XenuIsWatching XenuIsWatching left a comment

Choose a reason for hiding this comment

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

it looks like there are some unrelated CI failures... you may need to rebase to get around it... but the PR now LGTM!

@alvsun alvsun force-pushed the npcx_i3c_target_drv branch from 26920a9 to 16d754f Compare November 25, 2024 03:27
@alvsun alvsun requested a review from fabiobaltieri November 25, 2024 08:12
@kartben kartben merged commit c9a95dd into zephyrproject-rtos:main Nov 25, 2024
24 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.

8 participants