Rework 1-Wire implementation

.vscode/cmake-variants-DEVCONTAINER.json

@@ -91,7 +91,7 @@
           "API_System.IO.Ports": "ON",
           "API_Windows.Devices.Adc": "ON",
           "API_System.Device.Adc": "ON",
-          "API_nanoFramework.Devices.OneWire": "ON",
+          "API_nanoFramework.Device.OneWire": "ON",
           "API_nanoFramework.Devices.Can": "OFF",
           "API_nanoFramework.ResourceManager": "ON",
           "API_nanoFramework.System.Collections": "ON",
@@ -142,7 +142,7 @@
           "API_System.Device.Adc": "ON",
           "API_System.Device.Dac": "ON",
           "API_System.Net": "ON",
-          "API_nanoFramework.Devices.OneWire": "OFF",
+          "API_nanoFramework.Device.OneWire": "OFF",
           "API_nanoFramework.Devices.Can": "OFF",
           "API_Windows.Storage": "ON",
           "API_nanoFramework.ResourceManager": "ON",
@@ -191,7 +191,7 @@
           "API_System.Device.Adc": "ON",
           "API_System.Device.Dac": "ON",
           "API_System.Net": "ON",
-          "API_nanoFramework.Devices.OneWire": "ON",
+          "API_nanoFramework.Device.OneWire": "ON",
           "API_nanoFramework.Devices.Can": "ON",
           "API_Windows.Storage": "ON",
           "API_nanoFramework.ResourceManager": "ON",
@@ -285,7 +285,7 @@
           "API_System.Net": "ON",
           "API_System.Device.WiFi": "ON",
           "API_Hardware.Esp32": "ON",
-	  "API_nanoFramework.Devices.OneWire": "ON",
+	  "API_nanoFramework.Device.OneWire": "ON",
           "API_nanoFramework.ResourceManager": "ON",
           "API_nanoFramework.System.Collections": "ON",
           "API_nanoFramework.System.Text": "ON",
@@ -332,7 +332,7 @@
           "API_System.Net": "ON", 
           "API_System.Device.WiFi": "ON",
           "API_Hardware.Esp32": "ON",
-          "API_nanoFramework.Devices.OneWire": "ON",
+          "API_nanoFramework.Device.OneWire": "ON",
           "API_nanoFramework.ResourceManager": "ON",
           "API_nanoFramework.System.Collections": "ON",
           "API_nanoFramework.System.Text": "ON",
@@ -380,7 +380,7 @@
           "API_System.Net": "ON", 
           "API_System.Device.WiFi": "ON",
           "API_Hardware.Esp32": "ON",
-          "API_nanoFramework.Devices.OneWire": "ON",
+          "API_nanoFramework.Device.OneWire": "ON",
           "API_nanoFramework.ResourceManager": "ON",
           "API_nanoFramework.System.Collections": "ON",
           "API_nanoFramework.System.Text": "ON",
@@ -433,7 +433,7 @@
           "API_nanoFramework.System.Text": "ON",
           "API_nanoFramework.Hardware.Esp32.Rmt": "ON",
           "API_Windows.Devices.Adc": "ON",
-          "API_nanoFramework.Devices.OneWire": "ON",
+          "API_nanoFramework.Device.OneWire": "ON",
           "API_nanoFramework.Graphics": "OFF",
           "API_nanoFramework.Device.Bluetooth": "OFF"
         }
@@ -476,7 +476,7 @@
           "API_System.Net": "ON",
           "API_System.Device.WiFi": "ON",
           "API_Hardware.Esp32": "ON",
-          "API_nanoFramework.Devices.OneWire": "ON",
+          "API_nanoFramework.Device.OneWire": "ON",
           "API_nanoFramework.ResourceManager": "ON",
           "API_nanoFramework.System.Collections": "ON",
           "API_nanoFramework.System.Text": "ON",
@@ -533,7 +533,7 @@
           "API_nanoFramework.System.Text": "ON",
           "API_nanoFramework.Hardware.Esp32.Rmt": "ON",
           "API_System.Device.Dac": "ON",
-          "API_nanoFramework.Devices.OneWire": "OFF",
+          "API_nanoFramework.Device.OneWire": "OFF",
           "API_nanoFramework.Graphics": "OFF",
           "API_nanoFramework.Device.Bluetooth": "OFF"
         }
@@ -581,7 +581,7 @@
           "API_nanoFramework.Hardware.Esp32.Rmt": "ON",
           "API_System.Device.Dac": "ON",
           "API_nanoFramework.Device.Bluetooth": "OFF",
-          "API_nanoFramework.Devices.OneWire": "OFF",
+          "API_nanoFramework.Device.OneWire": "OFF",
           "API_nanoFramework.Graphics": "ON",
           "GRAPHICS_DISPLAY": "ILI9341_240x320_SPI.cpp",
           "GRAPHICS_DISPLAY_INTERFACE": "Spi_To_Display.cpp",
@@ -679,7 +679,7 @@
           "API_nanoFramework.System.Collections": "ON",
           "API_nanoFramework.System.Text": "ON",
           "API_nanoFramework.Hardware.Esp32.Rmt": "ON",
-          "API_nanoFramework.Devices.OneWire": "ON",
+          "API_nanoFramework.Device.OneWire": "ON",
           "API_System.Device.Dac": "ON",
           "API_nanoFramework.Graphics": "OFF",
           "API_nanoFramework.Device.Bluetooth": "OFF"

.vscode/cmake-variants.TEMPLATE.json

@@ -92,7 +92,7 @@
           "NF_SECURITY_MBEDTLS": "OFF-default-ON-to-add-network-security-from-mbedTLS",
           "MBEDTLS_SOURCE": "<path-to-mbedtls-source-mind-the-forward-slashes>",
           "API_nanoFramework.Devices.Can": "OFF-default-ON-to-add-this-API",
-          "API_nanoFramework.Devices.OneWire": "OFF-default-ON-to-add-this-API",
+          "API_nanoFramework.Device.OneWire": "OFF-default-ON-to-add-this-API",
           "API_nanoFramework.ResourceManager": "OFF-default-ON-to-add-this-API",
           "API_nanoFramework.System.Collections": "OFF-default-ON-to-add-this-API",
           "API_nanoFramework.System.Text": "OFF-default-ON-to-add-this-API",

CMake/Modules/FindNF_NativeAssemblies.cmake

@@ -10,7 +10,7 @@
 ###########################################################################################
 
 option(API_nanoFramework.Devices.Can            "option for nanoFramework.Devices.Can")
-option(API_nanoFramework.Devices.OneWire        "option for nanoFramework.Devices.OneWire")
+option(API_nanoFramework.Device.OneWire        "option for nanoFramework.Device.OneWire")
 option(API_nanoFramework.Networking.Sntp        "option for nanoFramework.Networking.Sntp")
 option(API_nanoFramework.Runtime.Events         "option for nanoFramework.Runtime.Events API")
 option(API_nanoFramework.ResourceManager        "option for nanoFramework.ResourceManager")
@@ -201,10 +201,10 @@ if(API_nanoFramework.Graphics)
     PerformSettingsForApiEntry("nanoFramework.Graphics")
 endif()
 
-# nanoFramework.Devices.OneWire
-if(API_nanoFramework.Devices.OneWire)
+# nanoFramework.Device.OneWire
+if(API_nanoFramework.Device.OneWire)
     ##### API name here (doted name)
-    PerformSettingsForApiEntry("nanoFramework.Devices.OneWire")
+    PerformSettingsForApiEntry("nanoFramework.Device.OneWire")
 endif()
 
 # nanoFramework.Networking.Sntp

CMake/Modules/FindnanoFramework.Device.OneWire.cmake

@@ -0,0 +1,49 @@
+#
+# Copyright (c) .NET Foundation and Contributors
+# See LICENSE file in the project root for full license information.
+#
+
+
+# native code directory
+set(BASE_PATH_FOR_THIS_MODULE "${BASE_PATH_FOR_CLASS_LIBRARIES_MODULES}/nanoFramework.Device.OneWire")
+
+
+# set include directories
+list(APPEND nanoFramework.Device.OneWire_INCLUDE_DIRS ${BASE_PATH_FOR_THIS_MODULE})
+list(APPEND nanoFramework.Device.OneWire_INCLUDE_DIRS ${TARGET_BASE_LOCATION})
+list(APPEND nanoFramework.Device.OneWire_INCLUDE_DIRS ${CMAKE_SOURCE_DIR}/src/nanoFramework.Device.OneWire)
+
+# source files
+set(nanoFramework.Device.OneWire_SRCS
+
+    nf_dev_onewire_nanoFramework_Device_OneWire_OneWireHost.cpp
+    nf_dev_onewire.cpp
+
+    target_nf_dev_onewire_config.cpp
+)
+
+foreach(SRC_FILE ${nanoFramework.Device.OneWire_SRCS})
+
+    set(nanoFramework.Device.OneWire_SRC_FILE SRC_FILE-NOTFOUND)
+
+    find_file(nanoFramework.Device.OneWire_SRC_FILE ${SRC_FILE}
+        PATHS 
+            ${BASE_PATH_FOR_THIS_MODULE}
+            ${TARGET_BASE_LOCATION}
+            ${CMAKE_SOURCE_DIR}/src/nanoFramework.Device.OneWire
+
+        CMAKE_FIND_ROOT_PATH_BOTH
+    )
+
+    if (BUILD_VERBOSE)
+        message("${SRC_FILE} >> ${nanoFramework.Device.OneWire_SRC_FILE}")
+    endif()
+
+    list(APPEND nanoFramework.Device.OneWire_SOURCES ${nanoFramework.Device.OneWire_SRC_FILE})
+
+endforeach()
+
+
+include(FindPackageHandleStandardArgs)
+
+FIND_PACKAGE_HANDLE_STANDARD_ARGS(nanoFramework.Device.OneWire DEFAULT_MSG nanoFramework.Device.OneWire_INCLUDE_DIRS nanoFramework.Device.OneWire_SOURCES)

CMakeLists.txt

@@ -494,14 +494,14 @@ endif()
 # for some APIs we need to enable the device in the HAL config
 # and/or manage other APIs that are required
 
-if( API_nanoFramework.Devices.OneWire OR
+if( API_nanoFramework.Device.OneWire OR
     API_Windows.Devices.SerialCommunication OR
     API_System.IO.Ports)
 
-    set(HAL_USE_UART_OPTION TRUE CACHE INTERNAL "HAL OneWire for nanoFramework.Devices.OneWire")
+    set(HAL_USE_UART_OPTION TRUE CACHE INTERNAL "HAL OneWire for nanoFramework.Device.OneWire")
 
 else()
-    set(HAL_USE_UART_OPTION FALSE CACHE INTERNAL "HAL OneWire for nanoFramework.Devices.OneWire")
+    set(HAL_USE_UART_OPTION FALSE CACHE INTERNAL "HAL OneWire for nanoFramework.Device.OneWire")
 endif()
 
 if( API_System.Net)
@@ -594,10 +594,10 @@ else()
 endif()
 
 
-if(API_nanoFramework.Devices.OneWire)
-    set(HAL_USE_STM32_ONEWIRE_OPTION TRUE CACHE INTERNAL "HAL STM32_ONEWIRE for nanoFramework.Devices.OneWire")
+if(API_nanoFramework.Device.OneWire)
+    set(HAL_USE_STM32_ONEWIRE_OPTION TRUE CACHE INTERNAL "HAL STM32_ONEWIRE for nanoFramework.Device.OneWire")
 else()
-    set(HAL_USE_STM32_ONEWIRE_OPTION FALSE CACHE INTERNAL "HAL STM32_ONEWIRE for nanoFramework.Devices.OneWire")
+    set(HAL_USE_STM32_ONEWIRE_OPTION FALSE CACHE INTERNAL "HAL STM32_ONEWIRE for nanoFramework.Device.OneWire")
 endif()
 
 #################################################################################
@@ -626,7 +626,7 @@ endif()
 #################################################################
 
 # include nanoFramework.Runtime.Events API
-if( API_nanoFramework.Devices.OneWire OR 
+if( API_nanoFramework.Device.OneWire OR 
     API_System.Net OR
     API_System.Device.Gpio OR
     API_Windows.Devices.Gpio OR

CMakeSettings.SAMPLE.json

@@ -164,7 +164,7 @@
           "type": "BOOL"
         },
         {
-          "name": "API_nanoFramework.Devices.OneWire",
+          "name": "API_nanoFramework.Device.OneWire",
           "value": "True",
           "type": "BOOL"
         },
@@ -449,7 +449,7 @@
           "type": "PATH"
         },
         {
-          "name": "API_nanoFramework.Devices.OneWire",
+          "name": "API_nanoFramework.Device.OneWire",
           "value": "True",
           "type": "BOOL"
         },

CMakeSettings.json

@@ -164,7 +164,7 @@
           "value": "ON"
         },
         {
-          "name": "API_nanoFramework.Devices.OneWire",
+          "name": "API_nanoFramework.Device.OneWire",
           "value": "OFF"
         },
         {
@@ -364,7 +364,7 @@
         },
         // LIBRARIES
         {
-          "name": "API_nanoFramework.Devices.OneWire:BOOL", //OFF-default-ON-to-add-this-API
+          "name": "API_nanoFramework.Device.OneWire:BOOL", //OFF-default-ON-to-add-this-API
           "value": "ON"
         },
         {

azure-pipelines-nightly.yml

@@ -368,7 +368,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON
@@ -404,7 +404,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON
@@ -441,7 +441,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON
@@ -477,7 +477,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON
@@ -591,7 +591,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON

azure-pipelines.yml

@@ -326,7 +326,7 @@ jobs:
           -DAPI_Windows.Devices.Pwm=ON -DAPI_System.Device.Pwm=ON
           -DAPI_Windows.Devices.SerialCommunication=ON -DAPI_System.IO.Ports=ON
           -DAPI_Windows.Devices.Adc=ON -DAPI_System.Device.Adc=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.Devices.Can=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
@@ -357,7 +357,7 @@ jobs:
           -DAPI_System.Device.Dac=ON
           -DAPI_System.Net=ON
           -DNF_SECURITY_MBEDTLS=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.Devices.Can=ON
           -DAPI_System.IO.FileSystem=ON
           -DAPI_nanoFramework.ResourceManager=ON
@@ -441,7 +441,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON
@@ -482,7 +482,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON
@@ -519,7 +519,7 @@ jobs:
           -DAPI_System.Net=ON
           -DAPI_System.Device.WiFi=ON
           -DAPI_Hardware.Esp32=ON
-          -DAPI_nanoFramework.Devices.OneWire=ON
+          -DAPI_nanoFramework.Device.OneWire=ON
           -DAPI_nanoFramework.ResourceManager=ON
           -DAPI_nanoFramework.System.Collections=ON
           -DAPI_nanoFramework.System.Text=ON