Removed few compilation warnings

Signed-off-by: Alexis Jeandet <[email protected]>

Author: jeandet

tests/registers_and_fields/main.cpp

@@ -17,16 +17,16 @@ struct reg_mock
     }
 
     template <typename U>
-    inline constexpr auto operator=(const U& bit_field_value) const noexcept -> decltype(
-        std::declval<U>().value, std::declval<U>().mask, std::declval<reg_mock<address, T>>())
+    inline constexpr auto operator=(const U& bit_field_value) const noexcept -> std::remove_reference_t<decltype(
+        std::declval<U>().value, std::declval<U>().mask, std::declval<reg_mock<address, T>>())>
     {
         reg_mock::value() = bit_field_value.value;
         return reg_mock<address, T> {};
     }
 
     template <typename U>
-    inline constexpr auto operator|=(const U& bit_field_value) const noexcept -> decltype(
-        std::declval<U>().value, std::declval<U>().mask, std::declval<reg_mock<address, T>>())
+    inline constexpr auto operator|=(const U& bit_field_value) const noexcept -> std::remove_reference_t<decltype(
+        std::declval<U>().value, std::declval<U>().mask, std::declval<reg_mock<address, T>>())>
     {
         reg_mock::value() = (reg_mock::value() & ~bit_field_value.mask) | bit_field_value.value;
         return reg_mock<address, T> {};

ucpp/register.hpp

@@ -64,15 +64,15 @@ struct reg_t
     }
 
     template<typename U>
-    inline constexpr auto operator=(const U& bit_field_value) const noexcept -> decltype (std::declval<U>().value, std::declval<U>().mask, std::declval<reg_t<T,address>>())
+    inline constexpr auto operator=(const U& bit_field_value) const noexcept -> std::remove_reference_t<decltype (std::declval<U>().value, std::declval<U>().mask, std::declval<reg_t<T,address>>())>
     {
         static_assert (!readonly,"this register is read-only");
         reg_t::value() = bit_field_value.value;
         return reg_t<T,address>{};
     }
 
     template<typename U>
-    inline constexpr auto operator|=(const U& bit_field_value) const noexcept -> decltype (std::declval<U>().value, std::declval<U>().mask, std::declval<reg_t<T,address>>())
+    inline constexpr auto operator|=(const U& bit_field_value) const noexcept -> std::remove_reference_t<decltype (std::declval<U>().value, std::declval<U>().mask, std::declval<reg_t<T,address>>())>
     {
         static_assert (!readonly,"this register is read-only");
         reg_t::value() = (reg_t::value() & ~bit_field_value.mask) | bit_field_value.value;
@@ -186,14 +186,14 @@ struct bitfield_array_t
             reg_t::value() = mask & (value << start_index);
         else if constexpr(std::conjunction_v<std::is_enum<T>, std::is_same<T,value_t>>)
         {
-            auto v = bitfield_cat<count-1>(static_cast<unsigned long long>(value), width);
+            auto v = bitfield_cat<count-1>(static_cast<typename reg_t::type>(value), width);
             reg_t::value() = mask & (v << start_index);
         }
         return bitfield_array_t<start_index,width,reg_t,count,value_t>{};
     }
 
-    constexpr operator typename reg_t::type() noexcept { return typename reg_t::type((int(reg_t::value())& mask)>>start_index); }
-    constexpr operator typename reg_t::type() const noexcept { return typename reg_t::type((int(reg_t::value())& mask)>>start_index); }
+    constexpr operator typename reg_t::type() noexcept { return (typename reg_t::type(reg_t::value())& mask)>>start_index; }
+    constexpr operator typename reg_t::type() const noexcept { return (typename reg_t::type(reg_t::value())& mask)>>start_index; }
 };
 
 }

ucpp/stm32/sdmmc.hpp

@@ -44,18 +44,18 @@ namespace
     {
         using namespace ucpp::sdcard::commands;
         using ICR = decltype(sdmmc_dev::ICR);
-        sdmmc_dev::ICR |= sdmmc_dev::ICR.CCRCFAILC.shift(1) | sdmmc_dev::ICR.CTIMEOUTC.shift(1)
-            | sdmmc_dev::ICR.CMDRENDC.shift(1) | sdmmc_dev::ICR.CMDSENTC.shift(1);
+        sdmmc_dev::ICR |= ICR::CCRCFAILC.shift(1) | ICR::CTIMEOUTC.shift(1)
+            | ICR::CMDRENDC.shift(1) | ICR::CMDSENTC.shift(1);
     }
 
     template <typename sdmmc_dev>
     inline constexpr void data_clear_icr_flags()
     {
         using namespace ucpp::sdcard::commands;
         using ICR = decltype(sdmmc_dev::ICR);
-        sdmmc_dev::ICR |= sdmmc_dev::ICR.DCRCFAILC.shift(1) | sdmmc_dev::ICR.DTIMEOUTC.shift(1)
-            | sdmmc_dev::ICR.TXUNDERRC.shift(1) | sdmmc_dev::ICR.RXOVERRC.shift(1)
-            | sdmmc_dev::ICR.DATAENDC.shift(1) | sdmmc_dev::ICR.DBCKENDC.shift(1);
+        sdmmc_dev::ICR |= ICR::DCRCFAILC.shift(1) | ICR::DTIMEOUTC.shift(1)
+            | ICR::TXUNDERRC.shift(1) | ICR::RXOVERRC.shift(1)
+            | ICR::DATAENDC.shift(1) | ICR::DBCKENDC.shift(1);
     }
 
     template <typename sdmmc_dev, typename CMD>