You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
7d7960e introduced externally visible speed signal in USBDevice. self.speed = Signal(2)
and it's updated from the reset sequencer: self.speed .eq(reset_sequencer.current_speed),
however, there is also an internal speed signal: speed = Signal(2, reset=USBSpeed.FULL)
This signal is passed into logic blocks, but is not updated at all.
It stays in its reset value.
IIUC, it should also be assigned to reset_sequencer.current_speed.
I wonder though, do we need two different signals in this case?
Probably better to just use the self.speed everywhere.
The text was updated successfully, but these errors were encountered:
Thank you for bringing this issue to our attention. I have talked with our development team and they have suggested I upgrade this issue to bug. I will leave this issue open until we have an updated solution to provide you.
7d7960e introduced externally visible
speed
signal in USBDevice.self.speed = Signal(2)
and it's updated from the reset sequencer:
self.speed .eq(reset_sequencer.current_speed),
however, there is also an internal speed signal:
speed = Signal(2, reset=USBSpeed.FULL)
This signal is passed into logic blocks, but is not updated at all.
It stays in its reset value.
IIUC, it should also be assigned to
reset_sequencer.current_speed
.I wonder though, do we need two different signals in this case?
Probably better to just use the
self.speed
everywhere.The text was updated successfully, but these errors were encountered: