Skip to content

Conversation

@tswaehn
Copy link
Contributor

@tswaehn tswaehn commented May 31, 2024

Before that fix, the default mrba was used; additionally the offset of "0" to the actual mcan memory location was used.

Fixes #68472

Copy link
Member

@henrikbrixandersen henrikbrixandersen left a comment

Choose a reason for hiding this comment

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

Please use regs as previously adviced.

@tswaehn
Copy link
Contributor Author

tswaehn commented Jun 1, 2024

Please use regs as previously adviced.

should be ready, as expected

@tswaehn
Copy link
Contributor Author

tswaehn commented Jun 2, 2024

@henrikbrixandersen ... understood, I will make the changes according your requestes.

@decsny decsny removed their request for review June 3, 2024 13:31
@tswaehn tswaehn force-pushed the test branch 2 times, most recently from b4a6be1 to a938238 Compare June 8, 2024 10:58
@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Jun 8, 2024
@zephyrbot zephyrbot requested review from aescolar, kartben and nashif June 8, 2024 11:49
@henrikbrixandersen
Copy link
Member

@tswaehn Ping?

@henrikbrixandersen
Copy link
Member

@tswaehn Do you plan to return to this PR? Zephyr v3.7.0 deadline for bug fixes is getting close. I'd like to see this one included.

@tswaehn
Copy link
Contributor Author

tswaehn commented Jun 27, 2024

@tswaehn Do you plan to return to this PR? Zephyr v3.7.0 deadline for bug fixes is getting close. I'd like to see this one included.

yes, I will try to get it done.

@tswaehn tswaehn force-pushed the test branch 3 times, most recently from d32fa43 to 4fde5d1 Compare June 27, 2024 17:06
@tswaehn
Copy link
Contributor Author

tswaehn commented Jun 27, 2024

I think, I covered all requested changes.

@tswaehn tswaehn force-pushed the test branch 2 times, most recently from 0da55c6 to 51d2644 Compare June 27, 2024 17:20
Before that fix, the default mrba was used; added
DMA Base Address to DTSI. Fixes zephyrproject-rtos#68472

Signed-off-by: Sven Ginka <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: CAN area: Devicetree Binding PR modifies or adds a Device Tree binding platform: Microchip SAM Microchip SAM Platform (formerly Atmel SAM) Release Notes To be mentioned in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[same70] CAN driver maybe broken?

6 participants