PYNQ
PYNQ copied to clipboard
Move DFX memory mapped regions to the `mem_dict`
Work done in this PR
- Include bram controller in the
ip_dictI think it would be nice if the bram controller is included in theip_dictso thereadandwritemethods fromDefaultIPcan be used directly. - Move DFX memory mapped regions to the mem_dict This goes hand in hand with #1352
This fix Xilinx/PYNQ-DEV#361
Codecov Report
Merging #1331 (85477d1) into image_v2.8 (b85f453) will decrease coverage by
0.04%. The diff coverage is0.00%.
@@ Coverage Diff @@
## image_v2.8 #1331 +/- ##
==============================================
- Coverage 46.47% 46.43% -0.05%
==============================================
Files 28 28
Lines 4953 4962 +9
Branches 971 974 +3
==============================================
+ Hits 2302 2304 +2
- Misses 2594 2601 +7
Partials 57 57
| Impacted Files | Coverage Δ | |
|---|---|---|
| pynq/overlay.py | 33.63% <ø> (ø) |
|
| pynq/pl_server/embedded_device.py | 0.00% <0.00%> (ø) |
|
| pynq/pl_server/hwh_parser.py | 17.24% <0.00%> (-0.41%) |
:arrow_down: |
| pynq/ps.py | 100.00% <0.00%> (ø) |
|
| pynq/pl_server/xrt_device.py | 29.36% <0.00%> (+0.18%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update b85f453...85477d1. Read the comment docs.
Move this to draft until we have a better solution for the AXI4-Lite interfaces on partial regions
TODO: test with HDL module in IPI