Skip to content

drivers/potentiometer : Driver for AD5161 #2597

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

Merged
merged 1 commit into from
May 27, 2025

Conversation

gprasadbng
Copy link
Collaborator

@gprasadbng gprasadbng commented May 12, 2025

Pull Request Description

This is the driver for the AD5161 digipot.

PR Type

  • Bug fix (change that fixes an issue)
  • [X ] New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • [X ] I have followed the Coding style guidelines
  • I have complied with the Submission Checklist
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • [X ] I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

Copy link
Contributor

@amiclaus amiclaus left a comment

Choose a reason for hiding this comment

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

please comply with the Submission Checklist as stated in the PR description.

@gprasadbng gprasadbng changed the title Driver for AD5161 drivers/potentiometer : Driver for AD5161 May 13, 2025
@gprasadbng gprasadbng force-pushed the Driver_for_AD5161 branch from 686f3c0 to 5612632 Compare May 13, 2025 11:25

/******************************************************************************/
/*************************** Types Declarations *******************************/
/******************************************************************************/
Copy link
Contributor

Choose a reason for hiding this comment

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

remove these comment blocks

};



Copy link
Contributor

@buha buha May 23, 2025

Choose a reason for hiding this comment

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

too many newlines


struct no_os_spi_desc *spi_desc;
/* Digital interface select GPIO descriptor */
struct no_os_gpio_desc *dis_gpio_desc;
Copy link
Contributor

Choose a reason for hiding this comment

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

be consistent in documenting these, some have a comment, some dont, also use /** comment */ syntax, to make it a doxygen comment

@gprasadbng gprasadbng force-pushed the Driver_for_AD5161 branch 2 times, most recently from b8118d5 to 15e3ee0 Compare May 27, 2025 05:43
Signed-off-by: gprasad <[email protected]>

Driver for potentiometer AD5161/AD5160. These devices uses the I2C
interface for programming.
@gprasadbng gprasadbng force-pushed the Driver_for_AD5161 branch from 15e3ee0 to 4f1286c Compare May 27, 2025 07:32
@buha buha merged commit e98557c into analogdevicesinc:main May 27, 2025
8 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants