Changed to using System.Device APIs

Author: Andrew

README.md

@@ -48,10 +48,8 @@ Please see our [docs folder here](/docs) for an API reference, pin layout and ot
 
 ### Dependencies
 
-This project relies on [RaspberryIO](https://github.com/unosquare/raspberryio).
-It's an easy-to-use .NET library for interacting with Raspberry Pi's IO functionality.
-RaspberryIO is built on [Wiring Pi](http://wiringpi.com/) -
-a pin based GPIO access library written in C.
+This project relies on [.NET Core IoT Libraries](https://github.com/dotnet/iot).
+It is a .NET library for interacting with Raspberry Pi's IO functionality.
 
 ## Installation
 
@@ -60,7 +58,6 @@ a pin based GPIO access library written in C.
 You can grab the latest version of PowerShell IoT by running:
 
 ```powershell
-sudo WIRINGPI_CODES=1 pwsh
 Install-Module Microsoft.PowerShell.IoT
 ```
 
@@ -133,14 +130,9 @@ _NOTE: If you'd rather not use the script, simply copy the `out/Microsoft.PowerS
 First, you must run pwsh with sudo:
 
 ```powershell
-sudo WIRINGPI_CODES=1 pwsh
+sudo pwsh
 ```
 
-##### About `WIRINGPI_CODES` environment variable
-
-`Microsoft.PowerShell.IoT` module internally uses [WiringPi library](http://wiringpi.com/reference/setup) which has a default behavior of terminating current process (in this case - PowerShell) even on non-critical errors in setup functions.
-To avoid such crashes define `WIRINGPI_CODES` environment variable either when starting PowerShell (see example above) or through configuration scripts - example for an interactive login shell - `echo "export WIRINGPI_CODES=1"|sudo tee -a /etc/profile.d/WiringPiCodes.sh`
-
 If you have the `Microsoft.PowerShell.IoT` module in your PSModulePath:
 
 ```powershell

ThirdPartyNotices.txt

@@ -1,8 +1,8 @@
 # Pin layout
 
-PowerShell IoT uses Wiring Pi under the hood.
-As such, it follow the Wiring Pi scheme for pin identification.
-For all of the `Pin` parameters in this module, use the WiringPi/wPi number listed in the following diagram:
+PowerShell IoT uses logical representation of the microcontroller's GPIOs.
+Refer to the microcontroller's datasheet to find this information.
+In case of Raspberry Pi use the BCM number listed in the following diagram:
 
 ## Image reference
 

docs/rpi3_pin_layout.md

@@ -3,75 +3,79 @@
 using System.Management.Automation;
 using System.Device.Gpio;
 
- [Cmdlet(VerbsCommon.Get, "GpioPin")]
-public class GetGpioPin : Cmdlet
+[Cmdlet(VerbsCommon.Get, "GpioPin")]
+public class GetGpioPin : GpioCmdletBase
 {
-	[Parameter(Mandatory = false, ValueFromPipeline = true, ValueFromPipelineByPropertyName = true, Position = 0)]
-	public int[] Id { get; set; }
+    [Parameter(Mandatory = false, ValueFromPipeline = true, ValueFromPipelineByPropertyName = true, Position = 0)]
+    public int[] Id { get; set; }
 
-	[Parameter(Mandatory = false, ValueFromPipelineByPropertyName = true, Position = 1)]
-	public PullMode? PullMode { get; set; }
+    [Parameter(Mandatory = false, ValueFromPipelineByPropertyName = true, Position = 1)]
+    public PullMode? PullMode { get; set; }
 
-	[Parameter(Mandatory = false, ValueFromPipelineByPropertyName = true)]
-	public SwitchParameter Raw { get; set; }
+    [Parameter(Mandatory = false, ValueFromPipelineByPropertyName = true)]
+    public SwitchParameter Raw { get; set; }
 
-	protected override void ProcessRecord()
-	{
-		ArrayList pinList = new ArrayList();
+    protected override void ProcessRecord()
+    {
+        ArrayList pinList = new ArrayList();
 
-		if ((this.Id == null) || (this.Id.Length <= 0))
-		{
-			// this is "gpio readall" functionality
-
-			/*foreach (var pin in Unosquare.RaspberryIO.Pi.Gpio.Pins)
-			{
-				pinList.Add(pin.PinNumber);
-			}*/
-		}
-		else
-		{
-			pinList.AddRange(this.Id);
-		}
+        if ((this.Id == null) || (this.Id.Length <= 0))
+        {
+            // TODO: this is "gpio readall" functionality
+        }
+        else
+        {
+            pinList.AddRange(this.Id);
+        }
+        
+        PinMode mode = PinMode.Input;
+        if (this.PullMode.HasValue)
+        {
+            switch (this.PullMode.Value)
+            {
+                case global::PullMode.PullDown: mode = PinMode.InputPullDown; break;
+                case global::PullMode.PullUp:   mode = PinMode.InputPullUp; break;
+                default:                        mode = PinMode.Input; break;
+            };
+        };
 
-		using (GpioController controller = new GpioController())
+        foreach (int pinId in pinList)
         {
-			PinMode mode = PinMode.Input;
-			if (this.PullMode.HasValue)
-			{
-				switch (this.PullMode.Value)
-				{
-					case global::PullMode.PullDown: mode = PinMode.InputPullDown; break;
-					case global::PullMode.PullUp:   mode = PinMode.InputPullUp; break;
-					default:                        mode = PinMode.Input; break;
-				};
-			};
+            SignalLevel slResult = SignalLevel.Low;
+
+             if (this.GpioController.IsPinOpen(pinId))
+            {
+                if (this.GpioController.GetPinMode(pinId) != mode)
+                {
+                    this.GpioController.SetPinMode(pinId, mode);
+                }
+            }
+            else
+            {
+                this.GpioController.OpenPin(pinId, mode);
+            }
 
-			foreach (int pinId in pinList)
-			{
-				SignalLevel slResult = SignalLevel.Low;
-				controller.OpenPin(pinId, mode); // pin will be closed in GpioController.Dispose()
-				if (controller.Read(pinId) == PinValue.High)
-				{
-					slResult = SignalLevel.High;
-				};
-				
-				if (this.Raw)
-				{
-					WriteObject(slResult);
-				}
-				else
-				{
-					GpioPinData pinData = new GpioPinData(pinId, slResult);
-					WriteObject(pinData);
-				}
-			}
-		}
-	}
+            if (this.GpioController.Read(pinId) == PinValue.High)
+            {
+                slResult = SignalLevel.High;
+            };
+            
+            if (this.Raw)
+            {
+                WriteObject(slResult);
+            }
+            else
+            {
+                GpioPinData pinData = new GpioPinData(pinId, slResult);
+                WriteObject(pinData);
+            }
+        }
+    }
 }
 
  public enum PullMode
 {
-	Off = 0,
-	PullDown = 1,
-	PullUp = 2
+    Off = 0,
+    PullDown = 1,
+    PullUp = 2
 }

src/Microsoft.PowerShell.IoT/Gpio/Get/GetGpioPin.cs

@@ -0,0 +1,36 @@
+using System;
+using System.Management.Automation;
+using System.Device.Gpio;
+
+public class GpioCmdletBase : Cmdlet
+{
+    protected static GpioController StaticGpioController;
+    protected GpioController GpioController
+    {
+        get
+        {
+            if (StaticGpioController == null)
+            {
+                StaticGpioController = new GpioController();
+            }
+            return StaticGpioController;
+        }
+        set
+        {
+            StaticGpioController = value;
+        }
+    }
+}
+
+[Cmdlet(VerbsCommon.Clear, "GpioResources")]
+public class ClearGpioResources : GpioCmdletBase
+{
+    protected override void ProcessRecord()
+    {
+        if (GpioCmdletBase.StaticGpioController != null)
+        {
+            GpioCmdletBase.StaticGpioController.Dispose();
+            GpioCmdletBase.StaticGpioController = null;
+        }
+    }
+}

src/Microsoft.PowerShell.IoT/Gpio/GpioCmdletBase.cs

@@ -1,17 +1,17 @@
  public class GpioPinData
 {
-	public int Id;
-	public SignalLevel Value;
+    public int Id;
+    public SignalLevel Value;
 
-	public GpioPinData(int id, SignalLevel value)
-	{
-		this.Id = id;
-		this.Value = value;
-	}
+    public GpioPinData(int id, SignalLevel value)
+    {
+        this.Id = id;
+        this.Value = value;
+    }
 }
 
 public enum SignalLevel
 {
-	Low = 0,
-	High = 1
+    Low = 0,
+    High = 1
 }