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

STUSB4500 USB-PD chip driver #574

Merged
merged 2 commits into from
Mar 10, 2021

Conversation

rleh
Copy link
Member

@rleh rleh commented Mar 9, 2021

Minimal but working implementation of a STUSB4500 driver.

Not actually tested on the STM32F072-Disco board, but on real hardware (because STUSB4500 package is too small for jumper wires).

cc @hshose

@rleh rleh requested a review from salkinium March 9, 2021 22:42
@rleh rleh force-pushed the feature/driver_stusb4500 branch from 596d439 to 20e188d Compare March 9, 2021 22:52
@hshose
Copy link
Contributor

hshose commented Mar 10, 2021

Just tested the module in the Otter Iron with STM32F072 - works as expected.

Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Thanks! I'm not convinced about the std::array usage.

src/modm/driver/usb/stusb4500.hpp Show resolved Hide resolved
src/modm/driver/usb/stusb4500.hpp Show resolved Hide resolved
src/modm/driver/usb/stusb4500.hpp Outdated Show resolved Hide resolved
src/modm/driver/usb/stusb4500.hpp Outdated Show resolved Hide resolved
src/modm/driver/usb/stusb4500.hpp Outdated Show resolved Hide resolved
src/modm/driver/usb/stusb4500.hpp Outdated Show resolved Hide resolved
src/modm/driver/usb/stusb4500.lb Outdated Show resolved Hide resolved
src/modm/driver/usb/stusb4500.lb Outdated Show resolved Hide resolved
@rleh rleh force-pushed the feature/driver_stusb4500 branch from 20e188d to fc79f79 Compare March 10, 2021 10:58
@rleh rleh requested a review from salkinium March 10, 2021 11:04
Copy link
Member

@salkinium salkinium left a comment

Choose a reason for hiding this comment

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

Nice!

@rleh rleh force-pushed the feature/driver_stusb4500 branch from fc79f79 to 1a11b08 Compare March 10, 2021 11:58
@rleh
Copy link
Member Author

rleh commented Mar 10, 2021

MacOS CI is drunk. Why don't we use a Docker image in MacOS?

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

You wrong about static asserts and arrays . Explain this and read the specifications.

src/modm/driver/usb/stusb4500.hpp Show resolved Hide resolved
@salkinium salkinium merged commit 1a11b08 into modm-io:develop Mar 10, 2021
@rleh rleh deleted the feature/driver_stusb4500 branch March 10, 2021 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants