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

AD4050 support was added by adding appropriate bit resolutions #104

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

Shetty-Anush
Copy link
Contributor

@Shetty-Anush Shetty-Anush commented Oct 29, 2024

  1. Add appropriate bit resolutions for the devices
  2. Bug fix - Move if directives to suitable places
  3. Remove reference of adc drivers present in app folder from .extSettings
  4. Migrate CubeMX to 6.11.1

SaikiranGudla
SaikiranGudla previously approved these changes Oct 30, 2024
@jansunil jansunil added the peer-reviewed Piece of code that has already been peer reviewed elsewhere label Oct 30, 2024
jansunil
jansunil previously approved these changes Oct 30, 2024
SaikiranGudla
SaikiranGudla previously approved these changes Nov 7, 2024
@@ -549,7 +539,7 @@ static int iio_ad405x_attr_get(void *device,
if (ret) {
return ret;
}
#endif

Copy link
Contributor

Choose a reason for hiding this comment

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

Could we remove this extra line?

@jansunil
Copy link
Contributor

jansunil commented Nov 7, 2024

@Shetty-Anush Astyle checks seem to be failing in the builds. Could you also please look into this?

1. Add appropriate bit resolutions for the devices
2. Bug fix - Move if directives to suitable places
3. Remove reference of adc drivers present in app folder from
   .extSettings
4. Migrate CubeMX to 6.11.1

Signed-off-by: Anush Shetty <[email protected]>
@SaikiranGudla SaikiranGudla merged commit 8f05208 into analogdevicesinc:main Nov 7, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
peer-reviewed Piece of code that has already been peer reviewed elsewhere
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants