luke
/
openFPGALoader
mirror of https://github.com/trabucayre/openFPGALoader.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix DNA read on Zynq devices 

Do not apply the Zynq non-volatile memory restriction when only reading FPGA DNA. Allow Zynq devices to use the existing 7-series DNA read path.

Tested on xc7z010 over XVC with --read-dna.
This commit is contained in:
MYJ695 2026-05-26 15:50:23 +08:00 committed by GitHub
parent 0818a404c9
commit fda4ef7b45
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/xilinx.cpp
View File

@ -337,7 +337,10 @@ Xilinx::Xilinx(Jtag *jtag, const std::string &filename,
_fpga_family = SPARTAN7_FAMILY; _fpga_family = SPARTAN7_FAMILY;
} else if (family == "zynq") { } else if (family == "zynq") {
_fpga_family = ZYNQ_FAMILY; _fpga_family = ZYNQ_FAMILY;
if (_mode != Device::MEM_MODE) { /* DNA read uses a 7-series PL JTAG instruction and should not
* be blocked by the Zynq PS-side SPI flash restriction.
*/
if (_mode != Device::MEM_MODE && !read_dna) {
char mess[256]; char mess[256];
snprintf(mess, 256, "Error: can't flash non-volatile memory for " snprintf(mess, 256, "Error: can't flash non-volatile memory for "
"Zynq7000 devices\n" "Zynq7000 devices\n"
@ -408,11 +411,12 @@ Xilinx::Xilinx(Jtag *jtag, const std::string &filename,
} }
if (read_dna) { if (read_dna) {
if (_fpga_family == ARTIX_FAMILY || _fpga_family == KINTEXUS_FAMILY) { if (_fpga_family == ARTIX_FAMILY || _fpga_family == KINTEXUS_FAMILY ||
_fpga_family == ZYNQ_FAMILY) {
uint64_t dna = Xilinx::fuse_dna_read(); uint64_t dna = Xilinx::fuse_dna_read();
printf("{\"dna\": \"0x%016" PRIx64 "\"}\n", dna); printf("{\"dna\": \"0x%016" PRIx64 "\"}\n", dna);
} else { } else {
throw std::runtime_error("Error: read_xadc only supported for Artix 7"); throw std::runtime_error("Error: read_dna only supported for 7-series style Xilinx FPGA");
} }
} }