Skip to content

Daq3 fixes#2353

Merged
mhennerich merged 3 commits intomasterfrom
daq3-fixes
Nov 22, 2023
Merged

Daq3 fixes#2353
mhennerich merged 3 commits intomasterfrom
daq3-fixes

Conversation

@mhennerich
Copy link
Copy Markdown
Contributor

No description provided.

Lane rate computation must use the JESD204 parameter N'(NP).

Fixes: 7d4673d ("iio: adc: ad9208: Initial HS ADC AD9208 family support")

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Only set vco_cfg when its initialized.

Fixes: 06897d6 ("iio: adc: ad9208: Enable ad9680 support in the ad9208 device driver")

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Add AD9528 to the topology since it declares itself as jesd204-device;
Since we have two separate typologies and only one clock chip.
Make sure we include it in the topology which gets initialized first.
Otherwise the CLK_SYNC state will reset the clocks and thus causing
a link drop. This requires that the clock chip defaults match the other
unhanded link.

Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
@mhennerich mhennerich requested a review from a team November 21, 2023 14:22
Copy link
Copy Markdown
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.

one minor nit....

jesd204-top-device = <0>; /* This is the TOP device */
jesd204-link-ids = <0>;
jesd204-inputs = <&axi_ad9680_core 0 0>;
//jesd204-ignore-errors;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: I would remove the line as it's a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's a useful comment for support purposes. If someone runs into this problem again. It's easier to just point to that line and instruct uncomment...

@mhennerich mhennerich merged commit 95e7fe8 into master Nov 22, 2023
@mhennerich mhennerich deleted the daq3-fixes branch November 22, 2023 09:00
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