dotnet / dotnet/iot

Mcp23017 initialization resets pin values

Open
#1,663 4 comments 0 reactions 0 assignees View on GitHub
bug Priority:2 up-for-grabs
Dominant language
C#
Stars
2.4k
Forks
630
Avg merge
11d 3h
Merged PRs (30d)
2

Description

**Describe the bug**

Every time a new instance of Mcp23017() is being created the pins are being reset to low. This is an issue when the written software reboots, eg when updating the software version.

**Steps to reproduce**

```csharp
var pinNumber = 0;

var i2cConnectionSettings = new I2cConnectionSettings(1, 0x20);

var i2cDevice = I2cDevice.Create(i2cConnectionSettings);

var device = new Mcp23017(i2cDevice);

var controller = new GpioController(PinNumberingScheme.Logical, Mcp23017Device);

controller.Write(pinNumber, PinValue.High);

var device2 = new Mcp23017(i2cDevice);

var pinValue = controller.Read(pinNumber);
```

**Expected behavior**

Pins need to keep their value after re-initializing the Mcp23017() adapter.

**Actual behavior**

Pins are being reset every time a new instance of Mcp23017() is being declared. Looks like it is embedded in the init code here:
https://github.com/dotnet/iot/blob/main/src/devices/Mcp23xxx/Mcp23xxx.cs - starting at line 84.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.