[cmake] Enable Werror

engdoreis · engdoreis · commit d88126739035 · 2026-05-28T10:35:33.000+01:00
Signed-off-by: Douglas Reis &lt;doreis@lowrisc.org&gt;
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -10,6 +10,7 @@ set(CMAKE_CXX_STANDARD 23)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
 set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+set(CMAKE_COMPILE_WARNING_AS_ERROR ON)
 
 find_package(argparse QUIET)
 find_package(magic_enum QUIET)
diff --git a/lib/ftdi/spi_host.cc b/lib/ftdi/spi_host.cc
@@ -222,6 +222,7 @@ bool SpiHost::set_clock(size_t clock) {
     std::cerr << std::format("failed to update the clock:{}\n", status);
     return false;
   }
+  return true;
 }
 
 static std::optional<SpiHost> new_ft4222(DeviceInfo& device) {

Original file line number	Diff line number	Diff line change
`@@ -222,6 +222,7 @@ bool SpiHost::set_clock(size_t clock) {`
`222`	`222`	`std::cerr << std::format("failed to update the clock:{}\n", status);`
`223`	`223`	`return false;`
`224`	`224`	`}`
	`225`	`+ return true;`
`225`	`226`	`}`
`226`	`227`
`227`	`228`	`static std::optional<SpiHost> new_ft4222(DeviceInfo& device) {`