Merge pull request #352 from taartspi/linuxfs_i2c_first_op_fail

Depending upon i2c method called the device may not be selected on th…
Pi4J · Apr 19, 2024 · 849775b · 849775b
2 parents c4ea33e + b2c8e3a
commit 849775b
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 5 deletions.
diff --git a/...pi4j-plugin-linuxfs/src/main/java/com/pi4j/plugin/linuxfs/provider/i2c/LinuxFsI2CBus.java b/...pi4j-plugin-linuxfs/src/main/java/com/pi4j/plugin/linuxfs/provider/i2c/LinuxFsI2CBus.java
@@ -79,9 +79,6 @@ protected void selectBusSlave(I2C i2c) throws IOException {
      */
     public int executeIOCTL(final I2C i2c, long command, ByteBuffer data, IntBuffer offsets){
         int rc = -1;
-        if (this.lastAddress != i2c.device()) {
-            this.lastAddress = i2c.device();
-        }
         try {
             if (this.lock.tryLock() || this.lock.tryLock(this.lockAquireTimeout, this.lockAquireTimeoutUnit)) {
 

diff --git a/...in-linuxfs/src/main/java/com/pi4j/plugin/linuxfs/provider/i2c/LinuxFsI2CProviderImpl.java b/...in-linuxfs/src/main/java/com/pi4j/plugin/linuxfs/provider/i2c/LinuxFsI2CProviderImpl.java
@@ -57,8 +57,6 @@ public synchronized I2C create(I2CConfig config) {
         LinuxFsI2CBus i2CBus = this.i2CBusMap.computeIfAbsent(config.getBus(), busNr -> new LinuxFsI2CBus(config));
         // create new I/O instance based on I/O config
         LinuxFsI2C i2C = new LinuxFsI2C(i2CBus, this, config);
-        // Workaround, needed if first LinuxFsI2C usage is ioctl (readRegister or writeRegister)
-        i2C.read();
         this.context.registry().add(i2C);
         return i2C;
     }