Skip to content

Fix clang build#2870

Merged
nunojsa merged 2 commits into
mainfrom
staging/misc-fixes
Aug 4, 2025
Merged

Fix clang build#2870
nunojsa merged 2 commits into
mainfrom
staging/misc-fixes

Conversation

@nunojsa
Copy link
Copy Markdown
Collaborator

@nunojsa nunojsa commented Aug 1, 2025

PR Description

Remove some older bindings files and fix compilation for clang.

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly (if there is the case)

As it turns out, -Wold-style-declaration does not exist for clang which
will cause compiling with it to fail. Therefore, only do the warning
hack in case we're building with GCC.

Fixes: 789db64 ("iio: adc: fix -Wmissing-prototypes")
Signed-off-by: Nuno Sá <nuno.sa@analog.com>
Copy link
Copy Markdown
Collaborator

@gastmaier gastmaier left a comment

Choose a reason for hiding this comment

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

In favor of
Documentation/devicetree/bindings/iio/adc/adi%2Cad4000.yaml

We already have a proper .yaml binding for adf4350 and the ad400x driver
was also dropped (in favor of the upstream ad4000) so that let's also drop
the bindings for it.

Signed-off-by: Nuno Sá <nuno.sa@analog.com>
@nunojsa nunojsa force-pushed the staging/misc-fixes branch from 44137d2 to 34c09f7 Compare August 1, 2025 15:12
@nunojsa nunojsa merged commit dd75474 into main Aug 4, 2025
19 checks passed
@nunojsa nunojsa deleted the staging/misc-fixes branch August 4, 2025 11:47
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