From 3d9717f7c3f3a21ba6b0b3054fe4c38d553c9cd7 Mon Sep 17 00:00:00 2001
From: hycrypt <79507881+hycrypt@users.noreply.github.com>
Date: Fri, 3 Mar 2023 11:18:08 +0100
Subject: [PATCH] add the skip_load_bridge and skip_reset arguments also for
 Xilinx devices

---
 src/main.cpp   |  2 +-
 src/xilinx.cpp | 24 ++++++++++++++++++++++--
 src/xilinx.hpp |  7 ++++---
 3 files changed, 27 insertions(+), 6 deletions(-)

diff --git a/src/main.cpp b/src/main.cpp
index d31558a..d494c94 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -540,7 +540,7 @@ int main(int argc, char **argv)
 		if (fab == "xilinx") {
 			fpga = new Xilinx(jtag, args.bit_file, args.secondary_bit_file,
 				args.file_type, args.prg_type, args.fpga_part, args.bridge_path,
-				args.target_flash, args.verify, args.verbose);
+				args.target_flash, args.verify, args.verbose, args.skip_load_bridge, args.skip_reset);
 		} else if (fab == "altera") {
 			fpga = new Altera(jtag, args.bit_file, args.file_type,
 				args.prg_type, args.fpga_part, args.bridge_path, args.verify,
diff --git a/src/xilinx.cpp b/src/xilinx.cpp
index 3b8c970..a52e878 100644
--- a/src/xilinx.cpp
+++ b/src/xilinx.cpp
@@ -128,9 +128,11 @@ Xilinx::Xilinx(Jtag *jtag, const std::string &filename,
 	Device::prog_type_t prg_type,
 	const std::string &device_package, const std::string &spiOverJtagPath,
 	const std::string &target_flash,
-	bool verify, int8_t verbose):
+	bool verify, int8_t verbose,
+	bool skip_load_bridge, bool skip_reset):
 	Device(jtag, filename, file_type, verify, verbose),
-	SPIInterface(filename, verbose, 256, verify),
+	SPIInterface(filename, verbose, 256, verify, skip_load_bridge,
+				 skip_reset),
 	_device_package(device_package), _spiOverJtagPath(spiOverJtagPath),
 	_irlen(6), _filename(filename), _secondary_filename(secondary_filename)
 {
@@ -434,6 +436,24 @@ void Xilinx::program(unsigned int offset, bool unprotect_flash)
 	delete bit;
 }
 
+bool Xilinx::post_flash_access()
+{
+	if (_skip_reset)
+		printInfo("Skip resetting device");
+	else
+		reset();
+	return true;
+}
+
+bool Xilinx::prepare_flash_access()
+{
+	if (_skip_load_bridge) {
+		printInfo("Skip loading bridge for spiOverjtag");
+		return true;
+	}
+	return load_bridge();
+}
+
 bool Xilinx::load_bridge()
 {
 	std::string bitname;
diff --git a/src/xilinx.hpp b/src/xilinx.hpp
index 2004a77..6ed120d 100644
--- a/src/xilinx.hpp
+++ b/src/xilinx.hpp
@@ -23,7 +23,8 @@ class Xilinx: public Device, SPIInterface {
 				const std::string &device_package,
 				const std::string &spiOverJtagPath,
 				const std::string &target_flash,
-				bool verify, int8_t verbose);
+				bool verify, int8_t verbose,
+				bool skip_load_bridge, bool skip_reset);
 		~Xilinx();
 
 		void program(unsigned int offset, bool unprotect_flash) override;
@@ -143,11 +144,11 @@ class Xilinx: public Device, SPIInterface {
 		/*!
 		 * \brief prepare SPI flash access (need to have bridge in RAM)
 		 */
-		virtual bool prepare_flash_access() override {return load_bridge();}
+		virtual bool prepare_flash_access() override;
 		/*!
 		 * \brief end of SPI flash access
 		 */
-		virtual bool post_flash_access() override {reset(); return true;}
+		virtual bool post_flash_access() override;
 
 	private:
 		/* list of xilinx family devices */