lgpio 0.1.10 → 0.1.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4da526bc3f239adf23035ca2c8923463667f979038b01fe0f8b63f0ad017a0e1
-  data.tar.gz: 6beafa4839875a6aac25f78a0b4b3e4fa25a28569269254ec4dcea27bfc06572
+  metadata.gz: e3b3d476891ddae4745580531d10dc7c58de9ac0f7c93f75ad2743597db0fd74
+  data.tar.gz: df0d8c8b767115c07c1d9174cb9cf69797a04278035fb6fbc41be68b4877a585
 SHA512:
-  metadata.gz: af9365d7c43fc0f7d5bab7c360e5a2e9325f2c3f7b7015ffff39d98ee3c4c2041123dfb81354d5d0113a06daab31910d18ed3fcee91dd75ded80c13b87543cba
-  data.tar.gz: aef0e2f50840a3b601627283f15df63f1f6ab5d065288f05275c07d04e0ad32def28d055acedfbba807c16f16b041d2df3bd5444f093bbb55c604aac2026c4ac
+  metadata.gz: affc87dba05d69d20589acbcffb3988018a5f2759516d9b55a452dc8baa7c00de4c1cc61bc511590503063a06b1a41efadcf589dfcc628a53704cde9eeafe423
+  data.tar.gz: 2ea4c0c0f9ba1f602dcaff830543fd69ca6e1c6c0555fec0961362be3e51482a38210158776e94e54a445c9a17a29198a6d541ade0cdbc8863e80bd5112372e6

data/Gemfile

@@ -0,0 +1,4 @@
+source 'https://rubygems.org'
+
+# Specify your gem's dependencies in denko.gemspec
+gemspec

data/README.md

@@ -38,7 +38,7 @@ These use the sysfs PWM interface, not lgpio C, but are a good fit for this gem.
   - Carrier generated by hardware PWM. Software modulated with monotonic clock timing.
   - Useful for sending infrared signals at 38kHz, for example.
 
-**Note:** Once hardware PWM is used on a pin, it stays PWM until reboot. The associated GPIO won't work.
+**Note:** Once a pin is bound to hardware PWM in the device tree, it shouldn't be used as regular GPIO. Behavior is inconsistent across different hardware.
 
 ## Installation
 On Debian-based Linuxes (RaspberryPi OS, Armbian, DietPi etc.):

data/examples/dht.rb

@@ -13,7 +13,7 @@ chip_handle = LGPIO.chip_open(GPIO_CHIP)
 #   Maximum number of pulses to read
 #   Timeout in milliseconds
 #
-data = LGPIO.gpio_read_pulses_us(chip_handle, DHT_PIN, 20_000, 0, 84, 100)
+data = LGPIO.gpio_read_pulses_us(chip_handle, DHT_PIN, 10_000, 0, 84, 100)
 
 # Handle errors.
 raise "error: DHT sensor not connected" unless data

data/examples/i2c_bb_ssd1306_bench.rb

@@ -35,8 +35,8 @@ LGPIO.chip_close(scl_handle)
 LGPIO.chip_close(sda_handle) unless (scl_handle == sda_handle)
 
 fps = FRAME_COUNT / (finish - start)
-# Also calculate C calls per second, using roughly 23 calls per byte written.
-cps = (START_ARRAY.length + ((PATTERN_1.length + PATTERN_2.length) / 2) + 2) * 23 * fps
+# Also calculate C calls per second, using roughly 24.5 GPIO calls per byte written.
+cps = (START_ARRAY.length + ((PATTERN_1.length + PATTERN_2.length) / 2) + 2) * 24.5 * fps
 cps = (cps / 1000.0).round
 
 puts "SSD1306 benchmark result: #{fps.round(2)} fps | #{cps}k C calls/s"

data/examples/spi_bb_ssd1306_bench.rb

@@ -25,8 +25,6 @@ pins.each_value do |hash|
   end
 end
 
-puts pins.inspect
-
 spi_bb = LGPIO::SPIBitBang.new(clock: pins[:clock], output: pins[:output])
 LGPIO.gpio_claim_output(pins[:reset][:handle], pins[:reset][:line], LGPIO::SET_PULL_NONE, LGPIO::LOW)
 LGPIO.gpio_claim_output(pins[:dc][:handle], pins[:dc][:line], LGPIO::SET_PULL_NONE, LGPIO::LOW)

data/examples/spi_hw_loopback.rb

@@ -12,7 +12,7 @@ puts "TX bytes: #{tx_bytes.inspect}"
 
 # rx_bytes == tx_bytes if MOSI looped back to MISO.
 # rx_byte all 255 when MOSI and MISO not connected.
-rx_bytes = LGPIO.spi_xfer(spi_handle, tx_bytes)
+rx_bytes = LGPIO.spi_xfer(spi_handle, tx_bytes, tx_bytes.length)
 puts "RX bytes: #{rx_bytes.inspect}"
 
 LGPIO.spi_close(spi_handle)

data/ext/lgpio/lgpio.c

@@ -42,6 +42,11 @@ static void microDelay(uint64_t micros) {
   nanoDelay(micros * 1000);
 }
 
+static VALUE rbMicroDelay(VALUE self, VALUE micros) {
+  microDelay(NUM2ULL(micros));
+  return Qnil;
+}
+
 /*****************************************************************************/
 /*                              CHIP & GPIO                                 */
 /*****************************************************************************/
@@ -86,6 +91,7 @@ static VALUE gpio_write(VALUE self, VALUE handle, VALUE gpio, VALUE level) {
 }
 
 static VALUE group_claim_input(VALUE self, VALUE handle, VALUE gpios, VALUE flags) {
+  Check_Type(gpios, T_ARRAY);
   int count = rb_array_len(gpios);
   int lgGpios[count];
   int i;
@@ -97,6 +103,7 @@ static VALUE group_claim_input(VALUE self, VALUE handle, VALUE gpios, VALUE flag
 }
 
 static VALUE group_claim_output(VALUE self, VALUE handle, VALUE gpios, VALUE flags, VALUE levels) {
+  Check_Type(gpios, T_ARRAY);
   int count = rb_array_len(gpios);
   int lgGpios[count];
   int lgLevels[count];
@@ -190,7 +197,7 @@ static VALUE tx_pulse(VALUE self, VALUE handle, VALUE gpio, VALUE on, VALUE off,
 }
 
 static VALUE tx_pwm(VALUE self, VALUE handle, VALUE gpio, VALUE freq, VALUE duty, VALUE offset, VALUE cycles) {
-  int result = lgTxPwm(NUM2INT(handle), NUM2INT(gpio), NUM2INT(freq), NUM2INT(duty), NUM2INT(offset), NUM2INT(cycles));
+  int result = lgTxPwm(NUM2INT(handle), NUM2INT(gpio), NUM2INT(freq), NUM2DBL(duty), NUM2INT(offset), NUM2INT(cycles));
   return INT2NUM(result);
 }
 
@@ -201,6 +208,7 @@ static VALUE tx_servo(VALUE self, VALUE handle, VALUE gpio, VALUE width, VALUE f
 
 static VALUE tx_wave(VALUE self, VALUE handle, VALUE lead_gpio, VALUE pulses) {
   // Copy Ruby array to array of lgPulse_t.
+  Check_Type(pulses, T_ARRAY);
   int       pulseCount = rb_array_len(pulses);
   lgPulse_t pulsesOut[pulseCount];
   VALUE     rbPulse;
@@ -225,9 +233,10 @@ static VALUE tx_wave_ook(VALUE self, VALUE dutyPath, VALUE dutyString, VALUE pul
   // The Ruby class LGPIO::HardwarePWM should have already set the PWM carrier frequency.
   //
   // Convert pulses from microseconds to nanoseconds.
+  Check_Type(pulses, T_ARRAY);
   uint32_t pulseCount = rb_array_len(pulses);
   uint64_t nanoPulses[pulseCount];
-  for (int i=0; i<pulseCount; i++) {
+  for (uint32_t i=0; i<pulseCount; i++) {
     nanoPulses[i] = NUM2UINT(rb_ary_entry(pulses, i)) * 1000;
   }
 
@@ -242,7 +251,7 @@ static VALUE tx_wave_ook(VALUE self, VALUE dutyPath, VALUE dutyString, VALUE pul
   const char *cDuty = StringValueCStr(dutyString);
 
   // Toggle duty cycle between given value and 0, to modulate the PWM carrier.
-  for (int i=0; i<pulseCount; i++) {
+  for (uint32_t i=0; i<pulseCount; i++) {
     if (i % 2 == 0) {
       dutyFile = fopen(filePath, "w");
       fputs(cDuty, dutyFile);
@@ -259,6 +268,7 @@ static VALUE tx_wave_ook(VALUE self, VALUE dutyPath, VALUE dutyString, VALUE pul
   dutyFile = fopen(filePath, "w");
   fputs("0", dutyFile);
   fclose(dutyFile);
+  return UINT2NUM(pulseCount);
 }
 
 /*****************************************************************************/
@@ -274,12 +284,13 @@ static VALUE i2c_close(VALUE self, VALUE handle){
   return INT2NUM(result);
 }
 
-static VALUE i2c_write_device(VALUE self, VALUE handle, VALUE byteArray){
-  int count = RARRAY_LEN(byteArray);
+static VALUE i2c_write_device(VALUE self, VALUE handle, VALUE txArray){
+  Check_Type(txArray, T_ARRAY);
+  int count = rb_array_len(txArray);
   uint8_t txBuf[count];
   VALUE currentByte;
   for(int i=0; i<count; i++){
-    currentByte = rb_ary_entry(byteArray, i);
+    currentByte = rb_ary_entry(txArray, i);
     Check_Type(currentByte, T_FIXNUM);
     txBuf[i] = NUM2CHR(currentByte);
   }
@@ -303,7 +314,8 @@ static VALUE i2c_read_device(VALUE self, VALUE handle, VALUE count){
 }
 
 static VALUE i2c_zip(VALUE self, VALUE handle, VALUE txArray, VALUE rb_rxCount){
-  int txCount = RARRAY_LEN(txArray);
+  Check_Type(txArray, T_ARRAY);
+  int txCount = rb_array_len(txArray);
   uint8_t txBuf[txCount];
   VALUE currentByte;
   for(int i=0; i<txCount; i++){
@@ -357,7 +369,8 @@ static VALUE spi_read(VALUE self, VALUE handle, VALUE rxCount){
 }
 
 static VALUE spi_write(VALUE self, VALUE handle, VALUE txArray){
-  int count = RARRAY_LEN(txArray);
+  Check_Type(txArray, T_ARRAY);
+  int count = rb_array_len(txArray);
   uint8_t txBuf[count];
   VALUE currentByte;
   for(int i=0; i<count; i++){
@@ -370,31 +383,42 @@ static VALUE spi_write(VALUE self, VALUE handle, VALUE txArray){
   return INT2NUM(result);
 }
 
-static VALUE spi_xfer(VALUE self, VALUE handle, VALUE txArray){
-  int count = RARRAY_LEN(txArray);
+static VALUE spi_xfer(VALUE self, VALUE handle, VALUE txArray, VALUE rxLength){
+  Check_Type(txArray, T_ARRAY);
+  int txCount = rb_array_len(txArray);
+  int rxCount = NUM2INT(rxLength);
+  int count   = txCount;
+  if (rxCount > txCount) count = rxCount;
+
   uint8_t txBuf[count];
+  // Not sure if this needs null termination like I2C. +1 won't hurt.
+  uint8_t rxBuf[count+1];
+
+  // Add given bytes to transmit.
   VALUE currentByte;
-  for(int i=0; i<count; i++){
+  for(int i=0; i<txCount; i++){
     currentByte = rb_ary_entry(txArray, i);
     Check_Type(currentByte, T_FIXNUM);
     txBuf[i] = NUM2CHR(currentByte);
   }
-
-  // Not sure if this needs null termination like I2C. +1 won't hurt.
-  uint8_t rxBuf[count+1];
+  // Extend with zeroes if reading more than writing.
+  if (count > txCount) {
+    for(int i=txCount; i<count; i++) txBuf[i] = 0;
+  }
 
   int result = lgSpiXfer(NUM2INT(handle), txBuf, rxBuf, count);
   if(result < 0) return INT2NUM(result);
 
-  VALUE retArray = rb_ary_new2(count);
-  for(int i=0; i<count; i++){
+  VALUE retArray = rb_ary_new2(rxCount);
+  for(int i=0; i<rxCount; i++){
     rb_ary_store(retArray, i, UINT2NUM(rxBuf[i]));
   }
   return retArray;
 }
 
 static VALUE spi_ws2812_write(VALUE self, VALUE handle, VALUE pixelArray){
-  int count = RARRAY_LEN(pixelArray);
+  Check_Type(pixelArray, T_ARRAY);
+  int count = rb_array_len(pixelArray);
 
   // Pull low for at least one byte at 2.4 Mhz before data, and 90 after.
   int zeroesBefore = 1;
@@ -414,8 +438,8 @@ static VALUE spi_ws2812_write(VALUE self, VALUE handle, VALUE pixelArray){
     Check_Type(currentByte_rb, T_FIXNUM);
     currentByte = NUM2CHR(currentByte_rb);
 
-    for (int i=7; i>=0; i--) {
-      currentBit = (currentByte & (1 << i));
+    for (int j=7; j>=0; j--) {
+      currentBit = (currentByte & (1 << j));
       temp = temp << 3;
       temp = (currentBit == 0) ? (temp | 0b100) : (temp | 0b110);
     }
@@ -522,7 +546,7 @@ static VALUE gpio_read_pulses_us(VALUE self, VALUE rbHandle, VALUE rbGPIO, VALUE
   // Return Ruby array of pulse as microseconds
   if (pulseIndex == 0) return Qnil;
   VALUE retArray = rb_ary_new2(pulseIndex);
-  for(int i=0; i<pulseIndex; i++){
+  for(uint32_t i=0; i<pulseIndex; i++){
     uint32_t pulse_us = round(pulses_ns[i] / 1000.0);
     rb_ary_store(retArray, i, UINT2NUM(pulse_us));
   }
@@ -581,6 +605,7 @@ static VALUE one_wire_reset(VALUE self, VALUE rbHandle, VALUE rbGPIO) {
   int handle = NUM2INT(rbHandle);
   int gpio   = NUM2INT(rbGPIO);
   struct timespec start;
+  struct timespec now;
   uint8_t presence = 1;
 
   // Hold low for 500us to reset, then go high.
@@ -591,13 +616,92 @@ static VALUE one_wire_reset(VALUE self, VALUE rbHandle, VALUE rbGPIO) {
 
   // Poll for 250us. If a device pulls the line low, return 0 (device present).
   clock_gettime(CLOCK_MONOTONIC, &start);
-  while(nanosSince(&start) < 250000){
-    if (lgGpioRead(handle, gpio) == 0) presence = 0;
+  now = start;
+  while(nanoDiff(&now, &start) < 250000){
+    if (digitalRead(gpio) == 0) presence = 0;
+    clock_gettime(CLOCK_MONOTONIC, &now);
   }
 
   return UINT2NUM(presence);
 }
 
+/*****************************************************************************/
+/*                          Bit-Bang I2C Helpers                             */
+/*****************************************************************************/
+static uint8_t bitReadU8(uint8_t* b, uint8_t i) {
+  return (*b >> i) & 0b1;
+}
+
+static void bitWriteU8(uint8_t* b, uint8_t i, uint8_t v) {
+  if (v == 0) {
+    *b &= ~(1 << i);
+  } else {
+    *b |=  (1 << i);
+  }
+}
+
+static void i2c_bb_set_sda(int handle, int gpio, uint8_t* state, uint8_t level) {
+  if (level == *state) return;
+  lgGpioWrite(handle, gpio, level);
+  *state = level;
+}
+
+static VALUE i2c_bb_read_byte(VALUE self, VALUE rbSCL_H, VALUE rbSCL_L, VALUE rbSDA_H, VALUE rbSDA_L, VALUE rbACK) {
+  int scl_H   = NUM2INT(rbSCL_H);
+  int scl_L   = NUM2INT(rbSCL_L);
+  int sda_H   = NUM2INT(rbSDA_H);
+  int sda_L   = NUM2INT(rbSDA_L);
+  uint8_t ack = RTEST(rbACK);
+
+  // Prevent caching by setting opposite to first state SDA will take.
+  uint8_t sdaState = 0;
+  uint8_t b;
+  uint8_t bit;
+
+  // Receive MSB first.
+  for (int i=7; i>=0; i--) {
+    i2c_bb_set_sda(sda_H, sda_L, &sdaState, 1);
+    lgGpioWrite(scl_H, scl_L, 1);
+    bit = lgGpioRead(sda_H, sda_L);
+    lgGpioWrite(scl_H, scl_L, 0);
+
+    bitWriteU8(&b, i, bit);
+  }
+
+  // Send ACK or NACK.
+  i2c_bb_set_sda(sda_H, sda_L, &sdaState, ack ^ 0b1);
+  lgGpioWrite(scl_H, scl_L, 1);
+  lgGpioWrite(scl_H, scl_L, 0);
+
+  return UINT2NUM(b);
+}
+
+static VALUE i2c_bb_write_byte(VALUE self, VALUE rbSCL_H, VALUE rbSCL_L, VALUE rbSDA_H, VALUE rbSDA_L, VALUE rbByte) {
+  int scl_H   = NUM2INT(rbSCL_H);
+  int scl_L   = NUM2INT(rbSCL_L);
+  int sda_H   = NUM2INT(rbSDA_H);
+  int sda_L   = NUM2INT(rbSDA_L);
+  uint8_t b   = NUM2UINT(rbByte);
+
+  // Prevent caching by setting opposite to first state SDA will take.
+  uint8_t sdaState = bitReadU8(&b, 7) ^ 0b1;
+  uint8_t ack;
+
+  // Write MSBFIRST.
+  for (int i=7; i>=0; i--) {
+    i2c_bb_set_sda(sda_H, sda_L, &sdaState, bitReadU8(&b, i));
+    lgGpioWrite(scl_H, scl_L, 1);
+    lgGpioWrite(scl_H, scl_L, 0);
+  }
+
+  // Read and return ACK.
+  i2c_bb_set_sda(sda_H, sda_L, &sdaState, 1);
+  lgGpioWrite(scl_H, scl_L, 1);
+  ack = lgGpioRead(sda_H, sda_L);
+  lgGpioWrite(scl_H, scl_L, 0);
+  return (ack == 0) ? Qtrue : Qfalse;
+}
+
 /*****************************************************************************/
 /*                           EXTENSION INIT                                  */
 /*****************************************************************************/
@@ -605,7 +709,7 @@ void Init_lgpio(void) {
   // Modules
   VALUE mLGPIO = rb_define_module("LGPIO");
 
-  // Basics
+  // Constants
   rb_define_const(mLGPIO, "SET_ACTIVE_LOW",   INT2NUM(LG_SET_ACTIVE_LOW));
   rb_define_const(mLGPIO, "SET_OPEN_DRAIN",   INT2NUM(LG_SET_OPEN_DRAIN));
   rb_define_const(mLGPIO, "SET_OPEN_SOURCE",  INT2NUM(LG_SET_OPEN_SOURCE));
@@ -615,6 +719,11 @@ void Init_lgpio(void) {
   rb_define_const(mLGPIO, "RISING_EDGE",      INT2NUM(LG_RISING_EDGE));
   rb_define_const(mLGPIO, "FALLING_EDGE",     INT2NUM(LG_FALLING_EDGE));
   rb_define_const(mLGPIO, "BOTH_EDGES",       INT2NUM(LG_BOTH_EDGES));
+
+  // Generic Helpers
+  rb_define_singleton_method(mLGPIO, "micro_delay",        rbMicroDelay,      1);
+
+  // Basic GPIO
   rb_define_singleton_method(mLGPIO, "chip_open",          chip_open,         1);
   rb_define_singleton_method(mLGPIO, "chip_close",         chip_close,        1);
   rb_define_singleton_method(mLGPIO, "gpio_get_mode",      gpio_get_mode,     2);
@@ -648,23 +757,23 @@ void Init_lgpio(void) {
   // Don't use this. Servo will jitter.
   rb_define_singleton_method(mLGPIO, "tx_servo", tx_servo, 6);
 
-  // Hardware PWM waves for on-off-keying.
+  // HW PWM waves, on-off-keying only.
   VALUE cHardwarePWM = rb_define_class_under(mLGPIO, "HardwarePWM", rb_cObject);
   rb_define_method(cHardwarePWM, "tx_wave_ook", tx_wave_ook, 3);
 
-  // I2C
+  // HW I2C
   rb_define_singleton_method(mLGPIO, "i2c_open",           i2c_open,          3);
   rb_define_singleton_method(mLGPIO, "i2c_close",          i2c_close,         1);
   rb_define_singleton_method(mLGPIO, "i2c_write_device",   i2c_write_device,  2);
   rb_define_singleton_method(mLGPIO, "i2c_read_device",    i2c_read_device,   2);
   rb_define_singleton_method(mLGPIO, "i2c_zip",            i2c_zip,           3);
 
-  // SPI
+  // HW SPI
   rb_define_singleton_method(mLGPIO, "spi_open",           spi_open,          4);
   rb_define_singleton_method(mLGPIO, "spi_close",          spi_close,         1);
   rb_define_singleton_method(mLGPIO, "spi_read",           spi_read,          2);
   rb_define_singleton_method(mLGPIO, "spi_write",          spi_write,         2);
-  rb_define_singleton_method(mLGPIO, "spi_xfer",           spi_xfer,          2);
+  rb_define_singleton_method(mLGPIO, "spi_xfer",           spi_xfer,          3);
   rb_define_singleton_method(mLGPIO, "spi_ws2812_write",   spi_ws2812_write,  2);
 
   // Bit-Bang Pulse Input
@@ -675,4 +784,8 @@ void Init_lgpio(void) {
   rb_define_singleton_method(mLGPIO, "one_wire_bit_read",   one_wire_bit_read,  2);
   rb_define_singleton_method(mLGPIO, "one_wire_bit_write",  one_wire_bit_write, 3);
   rb_define_singleton_method(mLGPIO, "one_wire_reset",      one_wire_reset,     2);
+
+  // Bit-bang I2C Helpers
+  rb_define_singleton_method(mLGPIO, "i2c_bb_write_byte",   i2c_bb_write_byte,  5);
+  rb_define_singleton_method(mLGPIO, "i2c_bb_read_byte",    i2c_bb_read_byte,   5);
 }

data/lgpio.gemspec

@@ -4,7 +4,7 @@ Gem::Specification.new do |s|
   s.name        = 'lgpio'
   s.version     = LGPIO::VERSION
   s.licenses    = ['MIT']
-  s.summary     = "Use Linux GPIO, I2C, SPI and PWM in Ruby"
+  s.summary     = "Ruby C extension for Linux GPIO"
   s.description = "Use Linux GPIO, I2C, SPI and PWM in Ruby"
 
   s.authors     = ["vickash"]
@@ -13,5 +13,6 @@ Gem::Specification.new do |s|
   s.homepage    = 'https://github.com/denko-rb/lgpio'
   s.metadata    = { "source_code_uri" => "https://github.com/denko-rb/lgpio" }
 
+  s.required_ruby_version = '>=3'
   s.extensions = %w[ext/lgpio/extconf.rb]
 end

data/lib/lgpio/hardware_pwm.rb

@@ -1,10 +1,10 @@
 module LGPIO
   class HardwarePWM
-    NS_PER_S  = 1_000_000_000
-    NS_PER_US = 1_000
+    NS_PER_S  = 10**9
+    NS_PER_US = 10**3
     SYS_FS_PWM_PATH = "/sys/class/pwm/"
 
-    attr_reader :period, :duty, :enabled
+    attr_reader :period, :duty, :polarity, :enabled
 
     def initialize(chip, channel, frequency: nil, period: nil)
       @chip    = chip
@@ -16,6 +16,11 @@ module LGPIO
       end
 
       period ? self.period = period : self.frequency = frequency
+
+      # Default to with 0 duty cycle and normal polarity.
+      self.duty = 0
+      self.polarity = :normal
+
       enable
     end
 
@@ -23,6 +28,10 @@ module LGPIO
       @path ||= "#{SYS_FS_PWM_PATH}pwmchip#{@chip}/pwm#{@channel}/"
     end
 
+    def polarity_path
+      @polarity_path ||= "#{path}polarity"
+    end
+
     def period_path
       @period_path ||= "#{path}period"
     end
@@ -35,16 +44,31 @@ module LGPIO
       @enable_path ||= "#{path}enable"
     end
 
+    def polarity=(p=:normal)
+      if p == :inversed
+        File.open(polarity_path, 'w') { |f| f.write("inversed") }
+        @polarity = :inversed
+      else
+        File.open(polarity_path, 'w') { |f| f.write("normal") }
+        @polarity = :normal
+      end
+    end
+
     def frequency=(freq)
       self.period = (NS_PER_S / freq.to_f).round
+      @frequency = freq
+    end
+
+    def frequency
+      # If not set explicitly, calculate from period, rounded to nearest Hz.
+      @frequency ||= (NS_PER_S / period.to_f).round
     end
 
     def period=(p)
-      old_period = File.read("#{path}period").strip.to_i
-      unless (old_period == 0)
-        File.open(duty_path, 'w') { |f| f.write("0") }
-      end
+      old_duty = duty || File.read(duty_path).strip.to_i
+      duty = 0 unless (old_duty == 0)
       File.open(period_path, 'w') { |f| f.write(p) }
+      @frequency = nil
       @period = p
     end
 

data/lib/lgpio/i2c_bitbang.rb

@@ -20,6 +20,10 @@ module LGPIO
       initialize_pins
     end
 
+    def config
+      @config ||= [scl_handle, scl_line, sda_handle, scl_line]
+    end
+
     def initialize_pins
       LGPIO.gpio_claim_output(scl_handle, scl_line, LGPIO::SET_PULL_NONE, LGPIO::HIGH)
       LGPIO.gpio_claim_output(sda_handle, sda_line, LGPIO::SET_OPEN_DRAIN | LGPIO::SET_PULL_UP, LGPIO::HIGH)
@@ -84,6 +88,14 @@ module LGPIO
       (read_bit == 0)
     end
 
+    def read_byte_c(ack)
+      LGPIO.i2c_bb_read_byte(@scl_handle, @scl_line, @sda_handle, @sda_line, ack)
+    end
+
+    def write_byte_c(byte)
+      LGPIO.i2c_bb_write_byte(@scl_handle, @scl_line, @sda_handle, @sda_line, byte)
+    end
+
     def read(address, length)
       raise ArgumentError, "invalid I2C address: #{address}. Range is 0x08..0x77" unless VALID_ADDRESSES.include?(address)
       raise ArgumentError, "invalid Integer for read length: #{length}" unless length.kind_of?(Integer)
@@ -94,10 +106,16 @@ module LGPIO
 
       # Read length bytes, and ACK for all but the last one.
       bytes = []
-      (length-1).times { bytes << read_byte(true) }
-      bytes << read_byte(false)
-      stop
 
+      # Bit-bang per-byte in Ruby.
+      # (length-1).times { bytes << read_byte(true) }
+      # bytes << read_byte(false)
+
+      # Bit-bang per-byte in C.
+      (length-1).times { bytes << read_byte_c(true) }
+      bytes << read_byte_c(false)
+
+      stop
       bytes
     end
 
@@ -107,7 +125,13 @@ module LGPIO
 
       start
       write_byte(write_form(address))
-      bytes.each { |byte| write_byte(byte) }
+
+      # Bit-bang per-byte in Ruby.
+      # bytes.each { |byte| write_byte(byte) }
+
+      # Bit-bang per-byte in C.
+      bytes.each { |byte| write_byte_c(byte) }
+
       stop
     end
 

data/lib/lgpio/infrared.rb

@@ -1,6 +1,16 @@
 module LGPIO
   class Infrared < HardwarePWM
-    def transmit(pulses, duty: 33.333)
+    FREQUENCY = 38_000
+    DUTY = 33.333
+
+    def initialize(*args, **kwargs)
+      new_kwargs = {frequency: FREQUENCY}.merge(kwargs)
+      super(*args, **new_kwargs)
+      # Avoid needing to call #enable and #disable, before and after each #transmit call.
+      # disable
+    end
+
+    def transmit(pulses, duty: DUTY)
       self.duty_percent = duty
       tx_wave_ook(duty_path, @duty.to_s, pulses)
     end

data/lib/lgpio/one_wire.rb

@@ -19,7 +19,7 @@ module LGPIO
       @handle = handle
       @gpio   = gpio
       @found_addresses = []
-      LGPIO.gpio_claim_output(handle, LGPIO::SET_OPEN_DRAIN | LGPIO::SET_PULL_UP, gpio, LGPIO::HIGH)
+      LGPIO.gpio_claim_output(handle, gpio, LGPIO::SET_OPEN_DRAIN | LGPIO::SET_PULL_UP, LGPIO::HIGH)
     end
 
     def reset
@@ -123,12 +123,12 @@ module LGPIO
       new_discrepancies = address ^ complement
 
       high_discrepancy = -1
-      (0..63).each { |i| high_discrepancy = i if new_discrepancies[i] == 0 }
+      (0..63).each { |i| high_discrepancy = i if ((new_discrepancies >> i) & 0b1 == 0) }
 
       [address, high_discrepancy]
     end
 
-    # Convert interleaved address/complement bytes to 64-bit numbers.
+    # Result is 16 bytes, 8 byte address and complement interleaved LSByte first.
     def split_search_result(data)
       address    = 0
       complement = 0
@@ -155,7 +155,7 @@ module LGPIO
       crc = 0b00000000
       bytes.take(bytes.length - 1).each do |byte|
         for bit in (0..7)
-          xor = byte[bit] ^ crc[0]
+          xor = ((byte >> bit) & 0b1) ^ (crc & 0b1)
           crc = crc ^ ((xor * (2 ** 3)) | (xor * (2 ** 4)))
           crc = crc >> 1
           crc = crc | (xor * (2 ** 7))
@@ -165,7 +165,9 @@ module LGPIO
     end
 
     def self.address_to_bytes(address)
-      [address].pack('Q<').split("").map(&:ord)
+      bytes = []
+      8.times { |i| bytes[i] = address >> (8*i) & 0xFF }
+      bytes
     end
   end
 end

data/lib/lgpio/spi_bitbang.rb

@@ -25,7 +25,7 @@ module LGPIO
     end
 
     def config
-      @config ||= { handle: handle, clock: clock, input: input, output: output }
+      @config ||= [clock_handle, clock_line, input_handle, input_line, output_handle, output_line]
     end
 
     def initialize_pins

data/lib/lgpio/version.rb

@@ -1,3 +1,3 @@
 module LGPIO
-  VERSION = "0.1.10"
+  VERSION = "0.1.12"
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: lgpio
 version: !ruby/object:Gem::Version
-  version: 0.1.10
+  version: 0.1.12
 platform: ruby
 authors:
 - vickash
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-28 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies: []
 description: Use Linux GPIO, I2C, SPI and PWM in Ruby
 email: mail@vickash.com
@@ -18,6 +17,7 @@ extensions:
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- Gemfile
 - LICENSE
 - README.md
 - Rakefile
@@ -67,7 +67,6 @@ licenses:
 - MIT
 metadata:
   source_code_uri: https://github.com/denko-rb/lgpio
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -75,15 +74,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.20
-signing_key:
+rubygems_version: 3.6.7
 specification_version: 4
-summary: Use Linux GPIO, I2C, SPI and PWM in Ruby
+summary: Ruby C extension for Linux GPIO
 test_files: []