dinyar / uGMTfirmware

Firmware for the CMS uGMT.
0 stars 4 forks source link

Phi offset not properly initialized for testbench #233

Closed dinyar closed 9 years ago

dinyar commented 9 years ago

Currently the phi offset is declared at 11 bit signed vector, but only the first 10 bits are actually relevant (as it will never be negative). This led to the MSB being initialized to 'X' in the testbench. This should be modified to use an unsigned vector for the phi offset and only do the extension/conversion to signed in the function that actually calculated absolute phi.