Skip to content

arm64: dts: zynqmp-zcu102-rev10-adrv904x: fix AD9528 reset GPIO polarity - #3464

Open
stefpopa wants to merge 1 commit into
mainfrom
fix/ad9528-adrv904x-reset-gpio-convention
Open

arm64: dts: zynqmp-zcu102-rev10-adrv904x: fix AD9528 reset GPIO polarity#3464
stefpopa wants to merge 1 commit into
mainfrom
fix/ad9528-adrv904x-reset-gpio-convention

Conversation

@stefpopa

@stefpopa stefpopa commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

PR Description

The AD9528 driver was updated in commit 907be26 ("iio: frequency: ad9528: Fix reset logic") to follow Linux GPIO convention where reset GPIOs are active-high from the driver's viewpoint, with active-low polarity communicated via devicetree properties.

Add GPIO_ACTIVE_LOW flag to the AD9528 reset-gpios to match the updated driver behavior. Without this fix, the AD9528 probe fails with error -5 (EIO) because the chip remains in reset.

Fixes: 907be26 ("iio: frequency: ad9528: Fix reset logic")

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 compiled my changes, including the documentation
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly
  • I have provided links for the relevant upstream lore

The AD9528 driver was updated in commit 907be26 ("iio: frequency:
ad9528: Fix reset logic") to follow Linux GPIO convention where reset
GPIOs are active-high from the driver's viewpoint, with active-low
polarity communicated via devicetree properties.

Add GPIO_ACTIVE_LOW flag to the AD9528 reset-gpios to match the updated
driver behavior. Without this fix, the AD9528 probe fails with error -5
(EIO) because the chip remains in reset.

Fixes: 907be26 ("iio: frequency: ad9528: Fix reset logic")
Signed-off-by: Stefan Popa <stefan.popa@analog.com>
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.

2 participants