Skip to content

Conversation

@ineskhou
Copy link
Contributor

@ineskhou ineskhou commented Nov 10, 2025

Load Switch Addition

Description

The LoadSwitch component is an active F' component that controls a single load switch output
through the gpioSet output port (connected to the platform's GPIO driver). It exposes two
async commands to turn the switch on and off, telemetry reporting the current state, and an
async Reset input which toggles the switch (off, short delay, on).

How Has This Been Tested?

Ran on the Cube. Lights turn on and off

  • Unit tests
  • Integration tests
  • Z Tests
  • Manual testing (describe steps)

Screenshots / Recordings (if applicable)

image

Checklist

  • Written detailed sdd with requirements, channels, ports, commands, telemetry defined and correctly formatted and spelled
  • Have written relevant integration tests and have documented them in the sdd
  • Have done a code review with
  • Have tested this PR on every supported board with correct board definitions

Further Notes / Considerations

kevinthegreat1 and others added 30 commits October 22, 2025 20:03
This reverts commit b5544e7.
@ineskhou ineskhou moved this to In review in v0.1-v0.3 Nov 11, 2025
@ineskhou
Copy link
Contributor Author

image

@hrfarmer hrfarmer mentioned this pull request Nov 11, 2025
8 tasks
Mikefly123
Mikefly123 previously approved these changes Nov 12, 2025
Copy link
Contributor

@Mikefly123 Mikefly123 left a comment

Choose a reason for hiding this comment

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

LGTM!

@moisesmata moisesmata merged commit f97253f into main Nov 12, 2025
2 of 3 checks passed
@moisesmata moisesmata deleted the load-switch branch November 12, 2025 01:18
@github-project-automation github-project-automation bot moved this from In review to Done in v0.1-v0.3 Nov 12, 2025
Comment on lines +308 to +318
charge: charge {

gpios = <&mcp23017 16 GPIO_ACTIVE_HIGH>; // GPB7
label = "CHARGE";
};

readonly: readonly {

gpios = <&mcp23017 15 GPIO_ACTIVE_HIGH>; // GPB6
label = "READONLY";
};
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't these two be 14 and 15 instead of 15 and 16?

Copy link
Contributor

@Mikefly123 Mikefly123 Nov 13, 2025

Choose a reason for hiding this comment

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

Oh you are right! Index at 0 and all that. @kevinthegreat1 if you want to make a quick PR to correct this please do!

image

@kevinthegreat1 kevinthegreat1 mentioned this pull request Nov 13, 2025
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

8 participants