Skip to content

Conversation

@BrentK-ADI
Copy link
Contributor

PR Description

Adds device tree parameter to enable the 3.3v logic levels for the ADF4030.

Register 0x5C (IRQ Masks and CMOS_OV) was part of the default register listing. This was moved to be manually configured in adf4030_startup to avoid having to configure CMOS_OV prior to testing the interface with the scratchpad then again after the default value was written.

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

Adds support to enable the 3.3v logic levels based on device tree
configuration through adi,cmos-3v3-enable.

Register 0x5C which contains interrupt masks and CMOS_OV field was
moved from the default register array to manually configured in
adf4030_startup.

Signed-off-by: Brent Kowal <brent.kowal@analog.com>
Adds the adi,cmos-3v3-enable property to the documentation.

Signed-off-by: Brent Kowal <brent.kowal@analog.com>
Copy link
Collaborator

@nunojsa nunojsa left a comment

Choose a reason for hiding this comment

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

Just one minor comment. Also, as a kind of process comment, bindings patches should come first.

description:
When set the CMOS output level is 3.3V. This applies to SPI
MUXOUT, and IRQB.(Default is 1.8V)
$ref: /schemas/types.yaml#/definitions/flag
Copy link
Collaborator

Choose a reason for hiding this comment

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

type: boolean instead of the $ref

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