Merge pull request #721 from YosysHQ/gatecat/mistral-cmake

Updates for latest libmistral

4 files changed, 4 insertions, 11 deletions

diff --git a/mistral/arch.cc b/mistral/arch.cc
index 70e8f806..ba7483e7 100644
--- a/mistral/arch.cc
+++ b/mistral/arch.cc
@@ -46,7 +46,7 @@ void IdString::initialize_arch(const BaseCtx *ctx)
 Arch::Arch(ArchArgs args)
 {
     this->args = args;
-    this->cyclonev = mistral::CycloneV::get_model(args.device, args.mistral_root);
+    this->cyclonev = mistral::CycloneV::get_model(args.device);
     NPNR_ASSERT(this->cyclonev != nullptr);
 
     // Setup fast identifier maps
diff --git a/mistral/arch.h b/mistral/arch.h
index 1a42530a..c00ee2a7 100644
--- a/mistral/arch.h
+++ b/mistral/arch.h
@@ -34,7 +34,6 @@ NEXTPNR_NAMESPACE_BEGIN
 struct ArchArgs
 {
     std::string device;
-    std::string mistral_root;
 };
 
 // These structures are used for fast ALM validity checking
diff --git a/mistral/family.cmake b/mistral/family.cmake
index 552c64ae..a18b5b27 100644
--- a/mistral/family.cmake
+++ b/mistral/family.cmake
@@ -1,13 +1,12 @@
 set(MISTRAL_ROOT "" CACHE STRING "Mistral install path")
+set(MISTRAL_DONT_INSTALL ON)
 
-aux_source_directory(${MISTRAL_ROOT}/lib MISTRAL_LIB_FILES)
-add_library(mistral STATIC ${MISTRAL_LIB_FILES})
-target_compile_options(mistral PRIVATE -Wno-maybe-uninitialized -Wno-uninitialized -Wno-unknown-warning-option)
+add_subdirectory(${MISTRAL_ROOT}/libmistral ${CMAKE_CURRENT_BINARY_DIR}/libmistral)
 
 find_package(LibLZMA REQUIRED)
 
 foreach(family_target ${family_targets})
-    target_include_directories(${family_target} PRIVATE ${MISTRAL_ROOT}/lib ${LIBLZMA_INCLUDE_DIRS})
+    target_include_directories(${family_target} PRIVATE ${MISTRAL_ROOT}/libmistral ${LIBLZMA_INCLUDE_DIRS})
     target_link_libraries(${family_target} PRIVATE mistral ${LIBLZMA_LIBRARIES})
 	# Currently required to avoid issues with mistral (LTO means the warnings can end up in nextpnr)
 	target_link_options(${family_target} PRIVATE -Wno-maybe-uninitialized -Wno-uninitialized -Wno-unknown-warning-option)
diff --git a/mistral/main.cc b/mistral/main.cc
index 0afba3d8..bd9ef591 100644
--- a/mistral/main.cc
+++ b/mistral/main.cc
@@ -47,7 +47,6 @@ MistralCommandHandler::MistralCommandHandler(int argc, char **argv) : CommandHan
 po::options_description MistralCommandHandler::getArchOptions()
 {
     po::options_description specific("Architecture specific options");
-    specific.add_options()("mistral", po::value<std::string>(), "path to mistral root");
     specific.add_options()("device", po::value<std::string>(), "device name (e.g. 5CSEBA6U23I7)");
     specific.add_options()("qsf", po::value<std::string>(), "path to QSF constraints file");
     specific.add_options()("rbf", po::value<std::string>(), "RBF bitstream to write");
@@ -74,13 +73,9 @@ void MistralCommandHandler::customBitstream(Context *ctx)
 std::unique_ptr<Context> MistralCommandHandler::createContext(dict<std::string, Property> &values)
 {
     ArchArgs chipArgs;
-    if (!vm.count("mistral")) {
-        log_error("mistral must be specified on the command line\n");
-    }
     if (!vm.count("device")) {
         log_error("device must be specified on the command line (e.g. --device 5CSEBA6U23I7)\n");
     }
-    chipArgs.mistral_root = vm["mistral"].as<std::string>();
     chipArgs.device = vm["device"].as<std::string>();
     auto ctx = std::unique_ptr<Context>(new Context(chipArgs));
     if (vm.count("compress-rbf"))