hdmi icon indicating copy to clipboard operation
hdmi copied to clipboard

improvement: replace parameters by inputs in module hdmi.sv

Open LMN128 opened this issue 4 years ago • 2 comments

Hi Sameer, I only propose to replace parameters in hdmi.sv module by inputs. It will be more handy for implementation. These values is useful to change as a option in project mainly:

VIDEO_ID_CODE
DVI_OUTPUT
VIDEO_REFRESH_RATE

Thanks a lot.

LMN128 avatar Apr 15 '20 14:04 LMN128

Hi Jan,

If VIDEO_ID_CODE is changed at runtime, are you going to update the pixel clock frequencies in the PLLs? Otherwise it would not work as an input.

For DVI_OUTPUT, are you deciding at runtime whether to output DVI or HDMI, depending on which the monitor supports?

The video refresh rate parameter does NOT change the refresh rate, it is used in audio clock calculations. If the refresh rate is 50Hz, 59,94Hz, 120Hz, etc. the audio will sound right as long as it is set correctly. It's floating point and I do not want to synthesize any floating point logic. It might be possible to encode all the possible refresh rates in an 8-bit coding though.

sameer avatar Apr 16 '20 17:04 sameer

i see, it is complicated change VIDEO_ID_CODE, but at least DVI_OUTPUT will be handy to possible change for me. Thanks

LMN128 avatar Apr 26 '20 08:04 LMN128