🐛 Fix build issues from last few PRs

.github/workflows/ci.yml

@@ -20,8 +20,10 @@ on:
   push:
     branches:
       - main
-  schedule:
-    - cron: "0 12 * * 0"
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
 
 jobs:
   library_check:

CMakeLists.txt

@@ -13,6 +13,9 @@
 # limitations under the License.
 cmake_minimum_required(VERSION 3.15)
 
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+set(CMAKE_COLOR_DIAGNOSTICS ON)
+
 project(libhal-sensor LANGUAGES CXX)
 
 libhal_test_and_make_library(

demos/applications/icm20948.cpp

@@ -41,14 +41,12 @@ void application(resource_list& p_map)
   auto& console = *p_map.console.value();
   auto& i2c = *p_map.i2c.value();
 
-  hal::print(console, "icm Application Starting...\n\n");
+  hal::print(console, "icm20948 Demo Application Starting...\n\n");
   hal::delay(clock, 200ms);
-  hal::sensor::icm20948 icm_device(i2c);
 
-  hal::delay(clock, 200ms);
-  icm_device.init_mag();
-  hal::delay(clock, 100ms);
+  hal::sensor::icm20948 icm_device(i2c, clock);
 
+  hal::delay(clock, 200ms);
   icm_device.auto_offsets();
 
   while (true) {

include/libhal-sensor/imu/icm20948.hpp

@@ -15,6 +15,7 @@
 #pragma once
 
 #include <libhal/i2c.hpp>
+#include <libhal/steady_clock.hpp>
 #include <libhal/timeout.hpp>
 #include <libhal/units.hpp>
 
@@ -224,9 +225,8 @@ class icm20948
    * @brief private constructor for icm20948 objects
    * @param p_i2c The I2C peripheral used for communication with the device.
    */
-  [[deprecated(
-    "Use the constructor with hal::steady_clock instead.")]] icm20948(hal::i2c&
-                                                                        p_i2c);
+  [[deprecated("Use the constructor with hal::steady_clock instead.")]]
+  icm20948(hal::i2c& p_i2c);
 
   /**
    * @brief private constructor for icm20948 objects

src/imu/icm20948.cpp

@@ -12,12 +12,16 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <array>
 #include <cmath>
 
+#include <array>
+#include <chrono>
+
 #include <libhal-sensor/imu/icm20948.hpp>
 #include <libhal-util/i2c.hpp>
+#include <libhal-util/steady_clock.hpp>
 #include <libhal/error.hpp>
+#include <libhal/steady_clock.hpp>
 
 namespace hal::sensor {
 
@@ -183,10 +187,10 @@ icm20948::icm20948(hal::i2c& p_i2c, hal::steady_clock& p_steady_clock)
   m_current_bank = 0;
 
   reset_icm20948();
-  hal::delay(*p_steady_clock, 100ms);
+  hal::delay(p_steady_clock, 100ms);
   set_clock_auto_select();
   sleep(false);
-  hal::delay(*p_steady_clock, 10ms);
+  hal::delay(p_steady_clock, 10ms);
   reset_mag();
 
   std::uint8_t tries = 0;
@@ -196,9 +200,8 @@ icm20948::icm20948(hal::i2c& p_i2c, hal::steady_clock& p_steady_clock)
     if (mag_whoami_ok()) {
       break;
     }
-
-    i2c_master_reset();
-    hal::delay(*p_steady_clock, 10ms);
+    reset_i2c_master();
+    hal::delay(p_steady_clock, 10ms);
   }
 
   if (auto id = whoami(); id != who_am_i_content) {