move extract_firmware_version_string() into ds::, d400-private

IntelRealSense · Dec 2, 2023 · c50c61a · c50c61a
1 parent 5a2749b
commit c50c61a
Show file tree

Hide file tree

Showing 6 changed files with 22 additions and 23 deletions.
diff --git a/src/ds/d400/d400-device.cpp b/src/ds/d400/d400-device.cpp
@@ -127,7 +127,7 @@ namespace librealsense
             throw librealsense::invalid_value_exception(
                 rsutils::string::from() << "Unsupported firmware binary image provided - " << image.size() << " bytes" );
 
-        std::string fw_version = firmware_check_interface::extract_firmware_version_string( image );
+        std::string fw_version = ds::extract_firmware_version_string( image );
 
         auto it = ds::d400_device_to_fw_min_version.find( _pid );
         if( it == ds::d400_device_to_fw_min_version.end() )

diff --git a/src/ds/d400/d400-fw-update-device.cpp b/src/ds/d400/d400-fw-update-device.cpp
@@ -26,7 +26,7 @@ ds_d400_update_device::ds_d400_update_device(
             throw librealsense::invalid_value_exception(
                 rsutils::string::from() << "Unsupported firmware binary image provided - " << image.size() << " bytes" );
 
-        std::string fw_version = extract_firmware_version_string(image);
+        std::string fw_version = ds::extract_firmware_version_string(image);
         auto it = ds::d400_device_to_fw_min_version.find(_usb_device->get_info().pid);
         if (it == ds::d400_device_to_fw_min_version.end())
             throw librealsense::invalid_value_exception(

diff --git a/src/ds/d400/d400-private.cpp b/src/ds/d400/d400-private.cpp
@@ -86,6 +86,23 @@ namespace librealsense
             return results;
         }
 
+        std::string extract_firmware_version_string( const std::vector< uint8_t > & fw_image )
+        {
+            auto version_offset = offsetof( platform::dfu_header, bcdDevice );
+            if( fw_image.size() < ( version_offset + sizeof( size_t ) ) )
+                throw std::runtime_error( "Firmware binary image might be corrupted - size is only: "
+                                          + std::to_string( fw_image.size() ) );
+
+            auto version = fw_image.data() + version_offset;
+            uint8_t major = *( version + 3 );
+            uint8_t minor = *( version + 2 );
+            uint8_t patch = *( version + 1 );
+            uint8_t build = *( version );
+
+            return std::to_string( major ) + "." + std::to_string( minor ) + "." + std::to_string( patch ) + "."
+                 + std::to_string( build );
+        }
+
         rs2_intrinsics get_d400_intrinsic_by_resolution(const vector<uint8_t>& raw_data, d400_calibration_table_id table_id, uint32_t width, uint32_t height)
         {
             switch (table_id)

diff --git a/src/ds/d400/d400-private.h b/src/ds/d400/d400-private.h
@@ -183,6 +183,8 @@ namespace librealsense
             return min_gvd_version <= cur_gvd_version;
         }
 
+        std::string extract_firmware_version_string( const std::vector< uint8_t > & fw_image );
+
         enum class d400_calibration_table_id
         {
             coefficients_table_id = 25,

diff --git a/src/ds/d500/d500-fw-update-device.cpp b/src/ds/d500/d500-fw-update-device.cpp
@@ -20,13 +20,7 @@ ds_d500_update_device::ds_d500_update_device( std::shared_ptr< const device_info
     bool ds_d500_update_device::check_fw_compatibility(const std::vector<uint8_t>& image) const
     {
         // Currently we cannot extract FW version from HKR FW image
-        bool result = true;
-
-        // TODO::: Once verified we can check against the minimal FW version map
-        std::string fw_version = extract_firmware_version_string(image);
-        LOG_INFO( "FW version extracted from the FW image is" + fw_version );
-
-        return result;
+        return true;
     }
 
     std::string ds_d500_update_device::parse_serial_number(const std::vector<uint8_t>& buffer) const

diff --git a/src/fw-update/fw-update-device-interface.h b/src/fw-update/fw-update-device-interface.h
@@ -14,20 +14,6 @@ namespace librealsense
     {
     public:
         virtual bool check_fw_compatibility(const std::vector<uint8_t>& image) const = 0;
-        static std::string extract_firmware_version_string(const std::vector<uint8_t>& fw_image)
-        {
-            auto version_offset = offsetof(platform::dfu_header, bcdDevice);
-            if (fw_image.size() < (version_offset + sizeof(size_t)))
-                throw std::runtime_error("Firmware binary image might be corrupted - size is only: " + std::to_string( fw_image.size() ));
-
-            auto version = fw_image.data() + version_offset;
-            uint8_t major = *(version + 3);
-            uint8_t minor = *(version + 2);
-            uint8_t patch = *(version + 1);
-            uint8_t build = *(version);
-
-            return std::to_string(major) + "." + std::to_string(minor) + "." + std::to_string(patch) + "." + std::to_string(build);
-        }
     };
 
     class updatable : public firmware_check_interface