From ca5f23c353d2e05ad47a2f11063bf4fe82e90041 Mon Sep 17 00:00:00 2001
From: Christoph Schweppe <info@cschweppe.de>
Date: Thu, 27 Oct 2022 10:38:19 +0200
Subject: [PATCH 1/2] Add timing helpers

---
 nanoFramework.Device.Can/CanTiming.cs         | 83 +++++++++++++++++++
 .../nanoFramework.Device.Can.nfproj           |  1 +
 2 files changed, 84 insertions(+)
 create mode 100644 nanoFramework.Device.Can/CanTiming.cs

diff --git a/nanoFramework.Device.Can/CanTiming.cs b/nanoFramework.Device.Can/CanTiming.cs
new file mode 100644
index 0000000..59b3f48
--- /dev/null
+++ b/nanoFramework.Device.Can/CanTiming.cs
@@ -0,0 +1,83 @@
+//
+// Copyright (c) .NET Foundation and Contributors
+// See LICENSE file in the project root for full license information.
+//
+
+namespace nanoFramework.Device.Can
+{
+    /// <summary>
+    /// Helper for common CAN timings for the STM32
+    /// </summary>
+    public static class CanTimingSTM32
+    {
+        /// <summary>
+        /// 25 KBaud.
+        /// </summary>
+        public static CanSettings Timing500KBaud () {
+            return new CanSettings(6, 8, 1, 0);
+        }
+
+    }
+
+    /// <summary>
+    /// Helper for common CAN timings for the ESP32. Values from https://github.com/espressif/esp-idf/blob/master/components/hal/include/hal/twai_types.h#L54
+    /// </summary>
+    public static class CanTimingESP32
+    {
+        /// <summary>
+        /// 25 KBaud.
+        /// </summary>
+        public static CanSettings Timing25KBaud () {
+            return new CanSettings(128, 16, 8, 3);
+        }
+
+        /// <summary>
+        /// 50 KBaud.
+        /// </summary>
+        public static CanSettings Timing50KBaud () {
+            return new CanSettings(80, 15, 4, 3);
+        }
+
+        /// <summary>
+        /// 100 KBaud.
+        /// </summary>
+        public static CanSettings Timing100KBaud () {
+            return new CanSettings(40, 15, 4, 3);
+        }
+
+        /// <summary>
+        /// 125 KBaud.
+        /// </summary>
+        public static CanSettings Timing125KBaud () {
+            return new CanSettings(32, 15, 4, 3);
+        }
+
+        /// <summary>
+        /// 250 KBaud.
+        /// </summary>
+        public static CanSettings Timing250KBaud () {
+            return new CanSettings(16, 15, 4, 3);
+        }
+
+        /// <summary>
+        /// 500 KBaud.
+        /// </summary>
+        public static CanSettings Timing500KBaud () {
+            return new CanSettings(8, 15, 4, 3);
+        }
+
+        /// <summary>
+        /// 800 KBaud.
+        /// </summary>
+        public static CanSettings Timing800KBaud() {
+            return new CanSettings(4, 16, 8, 3);
+        }
+
+        /// <summary>
+        /// 1000 KBaud.
+        /// </summary>
+        public static CanSettings Timing1000KBaud () {
+            return new CanSettings(4, 15, 4, 3);
+        }
+    }
+}
diff --git a/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj b/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj
index f96dc37..836e1a0 100644
--- a/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj
+++ b/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj
@@ -53,6 +53,7 @@
     <Compile Include="CanMessageReceivedEventArgs.cs" />
     <Compile Include="CanControllerEventListener.cs" />
     <Compile Include="CanMessageEvent.cs" />
+    <Compile Include="CanTiming.cs" />
     <None Include="key.snk" />
   </ItemGroup>
   <ItemGroup>

From 13917f5d87eeacf1e7df22f0d416afb631fc91f6 Mon Sep 17 00:00:00 2001
From: Christoph Schweppe <info@cschweppe.de>
Date: Thu, 27 Oct 2022 15:22:36 +0200
Subject: [PATCH 2/2] Use enum evaluated on native side

---
 nanoFramework.Device.Can/CanBitTiming.cs      | 58 +++++++++++++
 nanoFramework.Device.Can/CanSettings.cs       | 20 +++++
 nanoFramework.Device.Can/CanTiming.cs         | 83 -------------------
 .../nanoFramework.Device.Can.nfproj           |  2 +-
 4 files changed, 79 insertions(+), 84 deletions(-)
 create mode 100644 nanoFramework.Device.Can/CanBitTiming.cs
 delete mode 100644 nanoFramework.Device.Can/CanTiming.cs

diff --git a/nanoFramework.Device.Can/CanBitTiming.cs b/nanoFramework.Device.Can/CanBitTiming.cs
new file mode 100644
index 0000000..0ef96ab
--- /dev/null
+++ b/nanoFramework.Device.Can/CanBitTiming.cs
@@ -0,0 +1,58 @@
+//
+// Copyright (c) .NET Foundation and Contributors
+// See LICENSE file in the project root for full license information.
+//
+
+namespace nanoFramework.Device.Can
+{
+    /// <summary>
+    /// CAN bit timings.
+    /// </summary>
+    public enum CanBitTiming
+    {
+        /// <summary>
+        /// None. Manually setted values.
+        /// </summary>
+        _None,
+
+        /// <summary>
+        /// 25kBaud.
+        /// </summary>
+        _25kBaud,
+
+        /// <summary>
+        /// 50kBaud.
+        /// </summary>
+        _50kBaud,
+
+        /// <summary>
+        /// 100kBaud.
+        /// </summary>
+        _100kBaud,
+
+        /// <summary>
+        /// 125kBaud.
+        /// </summary>
+        _125kBaud,
+
+        /// <summary>
+        /// 250kBaud.
+        /// </summary>
+        _250kBaud,
+
+        /// <summary>
+        /// 500kBaud.
+        /// </summary>
+        _500kBaud,
+
+        /// <summary>
+        /// 800kBaud.
+        /// </summary>
+        _800kBaud,
+
+        /// <summary>
+        /// 1000kBaud.
+        /// </summary>
+        _1000kBaud,
+    }
+}
diff --git a/nanoFramework.Device.Can/CanSettings.cs b/nanoFramework.Device.Can/CanSettings.cs
index a7ac738..453b320 100644
--- a/nanoFramework.Device.Can/CanSettings.cs
+++ b/nanoFramework.Device.Can/CanSettings.cs
@@ -22,6 +22,9 @@ public sealed class CanSettings
         [System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
         private byte _syncJumpWidth;
 
+        [System.Diagnostics.DebuggerBrowsable(System.Diagnostics.DebuggerBrowsableState.Never)]
+        private CanBitTiming _timing = CanBitTiming._None;
+
         /// <summary>
         /// Initializes a new instance of <see cref="CanSettings"/>.
         /// </summary>
@@ -37,6 +40,14 @@ public sealed class CanSettings
             _syncJumpWidth = syncJumpWidth;
         }
 
+        /// <summary>
+        /// Initializes a new instance of <see cref="CanSettings"/>.
+        /// </summary>
+        /// <param name="timing">Bus baud rate timing preset.</param>
+        public CanSettings (CanBitTiming timing) {
+            _timing = timing;
+        }
+
         /// <summary>
         /// Initializes a copy of a <see cref="CanSettings"/> object.
         /// </summary>
@@ -84,5 +95,14 @@ public byte SyncJumpWidth
             get { return _syncJumpWidth; }
             set { _syncJumpWidth = value; }
         }
+
+        /// <summary>
+        /// Gets or sets the <see cref="CanBitTiming"/>.
+        /// </summary>
+        public CanBitTiming Timing
+        {
+            get { return _timing; }
+            set { _timing = value; }
+        }
     }
 }
diff --git a/nanoFramework.Device.Can/CanTiming.cs b/nanoFramework.Device.Can/CanTiming.cs
deleted file mode 100644
index 59b3f48..0000000
--- a/nanoFramework.Device.Can/CanTiming.cs
+++ /dev/null
@@ -1,83 +0,0 @@
-//
-// Copyright (c) .NET Foundation and Contributors
-// See LICENSE file in the project root for full license information.
-//
-
-namespace nanoFramework.Device.Can
-{
-    /// <summary>
-    /// Helper for common CAN timings for the STM32
-    /// </summary>
-    public static class CanTimingSTM32
-    {
-        /// <summary>
-        /// 25 KBaud.
-        /// </summary>
-        public static CanSettings Timing500KBaud () {
-            return new CanSettings(6, 8, 1, 0);
-        }
-
-    }
-
-    /// <summary>
-    /// Helper for common CAN timings for the ESP32. Values from https://github.com/espressif/esp-idf/blob/master/components/hal/include/hal/twai_types.h#L54
-    /// </summary>
-    public static class CanTimingESP32
-    {
-        /// <summary>
-        /// 25 KBaud.
-        /// </summary>
-        public static CanSettings Timing25KBaud () {
-            return new CanSettings(128, 16, 8, 3);
-        }
-
-        /// <summary>
-        /// 50 KBaud.
-        /// </summary>
-        public static CanSettings Timing50KBaud () {
-            return new CanSettings(80, 15, 4, 3);
-        }
-
-        /// <summary>
-        /// 100 KBaud.
-        /// </summary>
-        public static CanSettings Timing100KBaud () {
-            return new CanSettings(40, 15, 4, 3);
-        }
-
-        /// <summary>
-        /// 125 KBaud.
-        /// </summary>
-        public static CanSettings Timing125KBaud () {
-            return new CanSettings(32, 15, 4, 3);
-        }
-
-        /// <summary>
-        /// 250 KBaud.
-        /// </summary>
-        public static CanSettings Timing250KBaud () {
-            return new CanSettings(16, 15, 4, 3);
-        }
-
-        /// <summary>
-        /// 500 KBaud.
-        /// </summary>
-        public static CanSettings Timing500KBaud () {
-            return new CanSettings(8, 15, 4, 3);
-        }
-
-        /// <summary>
-        /// 800 KBaud.
-        /// </summary>
-        public static CanSettings Timing800KBaud() {
-            return new CanSettings(4, 16, 8, 3);
-        }
-
-        /// <summary>
-        /// 1000 KBaud.
-        /// </summary>
-        public static CanSettings Timing1000KBaud () {
-            return new CanSettings(4, 15, 4, 3);
-        }
-    }
-}
diff --git a/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj b/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj
index 836e1a0..68ac034 100644
--- a/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj
+++ b/nanoFramework.Device.Can/nanoFramework.Device.Can.nfproj
@@ -53,7 +53,7 @@
     <Compile Include="CanMessageReceivedEventArgs.cs" />
     <Compile Include="CanControllerEventListener.cs" />
     <Compile Include="CanMessageEvent.cs" />
-    <Compile Include="CanTiming.cs" />
+    <Compile Include="CanBitTiming.cs" />
     <None Include="key.snk" />
   </ItemGroup>
   <ItemGroup>