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

migrate some projects to new example infrastructure #2420

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

Conversation

buha
Copy link
Contributor

@buha buha commented Jan 23, 2025

This migrates several projects to the new example infrastructure, most important being adxl355 which was used as template for many others.

PR Type

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

PR Checklist

  • I have followed the Coding style guidelines
  • 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
  • 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

buha added 2 commits January 22, 2025 15:35
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
struct no_os_irq_ctrl_desc *nvic_desc;
struct no_os_irq_init_param nvic_ip = {
.platform_ops = &max_irq_ops,
};
struct no_os_gpio_desc *adxl355_gpio_desc;

/* Initialize DATA READY pin */
ret = no_os_gpio_get_optional(&adxl355_gpio_desc, &adxl355_gpio_drdy_ip);
Copy link
Contributor Author

@buha buha Jan 23, 2025

Choose a reason for hiding this comment

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

this DRDY pin is really weird, it's initialized for the iio_trigger_example (aducm3029 and maxim BUT NOT stm32!) but it's not used anywhere, so if anyone has an explanation for this let me know

buha added 5 commits January 27, 2025 11:00
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
@buha buha force-pushed the staging/examples3 branch from 7fd64a7 to 6a0c60c Compare January 27, 2025 09:01
Make the necessary changes to use the new example infrastructure.

Signed-off-by: Darius Berghe <[email protected]>
@buha buha force-pushed the staging/examples3 branch from 6a0c60c to 52820bb Compare January 27, 2025 12:45
@buha
Copy link
Contributor Author

buha commented Jan 27, 2025

V2:

  • attempt to fix CI build issues

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.

1 participant