- ReplyType: The driver already checks for NACK elsewhere.
- RequestId: The link address reply will always carry the LINK_ADDRESS tag.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
Both RX and TX cases use the CRC functions so having *Tx* in the function name
is incorrect.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
Larger messages will need to be split into multiple fragments.
- The MsgData member carries all information.
- The FragmentNum member indicates the last fragment processed.
- The header reflects the latest fragment.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
1) Down request for MST sideband messages TX to RX.
2) Down reply for MST sideband messages from RX to TX.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
The IP documentation "recommends" enabling the following:
- Issuing reset at every training iteration.
- Issuing reset at every link rate change.
- Issuing reset at start of training pattern 1.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
No need for this since the handler will never be called if the interrupts are
masked in the first place.
Also, by default, nested interrupts are disabled and it is up to the user to
explicitly request nested interrupts - allow this.
The interrupt mask should already apply in hardware.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
Even if link training succeeds, sometimes, the link will fall if the link is
enabled prior to the training pattern being turned off.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
As required by DisplayPort compliance, the clock recovery sequence should
fail if all lanes haven't completed clock recovery on the first attempt
using the maximum voltage swing supported.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
Prior to initiating a new AUX transaction, always ensure that any pending
transactions have completed.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
AUX reply now makes use of the REPLY_STATUS register rather than
INTERRUPT_STATUS. It seems that INTERRUPT_STATUS may be updated before
REPLY_STATUS gets updated, resulting in a race condition.
Signed-off-by: Andrei-Liviu Simion <andrei.simion@xilinx.com>
