From 26a187e5ebcd1bdb6e079a91e25b610dc603aab2 Mon Sep 17 00:00:00 2001 From: Keith Rothman <537074+litghost@users.noreply.github.com> Date: Tue, 16 Feb 2021 14:00:01 -0800 Subject: [PATCH] Require `--package` when arch BBA contains multiple packages. Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com> --- fpga_interchange/arch.cc | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/fpga_interchange/arch.cc b/fpga_interchange/arch.cc index 8c8224ca..dc5b5c77 100644 --- a/fpga_interchange/arch.cc +++ b/fpga_interchange/arch.cc @@ -126,9 +126,17 @@ Arch::Arch(ArchArgs args) : args(args) } else { // Default to first package. NPNR_ASSERT(chip_info->packages.size() > 0); - IdString package_name(chip_info->packages[0].package); - this->args.package = package_name.str(this); - package_index = 0; + if(chip_info->packages.size() == 1) { + IdString package_name(chip_info->packages[0].package); + this->args.package = package_name.str(this); + package_index = 0; + } else { + log_info("Package must be specified (with --package arg) when multiple packages are available, packages:\n"); + for(const auto &package : chip_info->packages) { + log_info(" - %s\n", IdString(package.package).c_str(this)); + } + log_error("--package is required!\n"); + } } std::unordered_set site_bel_pads;