diff options
author | Evan Wang <[email protected]> | 2021-07-22 16:40:38 +0200 |
---|---|---|
committer | Lorenzo Pieralisi <[email protected]> | 2021-08-05 10:51:49 +0100 |
commit | fcb461e2bc8b83b7eaca20cb2221e8b940f2189c (patch) | |
tree | bce95b4d1647cd741914d4266cec34e5115e9ff1 /lib/test_overflow.c | |
parent | e73f0f0ee7541171d89f2e2491130c7771ba58d3 (diff) |
PCI: aardvark: Fix checking for PIO status
There is an issue that when PCIe switch is connected to an Armada 3700
board, there will be lots of warnings about PIO errors when reading the
config space. According to Aardvark PIO read and write sequence in HW
specification, the current way to check PIO status has the following
issues:
1) For PIO read operation, it reports the error message, which should be
avoided according to HW specification.
2) For PIO read and write operations, it only checks PIO operation complete
status, which is not enough, and error status should also be checked.
This patch aligns the code with Aardvark PIO read and write sequence in HW
specification on PIO status check and fix the warnings when reading config
space.
[pali: Fix CRS handling when CRSSVE is not enabled]
Link: https://lore.kernel.org/r/[email protected]
Tested-by: Victor Gu <[email protected]>
Signed-off-by: Evan Wang <[email protected]>
Signed-off-by: Pali Rohár <[email protected]>
Signed-off-by: Lorenzo Pieralisi <[email protected]>
Reviewed-by: Victor Gu <[email protected]>
Reviewed-by: Marek Behún <[email protected]>
Cc: [email protected] # b1bd5714472c ("PCI: aardvark: Indicate error in 'val' when config read fails")
Diffstat (limited to 'lib/test_overflow.c')
0 files changed, 0 insertions, 0 deletions