Skip to content

Conversation

@maass-hamburg
Copy link
Member

add ti tps55287 regulator

@maass-hamburg maass-hamburg force-pushed the regulator_tps55287 branch 3 times, most recently from 1795b6e to c253c8d Compare November 3, 2025 11:14
regulator fix i2c.dtsi format

Signed-off-by: Fin Maaß <f.maass@vogl-electronic.com>
@maass-hamburg maass-hamburg force-pushed the regulator_tps55287 branch 2 times, most recently from e973246 to 0bb5b81 Compare November 10, 2025 17:50
@maass-hamburg maass-hamburg marked this pull request as ready for review November 10, 2025 17:51
@zephyrbot zephyrbot added area: Devicetree Bindings platform: TI SimpleLink Texas Instruments SimpleLink MCU area: Boards/SoCs area: Regulators area: Tests Issues related to a particular existing or missing test labels Nov 10, 2025
Copy link
Member

@dsseng dsseng left a comment

Choose a reason for hiding this comment

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

Thanks! Please see some of the non-critical, non-blocking comments and implement ones you consider worthwhile

Copy link
Member

@dsseng dsseng left a comment

Choose a reason for hiding this comment

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

Good, thank you for improvements. Please feel free to resolve my review threads

add ti tps55287 regulator

Signed-off-by: Fin Maaß <f.maass@vogl-electronic.com>
@maass-hamburg
Copy link
Member Author

init prio can be equal to the one of the i2c controller, because of #60410 as the devicetree ordinal is used as a sub-priority and the devicetree ordinal of a child is always higher that the parent, so it will be init after the parent.

@sonarqubecloud
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Boards/SoCs area: Devicetree Bindings area: Regulators area: Tests Issues related to a particular existing or missing test platform: TI SimpleLink Texas Instruments SimpleLink MCU

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants