Skip to content
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

Add mcxn236 i3c support #80488

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

NeilChen93
Copy link
Collaborator

Add mcxn236 i3c support

@@ -131,6 +131,10 @@
status = "okay";
};

&i3c1 {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Add I3C to the board YAML as well, like here:

Also, does this board have any sensor on the I3C bus we could enable here, to provide an easy way to validate the bus works?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hi @danieldegrasse , frdmmcxn236 board don't have I3C sensor.

Copy link
Contributor

Choose a reason for hiding this comment

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

FYI, some Freedom boards include the P3T1755 I3C temperature sensor. Mahesh just enabled this sensor in Add I3C support for FRDM-MCXN947. The N947 is a very similar SOC and shares the same I3C peripheral, so the driver is validated on that board.

For other boards without this sensor, NXP offers the P3T1755DP-ARD Arduino shield. @hakehuang has one of these shields so we can test boards without I3C sensors if needed.

Best regards

Copy link
Collaborator

@hakehuang hakehuang Nov 6, 2024

Choose a reason for hiding this comment

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

in P3T1755, the pin on arduino is lay as below: is it possible to align? as I can see the P1_11 is not compatible
image

image

@dleach02
Copy link
Member

dleach02 commented Nov 5, 2024

Update the board doc page with this added feature

Add i3c nodes to NXP MCXN23x dtsi file

Signed-off-by: Neil Chen <cheng.chen_1@nxp.com>
Support i3c for NXP frdm_mcxn236 board.

Signed-off-by: Neil Chen <cheng.chen_1@nxp.com>
Support the shield on FRDM_MCXN236 board by I2C and I3C way.

Signed-off-by: Neil Chen <cheng.chen_1@nxp.com>
@NeilChen93
Copy link
Collaborator Author

Hi @hakehuang , I already enabled MCXN236 I3C support according to Lucien's PR. But I also don't have local environment to test, could you help test it on your side? Thanks a lot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Samples Samples area: Sensors Sensors area: Shields Shields (add-on boards) platform: NXP NXP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants