Add rt700 i3c support on cm33_cpu1 core #86833
Conversation
zephyrbot
requested review from
asemjonovs,
avisconti,
butok,
decsny,
DerekSnell,
dleach02,
EmilioCBen,
jeppenodgaard,
kartben,
MaureenHelm,
mmahadevan108,
teburd,
tristan-google,
ubieda and
yperess
| / { | ||
| aliases { | ||
| ambient-temp0 = &p3t1755; | ||
| }; | ||
| }; | ||
|
|
||
| &i3c3 { | ||
| status = "okay"; | ||
|
|
||
| i2c-scl-hz = <DT_FREQ_K(400)>; | ||
| i3c-scl-hz = <DT_FREQ_M(4)>; | ||
| i3c-od-scl-hz = <DT_FREQ_K(1500)>; | ||
|
|
||
| p3t1755: p3t1755@4800000236152a0090 { | ||
| compatible = "nxp,p3t1755"; | ||
| reg = <0x48 0x0236 0x152a0090>; | ||
| status = "okay"; | ||
| }; | ||
| }; |
There was a problem hiding this comment.
No need for an overlay, this can go in the board's dts
There was a problem hiding this comment.
Move content from .overlay to board's dts. Thank you
There was a problem hiding this comment.
ambient-temp0 still need to be defined in .overlay. Otherwise secure_mqtt_sensor_actuator will be included into CI because the filter condition is "ambient-temp0".
tests: sample.net.secure_mqtt_sensor_actuator: harness: net tags: - net - mqtt - sensors filter: dt_alias_exists("ambient-temp0") integration_platforms: - adi_eval_adin1110ebz
There was a problem hiding this comment.
Otherwise secure_mqtt_sensor_actuator will be included into CI because the filter condition is "ambient-temp0".
Why is this a problem?
There was a problem hiding this comment.
At first, I moved all overlay related content into the board level, but CI could not compile successfully on "secure_mqtt_sensor_actuator" case because this case was not supported now, but this case filter only whether there was any ambient_temp0 enabled.
lucien-nxp
force-pushed
the
add_rt700_i3c_support
branch
2 times, most recently
from
57b3dd1 to
96e894c
Compare
lucien-nxp
force-pushed
the
add_rt700_i3c_support
branch
from
96e894c to
3d9ac79
Compare
lucien-nxp
force-pushed
the
add_rt700_i3c_support
branch
from
f5b8edc to
dd30fc4
Compare
|
Blocked by #88305, waiting for merge. |
add i3c2/3 instances for cm33_cpu1 Signed-off-by: Lucien Zhao <[email protected]>
Add support for p3t1755 temperature sensor on RT700 cm33_cpu1 core Signed-off-by: Lucien Zhao <[email protected]>
lucien-nxp
force-pushed
the
add_rt700_i3c_support
branch
from
dd30fc4 to
bcef22f
Compare
Add support for p3t1755 temperature sensor on RT700 cm33_cpu1 core