ILAS(with_counter=True) overflow

enjoy-digital / litejesd204b

Small footprint and configurable JESD204B core

Other

40 stars 12 forks source link

ILAS(with_counter=True) overflow #2

Closed michael-betz closed 4 years ago

michael-betz commented 4 years ago

for octets_per_multiframe > 64, the counter producing dummy data overflows > 0xFF, which corrupts the ILAS sequence.

This prevented me from getting the link up on an AD9174.

This fixed it in link.py:

        for i in range(4):
            if with_counter:
                multiframe = [(i * octets_per_multiframe + j) & 0xFF
                    for j in range(octets_per_multiframe)]

enjoy-digital commented 4 years ago

Thanks for the feedback and fix, this is integrated with https://github.com/enjoy-digital/litejesd204b/commit/2575896e59e608d6b7d661a13bf7be977d079f8f.