search

Digilent / vivado-library

MIT License
563 stars 309 forks source link

ip: axi_dynclk: Make BUFMR optional #3

Closed sbobrowicz closed 8 years ago

sbobrowicz commented 8 years ago

A BUFMR was added on the output of the MMCM in order to get this core to work with Nexys Video. Doing this allows the MMCM that is instantiated to be in the bank above or below the bank attached to the HDMI port. This was necessary on the Nexys Video because both the HDMI input and output are on the same bank, and HDMI input requires an MMCM too. The problem with this work around is that it causes a timing error due to pulse width on the BUFMR for typical video frequencies, despite still working (at least in the case of the Nexys Video).

Adding the BUFMR needs to be brought up to the GUI as a parameter that can be disabled, since it is not necessary in most cases.

sbobrowicz commented 8 years ago

Fixed locally, still need to commit

sbobrowicz commented 8 years ago

pushed and tested