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

AD1939 bugs #2

Open
1 of 2 tasks
tvannoy opened this issue Apr 1, 2020 · 1 comment
Open
1 of 2 tasks

AD1939 bugs #2

tvannoy opened this issue Apr 1, 2020 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@tvannoy
Copy link
Member

tvannoy commented Apr 1, 2020

During testing of the programmable FIR filter, we've found a couple bugs in the AD1939 component

  • maxChannel was set to 0 in the tcl script, which caused any incoming channel greater than 0 to not be valid; this was causing the valid signal to never go high for the right channel.
  • left and right DAC channels get swapped in the VHDL code
@tvannoy
Copy link
Member Author

tvannoy commented Sep 14, 2020

It turns out the left and right DAC channels being swapped is more involved than just switching a few signals.

Right now, the left channel gets loaded in when lrclk = 1, even though the channel should be active when lrclk = 0. Vice versa for the right channel. The data from the parallel to serial converts gets shifted out after the load signal is dessarted. Changing the polarity of the load signals in the parallel to serial converters results in no audio output. It might be the case that the polarity of these signals should be inverted as it is right now, however the audio channels are currently being swapped in the output.

In simulation, the data that arrives while lrclk = 0 gets put onto the avalon bus when channel = 1. This might be the true source of the problem, rather than the inverted polarity of the parallel to serial converters. If that's the case, this suggests that something is off in the timing of the state machines that capture the incoming data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants