Apparent typo in dts/arm/rpi_pico/rp2040.dtsi #70959
Assignees
Labels
bug
The issue is a bug, or the PR is fixing a bug
platform: Raspberry Pi Pico
Raspberry Pi Pico (RPi Pico)
priority: low
Low impact/importance bug
Comments
|
Hi @ThreeEights! We appreciate you submitting your first issue for our open-source project. 🌟 Even though I'm a bot, I can assure you that the whole community is genuinely grateful for your time and effort. 🤖💙 |
ThreeEights
added a commit
to ThreeEights/zephyr
that referenced
this issue
Apr 1, 2024
It was RPI_PICO_RESETS_RESET_I2C0, but should be RPI_PICO_RESETS_RESET_I2C1. Fixes: zephyrproject-rtos#70959 Signed-off-by: Steve Boylan <[email protected]>
fabiobaltieri
pushed a commit
that referenced
this issue
Apr 3, 2024
It was RPI_PICO_RESETS_RESET_I2C0, but should be RPI_PICO_RESETS_RESET_I2C1. Fixes: #70959 Signed-off-by: Steve Boylan <[email protected]>
Lahyllas
pushed a commit
to tsnlab/zephyr
that referenced
this issue
May 16, 2024
It was RPI_PICO_RESETS_RESET_I2C0, but should be RPI_PICO_RESETS_RESET_I2C1. Fixes: zephyrproject-rtos#70959 Signed-off-by: Steve Boylan <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The "resets" for i2c1 in dts/arm/rpi_pico/rp2040.dtsi is set to RPI_PICO_RESETS_RESET_I2C0, but that is the reset bit for i2c0.
Expected behavior
The correct (local) define for the reset bit for i2c1 is RPI_PICO_RESETS_RESET_I2C1.
Impact
This appears to be a minor issue, but can affect the interaction between the two I2C devices.
Additional context
Information on the RESET register is in section 2.14 of the RP2040 Datasheet.
The text was updated successfully, but these errors were encountered: