Skip to content

Add ADC support to TI cc23x0 SoC #84520

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

jpanisbl
Copy link
Contributor

@jpanisbl jpanisbl commented Jan 24, 2025

This series adds ADC support to TI cc23x0 SoC.

Datasheet: https://www.ti.com/lit/ds/symlink/cc2340r5.pdf

@zephyrbot
Copy link
Collaborator

zephyrbot commented Jan 24, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@jpanisbl jpanisbl force-pushed the baylibre/upstream-cc23x0-adc branch from 6c2ddc0 to ffb8328 Compare February 4, 2025 14:05
@jpanisbl jpanisbl marked this pull request as ready for review February 4, 2025 16:23
@zephyrbot zephyrbot added area: ADC Analog-to-Digital Converter (ADC) platform: TI SimpleLink Texas Instruments SimpleLink MCU area: Samples Samples labels Feb 4, 2025
@kartben kartben added this to the v4.2.0 milestone Feb 17, 2025
@jpanisbl jpanisbl force-pushed the baylibre/upstream-cc23x0-adc branch 5 times, most recently from e3b0190 to 141167b Compare March 4, 2025 16:56
@jpanisbl jpanisbl force-pushed the baylibre/upstream-cc23x0-adc branch from 141167b to a020590 Compare March 6, 2025 17:35
@jpanisbl jpanisbl force-pushed the baylibre/upstream-cc23x0-adc branch from a020590 to 1f96440 Compare March 10, 2025 09:42
@jpanisbl jpanisbl force-pushed the baylibre/upstream-cc23x0-adc branch from 1f96440 to 2a33204 Compare March 31, 2025 08:14
Add support for 16-channel ADC to cc23x0 SoC. The driver supports the
following conversion modes:
- Single channel | Single conversion,
- Sequence of channels (up to 4) | Single conversion.

Signed-off-by: Julien Panis <[email protected]>
Add support for ADC to cc23x0 SoC.

Signed-off-by: Julien Panis <[email protected]>
Enable ADC.

Signed-off-by: Julien Panis <[email protected]>
Add LP-EM-CC2340R5 board support to ADC sample.

Signed-off-by: Julien Panis <[email protected]>
Add LP-EM-CC2340R5 board support to ADC sample.

Signed-off-by: Julien Panis <[email protected]>
@jpanisbl jpanisbl force-pushed the baylibre/upstream-cc23x0-adc branch from 2a33204 to 21175f8 Compare April 17, 2025 08:38
@jpanisbl
Copy link
Contributor Author

jpanisbl commented Apr 22, 2025

@kartben Hi. Would it be possible to add a couple of reviewers for this PR please ?
The feedback above were interesting (and I took them into account) but they were made by someone who does not have write access.

@kartben kartben requested a review from Copilot April 24, 2025 11:04
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds ADC support for the TI cc23x0 SoC by introducing a new device tree binding definition.

  • Introduces a new binding file for the TI cc23x0 ADC.
  • Specifies required properties including reg, interrupts, pinctrl-0, pinctrl-names, and io-channel definitions.
Files not reviewed (9)
  • boards/ti/lp_em_cc2340r5/lp_em_cc2340r5-pinctrl.dtsi: Language not supported
  • boards/ti/lp_em_cc2340r5/lp_em_cc2340r5.dts: Language not supported
  • drivers/adc/CMakeLists.txt: Language not supported
  • drivers/adc/Kconfig: Language not supported
  • drivers/adc/Kconfig.cc23x0: Language not supported
  • dts/arm/ti/cc23x0.dtsi: Language not supported
  • samples/drivers/adc/adc_dt/boards/lp_em_cc2340r5.overlay: Language not supported
  • samples/drivers/adc/adc_sequence/boards/lp_em_cc2340r5.conf: Language not supported
  • samples/drivers/adc/adc_sequence/boards/lp_em_cc2340r5.overlay: Language not supported

@jpanisbl
Copy link
Contributor Author

Hi @anangl @nashif Just pinging you for a review, when you are available for that (I know that you can be busy with other stuffs). Thank you.

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: Samples Samples platform: TI SimpleLink Texas Instruments SimpleLink MCU
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants