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

drivers: adc: ltc2451: Add ltc2451 driver #64390

Merged

Conversation

ethan-duckett-brill
Copy link
Contributor

Adds support for the Linear Technologies LTC2451 ADC.

@zephyrbot zephyrbot added platform: ADI Analog Devices, Inc. area: ADC Analog-to-Digital Converter (ADC) area: Devicetree Binding PR modifies or adds a Device Tree binding labels Oct 25, 2023
Copy link
Member

@martinjaeger martinjaeger left a comment

Choose a reason for hiding this comment

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

Generally looks good. Only got some minor suggestions for improvement.

drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
dts/bindings/adc/lltc,ltc2451.yaml Outdated Show resolved Hide resolved
drivers/adc/Kconfig.ltc2451 Outdated Show resolved Hide resolved
drivers/adc/Kconfig.ltc2451 Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
dts/bindings/adc/lltc,ltc2451.yaml Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
@ethan-duckett-brill ethan-duckett-brill force-pushed the add_ltc2451_driver branch 4 times, most recently from 0da7d8d to a6ccdef Compare November 8, 2023 12:52
@ethan-duckett-brill ethan-duckett-brill marked this pull request as ready for review November 8, 2023 13:43
Copy link
Member

@martinjaeger martinjaeger left a comment

Choose a reason for hiding this comment

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

The driver needs to be added to the build_all unit tests by adding a devicetree node to tests/drivers/build_all/adc/app.overlay. See the recently merged #64592 as an example.

drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
@ethan-duckett-brill ethan-duckett-brill marked this pull request as draft November 8, 2023 14:30
@ethan-duckett-brill ethan-duckett-brill force-pushed the add_ltc2451_driver branch 2 times, most recently from 7c3e540 to 13fc81e Compare November 8, 2023 15:15
drivers/adc/Kconfig.ltc2451 Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
martinjaeger
martinjaeger previously approved these changes Nov 8, 2023
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
drivers/adc/adc_ltc2451.c Outdated Show resolved Hide resolved
Adds support for the Linear Technologies LTC2451 ADC.

Signed-off-by: Ethan Duckett <ethan.duckett@brillpower.com>
@ethan-duckett-brill
Copy link
Contributor Author

Thank you for the reviews guys, hopefully once this pipeline has finished we should be good to go!

Copy link
Member

@martinjaeger martinjaeger left a comment

Choose a reason for hiding this comment

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

Thanks @ethan-duckett-brill for the patience :)

Copy link
Contributor

@marcusfolkesson marcusfolkesson left a comment

Choose a reason for hiding this comment

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

LGTM

@carlescufi carlescufi merged commit da4e3e7 into zephyrproject-rtos:main Nov 13, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC) area: Devicetree Binding PR modifies or adds a Device Tree binding platform: ADI Analog Devices, Inc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants