whoops, forgot to set isOut

CHANGELOG.md

@@ -1,55 +1,3 @@
-0.33.0
----
-- **new devices**
-    - **ens160**
-        - Add ens160 i2c driver
-    - **lsm303dlhc**
-        - added support for LSM303DLHC e-Compass; (#783)
-    - **seesaw**
-        - add support for Adafruit Seesaw encoders
-
-- **enhancements**
-    - **ws2812**
-        - add RP2350 support
-    - **ssd1306**
-        - avoid unnecessary heap allocations (#767)
-    - **gps**
-        - allow gps init with address
-    - **lsm6ds3tr**
-        - avoid unnecessary heap allocations (#766)
-
-- **bugfixes**
-    - **gps**
-        - Fix gps time calculation (#785)
-
-
-0.32.0
----
-- **enhancements**
-    - **bmp280**
-        - remove alloc on read sensor data
-    - **ws2812**
-        - add 200MHz support for the Cortex-M0/rp2040
-
-- **bugfixes**
-    - **ssd1306**
-        - remove time.Sleep from SSD1306 SPI transfer code
-    - **tmc2209**
-        - tmc2209 bug fixes (#755)
-
-- **docs**
-    - **contributing**
-        - add driver design pointer to CONTRIBUTING.md
-
-
-0.31.0
----
----
-- **enhancements**
-    - **spi**
-        - update all SPI usage to use either *machine.SPI or drivers.SPI
-
-
 0.30.0
 ---
 - **new devices**

CONTRIBUTING.md

@@ -8,9 +8,6 @@ We would like your help to make this project better, so we appreciate any contri
 
 We'd love to get your feedback on getting started with TinyGo. Run into any difficulty, confusion, or anything else? You are not alone. We want to know about your experience, so we can help the next people. Please open a Github issue with your questions, or you can also get in touch directly with us on our Slack channel at [https://gophers.slack.com/messages/CDJD3SUP6](https://gophers.slack.com/messages/CDJD3SUP6).
 
-### Driver design
-Before porting or writing a driver from scratch please read **[Driver Design for TinyGo](https://tinygo.org/docs/guides/driver-design)**.
-
 ### One of the TinyGo drivers is not working as you expect
 
 Please open a Github issue with your problem, and we will be happy to assist.

apa102/apa102.go

@@ -5,9 +5,10 @@ package apa102 // import "tinygo.org/x/drivers/apa102"
 
 import (
 	"image/color"
-	"machine"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 const (
@@ -37,8 +38,11 @@ func New(b drivers.SPI) *Device {
 
 // NewSoftwareSPI returns a new APA102 driver that will use a software based
 // implementation of the SPI protocol.
-func NewSoftwareSPI(sckPin, sdoPin machine.Pin, delay uint32) *Device {
-	return New(&bbSPI{SCK: sckPin, SDO: sdoPin, Delay: delay})
+func NewSoftwareSPI(sckPin, sdoPin pin.Output, delay uint32) *Device {
+	return New(&bbSPI{SCK: sckPin.Set, SDO: sdoPin.Set, Delay: delay, configurePins: func() {
+		legacy.ConfigurePinOut(sckPin)
+		legacy.ConfigurePinOut(sdoPin)
+	}})
 }
 
 // WriteColors writes the given RGBA color slice out using the APA102 protocol.

apa102/softspi.go

@@ -1,6 +1,9 @@
 package apa102
 
-import "machine"
+import (
+	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+)
 
 // bbSPI is a dumb bit-bang implementation of SPI protocol that is hardcoded
 // to mode 0 and ignores trying to receive data. Just enough for the APA102.
@@ -8,15 +11,18 @@ import "machine"
 // most purposes other than the APA102 package. It might be desirable to make
 // this more generic and include it in the TinyGo "machine" package instead.
 type bbSPI struct {
-	SCK   machine.Pin
-	SDO   machine.Pin
-	Delay uint32
+	SCK           drivers.PinOutput
+	SDO           drivers.PinOutput
+	Delay         uint32
+	configurePins func()
 }
 
 // Configure sets up the SCK and SDO pins as outputs and sets them low
 func (s *bbSPI) Configure() {
-	s.SCK.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	s.SDO.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	if s.configurePins == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
+	s.configurePins()
 	s.SCK.Low()
 	s.SDO.Low()
 	if s.Delay == 0 {

bmi160/bmi160.go

@@ -1,31 +1,36 @@
 package bmi160
 
 import (
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // DeviceSPI is the SPI interface to a BMI160 accelerometer/gyroscope. There is
 // also an I2C interface, but it is not yet supported.
 type DeviceSPI struct {
 	// Chip select pin
-	CSB machine.Pin
+	csb drivers.PinOutput
 
 	buf [7]byte
 
 	// SPI bus (requires chip select to be usable).
-	Bus drivers.SPI
+	bus           drivers.SPI
+	configurePins func()
 }
 
 // NewSPI returns a new device driver. The pin and SPI interface are not
 // touched, provide a fully configured SPI object and call Configure to start
 // using this device.
-func NewSPI(csb machine.Pin, spi drivers.SPI) *DeviceSPI {
+func NewSPI(csb pin.Output, spi drivers.SPI) *DeviceSPI {
 	return &DeviceSPI{
-		CSB: csb, // chip select
-		Bus: spi,
+		csb: csb.Set, // chip select
+		bus: spi,
+		configurePins: func() {
+			legacy.ConfigurePinOut(csb)
+		},
 	}
 }
 
@@ -33,8 +38,11 @@ func NewSPI(csb machine.Pin, spi drivers.SPI) *DeviceSPI {
 // configures the BMI160, but it does not configure the SPI interface (it is
 // assumed to be up and running).
 func (d *DeviceSPI) Configure() error {
-	d.CSB.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.CSB.High()
+	if d.configurePins == nil {
+		return legacy.ErrConfigBeforeInstantiated
+	}
+	d.configurePins()
+	d.csb.High()
 
 	// The datasheet recommends doing a register read from address 0x7F to get
 	// SPI communication going:
@@ -86,9 +94,9 @@ func (d *DeviceSPI) ReadTemperature() (temperature int32, err error) {
 	data[0] = 0x80 | reg_TEMPERATURE_0
 	data[1] = 0
 	data[2] = 0
-	d.CSB.Low()
-	err = d.Bus.Tx(data, data)
-	d.CSB.High()
+	d.csb.Low()
+	err = d.bus.Tx(data, data)
+	d.csb.High()
 	if err != nil {
 		return
 	}
@@ -123,9 +131,9 @@ func (d *DeviceSPI) ReadAcceleration() (x int32, y int32, z int32, err error) {
 	for i := 1; i < len(data); i++ {
 		data[i] = 0
 	}
-	d.CSB.Low()
-	err = d.Bus.Tx(data, data)
-	d.CSB.High()
+	d.csb.Low()
+	err = d.bus.Tx(data, data)
+	d.csb.High()
 	if err != nil {
 		return
 	}
@@ -153,9 +161,9 @@ func (d *DeviceSPI) ReadRotation() (x int32, y int32, z int32, err error) {
 	for i := 1; i < len(data); i++ {
 		data[i] = 0
 	}
-	d.CSB.Low()
-	err = d.Bus.Tx(data, data)
-	d.CSB.High()
+	d.csb.Low()
+	err = d.bus.Tx(data, data)
+	d.csb.High()
 	if err != nil {
 		return
 	}
@@ -201,9 +209,9 @@ func (d *DeviceSPI) readRegister(address uint8) uint8 {
 	data := d.buf[:2]
 	data[0] = 0x80 | address
 	data[1] = 0
-	d.CSB.Low()
-	d.Bus.Tx(data, data)
-	d.CSB.High()
+	d.csb.Low()
+	d.bus.Tx(data, data)
+	d.csb.High()
 	return data[1]
 }
 
@@ -217,7 +225,7 @@ func (d *DeviceSPI) writeRegister(address, data uint8) {
 	buf[0] = address
 	buf[1] = data
 
-	d.CSB.Low()
-	d.Bus.Tx(buf, buf)
-	d.CSB.High()
+	d.csb.Low()
+	d.bus.Tx(buf, buf)
+	d.csb.High()
 }

bmp280/bmp280.go

@@ -23,7 +23,6 @@ type Filter uint
 type Device struct {
 	bus         drivers.I2C
 	Address     uint16
-	buf         [6]byte
 	cali        calibrationCoefficients
 	Temperature Oversampling
 	Pressure    Oversampling
@@ -135,8 +134,8 @@ func (d *Device) PrintCali() {
 
 // ReadTemperature returns the temperature in celsius milli degrees (°C/1000).
 func (d *Device) ReadTemperature() (temperature int32, err error) {
-	data := d.buf[:3]
-	if err = d.readData(REG_TEMP, data); err != nil {
+	data, err := d.readData(REG_TEMP, 3)
+	if err != nil {
 		return
 	}
 
@@ -159,8 +158,8 @@ func (d *Device) ReadTemperature() (temperature int32, err error) {
 // ReadPressure returns the pressure in milli pascals (mPa).
 func (d *Device) ReadPressure() (pressure int32, err error) {
 	// First 3 bytes are Pressure, last 3 bytes are Temperature
-	data := d.buf[:6]
-	if err = d.readData(REG_PRES, data); err != nil {
+	data, err := d.readData(REG_PRES, 6)
+	if err != nil {
 		return
 	}
 
@@ -204,7 +203,7 @@ func (d *Device) ReadPressure() (pressure int32, err error) {
 }
 
 // readData reads n number of bytes of the specified register
-func (d *Device) readData(register int, data []byte) error {
+func (d *Device) readData(register int, n int) ([]byte, error) {
 	// If not in normal mode, set the mode to FORCED mode, to prevent incorrect measurements
 	// After the measurement in FORCED mode, the sensor will return to SLEEP mode
 	if d.Mode != MODE_NORMAL {
@@ -219,7 +218,9 @@ func (d *Device) readData(register int, data []byte) error {
 	}
 
 	// Read the requested register
-	return legacy.ReadRegister(d.bus, uint8(d.Address), uint8(register), data[:])
+	data := make([]byte, n)
+	err := legacy.ReadRegister(d.bus, uint8(d.Address), uint8(register), data[:])
+	return data, err
 }
 
 // convert3Bytes converts three bytes to int32

buzzer/buzzer.go

@@ -2,22 +2,23 @@
 package buzzer // import "tinygo.org/x/drivers/buzzer"
 
 import (
-	"machine"
-
 	"time"
+
+	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // Device wraps a GPIO connection to a buzzer.
 type Device struct {
-	pin  machine.Pin
+	pin  drivers.PinOutput
 	High bool
 	BPM  float64
 }
 
 // New returns a new buzzer driver given which pin to use
-func New(pin machine.Pin) Device {
+func New(pin pin.Output) Device {
 	return Device{
-		pin:  pin,
+		pin:  pin.Set,
 		High: false,
 		BPM:  96.0,
 	}
@@ -25,14 +26,14 @@ func New(pin machine.Pin) Device {
 
 // On sets the buzzer to a high state.
 func (l *Device) On() (err error) {
-	l.pin.Set(true)
+	l.pin.High()
 	l.High = true
 	return
 }
 
 // Off sets the buzzer to a low state.
 func (l *Device) Off() (err error) {
-	l.pin.Set(false)
+	l.pin.Low()
 	l.High = false
 	return
 }

easystepper/easystepper.go

@@ -2,9 +2,9 @@
 package easystepper // import "tinygo.org/x/drivers/easystepper"
 
 import (
-	"errors"
-	"machine"
 	"time"
+
+	"tinygo.org/x/drivers"
 )
 
 // StepMode determines the coil sequence used to perform a single step
@@ -30,28 +30,10 @@ func (sm StepMode) stepCount() uint {
 	}
 }
 
-// DeviceConfig contains the configuration data for a single easystepper driver
-type DeviceConfig struct {
-	// Pin1 ... Pin4 determines the pins to configure and use for the device
-	Pin1, Pin2, Pin3, Pin4 machine.Pin
-	// StepCount is the number of steps required to perform a full revolution of the stepper motor
-	StepCount uint
-	// RPM determines the speed of the stepper motor in 'Revolutions per Minute'
-	RPM uint
-	// Mode determines the coil sequence used to perform a single step
-	Mode StepMode
-}
-
-// DualDeviceConfig contains the configuration data for a dual easystepper driver
-type DualDeviceConfig struct {
-	DeviceConfig
-	// Pin5 ... Pin8 determines the pins to configure and use for the second device
-	Pin5, Pin6, Pin7, Pin8 machine.Pin
-}
-
 // Device holds the pins and the delay between steps
 type Device struct {
-	pins       [4]machine.Pin
+	pins       [4]drivers.PinOutput
+	config     func()
 	stepDelay  time.Duration
 	stepNumber uint8
 	stepMode   StepMode
@@ -62,51 +44,6 @@ type DualDevice struct {
 	devices [2]*Device
 }
 
-// New returns a new single easystepper driver given a DeviceConfig
-func New(config DeviceConfig) (*Device, error) {
-	if config.StepCount == 0 || config.RPM == 0 {
-		return nil, errors.New("config.StepCount and config.RPM must be > 0")
-	}
-	return &Device{
-		pins:      [4]machine.Pin{config.Pin1, config.Pin2, config.Pin3, config.Pin4},
-		stepDelay: time.Second * 60 / time.Duration((config.StepCount * config.RPM)),
-		stepMode:  config.Mode,
-	}, nil
-}
-
-// Configure configures the pins of the Device
-func (d *Device) Configure() {
-	for _, pin := range d.pins {
-		pin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	}
-}
-
-// NewDual returns a new dual easystepper driver given 8 pins, number of steps and rpm
-func NewDual(config DualDeviceConfig) (*DualDevice, error) {
-	// Create the first device
-	dev1, err := New(config.DeviceConfig)
-	if err != nil {
-		return nil, err
-	}
-	// Create the second device
-	config.DeviceConfig.Pin1 = config.Pin5
-	config.DeviceConfig.Pin2 = config.Pin6
-	config.DeviceConfig.Pin3 = config.Pin7
-	config.DeviceConfig.Pin4 = config.Pin8
-	dev2, err := New(config.DeviceConfig)
-	if err != nil {
-		return nil, err
-	}
-	// Return composite dual device
-	return &DualDevice{devices: [2]*Device{dev1, dev2}}, nil
-}
-
-// Configure configures the pins of the DualDevice
-func (d *DualDevice) Configure() {
-	d.devices[0].Configure()
-	d.devices[1].Configure()
-}
-
 // Move rotates the motor the number of given steps
 // (negative steps will rotate it the opposite direction)
 func (d *Device) Move(steps int32) {

easystepper/easystepper_go.go

@@ -0,0 +1,26 @@
+package easystepper
+
+import (
+	"errors"
+	"time"
+
+	"tinygo.org/x/drivers"
+)
+
+func NewCrossPlatform(stepcount, rpm uint, mode StepMode, pins [4]drivers.PinOutput) (*Device, error) {
+	if stepcount == 0 || rpm == 0 {
+		return nil, errors.New("zero rpm and/or stepcount")
+	}
+	for i := range pins {
+		if pins[i] == nil {
+			return nil, errors.New("nil pin")
+		}
+	}
+	d := &Device{
+		pins:      pins,
+		stepDelay: time.Second * 60 / time.Duration((stepcount * rpm)),
+		stepMode:  mode,
+		config:    func() {},
+	}
+	return d, nil
+}

easystepper/easystepper_tinygo.go

@@ -0,0 +1,83 @@
+//go:build baremetal
+
+package easystepper
+
+import (
+	"errors"
+	"machine"
+	"time"
+
+	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+)
+
+// New returns a new single easystepper driver given a DeviceConfig
+func New(config DeviceConfig) (*Device, error) {
+	if config.StepCount == 0 || config.RPM == 0 {
+		return nil, errors.New("config.StepCount and config.RPM must be > 0")
+	}
+	return &Device{
+		pins:      [4]drivers.PinOutput{config.Pin1.Set, config.Pin2.Set, config.Pin3.Set, config.Pin4.Set},
+		stepDelay: time.Second * 60 / time.Duration((config.StepCount * config.RPM)),
+		stepMode:  config.Mode,
+		config: func() {
+			legacy.ConfigurePinOut(config.Pin1)
+			legacy.ConfigurePinOut(config.Pin2)
+			legacy.ConfigurePinOut(config.Pin3)
+			legacy.ConfigurePinOut(config.Pin4)
+		},
+	}, nil
+}
+
+// Configure configures the pins of the Device
+func (d *Device) Configure() {
+	if d.config == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
+	d.config()
+}
+
+// Configure configures the pins of the DualDevice
+func (d *DualDevice) Configure() {
+	d.devices[0].Configure()
+	d.devices[1].Configure()
+}
+
+// NewDual returns a new dual easystepper driver given 8 pins, number of steps and rpm
+func NewDual(config DualDeviceConfig) (*DualDevice, error) {
+	// Create the first device
+	dev1, err := New(config.DeviceConfig)
+	if err != nil {
+		return nil, err
+	}
+	// Create the second device
+	config.DeviceConfig.Pin1 = config.Pin5
+	config.DeviceConfig.Pin2 = config.Pin6
+	config.DeviceConfig.Pin3 = config.Pin7
+	config.DeviceConfig.Pin4 = config.Pin8
+	dev2, err := New(config.DeviceConfig)
+	if err != nil {
+		return nil, err
+	}
+	// Return composite dual device
+	return &DualDevice{devices: [2]*Device{dev1, dev2}}, nil
+}
+
+// DeviceConfig contains the configuration data for a single easystepper driver
+type DeviceConfig struct {
+	// Pin1 ... Pin4 determines the pins to configure and use for the device
+	Pin1, Pin2, Pin3, Pin4 machine.Pin
+	// StepCount is the number of steps required to perform a full revolution of the stepper motor
+	StepCount uint
+	// RPM determines the speed of the stepper motor in 'Revolutions per Minute'
+	RPM uint
+	// Mode determines the coil sequence used to perform a single step
+	Mode StepMode
+}
+
+// DualDeviceConfig contains the configuration data for a dual easystepper driver
+type DualDeviceConfig struct {
+	DeviceConfig
+	// Pin5 ... Pin8 determines the pins to configure and use for the second device
+	Pin5, Pin6, Pin7, Pin8 machine.Pin
+}

encoders/quadrature_interrupt.go

@@ -1,4 +1,4 @@
-//go:build tinygo && (rp2040 || rp2350 || stm32 || k210 || esp32c3 || nrf || sam || (avr && (atmega328p || atmega328pb)))
+//go:build tinygo && (rp2040 || stm32 || k210 || esp32c3 || nrf || sam || (avr && (atmega328p || atmega328pb)))
 
 // Implementation based on:
 // https://gist.github.com/aykevl/3fc1683ed77bb0a9c07559dfe857304a

ens160/ens160.go

@@ -1,225 +0,0 @@
-// Package ens160 provides a driver for the ScioSense ENS160 digital gas sensor.
-//
-// Datasheet: https://www.sciosense.com/wp-content/uploads/2023/12/ENS160-Datasheet.pdf
-package ens160
-
-import (
-	"encoding/binary"
-	"errors"
-	"time"
-
-	"tinygo.org/x/drivers"
-)
-
-const (
-	defaultTimeout = 30 * time.Millisecond
-	shortTimeout   = 1 * time.Millisecond
-)
-
-// Conversion constants for environment data compensation.
-const (
-	kelvinOffsetMilli = 273150          // 273.15 K in milli-units
-	tempRawFactor     = 64              // As per datasheet for TEMP_IN
-	humRawFactor      = 512             // As per datasheet for RH_IN
-	milliFactor       = 1000            // For converting from milli-units
-	roundingTerm      = milliFactor / 2 // For rounding before integer division
-)
-
-// validityStrings provides human-readable descriptions for validity flags.
-var validityStrings = [...]string{
-	ValidityNormalOperation:     "normal operation",
-	ValidityWarmUpPhase:         "warm-up phase, wait ~3 minutes for valid data",
-	ValidityInitialStartUpPhase: "initial start-up phase, wait ~1 hour for valid data",
-	ValidityInvalidOutput:       "invalid output",
-}
-
-// Device wraps an I2C connection to an ENS160 device.
-type Device struct {
-	bus  drivers.I2C // I²C implementation
-	addr uint16      // 7‑bit bus address, promoted to uint16 per drivers.I2C
-
-	// shadow registers / last measurements
-	lastTvocPPB  uint16
-	lastEco2PPM  uint16
-	lastAqiUBA   uint8
-	lastValidity uint8 // Store the latest validity status
-
-	// pre‑allocated buffers
-	wbuf [5]byte // longest write: reg + 4 bytes (TEMP+RH)
-	rbuf [5]byte // longest read: DATA burst (5 bytes)
-}
-
-// New returns a new ENS160 driver.
-func New(bus drivers.I2C, addr uint16) *Device {
-	if addr == 0 {
-		addr = DefaultAddress
-	}
-	return &Device{
-		bus:          bus,
-		addr:         addr,
-		lastValidity: ValidityInvalidOutput,
-	}
-}
-
-// Connected returns whether a ENS160 has been found.
-func (d *Device) Connected() bool {
-	d.wbuf[0] = regPartID
-	err := d.bus.Tx(d.addr, d.wbuf[:1], d.rbuf[:2])
-	return err == nil && d.rbuf[0] == LowPartID && d.rbuf[1] == HighPartID
-}
-
-// Configure sets up the device for reading.
-func (d *Device) Configure() error {
-	// 1. Soft-reset. The device will automatically enter IDLE mode.
-	if err := d.write1(regOpMode, ModeReset); err != nil {
-		return err
-	}
-	time.Sleep(defaultTimeout)
-
-	// 2. Clear GPR registers, then go to STANDARD mode.
-	if err := d.write1(regCommand, cmdClrGPR); err != nil {
-		return err
-	}
-	time.Sleep(defaultTimeout)
-
-	if err := d.write1(regOpMode, ModeStandard); err != nil {
-		return err
-	}
-	time.Sleep(defaultTimeout)
-
-	return nil
-}
-
-// calculateTempRaw converts temperature from milli-degrees Celsius to the sensor's raw format.
-func calculateTempRaw(tempMilliC int32) uint16 {
-	// Clip temperature
-	const (
-		minC = -40 * 1000
-		maxC = 85 * 1000
-	)
-	if tempMilliC < minC {
-		tempMilliC = minC
-	} else if tempMilliC > maxC {
-		tempMilliC = maxC
-	}
-
-	// Integer fixed-point conversion to format required by the sensor.
-	// Formula from datasheet: T_IN = (T_ambient_C + 273.15) * 64
-	return uint16((((tempMilliC + kelvinOffsetMilli) * tempRawFactor) + roundingTerm) / milliFactor)
-}
-
-// calculateHumRaw converts relative humidity from milli-percent to the sensor's raw format.
-func calculateHumRaw(rhMilliPct int32) uint16 {
-	// Clip humidity
-	if rhMilliPct < 0 {
-		rhMilliPct = 0
-	} else if rhMilliPct > 100*1000 {
-		rhMilliPct = 100 * 1000
-	}
-
-	// Integer fixed-point conversion to format required by the sensor.
-	// Formula from datasheet: RH_IN = (RH_ambient_% * 512)
-	return uint16(((rhMilliPct * humRawFactor) + roundingTerm) / milliFactor)
-}
-
-// SetEnvDataMilli sets the ambient temperature and humidity for compensation.
-//
-// tempMilliC is the temperature in milli-degrees Celsius.
-// rhMilliPct is the relative humidity in milli-percent.
-func (d *Device) SetEnvDataMilli(tempMilliC, rhMilliPct int32) error {
-	tempRaw := calculateTempRaw(tempMilliC)
-	humRaw := calculateHumRaw(rhMilliPct)
-
-	d.wbuf[0] = regTempIn // start address (auto‑increment)
-	binary.LittleEndian.PutUint16(d.wbuf[1:3], tempRaw)
-	binary.LittleEndian.PutUint16(d.wbuf[3:5], humRaw)
-
-	return d.bus.Tx(d.addr, d.wbuf[:5], nil)
-}
-
-// Update refreshes the concentration measurements.
-func (d *Device) Update(which drivers.Measurement) error {
-	if which&drivers.Concentration == 0 {
-		return nil // nothing requested
-	}
-
-	const maxTries = 1000
-	var (
-		status   uint8
-		validity uint8
-	)
-	var gotData bool
-
-	// Poll DEVICE_STATUS until NEWDAT or timeout
-	for range maxTries {
-		var err error
-		status, err = d.read1(regStatus)
-		if err != nil {
-			return err
-		}
-		if status&statusSTATER != 0 {
-			return errors.New("ENS160: error (STATER set)")
-		}
-		validity = (status & statusValidityMask) >> statusValidityShift
-
-		if status&statusNEWDAT != 0 {
-			gotData = true
-			break // Always break when data available
-		}
-		time.Sleep(shortTimeout)
-	}
-	if !gotData {
-		return errors.New("ENS160: timeout waiting for NEWDAT")
-	}
-
-	// Burst-read data regardless of validity state
-	d.wbuf[0] = regAQI
-	if err := d.bus.Tx(d.addr, d.wbuf[:1], d.rbuf[:5]); err != nil {
-		return errors.New("ENS160: burst read failed")
-	}
-
-	d.lastAqiUBA = d.rbuf[0]
-	d.lastTvocPPB = binary.LittleEndian.Uint16(d.rbuf[1:3])
-	d.lastEco2PPM = binary.LittleEndian.Uint16(d.rbuf[3:5])
-	d.lastValidity = validity // Store the validity status
-
-	return nil
-}
-
-// TVOC returns the last total‑VOC concentration in parts‑per‑billion.
-func (d *Device) TVOC() uint16 { return d.lastTvocPPB }
-
-// ECO2 returns the last equivalent CO₂ concentration in parts‑per‑million.
-func (d *Device) ECO2() uint16 { return d.lastEco2PPM }
-
-// AQI returns the last Air‑Quality Index according to UBA (1–5).
-func (d *Device) AQI() uint8 { return d.lastAqiUBA }
-
-// Validity returns the current operating state of the sensor.
-func (d *Device) Validity() uint8 {
-	return d.lastValidity
-}
-
-// ValidityString returns a human-readable string describing the current validity status.
-func (d *Device) ValidityString() string {
-	if int(d.lastValidity) < len(validityStrings) {
-		return validityStrings[d.lastValidity]
-	}
-	return "unknown"
-}
-
-// write1 writes a single byte to a register.
-func (d *Device) write1(reg, val uint8) error {
-	d.wbuf[0] = reg
-	d.wbuf[1] = val
-	return d.bus.Tx(d.addr, d.wbuf[:2], nil)
-}
-
-// read1 reads a single byte from a register.
-func (d *Device) read1(reg uint8) (uint8, error) {
-	d.wbuf[0] = reg
-	if err := d.bus.Tx(d.addr, d.wbuf[:1], d.rbuf[:1]); err != nil {
-		return 0, err
-	}
-	return d.rbuf[0], nil
-}

ens160/ens160_test.go

@@ -1,54 +0,0 @@
-package ens160
-
-import (
-	"testing"
-)
-
-func TestCalculateTempRaw(t *testing.T) {
-	testCases := []struct {
-		name        string
-		tempMilliC  int32
-		expectedRaw uint16
-	}{
-		{"25°C", 25000, 19082},
-		{"-10.5°C", -10500, 16810},
-		{"Min temp", -40000, 14922},
-		{"Below min", -50000, 14922},
-		{"Max temp", 85000, 22922},
-		{"Above max", 90000, 22922},
-		{"Zero", 0, 17482},
-	}
-
-	for _, tc := range testCases {
-		t.Run(tc.name, func(t *testing.T) {
-			raw := calculateTempRaw(tc.tempMilliC)
-			if raw != tc.expectedRaw {
-				t.Errorf("expected %d, got %d", tc.expectedRaw, raw)
-			}
-		})
-	}
-}
-
-func TestCalculateHumRaw(t *testing.T) {
-	testCases := []struct {
-		name        string
-		rhMilliPct  int32
-		expectedRaw uint16
-	}{
-		{"50%", 50000, 25600},
-		{"0%", 0, 0},
-		{"100%", 100000, 51200},
-		{"Below 0%", -10000, 0},
-		{"Above 100%", 110000, 51200},
-		{"33.3%", 33300, 17050},
-	}
-
-	for _, tc := range testCases {
-		t.Run(tc.name, func(t *testing.T) {
-			raw := calculateHumRaw(tc.rhMilliPct)
-			if raw != tc.expectedRaw {
-				t.Errorf("expected %d, got %d", tc.expectedRaw, raw)
-			}
-		})
-	}
-}

ens160/registers.go

@@ -1,65 +0,0 @@
-package ens160
-
-// DefaultAddress is the default I2C address for the ENS160 when the ADDR pin is
-// connected to high (3.3V). When connected to low (GND), the address is 0x52.
-const DefaultAddress = 0x53
-
-// Registers
-const (
-	regPartID   = 0x00
-	regOpMode   = 0x10
-	regConfig   = 0x11
-	regCommand  = 0x12
-	regTempIn   = 0x13
-	regRhIn     = 0x15
-	regStatus   = 0x20
-	regAQI      = 0x21
-	regTVOC     = 0x22
-	regECO2     = 0x24
-	regDataT    = 0x30
-	regDataRH   = 0x32
-	regMISR     = 0x38
-	regGPRWrite = 0x40
-	regGPRRead  = 0x48
-)
-
-// Operating modes
-const (
-	ModeDeepSleep = 0x00
-	ModeIdle      = 0x01
-	ModeStandard  = 0x02
-	ModeReset     = 0xF0
-)
-
-// Status register bits
-const (
-	statusSTATAS = 1 << 7
-	statusSTATER = 1 << 6
-
-	statusValidityMask  = 0x0C
-	statusValidityShift = 2
-
-	statusNEWDAT = 1 << 1
-	statusNEWGPR = 1 << 0
-)
-
-// Validity flags
-const (
-	ValidityNormalOperation     = 0x00
-	ValidityWarmUpPhase         = 0x01 // need ~3 minutes until valid data
-	ValidityInitialStartUpPhase = 0x02 // need ~1 hour until valid data
-	ValidityInvalidOutput       = 0x03
-)
-
-// Commands
-const (
-	cmdNOP       = 0x00
-	cmdGetAppVer = 0x0E
-	cmdClrGPR    = 0xCC
-)
-
-// Part IDs
-const (
-	LowPartID  = 0x60
-	HighPartID = 0x01
-)

examples/ens160/main.go

@@ -1,56 +0,0 @@
-// This example demonstrates ENS160 usage.
-//
-// Wiring:
-// - VCC to 3.3V, GND to ground
-// - SDA to board SDA, SCL to board SCL
-
-package main
-
-import (
-	"time"
-
-	"machine"
-
-	"tinygo.org/x/drivers"
-	"tinygo.org/x/drivers/ens160"
-)
-
-func main() {
-	err := machine.I2C0.Configure(machine.I2CConfig{
-		Frequency: 400 * machine.KHz,
-	})
-	if err != nil {
-		println("Failed to configure I2C:", err)
-	}
-
-	dev := ens160.New(machine.I2C0, ens160.DefaultAddress)
-
-	connected := dev.Connected()
-	if !connected {
-		println("ENS160 not detected")
-		return
-	}
-	println("ENS160 detected")
-
-	if err := dev.Configure(); err != nil {
-		println("Failed to configure ENS160:", err)
-	}
-
-	for {
-		err := dev.Update(drivers.Concentration)
-		if err != nil {
-			println("Error reading ENS160: %v\n", err)
-			time.Sleep(5 * time.Second)
-			continue
-		}
-
-		println(
-			"AQI:", dev.AQI(),
-			"TVOC:", dev.TVOC(),
-			"eCO2:", dev.ECO2(),
-			"Validity:", dev.ValidityString(),
-		)
-
-		time.Sleep(2 * time.Second)
-	}
-}

examples/gps/i2c/main.go

@@ -10,7 +10,7 @@ import (
 func main() {
 	println("GPS I2C Example")
 	machine.I2C0.Configure(machine.I2CConfig{})
-	ublox := gps.NewI2CWithAddress(machine.I2C0, gps.UBLOX_I2C_ADDRESS)
+	ublox := gps.NewI2C(machine.I2C0)
 	parser := gps.NewParser()
 	var fix gps.Fix
 	for {

examples/honeyhsc/example.go

@@ -1,49 +0,0 @@
-package main
-
-import (
-	"machine"
-	"time"
-
-	"tinygo.org/x/drivers"
-	"tinygo.org/x/drivers/honeyhsc"
-)
-
-// Data taken from https://github.com/rodan/honeywell_hsc_ssc_i2c/blob/master/hsc_ssc_i2c.cpp
-// these defaults are valid for the HSCMRNN030PA2A3 chip
-const (
-	i2cAddress = 0x28
-	// 10%
-	outputMinimum = 0x666
-	// 90% of 2^14 - 1
-	outputMax = 0x399A
-	// min is 0 for sensors that give absolute values
-	pressureMin = 0
-	// 30psi (and we want results in millipascals)
-	// pressureMax = 206842.7
-	pressureMax = 206843 * 1000
-)
-
-func main() {
-	bus := machine.I2C0
-	err := bus.Configure(machine.I2CConfig{
-		Frequency: 400_000, // 100kHz minimum and 400kHz I2C maximum clock. 50 to 800 for SPI.
-		SDA:       machine.I2C0_SDA_PIN,
-		SCL:       machine.I2C0_SCL_PIN,
-	})
-	if err != nil {
-		panic(err.Error())
-	}
-	sensor := honeyhsc.NewDevI2C(bus, i2cAddress, outputMinimum, outputMax, pressureMin, pressureMax)
-	for {
-		time.Sleep(time.Second)
-		const measuremask = drivers.Pressure | drivers.Temperature
-		err := sensor.Update(measuremask)
-		if err != nil {
-			println("error updating measurements:", err.Error())
-			continue
-		}
-		P := sensor.Pressure()
-		T := sensor.Temperature()
-		println("pressure:", P, "temperature:", T)
-	}
-}

examples/lis3dh/main.go

@@ -14,18 +14,9 @@ func main() {
 	i2c.Configure(machine.I2CConfig{SCL: machine.SCL1_PIN, SDA: machine.SDA1_PIN})
 
 	accel := lis3dh.New(i2c)
-	err := accel.Configure(lis3dh.Config{
-		Address: lis3dh.Address1, // address on the Circuit Playground Express
-	})
-	for err != nil {
-		println("could not configure LIS3DH:", err)
-		time.Sleep(time.Second)
-	}
-	err = accel.SetRange(lis3dh.RANGE_2_G)
-	for err != nil {
-		println("could not set acceleration range:", err)
-		time.Sleep(time.Second)
-	}
+	accel.Address = lis3dh.Address1 // address on the Circuit Playground Express
+	accel.Configure()
+	accel.SetRange(lis3dh.RANGE_2_G)
 
 	println(accel.Connected())
 

examples/lsm303dlhc/main.go

@@ -1,58 +0,0 @@
-package main
-
-import (
-	"machine"
-	"time"
-
-	"tinygo.org/x/drivers/lsm303dlhc"
-)
-
-func main() {
-
-	// LSM303DLHC is connected to the I2C0 bus on Adafruit Feather M4 via pins: 20(SDA) and 21(SCL).
-	machine.I2C0.Configure(machine.I2CConfig{})
-
-	sensor := lsm303dlhc.New(machine.I2C0)
-	//default settings
-	err := sensor.Configure(lsm303dlhc.Configuration{
-		AccelPowerMode: lsm303dlhc.ACCEL_POWER_NORMAL,
-		AccelRange:     lsm303dlhc.ACCEL_RANGE_2G,
-		AccelDataRate:  lsm303dlhc.ACCEL_DATARATE_100HZ,
-		MagPowerMode:   lsm303dlhc.MAG_POWER_NORMAL,
-		MagSystemMode:  lsm303dlhc.MAG_SYSTEM_CONTINUOUS,
-		MagDataRate:    lsm303dlhc.MAG_DATARATE_10HZ,
-	})
-	if err != nil {
-		for {
-			println("Failed to configure", err.Error())
-			time.Sleep(time.Second)
-		}
-	}
-
-	for {
-		accel_x, accel_y, accel_z, err := sensor.ReadAcceleration()
-		if err != nil {
-			println("Failed to read accel", err.Error())
-		}
-		println("ACCEL_X:", accel_x, " ACCEL_Y:", accel_y, " ACCEL_Z:", accel_z)
-
-		mag_x, mag_y, mag_z, err := sensor.ReadMagneticField()
-		if err != nil {
-			println("Failed to read mag", err.Error())
-		}
-		println("MAG_X:", mag_x, " MAG_Y:", mag_y, " MAG_Z:", mag_z)
-
-		pitch, roll, _ := sensor.ReadPitchRoll()
-		println("Pitch:", float32(pitch), " Roll:", float32(roll))
-
-		heading, _ := sensor.ReadCompass()
-		println("Heading:", float32(heading), "degrees")
-
-		temp, _ := sensor.ReadTemperature()
-		println("Temperature:", float32(temp)/1000, "*C")
-
-		println("\n")
-		time.Sleep(time.Millisecond * 250)
-	}
-
-}

examples/seesaw/rotary-encoder/main.go

@@ -1,35 +0,0 @@
-package main
-
-import (
-	"machine"
-	"time"
-
-	"tinygo.org/x/drivers/seesaw"
-)
-
-// example reading the position of a rotary encoder (4991) powered by a seesaw
-// https://learn.adafruit.com/adafruit-i2c-qt-rotary-encoder/arduino
-func main() {
-	// This assumes you are using an Adafruit QT Py RP2040 for its Stemma QT connector
-	// https://www.adafruit.com/product/4900
-	i2c := machine.I2C1
-	i2c.Configure(machine.I2CConfig{
-		SCL: machine.I2C1_QT_SCL_PIN,
-		SDA: machine.I2C1_QT_SDA_PIN,
-	})
-
-	dev := seesaw.New(i2c)
-	dev.Address = 0x36
-
-	for {
-		time.Sleep(time.Second)
-
-		pos, err := dev.GetEncoderPosition(0, false)
-		if err != nil {
-			println(err)
-			continue
-		}
-
-		println(pos)
-	}
-}

examples/ssd1306/i2c_128x32/main.go

@@ -0,0 +1,51 @@
+package main
+
+import (
+	"machine"
+
+	"image/color"
+	"time"
+
+	"tinygo.org/x/drivers/ssd1306"
+)
+
+func main() {
+	machine.I2C0.Configure(machine.I2CConfig{
+		Frequency: machine.TWI_FREQ_400KHZ,
+	})
+
+	display := ssd1306.NewI2C(machine.I2C0)
+	display.Configure(ssd1306.Config{
+		Address: ssd1306.Address_128_32,
+		Width:   128,
+		Height:  32,
+	})
+
+	display.ClearDisplay()
+
+	x := int16(0)
+	y := int16(0)
+	deltaX := int16(1)
+	deltaY := int16(1)
+	for {
+		pixel := display.GetPixel(x, y)
+		c := color.RGBA{255, 255, 255, 255}
+		if pixel {
+			c = color.RGBA{0, 0, 0, 255}
+		}
+		display.SetPixel(x, y, c)
+		display.Display()
+
+		x += deltaX
+		y += deltaY
+
+		if x == 0 || x == 127 {
+			deltaX = -deltaX
+		}
+
+		if y == 0 || y == 31 {
+			deltaY = -deltaY
+		}
+		time.Sleep(1 * time.Millisecond)
+	}
+}

examples/ssd1306/i2c_128x64/main.go

@@ -0,0 +1,60 @@
+// This example shows how to use 128x64 display over I2C
+// Tested on Seeeduino XIAO Expansion Board https://wiki.seeedstudio.com/Seeeduino-XIAO-Expansion-Board/
+//
+// According to manual, I2C address of the display is 0x78, but that's 8-bit address.
+// TinyGo operates on 7-bit addresses and respective 7-bit address would be 0x3C, which we use below.
+//
+// To learn more about different types of I2C addresses, please see following page
+// https://www.totalphase.com/support/articles/200349176-7-bit-8-bit-and-10-bit-I2C-Slave-Addressing
+
+package main
+
+import (
+	"machine"
+
+	"image/color"
+	"time"
+
+	"tinygo.org/x/drivers/ssd1306"
+)
+
+func main() {
+	machine.I2C0.Configure(machine.I2CConfig{
+		Frequency: machine.TWI_FREQ_400KHZ,
+	})
+
+	display := ssd1306.NewI2C(machine.I2C0)
+	display.Configure(ssd1306.Config{
+		Address: 0x3C,
+		Width:   128,
+		Height:  64,
+	})
+
+	display.ClearDisplay()
+
+	x := int16(0)
+	y := int16(0)
+	deltaX := int16(1)
+	deltaY := int16(1)
+	for {
+		pixel := display.GetPixel(x, y)
+		c := color.RGBA{255, 255, 255, 255}
+		if pixel {
+			c = color.RGBA{0, 0, 0, 255}
+		}
+		display.SetPixel(x, y, c)
+		display.Display()
+
+		x += deltaX
+		y += deltaY
+
+		if x == 0 || x == 127 {
+			deltaX = -deltaX
+		}
+
+		if y == 0 || y == 63 {
+			deltaY = -deltaY
+		}
+		time.Sleep(1 * time.Millisecond)
+	}
+}

examples/ssd1306/main.go

@@ -1,59 +0,0 @@
-package main
-
-// This example shows how to use SSD1306 OLED display driver over I2C and SPI.
-//
-// Check the `newSSD1306Display()` functions for I2C and SPI initializations.
-
-import (
-	"runtime"
-
-	"image/color"
-	"time"
-)
-
-func main() {
-
-	display := newSSD1306Display()
-	display.ClearDisplay()
-
-	w, h := display.Size()
-	x := int16(0)
-	y := int16(0)
-	deltaX := int16(1)
-	deltaY := int16(1)
-
-	traceTime := time.Now().UnixMilli() + 1000
-	frames := 0
-	ms := runtime.MemStats{}
-
-	for {
-		pixel := display.GetPixel(x, y)
-		c := color.RGBA{255, 255, 255, 255}
-		if pixel {
-			c = color.RGBA{0, 0, 0, 255}
-		}
-		display.SetPixel(x, y, c)
-		display.Display()
-
-		x += deltaX
-		y += deltaY
-
-		if x == 0 || x == w-1 {
-			deltaX = -deltaX
-		}
-
-		if y == 0 || y == h-1 {
-			deltaY = -deltaY
-		}
-
-		frames++
-		now := time.Now().UnixMilli()
-		if now >= traceTime {
-			runtime.ReadMemStats(&ms)
-			println("TS", now, "| FPS", frames, "| HeapInuse", ms.HeapInuse)
-			traceTime = now + 1000
-			frames = 0
-		}
-	}
-
-}

examples/ssd1306/main_i2c_xiao-ble.go

@@ -1,38 +0,0 @@
-//go:build xiao_ble
-
-// This initializes SSD1306 OLED display driver over I2C.
-//
-// Seeed XIAO BLE board + SSD1306 128x32 I2C OLED display.
-//
-// Wiring:
-// - XIAO GND      -> OLED GND
-// - XIAO 3v3      -> OLED VCC
-// - XIAO D4 (SDA) -> OLED SDA
-// - XIAO D5 (SCL) -> OLED SCK
-//
-// For your case:
-// - Connect the display to I2C pins on your board.
-// - Adjust I2C address and display size as needed.
-
-package main
-
-import (
-	"machine"
-
-	"tinygo.org/x/drivers/ssd1306"
-)
-
-func newSSD1306Display() *ssd1306.Device {
-	machine.I2C0.Configure(machine.I2CConfig{
-		Frequency: 400 * machine.KHz,
-		SDA:       machine.SDA0_PIN,
-		SCL:       machine.SCL0_PIN,
-	})
-	display := ssd1306.NewI2C(machine.I2C0)
-	display.Configure(ssd1306.Config{
-		Address: ssd1306.Address_128_32, // or ssd1306.Address
-		Width:   128,
-		Height:  32, // or 64
-	})
-	return display
-}

examples/ssd1306/main_spi_thumby.go

@@ -1,27 +0,0 @@
-//go:build thumby
-
-// This initializes SSD1306 OLED display driver over SPI.
-//
-// Thumby board has a tiny built-in 72x40 display.
-//
-// As the display is built-in, no wiring is needed.
-
-package main
-
-import (
-	"machine"
-
-	"tinygo.org/x/drivers/ssd1306"
-)
-
-func newSSD1306Display() *ssd1306.Device {
-	machine.SPI0.Configure(machine.SPIConfig{})
-	display := ssd1306.NewSPI(machine.SPI0, machine.THUMBY_DC_PIN, machine.THUMBY_RESET_PIN, machine.THUMBY_CS_PIN)
-	display.Configure(ssd1306.Config{
-		Width:     72,
-		Height:    40,
-		ResetCol:  ssd1306.ResetValue{28, 99},
-		ResetPage: ssd1306.ResetValue{0, 5},
-	})
-	return display
-}

examples/ssd1306/main_spi_xiao-rp2040.go

@@ -1,40 +0,0 @@
-//go:build xiao_rp2040
-
-// This initializes SSD1306 OLED display driver over SPI.
-//
-// Seeed XIAO RP2040 board + SSD1306 128x64 SPI OLED display.
-//
-// Wiring:
-// - XIAO GND       -> OLED GND
-// - XIAO 3v3       -> OLED VCC
-// - XIAO D8 (SCK)  -> OLED D0
-// - XIAO D10 (SDO) -> OLED D1
-// - XIAO D4        -> OLED RES
-// - XIAO D5        -> OLED DC
-// - XIAO D6        -> OLED CS
-//
-// For your case:
-// - Connect the display to SPI pins on your board.
-// - Adjust RES, DC and CS pins as needed.
-// - Adjust SPI frequency as needed.
-// - Adjust display size as needed.
-
-package main
-
-import (
-	"machine"
-
-	"tinygo.org/x/drivers/ssd1306"
-)
-
-func newSSD1306Display() *ssd1306.Device {
-	machine.SPI0.Configure(machine.SPIConfig{
-		Frequency: 50 * machine.MHz,
-	})
-	display := ssd1306.NewSPI(machine.SPI0, machine.D5, machine.D4, machine.D6)
-	display.Configure(ssd1306.Config{
-		Width:  128,
-		Height: 64,
-	})
-	return display
-}

examples/ssd1306/spi_128x64/main.go

@@ -0,0 +1,48 @@
+package main
+
+import (
+	"image/color"
+	"machine"
+	"time"
+
+	"tinygo.org/x/drivers/ssd1306"
+)
+
+func main() {
+	machine.SPI0.Configure(machine.SPIConfig{
+		Frequency: 8000000,
+	})
+	display := ssd1306.NewSPI(machine.SPI0, machine.P8, machine.P7, machine.P9)
+	display.Configure(ssd1306.Config{
+		Width:  128,
+		Height: 64,
+	})
+
+	display.ClearDisplay()
+
+	x := int16(64)
+	y := int16(32)
+	deltaX := int16(1)
+	deltaY := int16(1)
+	for {
+		pixel := display.GetPixel(x, y)
+		c := color.RGBA{255, 255, 255, 255}
+		if pixel {
+			c = color.RGBA{0, 0, 0, 255}
+		}
+		display.SetPixel(x, y, c)
+		display.Display()
+
+		x += deltaX
+		y += deltaY
+
+		if x == 0 || x == 127 {
+			deltaX = -deltaX
+		}
+
+		if y == 0 || y == 63 {
+			deltaY = -deltaY
+		}
+		time.Sleep(1 * time.Millisecond)
+	}
+}

examples/ssd1306/spi_thumby/main.go

@@ -0,0 +1,50 @@
+// This example using the SSD1306 OLED display over SPI on the Thumby board
+// A very tiny 72x40 display.
+package main
+
+import (
+	"image/color"
+	"machine"
+	"time"
+
+	"tinygo.org/x/drivers/ssd1306"
+)
+
+func main() {
+	machine.SPI0.Configure(machine.SPIConfig{})
+	display := ssd1306.NewSPI(machine.SPI0, machine.THUMBY_DC_PIN, machine.THUMBY_RESET_PIN, machine.THUMBY_CS_PIN)
+	display.Configure(ssd1306.Config{
+		Width:     72,
+		Height:    40,
+		ResetCol:  ssd1306.ResetValue{28, 99},
+		ResetPage: ssd1306.ResetValue{0, 5},
+	})
+
+	display.ClearDisplay()
+
+	x := int16(36)
+	y := int16(20)
+	deltaX := int16(1)
+	deltaY := int16(1)
+	for {
+		pixel := display.GetPixel(x, y)
+		c := color.RGBA{255, 255, 255, 255}
+		if pixel {
+			c = color.RGBA{0, 0, 0, 255}
+		}
+		display.SetPixel(x, y, c)
+		display.Display()
+
+		x += deltaX
+		y += deltaY
+
+		if x == 0 || x == 71 {
+			deltaX = -deltaX
+		}
+
+		if y == 0 || y == 39 {
+			deltaY = -deltaY
+		}
+		time.Sleep(1 * time.Millisecond)
+	}
+}

flash/devices.go

@@ -46,10 +46,6 @@ var DefaultDeviceIdentifier = DeviceIdentifierFunc(func(id JedecID) Attrs {
 		return GD25Q16C()
 	case 0xC84017:
 		return GD25Q64C()
-	case 0x856015:
-		return P25Q16H()
-	case 0xEF4014:
-		return W25Q80DV()
 	case 0xEF4015:
 		return W25Q16JVIQ()
 	case 0xEF4016:
@@ -243,24 +239,6 @@ func GD25Q64C() Attrs {
 	}
 }
 
-// Settings for the Puya P25Q16H 2MiB SPI flash.
-// Datasheet: https://files.seeedstudio.com/wiki/github_weiruanexample/Flash_P25Q16H-UXH-IR_Datasheet.pdf
-func P25Q16H() Attrs {
-	return Attrs{
-		TotalSize:           1 << 21, // 2 MiB
-		StartUp:             5000 * time.Microsecond,
-		JedecID:             JedecID{0x85, 0x60, 0x15},
-		MaxClockSpeedMHz:    55,
-		QuadEnableBitMask:   0x02,
-		HasSectorProtection: true,
-		SupportsFastRead:    true,
-		SupportsQSPI:        true,
-		SupportsQSPIWrites:  true,
-		WriteStatusSplit:    true,
-		SingleStatusByte:    false,
-	}
-}
-
 // Settings for the Winbond W25Q16JV-IQ 2MiB SPI flash. Note that JV-IM has a
 // different .memory_type (0x70) Datasheet:
 // https://www.winbond.com/resource-files/w25q16jv%20spi%20revf%2005092017.pdf
@@ -402,25 +380,6 @@ func W25Q80DL() Attrs {
 		TotalSize:           1 << 20, // 1 MiB
 		StartUp:             5000 * time.Microsecond,
 		JedecID:             JedecID{0xEF, 0x60, 0x14},
-		MaxClockSpeedMHz:    80,
-		QuadEnableBitMask:   0x02,
-		HasSectorProtection: false,
-		SupportsFastRead:    true,
-		SupportsQSPI:        true,
-		SupportsQSPIWrites:  false,
-		WriteStatusSplit:    false,
-		SingleStatusByte:    false,
-	}
-}
-
-// Settings for the Winbond W25Q80DV 2MiB SPI flash.
-// Datasheet:
-// https://www.winbond.com/resource-files/w25q80dv%20dl_revh_10022015.pdf
-func W25Q80DV() Attrs {
-	return Attrs{
-		TotalSize:           1 << 21, // 2 MiB
-		StartUp:             5000 * time.Microsecond,
-		JedecID:             JedecID{0xEF, 0x40, 0x14},
 		MaxClockSpeedMHz:    104,
 		QuadEnableBitMask:   0x02,
 		HasSectorProtection: false,

ft6336/ft6336.go

@@ -5,28 +5,29 @@
 package ft6336
 
 import (
-	"machine"
-
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 	"tinygo.org/x/drivers/touch"
 )
 
 // Device wraps FT6336 I2C Self-Capacitive touch
 type Device struct {
-	bus     drivers.I2C
-	buf     []byte
-	Address uint8
-	intPin  machine.Pin
+	bus           drivers.I2C
+	buf           []byte
+	Address       uint8
+	configurePins func()
 }
 
 // New returns FT6336 device for the provided I2C bus using default address.
-func New(i2c drivers.I2C, intPin machine.Pin) *Device {
+func New(i2c drivers.I2C, intPin pin.Input) *Device {
 	return &Device{
 		bus:     i2c,
 		buf:     make([]byte, 11),
 		Address: Address,
-		intPin:  intPin,
+		configurePins: func() {
+			legacy.ConfigurePinInputPulldown(intPin)
+		},
 	}
 }
 
@@ -36,8 +37,11 @@ type Config struct {
 
 // Configure the FT6336 device.
 func (d *Device) Configure(config Config) error {
+	if d.configurePins == nil {
+		return legacy.ErrConfigBeforeInstantiated
+	}
 	d.write1Byte(0xA4, 0x00)
-	d.intPin.Configure(machine.PinConfig{Mode: machine.PinInputPulldown})
+	d.configurePins()
 	return nil
 }
 

gc9a01/gc9a01.go

@@ -5,12 +5,13 @@ package gc9a01 // import "tinygo.org/x/drivers/gc9a01"
 
 import (
 	"image/color"
-	"machine"
 	"time"
 
 	"errors"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // Rotation controls the rotation used by the display.
@@ -22,10 +23,10 @@ type FrameRate uint8
 // Device wraps an SPI connection.
 type Device struct {
 	bus             drivers.SPI
-	dcPin           machine.Pin
-	resetPin        machine.Pin
-	csPin           machine.Pin
-	blPin           machine.Pin
+	dcPin           drivers.PinOutput
+	resetPin        drivers.PinOutput
+	csPin           drivers.PinOutput
+	blPin           drivers.PinOutput
 	width           int16
 	height          int16
 	columnOffsetCfg int16
@@ -52,17 +53,17 @@ type Config struct {
 }
 
 // New creates a new ST7789 connection. The SPI wire must already be configured.
-func New(bus drivers.SPI, resetPin, dcPin, csPin, blPin machine.Pin) Device {
-	resetPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	blPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+func New(bus drivers.SPI, resetPin, dcPin, csPin, blPin pin.Output) Device {
+	legacy.ConfigurePinOut(resetPin)
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(csPin)
+	legacy.ConfigurePinOut(blPin)
 	return Device{
 		bus:      bus,
-		resetPin: resetPin,
-		dcPin:    dcPin,
-		csPin:    csPin,
-		blPin:    blPin,
+		resetPin: resetPin.Set,
+		dcPin:    dcPin.Set,
+		csPin:    csPin.Set,
+		blPin:    blPin.Set,
 	}
 }
 
@@ -226,7 +227,7 @@ func (d *Device) Data(data uint8) {
 
 // Tx sends data to the display
 func (d *Device) Tx(data []byte, isCommand bool) {
-	d.dcPin.Set(!isCommand)
+	d.dcPin(!isCommand)
 	d.bus.Tx(data, nil)
 }
 

gps/gps.go

@@ -69,16 +69,10 @@ func NewUART(uart drivers.UART) Device {
 }
 
 // NewI2C creates a new I2C GPS connection.
-// Uses the default i2c address (0x42) for backward compatibility reasons.
 func NewI2C(bus drivers.I2C) Device {
-	return NewI2CWithAddress(bus, I2C_ADDRESS)
-}
-
-// NewI2CWithAddress creates a new I2C GPS connection on the provided address
-func NewI2CWithAddress(bus drivers.I2C, i2cAddress uint16) Device {
 	return Device{
 		bus:      bus,
-		address:  i2cAddress,
+		address:  I2C_ADDRESS,
 		buffer:   make([]byte, bufferSize),
 		bufIdx:   bufferSize,
 		sentence: strings.Builder{},

gps/gpsparser.go

@@ -96,10 +96,7 @@ func (parser *Parser) Parse(sentence string) (Fix, error) {
 		fix.Speed = findSpeed(fields[7])
 		fix.Heading = findHeading(fields[8])
 		date := findDate(fields[9])
-		fix.Time = date.Add(time.Duration(fix.Time.Hour())*time.Hour +
-			time.Duration(fix.Time.Minute())*time.Minute +
-			time.Duration(fix.Time.Second())*time.Second +
-			time.Duration(fix.Time.Nanosecond())*time.Nanosecond)
+		fix.Time = fix.Time.AddDate(date.Year(), int(date.Month()), date.Day())
 
 		return fix, nil
 	}

gps/gpsparser_test.go

@@ -70,15 +70,15 @@ func TestParseRMC(t *testing.T) {
 		t.Error("should have errInvalidRMCSentence error")
 	}
 
-	val = "$GPRMC,203522.00,A,5109.0262308,N,11401.8407342,W,0.004,133.4,010622,0.0,E,D*2B"
+	val = "$GPRMC,203522.00,A,5109.0262308,N,11401.8407342,W,0.004,133.4,130522,0.0,E,D*2B"
 	fix, err := p.Parse(val)
 	if err != nil {
 		t.Error("should have parsed")
 	}
 
 	c.Assert(fix.Time.Year(), qt.Equals, 2022)
-	c.Assert(fix.Time.Month(), qt.Equals, time.June)
-	c.Assert(fix.Time.Day(), qt.Equals, 1)
+	c.Assert(fix.Time.Month(), qt.Equals, time.May)
+	c.Assert(fix.Time.Day(), qt.Equals, 13)
 	c.Assert(fix.Time.Hour(), qt.Equals, 20)
 	c.Assert(fix.Time.Minute(), qt.Equals, 35)
 	c.Assert(fix.Time.Second(), qt.Equals, 22)

gps/registers.go

@@ -4,11 +4,7 @@ package gps
 
 // The I2C address which this device listens to.
 const (
-	// To ensure backward compatibility
-	I2C_ADDRESS = UBLOX_I2C_ADDRESS
-
-	UBLOX_I2C_ADDRESS   = 0x42
-	PA1010D_I2C_ADDRESS = 0x10
+	I2C_ADDRESS = 0x42
 )
 
 const (

hcsr04/hcsr04.go

@@ -5,30 +5,40 @@
 package hcsr04
 
 import (
-	"machine"
 	"time"
+
+	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 const TIMEOUT = 23324 // max sensing distance (4m)
 
 // Device holds the pins
 type Device struct {
-	trigger machine.Pin
-	echo    machine.Pin
+	trigger       drivers.PinOutput
+	echo          drivers.PinInput
+	configurePins func()
 }
 
 // New returns a new ultrasonic driver given 2 pins
-func New(trigger, echo machine.Pin) Device {
+func New(trigger pin.Output, echo pin.Input) Device {
 	return Device{
-		trigger: trigger,
-		echo:    echo,
+		trigger: trigger.Set,
+		echo:    echo.Get,
+		configurePins: func() {
+			legacy.ConfigurePinOut(trigger)
+			legacy.ConfigurePinInput(echo)
+		},
 	}
 }
 
 // Configure configures the pins of the Device
 func (d *Device) Configure() {
-	d.trigger.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.echo.Configure(machine.PinConfig{Mode: machine.PinInput})
+	if d.configurePins == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
+	d.configurePins()
 }
 
 // ReadDistance returns the distance of the object in mm
@@ -52,7 +62,7 @@ func (d *Device) ReadPulse() int32 {
 	d.trigger.Low()
 	i := uint8(0)
 	for {
-		if d.echo.Get() {
+		if d.echo() {
 			t = time.Now()
 			break
 		}
@@ -66,7 +76,7 @@ func (d *Device) ReadPulse() int32 {
 	}
 	i = 0
 	for {
-		if !d.echo.Get() {
+		if !d.echo() {
 			return int32(time.Since(t).Microseconds())
 		}
 		i++

hd44780/hd44780.go

@@ -210,7 +210,7 @@ func (d *Device) SendCommand(command byte) {
 	d.bus.SetCommandMode(true)
 	d.bus.Write([]byte{command})
 
-	for d.busy(command == DISPLAY_CLEAR || command == CURSOR_HOME) {
+	for d.isBusy(command == DISPLAY_CLEAR || command == CURSOR_HOME) {
 	}
 }
 
@@ -219,7 +219,7 @@ func (d *Device) sendData(data byte) {
 	d.bus.SetCommandMode(false)
 	d.bus.Write([]byte{data})
 
-	for d.busy(false) {
+	for d.isBusy(false) {
 	}
 }
 
@@ -231,9 +231,9 @@ func (d *Device) CreateCharacter(cgramAddr uint8, data []byte) {
 	}
 }
 
-// busy returns true when hd447890 is busy
+// isBusy returns true when hd447890 is isBusy
 // or after the timeout specified
-func (d *Device) busy(longDelay bool) bool {
+func (d *Device) isBusy(longDelay bool) bool {
 	if d.bus.WriteOnly() {
 		// Can't read busy flag if write only, so sleep a bit then return
 		if longDelay {
@@ -261,7 +261,7 @@ func (d *Device) busy(longDelay bool) bool {
 
 // Busy returns true when hd447890 is busy
 func (d *Device) Busy() bool {
-	return d.busy(false)
+	return d.isBusy(false)
 }
 
 // Size returns the current size of the display.

honeyhsc/hsc.go

@@ -1,191 +0,0 @@
-package honeyhsc
-
-import (
-	"errors"
-	"math"
-
-	"tinygo.org/x/drivers"
-)
-
-var (
-	errSensorMissing = errors.New("hsc: not connected")
-	errDiagnostic    = errors.New("hsc: diagnostic error")
-)
-
-const (
-	measuremask  = drivers.Pressure | drivers.Temperature
-	statusMask   = 0b1100_0000
-	statusOffset = 6
-)
-
-// DevI2C is the TruStability® High Accuracy Silicon Ceramic (HSC) Series is a piezoresistive silicon pressure sensor offering a ratiometric
-// analog or digital output for reading pressure over the specified full scale pressure span and temperature range.
-type DevI2C struct {
-	bus drivers.I2C
-	dev
-	addr uint8
-	buf  [6]byte
-}
-
-// NewDevI2C creates and returns a new DevI2C that communicates with an HSC device over the provided I2C bus.
-// Parameters:
-//   - bus: the I2C bus to use.
-//   - addr: the 7-bit I2C address of the sensor.
-//   - outMin, outMax: raw output code range (counts) corresponding to the pressure span. Depends on sensor model.
-//   - pMin, pMax: pressure range endpoints in millipascals (mPa). Depends on sensor model.
-//
-// The returned DevI2C will use these calibration parameters to convert raw bridge counts to pressure.
-func NewDevI2C(bus drivers.I2C, addr, outMin, outMax uint16, pMin, pMax int32) *DevI2C {
-	h := &DevI2C{
-		bus:  bus,
-		addr: uint8(addr),
-		dev: dev{
-			cmin: outMin,
-			cmax: outMax,
-			pmin: pMin,
-			pmax: pMax,
-		},
-	}
-	return h
-}
-
-// ReadTemperature reads and returns the temperature in milliKelvin (mC) from the I2C-attached HSC device.
-// It performs an Update internally to get the latest temperature value.
-func (h *DevI2C) ReadTemperature() (int32, error) {
-	err := h.Update(drivers.Temperature)
-	if err != nil {
-		return 0, err
-	}
-	return h.Temperature(), nil
-}
-
-// Update reads both temperature and pressure data from the I2C-attached HSC device when
-// the requested measurement mask includes pressure or temperature.
-// If neither pressure nor temperature is requested, Update is a no-op.
-func (d *DevI2C) Update(which drivers.Measurement) error {
-	// Update performs an I2C transaction to read 4 bytes, parses the status bits, 14-bit bridge data and
-	// temperature bits, and forwards them to the internal update routine. Any I2C transport error is returned,
-	// as well as errors produced by the internal update (e.g. errSensorMissing, errDiagnostic).
-	if which&measuremask == 0 {
-		return nil
-	}
-	rbuf := d.buf[:4]
-	wbuf := d.buf[4:6]
-	const reg = 0
-	value := (d.addr << 1) | 1
-	wbuf[0] = reg
-	wbuf[1] = value
-	err := d.bus.Tx(uint16(d.addr), wbuf, rbuf)
-	if err != nil {
-		return err
-	}
-	status := (rbuf[0] & statusMask) >> statusOffset
-	bridgeData := (uint16(rbuf[0]&^statusMask) << 8) | uint16(rbuf[1])
-	tempData := uint16(rbuf[2])<<8 | uint16(rbuf[3]&0xe0)>>5
-	return d.dev.update(status, bridgeData, tempData)
-}
-
-type pinout func(level bool)
-
-// DevI2C is the TruStability® High Accuracy Silicon Ceramic (HSC) Series is a piezoresistive silicon pressure sensor offering a ratiometric
-// analog or digital output for reading pressure over the specified full scale pressure span and temperature range.
-type DevSPI struct {
-	spi drivers.SPI
-	cs  pinout
-	dev
-	buf [4]byte
-}
-
-// NewDevSPI creates and returns a new DevSPI that communicates with an HSC device over SPI.
-// Parameters:
-//   - conn: the SPI connection to use.
-//   - cs: a chip-select function that drives the device select line low/high.
-//   - outMin, outMax: raw output code range (counts) corresponding to the pressure span. Depends on sensor model.
-//   - pMin, pMax: pressure range endpoints in millipascals (mPa). Depends on sensor model.
-//
-// The function returns the constructed DevSPI and an error value (currently always nil).
-func NewDevSPI(conn drivers.SPI, cs pinout, outMin, outMax uint16, pMin, pMax int32) (*DevSPI, error) {
-	h := &DevSPI{
-		spi: conn,
-		cs:  cs,
-		dev: dev{
-			cmin: outMin,
-			cmax: outMax,
-			pmin: pMin,
-			pmax: pMax,
-		},
-	}
-	return h, nil
-}
-
-// ReadTemperature reads and returns the temperature in milliKelvin (mC) from the SPI-attached HSC device.
-// It performs an Update internally to get the latest temperature value.
-func (h *DevSPI) ReadTemperature() (int32, error) {
-	err := h.Update(drivers.Temperature)
-	if err != nil {
-		return 0, err
-	}
-	return h.Temperature(), nil
-}
-
-// Update reads pressure and temperature data from the SPI-attached HSC device when the requested measurement mask includes
-// pressure or temperature. If neither pressure nor temperature is requested, Update is a no-op.
-func (h *DevSPI) Update(which drivers.Measurement) error {
-	// It toggles the provided chip-select, performs an SPI transfer to read 4 bytes, parses the status bits,
-	// 14-bit bridge data and temperature bits, and forwards them to the internal update routine. Any SPI
-	// transport error is returned, as well as errors produced by the internal update (e.g. errSensorMissing, errDiagnostic).
-	if which&measuremask == 0 {
-		return nil
-	}
-	buf := &h.buf
-	h.cs(false)
-	err := h.spi.Tx(nil, buf[:4])
-	h.cs(true)
-	if err != nil {
-		return err
-	}
-	// First two bits are status bits.
-	status := (buf[0] & statusMask) >> statusOffset
-	bridgeData := (uint16(buf[0]&^statusMask) << 8) | uint16(buf[1])
-
-	tempData := uint16(buf[2])<<8 | uint16(buf[3]&0xe0)>>5
-	return h.dev.update(status, bridgeData, tempData)
-}
-
-type dev struct {
-	pressure   int32
-	temp       int32
-	cmin, cmax uint16
-	pmin, pmax int32
-}
-
-// Pressure returns the most recently computed pressure value in millipascals (mPa).
-// The value is taken from the last successful Update.
-func (d *dev) Pressure() int32 {
-	return d.pressure
-}
-
-// Temperature returns the most recently read temperature value in milliKelvin (mC).
-// The value is taken from the last successful Update.
-func (d *dev) Temperature() int32 {
-	return d.temp + 273_150
-}
-
-// update interprets raw sensor fields (status, bridgeData, tempData) and updates the dev's stored
-// pressure and temperature. It returns errSensorMissing when the temperature raw value indicates no sensor
-// (tempData == math.MaxUint16), errDiagnostic when the status indicates a device diagnostic condition
-// (status == 3), or nil on success. Pressure is computed with integer arithmetic using the configured
-// cmin/cmax -> pmin/pmax linear mapping in order to avoid overflows.
-func (d *dev) update(status uint8, bridgeData, tempData uint16) error {
-	if tempData == math.MaxUint16 {
-		return errSensorMissing
-	} else if status == 3 {
-		return errDiagnostic
-	}
-
-	// Take care not to overflow here.
-	p := (int32(bridgeData)-int32(d.cmin))*(d.pmax-d.pmin)/int32(d.cmax-d.cmin) + d.pmin
-	d.temp = int32(tempData)
-	d.pressure = p
-	return nil
-}

internal/legacy/pinhal.go

@@ -6,14 +6,6 @@ import (
 	"tinygo.org/x/drivers/internal/pin"
 )
 
-// The pingconfig group of files serve to abstract away
-// pin configuration calls on the machine.Pin type.
-// It was observed this way of developing drivers was
-// non-portable and unusable on "big" Go projects so
-// future projects should NOT configure pins in driver code.
-// Users must configure pins before passing them as arguments
-// to drivers.
-
 // ConfigurePinOut is a legacy function used to configure pins as outputs.
 //
 // Deprecated: Do not configure pins in drivers.

internal/legacy/pinhal_go.go

@@ -1,13 +1,9 @@
-//go:build !tinygo
+//go:build !baremetal
 
 package legacy
 
 import "tinygo.org/x/drivers/internal/pin"
 
-// This file compiles for non-tinygo builds
-// for use with "big" or "upstream" Go where
-// there is no machine package.
-
 func configurePinOut(p pin.Output)          {}
 func configurePinInput(p pin.Input)         {}
 func configurePinInputPulldown(p pin.Input) {}

internal/pin/internalpin.go

@@ -1,72 +0,0 @@
-// package pin implements a TinyGo Pin HAL.
-// It serves to eliminate machine.Pin from driver constructors
-// so that drivers can be used in "big" Go projects where
-// there is no machine package.
-// This file contains both function and interface-style Pin HAL definitions.
-package pin
-
-// OutputFunc is hardware abstraction for a pin which outputs a
-// digital signal (high or low level).
-//
-//	// Code conversion demo: from machine.Pin to pin.OutputFunc
-//	led := machine.LED
-//	led.Configure(machine.PinConfig{Mode: machine.Output})
-//	var pin pin.OutputFunc = led.Set // Going from a machine.Pin to a pin.OutputFunc
-//
-// This is an alternative to [Output] which is an interface type.
-type OutputFunc func(level bool)
-
-// High sets the underlying pin's level to high. This is equivalent to calling PinOutput(true).
-func (setPin OutputFunc) High() {
-	setPin(true)
-}
-
-// Low sets the underlying pin's level to low. This is equivalent to calling PinOutput(false).
-func (setPin OutputFunc) Low() {
-	setPin(false)
-}
-
-// InputFunc is hardware abstraction for a pin which receives a
-// digital signal and reads it (high or low level).
-//
-//	// Code conversion demo: from machine.Pin to pin.InputFunc
-//	input := machine.LED
-//	input.Configure(machine.PinConfig{Mode: machine.PinInputPulldown}) // or use machine.PinInputPullup or machine.Input
-//	var pin pin.InputFunc = input.Get // Going from a machine.Pin to a pin.InputFunc
-//
-// This is an alternative to [Input] which is an interface type.
-type InputFunc func() (level bool)
-
-// // Below is an example on how to define a input/output pin HAL for a
-// // pin that must switch between input and output mode:
-//
-// 	var pinIsOutput bool
-// 	var po PinOutputFunc = func(b bool) {
-// 		if !pinIsOutput {
-// 			pin.Configure(outputMode)
-// 			pinIsOutput = true
-// 		}
-// 		pin.Set(b)
-// 	}
-//
-// 	var pi PinInputFunc = func() bool {
-// 		if pinIsOutput {
-// 			pin.Configure(inputMode)
-// 			pinIsOutput = false
-// 		}
-// 		return pin.Get()
-// 	}
-
-// Output interface represents a pin hardware abstraction layer for a pin that can output a digital signal.
-//
-// This is an alternative to [OutputFunc] abstraction which is a function type.
-type Output interface {
-	Set(level bool)
-}
-
-// Input interface represents a pin hardware abstraction layer for a pin that can read a digital signal.
-//
-// This is an alternative to [InputFunc] abstraction which is a function type.
-type Input interface {
-	Get() (level bool)
-}

internal/pin/pin.go

@@ -0,0 +1,23 @@
+package pin
+
+import "tinygo.org/x/drivers"
+
+// Here to aid relevant documentation links of [drivers.PinOutput] and [drivers.PinInput].
+var _ drivers.PinOutput
+
+// Output represents a pin hardware abstraction layer for a pin that can output a digital signal.
+//
+// This is an alternative to [drivers.PinOutput] abstraction which is a function type and has
+// not been standardized as of yet as a standard HAL in the drivers package,
+// [discussion ongoing here].
+//
+// [discussion ongoing here]: https://github.com/orgs/tinygo-org/discussions/5043
+type Output interface {
+	Set(level bool)
+}
+
+// Input represents a pin hardware abstraction layer.
+// See [Output] for more information on why this type exists separate to drivers.
+type Input interface {
+	Get() (level bool)
+}

internal/regmap/device8.go

@@ -1,143 +0,0 @@
-package regmap
-
-import (
-	"encoding/binary"
-	"io"
-
-	"tinygo.org/x/drivers"
-)
-
-// Device8 implements common logic to most 8-bit peripherals with an I2C or SPI bus.
-// All methods expect the target to support conventional register read and write operations
-// where the first byte sent is the register address being accessed.
-//
-// All methods use an internal buffer and perform no dynamic memory allocation.
-type Device8 struct {
-	buf [10]byte
-}
-
-// clear zeroes Device8's buffers.
-func (d *Device8) clear() {
-	d.buf = [10]byte{}
-}
-
-// I2C methods.
-
-// Read8I2C reads a single byte from register addr of the device at i2cAddr using the provided I2C bus.
-func (d *Device8) Read8I2C(bus drivers.I2C, i2cAddr uint16, addr uint8) (byte, error) {
-	d.buf[0] = addr
-	err := bus.Tx(i2cAddr, d.buf[0:1], d.buf[1:2])
-	return d.buf[1], err
-}
-
-// Read16I2C reads a 16-bit value from register addr of the device at i2cAddr using the provided I2C bus.
-// The byte order is specified by order.
-func (d *Device8) Read16I2C(bus drivers.I2C, i2cAddr uint16, addr uint8, order binary.ByteOrder) (uint16, error) {
-	d.buf[0] = addr
-	err := bus.Tx(i2cAddr, d.buf[0:1], d.buf[1:3])
-	return order.Uint16(d.buf[1:3]), err
-}
-
-// Read32I2C reads a 32-bit value from register addr of the device at i2cAddr using the provided I2C bus.
-// The byte order is specified by order.
-func (d *Device8) Read32I2C(bus drivers.I2C, i2cAddr uint16, addr uint8, order binary.ByteOrder) (uint32, error) {
-	d.buf[0] = addr
-	err := bus.Tx(i2cAddr, d.buf[0:1], d.buf[1:5])
-	return order.Uint32(d.buf[1:5]), err
-}
-
-// ReadDataI2C reads dataLength bytes from register addr of the device at i2cAddr using the provided I2C bus.
-// The data is stored in dataDestination.
-func (d *Device8) ReadDataI2C(bus drivers.I2C, i2cAddr uint16, addr uint8, dataDestination []byte) error {
-	d.buf[0] = addr
-	return bus.Tx(i2cAddr, d.buf[:1], dataDestination)
-}
-
-// Write8I2C writes a single byte value to register addr of the device at i2cAddr using the provided I2C bus.
-func (d *Device8) Write8I2C(bus drivers.I2C, i2cAddr uint16, addr, value uint8) error {
-	d.buf[0] = addr
-	d.buf[1] = value
-	return bus.Tx(i2cAddr, d.buf[:2], nil)
-}
-
-// Write16I2C writes a 16-bit value to register addr of the device at i2cAddr using the provided I2C bus.
-// The byte order is specified by order.
-func (d *Device8) Write16I2C(bus drivers.I2C, i2cAddr uint16, addr uint8, value uint16, order binary.ByteOrder) error {
-	d.buf[0] = addr
-	order.PutUint16(d.buf[1:3], value)
-	return bus.Tx(i2cAddr, d.buf[0:3], nil)
-}
-
-// Write32I2C writes a 32-bit value to register addr of the device at i2cAddr using the provided I2C bus.
-// The byte order is specified by order.
-func (d *Device8) Write32I2C(bus drivers.I2C, i2cAddr uint16, addr uint8, value uint32, order binary.ByteOrder) error {
-	d.buf[0] = addr
-	order.PutUint32(d.buf[1:5], value)
-	return bus.Tx(i2cAddr, d.buf[0:5], nil)
-}
-
-// SPI methods.
-
-// Read8SPI reads a single byte from register addr using the provided SPI bus.
-func (d *Device8) Read8SPI(bus drivers.SPI, addr uint8) (byte, error) {
-	d.clear()
-	d.buf[0] = addr
-	err := bus.Tx(d.buf[0:1], d.buf[1:2]) // We suppose data is returned after first byte in SPI.
-	return d.buf[1], err
-}
-
-// Read16SPI reads a 16-bit value from register addr using the provided SPI bus. The byte order is specified by order.
-func (d *Device8) Read16SPI(bus drivers.SPI, addr uint8, order binary.ByteOrder) (uint16, error) {
-	d.clear()
-	d.buf[0] = addr
-	err := bus.Tx(d.buf[0:3], d.buf[3:6]) // We suppose data is returned after first byte in SPI.
-	return order.Uint16(d.buf[4:6]), err
-}
-
-// Read32SPI reads a 32-bit value from register addr using the provided SPI bus. The byte order is specified by order.
-func (d *Device8) Read32SPI(bus drivers.SPI, addr uint8, order binary.ByteOrder) (uint32, error) {
-	d.clear()
-	d.buf[0] = addr
-	err := bus.Tx(d.buf[0:5], d.buf[5:10]) // We suppose data is returned after first byte in SPI.
-	return order.Uint32(d.buf[6:10]), err
-}
-
-// ReadDataSPI reads data from a 8bit device address. It assumes data at register address is sent back
-// from device after first byte is written as address.
-// It needs the auxiliary buffer length to be large enough to contain both the write and read portions of buffer,
-// so 2*(dataLength+1) < len(auxiliaryBuf) must hold.
-func (d *Device8) ReadDataSPI(bus drivers.SPI, addr uint8, dataLength int, auxiliaryBuf []byte) ([]byte, error) {
-	split := len(auxiliaryBuf) / 2
-	if split < dataLength+1 {
-		return nil, io.ErrShortBuffer
-	}
-
-	wbuf, rbuf := auxiliaryBuf[:split], auxiliaryBuf[split:]
-	wbuf[0] = addr
-	err := bus.Tx(wbuf, rbuf)
-	return rbuf[1:], err
-}
-
-// Write8SPI writes a single byte value to register addr using the provided SPI bus.
-func (d *Device8) Write8SPI(bus drivers.SPI, addr, value uint8) error {
-	d.clear()
-	d.buf[0] = addr
-	d.buf[1] = value
-	return bus.Tx(d.buf[:2], nil)
-}
-
-// Write16SPI writes a 16-bit value to register addr using the provided SPI bus. The byte order is specified by order.
-func (d *Device8) Write16SPI(bus drivers.SPI, addr uint8, value uint16, order binary.ByteOrder) error {
-	d.clear()
-	d.buf[0] = addr
-	order.PutUint16(d.buf[1:3], value)
-	return bus.Tx(d.buf[:3], nil)
-}
-
-// Write32SPI writes a 32-bit value to register addr using the provided SPI bus. The byte order is specified by order.
-func (d *Device8) Write32SPI(bus drivers.SPI, addr uint8, value uint32, order binary.ByteOrder) error {
-	d.clear()
-	d.buf[0] = addr
-	order.PutUint32(d.buf[1:5], value)
-	return bus.Tx(d.buf[:5], nil)
-}

internal/regmap/device8tx.go

@@ -1,196 +0,0 @@
-// Package regmap provides transaction-based interfaces for reading and writing
-// to device registers over I2C and SPI buses with pre-allocated buffers.
-package regmap
-
-import (
-	"errors"
-
-	"tinygo.org/x/drivers"
-)
-
-var (
-	// errNotInTx indicates an operation was attempted outside of an active transaction.
-	errNotInTx = errors.New("device not in Tx")
-
-	// errInTx indicates a transaction was started while another is still active.
-	errInTx = errors.New("device already in Tx")
-
-	// errShortWriteBuffer indicates the write buffer is too small for the requested operation.
-	errShortWriteBuffer = errors.New("device write buffer too short")
-
-	// errShortReadBuffer indicates the read buffer is too small for the requested operation.
-	errShortReadBuffer = errors.New("device read buffer too short")
-)
-
-// Device8Txer wraps a Device8 to provide buffered transaction support for
-// I2C and SPI operations. It maintains pre-allocated buffers to avoid heap
-// allocations during register access operations.
-//
-// Users must call SetBuffers to configure the write and read buffers before
-// initiating transactions.
-type Device8Txer struct {
-	Device8
-	writeBuf []byte // Pre-allocated buffer for write operations
-	readBuf  []byte // Pre-allocated buffer for read operations
-	inTx     bool   // Tracks whether a transaction is currently active
-}
-
-// SetTxBuffers configures the write and read buffers for this device.
-// These buffers are reused across transactions to avoid heap allocations.
-//
-// The writebuf should be large enough to hold the register address plus
-// all data bytes to be written in a single transaction.
-func (d *Device8Txer) SetTxBuffers(writebuf, readbuf []byte) {
-	d.readBuf = readbuf
-	d.writeBuf = writebuf
-}
-
-// Tx8 represents an active transaction for an 8-bit register device.
-// It tracks the write buffer and current offset as data is added to the transaction.
-//
-// Use AddWriteByte or AddWriteData to add data to the transaction, then call
-// DoTxI2C or DoTxSPI to execute the transaction over the bus.
-type Tx8 struct {
-	dw  *Device8Txer // Reference to the parent device
-	off int          // Current offset in the write buffer
-}
-
-// Tx initiates a new transaction for writing to the specified register address.
-//
-// Parameters:
-//   - writeAddr: The 8-bit register address to write to
-//
-// Returns a Tx8 handle that can be used to add data and execute the transaction.
-//
-// Returns an error if:
-//   - A transaction is already active (errInTx)
-//   - The write buffer is too short (errShortWriteBuffer)
-func (dw *Device8Txer) Tx(writeAddr uint8) (Tx8, error) {
-	if dw.inTx {
-		return Tx8{}, errInTx
-	} else if len(dw.writeBuf) < 1 {
-		return Tx8{}, errShortWriteBuffer
-	}
-	dw.writeBuf[0] = writeAddr
-	return Tx8{dw: dw, off: 1}, nil
-}
-
-// AddWriteData appends multiple bytes to the current transaction's write buffer.
-//
-// Parameters:
-//   - buf: Variable number of bytes to add to the transaction
-//
-// Returns an error if:
-//   - No transaction is active (errNotInTx)
-//   - The write buffer doesn't have enough space (errShortWriteBuffer)
-func (tx *Tx8) AddWriteData(buf ...byte) error {
-	if !tx.dw.inTx {
-		return errNotInTx
-	}
-	avail := tx.dw.writeBuf[tx.off:]
-	if len(avail) < len(buf) {
-		return errShortWriteBuffer
-	}
-	n := copy(avail, buf)
-	tx.off += n
-	return nil
-}
-
-// AddWriteByte appends a single byte to the current transaction's write buffer.
-//
-// Parameters:
-//   - b: The byte to add to the transaction
-//
-// Returns an error if:
-//   - No transaction is active (errNotInTx)
-//   - The write buffer doesn't have enough space (errShortWriteBuffer)
-func (tx *Tx8) AddWriteByte(b byte) error {
-	if !tx.dw.inTx {
-		return errNotInTx
-	}
-	avail := tx.dw.writeBuf[tx.off:]
-	if len(avail) < 1 {
-		return errShortWriteBuffer
-	}
-	avail[0] = b
-	tx.off++
-	return nil
-}
-
-// DoTxI2C executes the transaction over an I2C bus.
-//
-// This performs a combined write-read I2C transaction, first sending the
-// register address and any data added to the transaction, then reading
-// the specified number of bytes from the device.
-//
-// Parameters:
-//   - bus: The I2C bus to communicate over
-//   - deviceAddr: The I2C address of the target device
-//   - readLength: Number of bytes to read from the device
-//
-// Returns the read data as a slice of the internal read buffer, valid until
-// the next transaction. The transaction is automatically freed after execution.
-//
-// Returns an error if:
-//   - No transaction is active (errNotInTx)
-//   - The read buffer is too short (errShortReadBuffer)
-//   - The I2C transaction fails
-func (tx *Tx8) DoTxI2C(bus drivers.I2C, deviceAddr uint16, readLength int) ([]byte, error) {
-	if tx.off == 0 || !tx.dw.inTx {
-		return nil, errNotInTx
-	}
-	defer tx.freeTx()
-	if len(tx.dw.readBuf) < readLength {
-		return nil, errShortReadBuffer
-	}
-	rbuf := tx.dw.readBuf[:readLength]
-	err := bus.Tx(deviceAddr, tx.dw.writeBuf[:tx.off], rbuf)
-	if err != nil {
-		return nil, err
-	}
-	return rbuf, err
-}
-
-// DoTxSPI executes the transaction over an SPI bus.
-//
-// This performs a full-duplex SPI transaction, simultaneously writing the
-// register address and data while reading the same number of bytes from the device.
-//
-// If no read buffer was configured (readBuf is nil), this performs a write-only
-// transaction and returns nil without error.
-//
-// Parameters:
-//   - bus: The SPI bus to communicate over
-//
-// Returns the read data as a slice of the internal read buffer (same length as
-// the write data), valid until the next transaction. The transaction is
-// automatically freed after execution.
-//
-// Returns an error if:
-//   - No transaction is active (errNotInTx)
-//   - The read buffer is too short (errShortReadBuffer)
-//   - The SPI transaction fails
-func (tx *Tx8) DoTxSPI(bus drivers.SPI) (readBuf []byte, err error) {
-	if tx.off == 0 || !tx.dw.inTx {
-		return nil, errNotInTx
-	}
-	defer tx.freeTx()
-	if tx.dw.readBuf == nil {
-		err = bus.Tx(tx.dw.writeBuf[:tx.off], nil) // Special case, only use write buffer functionality.
-		return nil, err
-	} else if len(readBuf) < tx.off {
-		return nil, errShortReadBuffer
-	}
-	rbuf := tx.dw.readBuf[:tx.off]
-	err = bus.Tx(tx.dw.writeBuf[:tx.off], rbuf)
-	if err != nil {
-		return nil, err
-	}
-	return rbuf, err
-}
-
-// freeTx marks the transaction as complete, allowing a new transaction to be started.
-// This is called internally by DoTxI2C and DoTxSPI after the transaction completes.
-func (tx *Tx8) freeTx() {
-	tx.dw.inTx = false
-}

internal/regmap/i2c-spi.go

@@ -1,123 +0,0 @@
-package regmap
-
-import (
-	"encoding/binary"
-
-	"tinygo.org/x/drivers"
-)
-
-// Device8SPI implements common logic to most 8-bit peripherals with an SPI bus.
-// All methods expect the target to support conventional register read and write operations
-// where the first byte sent is the register address being accessed.
-//
-// All methods use an internal buffer and perform no dynamic memory allocation.
-type Device8SPI struct {
-	bus   drivers.SPI
-	order binary.ByteOrder
-	d     Device8
-}
-
-// SetBus sets the SPI bus and byte order for the Device8SPI.
-//
-// As a hint, most SPI devices use big-endian (MSB) byte order.
-//   - Big endian: A value of 0x1234 is transmitted as 0x12 followed by 0x34.
-//   - Little endian: A value of 0x1234 is transmitted as 0x34 followed by 0x12.
-func (d *Device8SPI) SetBus(bus drivers.SPI, order binary.ByteOrder) {
-	d.bus = bus
-	d.order = order
-}
-
-// Read8 reads a single byte from register addr.
-func (d *Device8SPI) Read8(addr uint8) (byte, error) {
-	return d.d.Read8SPI(d.bus, addr)
-}
-
-// Read16 reads a 16-bit value from register addr.
-func (d *Device8SPI) Read16(addr uint8) (uint16, error) {
-	return d.d.Read16SPI(d.bus, addr, d.order)
-}
-
-// Read32 reads a 32-bit value from register addr.
-func (d *Device8SPI) Read32(addr uint8) (uint32, error) {
-	return d.d.Read32SPI(d.bus, addr, d.order)
-}
-
-// ReadData reads dataLength bytes from register addr. Due to the internal functioning of
-// SPI, an auxiliary buffer must be provided to perform the operation and avoid memory allocation.
-// The returned slice is a subslice of auxBuffer containing the read data.
-func (d *Device8SPI) ReadData(addr uint8, datalength int, auxBuffer []byte) ([]byte, error) {
-	return d.d.ReadDataSPI(d.bus, addr, datalength, auxBuffer)
-}
-
-// Write8 writes a single byte value to register addr.
-func (d *Device8SPI) Write8(addr, value uint8) error {
-	return d.d.Write8SPI(d.bus, addr, value)
-}
-
-// Write16 writes a 16-bit value to register addr.
-func (d *Device8SPI) Write16(addr uint8, value uint16) error {
-	return d.d.Write16SPI(d.bus, addr, value, d.order)
-}
-
-// Write32 writes a 32-bit value to register addr.
-func (d *Device8SPI) Write32(addr uint8, value uint32) error {
-	return d.d.Write32SPI(d.bus, addr, value, d.order)
-}
-
-// Device8I2C implements common logic to most 8-bit peripherals with an I2C bus.
-// All methods expect the target to support conventional register read and write operations
-// where the first byte sent is the register address being accessed.
-//
-// All methods use an internal buffer and perform no dynamic memory allocation.
-type Device8I2C struct {
-	bus     drivers.I2C
-	i2cAddr uint16
-	order   binary.ByteOrder
-	d       Device8
-}
-
-// SetBus sets the I2C bus, device address, and byte order for the Device8I2C.
-//
-// As a hint, most I2C devices use big-endian (MSB) byte order.
-//   - Big endian: A value of 0x1234 is transmitted as 0x12 followed by 0x34.
-//   - Little endian: A value of 0x1234 is transmitted as 0x34 followed by 0x12.
-func (d *Device8I2C) SetBus(bus drivers.I2C, i2cAddr uint16, order binary.ByteOrder) {
-	d.bus = bus
-	d.i2cAddr = i2cAddr
-	d.order = order
-}
-
-// Read8 reads a single byte from register addr.
-func (d *Device8I2C) Read8(addr uint8) (byte, error) {
-	return d.d.Read8I2C(d.bus, d.i2cAddr, addr)
-}
-
-// Read16 reads a 16-bit value from register addr.
-func (d *Device8I2C) Read16(addr uint8) (uint16, error) {
-	return d.d.Read16I2C(d.bus, d.i2cAddr, addr, d.order)
-}
-
-// Read32 reads a 32-bit value from register addr.
-func (d *Device8I2C) Read32(addr uint8) (uint32, error) {
-	return d.d.Read32I2C(d.bus, d.i2cAddr, addr, d.order)
-}
-
-// ReadData reads dataLength bytes from register addr.
-func (d *Device8I2C) ReadData(addr uint8, dataDestination []byte) error {
-	return d.d.ReadDataI2C(d.bus, d.i2cAddr, addr, dataDestination)
-}
-
-// Write8 writes a single byte value to register addr.
-func (d *Device8I2C) Write8(addr, value uint8) error {
-	return d.d.Write8I2C(d.bus, d.i2cAddr, addr, value)
-}
-
-// Write16 writes a 16-bit value to register addr.
-func (d *Device8I2C) Write16(addr uint8, value uint16) error {
-	return d.d.Write16I2C(d.bus, d.i2cAddr, addr, value, d.order)
-}
-
-// Write32 writes a 32-bit value to register addr.
-func (d *Device8I2C) Write32(addr uint8, value uint32) error {
-	return d.d.Write32I2C(d.bus, d.i2cAddr, addr, value, d.order)
-}

internal/regmap/regmap_test.go

@@ -1,34 +0,0 @@
-package regmap
-
-import (
-	"fmt"
-
-	"tinygo.org/x/drivers"
-)
-
-func ExampleDevice8Txer() {
-	// Initialization.
-	var dtx Device8Txer
-	dtx.SetTxBuffers(make([]byte, 256), make([]byte, 256))
-
-	// Usage.
-	const (
-		defaultAddr = 65
-		REG_WRITE   = 0x1f
-		IOCTL_CALL  = 0xc0
-	)
-	tx, err := dtx.Tx(REG_WRITE)
-	if err != nil {
-		panic(err)
-	}
-	err = tx.AddWriteData(IOCTL_CALL, 0x80, 0x80)
-	if err != nil {
-		panic(err)
-	}
-	var bus drivers.I2C
-	readData, err := tx.DoTxI2C(bus, defaultAddr, 20)
-	if err != nil {
-		panic(err)
-	}
-	fmt.Println(readData)
-}

lis3dh/lis3dh.go

@@ -11,57 +11,37 @@ import (
 // Device wraps an I2C connection to a LIS3DH device.
 type Device struct {
 	bus     drivers.I2C
-	address uint16
-	r       Range
-	accel   [6]byte // stored acceleration data (from the Update call)
-}
-
-// Driver configuration, used for the Configure call. All fields are optional.
-type Config struct {
 	Address uint16
+	r       Range
 }
 
 // New creates a new LIS3DH connection. The I2C bus must already be configured.
 //
 // This function only creates the Device object, it does not touch the device.
 func New(bus drivers.I2C) Device {
-	return Device{bus: bus, address: Address0}
+	return Device{bus: bus, Address: Address0}
 }
 
 // Configure sets up the device for communication
-func (d *Device) Configure(config Config) error {
-	if config.Address != 0 {
-		d.address = config.Address
-	}
-
+func (d *Device) Configure() {
 	// enable all axes, normal mode
-	err := legacy.WriteRegister(d.bus, uint8(d.address), REG_CTRL1, []byte{0x07})
-	if err != nil {
-		return err
-	}
+	legacy.WriteRegister(d.bus, uint8(d.Address), REG_CTRL1, []byte{0x07})
 
 	// 400Hz rate
-	err = d.SetDataRate(DATARATE_400_HZ)
-	if err != nil {
-		return err
-	}
+	d.SetDataRate(DATARATE_400_HZ)
 
 	// High res & BDU enabled
-	err = legacy.WriteRegister(d.bus, uint8(d.address), REG_CTRL4, []byte{0x88})
-	if err != nil {
-		return err
-	}
+	legacy.WriteRegister(d.bus, uint8(d.Address), REG_CTRL4, []byte{0x88})
 
 	// get current range
-	d.r, err = d.ReadRange()
-	return err
+	d.r = d.ReadRange()
 }
 
 // Connected returns whether a LIS3DH has been found.
 // It does a "who am I" request and checks the response.
 func (d *Device) Connected() bool {
 	data := []byte{0}
-	err := legacy.ReadRegister(d.bus, uint8(d.address), WHO_AM_I, data)
+	err := legacy.ReadRegister(d.bus, uint8(d.Address), WHO_AM_I, data)
 	if err != nil {
 		return false
 	}
@@ -69,51 +49,46 @@ func (d *Device) Connected() bool {
 }
 
 // SetDataRate sets the speed of data collected by the LIS3DH.
-func (d *Device) SetDataRate(rate DataRate) error {
+func (d *Device) SetDataRate(rate DataRate) {
 	ctl1 := []byte{0}
-	err := legacy.ReadRegister(d.bus, uint8(d.address), REG_CTRL1, ctl1)
+	err := legacy.ReadRegister(d.bus, uint8(d.Address), REG_CTRL1, ctl1)
 	if err != nil {
-		return err
+		println(err.Error())
 	}
 	// mask off bits
 	ctl1[0] &^= 0xf0
 	ctl1[0] |= (byte(rate) << 4)
-	return legacy.WriteRegister(d.bus, uint8(d.address), REG_CTRL1, ctl1)
+	legacy.WriteRegister(d.bus, uint8(d.Address), REG_CTRL1, ctl1)
 }
 
 // SetRange sets the G range for LIS3DH.
-func (d *Device) SetRange(r Range) error {
+func (d *Device) SetRange(r Range) {
 	ctl := []byte{0}
-	err := legacy.ReadRegister(d.bus, uint8(d.address), REG_CTRL4, ctl)
+	err := legacy.ReadRegister(d.bus, uint8(d.Address), REG_CTRL4, ctl)
 	if err != nil {
-		return err
+		println(err.Error())
 	}
 	// mask off bits
 	ctl[0] &^= 0x30
 	ctl[0] |= (byte(r) << 4)
-	err = legacy.WriteRegister(d.bus, uint8(d.address), REG_CTRL4, ctl)
-	if err != nil {
-		return err
-	}
+	legacy.WriteRegister(d.bus, uint8(d.Address), REG_CTRL4, ctl)
 
 	// store the new range
 	d.r = r
-
-	return nil
 }
 
 // ReadRange returns the current G range for LIS3DH.
-func (d *Device) ReadRange() (r Range, err error) {
+func (d *Device) ReadRange() (r Range) {
 	ctl := []byte{0}
-	err = legacy.ReadRegister(d.bus, uint8(d.address), REG_CTRL4, ctl)
+	err := legacy.ReadRegister(d.bus, uint8(d.Address), REG_CTRL4, ctl)
 	if err != nil {
-		return 0, err
+		println(err.Error())
 	}
 	// mask off bits
 	r = Range(ctl[0] >> 4)
 	r &= 0x03
 
-	return r, nil
+	return r
 }
 
 // ReadAcceleration reads the current acceleration from the device and returns
@@ -121,17 +96,28 @@ func (d *Device) ReadRange() (r Range, err error) {
 // and the sensor is not moving the returned value will be around 1000000 or
 // -1000000.
 func (d *Device) ReadAcceleration() (int32, int32, int32, error) {
-	rawX, rawY, rawZ := d.ReadRawAcceleration()
-	x, y, z := normalizeRange(rawX, rawY, rawZ, d.r)
-	return x, y, z, nil
+	x, y, z := d.ReadRawAcceleration()
+	divider := float32(1)
+	switch d.r {
+	case RANGE_16_G:
+		divider = 1365
+	case RANGE_8_G:
+		divider = 4096
+	case RANGE_4_G:
+		divider = 8190
+	case RANGE_2_G:
+		divider = 16380
+	}
+
+	return int32(float32(x) / divider * 1000000), int32(float32(y) / divider * 1000000), int32(float32(z) / divider * 1000000), nil
 }
 
 // ReadRawAcceleration returns the raw x, y and z axis from the LIS3DH
 func (d *Device) ReadRawAcceleration() (x int16, y int16, z int16) {
-	legacy.WriteRegister(d.bus, uint8(d.address), REG_OUT_X_L|0x80, nil)
+	legacy.WriteRegister(d.bus, uint8(d.Address), REG_OUT_X_L|0x80, nil)
 
 	data := []byte{0, 0, 0, 0, 0, 0}
-	d.bus.Tx(d.address, nil, data)
+	d.bus.Tx(d.Address, nil, data)
 
 	x = int16((uint16(data[1]) << 8) | uint16(data[0]))
 	y = int16((uint16(data[3]) << 8) | uint16(data[2]))
@@ -139,68 +125,3 @@ func (d *Device) ReadRawAcceleration() (x int16, y int16, z int16) {
 
 	return
 }
-
-// Update the sensor values of the 'which' parameter. Only acceleration is
-// supported at the moment.
-func (d *Device) Update(which drivers.Measurement) error {
-	if which&drivers.Acceleration != 0 {
-		// Read raw acceleration values and store them in the driver.
-		err := legacy.WriteRegister(d.bus, uint8(d.address), REG_OUT_X_L|0x80, nil)
-		if err != nil {
-			return err
-		}
-		err = d.bus.Tx(d.address, nil, d.accel[:])
-		if err != nil {
-			return err
-		}
-	}
-	return nil
-}
-
-// Acceleration returns the last read acceleration in µg (micro-gravity).
-// When one of the axes is pointing straight to Earth and the sensor is not
-// moving the returned value will be around 1000000 or -1000000.
-func (d *Device) Acceleration() (x, y, z int32) {
-	// Extract the raw 16-bit values.
-	rawX := int16((uint16(d.accel[1]) << 8) | uint16(d.accel[0]))
-	rawY := int16((uint16(d.accel[3]) << 8) | uint16(d.accel[2]))
-	rawZ := int16((uint16(d.accel[5]) << 8) | uint16(d.accel[4]))
-
-	// Normalize these values, to be in µg (micro-gravity).
-	return normalizeRange(rawX, rawY, rawZ, d.r)
-}
-
-// Convert raw 16-bit values to normalized 32-bit values while avoiding floats
-// and divisions.
-func normalizeRange(rawX, rawY, rawZ int16, r Range) (x, y, z int32) {
-	// We're going to convert the 16-bit raw values to values in the range
-	// -1000_000..1000_000. For now we're going to assume a range of 16G, we'll
-	// adjust that range later.
-	// The formula is derived as follows, and carefully selected to avoid
-	// overflow and integer divisions (the division will be optimized to a
-	// bitshift):
-	//   x = x * 1000_000      / 2048
-	//   x = x * (1000_000/64) / (2048/64)
-	//   x = x * 15625         / 32
-	x = int32(rawX) * 15625 / 32
-	y = int32(rawY) * 15625 / 32
-	z = int32(rawZ) * 15625 / 32
-
-	// Now we need to normalize the three values, since we assumed 16G before.
-	shift := uint32(0)
-	switch r {
-	case RANGE_16_G:
-		shift = 0
-	case RANGE_8_G:
-		shift = 1
-	case RANGE_4_G:
-		shift = 2
-	case RANGE_2_G:
-		shift = 3
-	}
-	x >>= shift
-	y >>= shift
-	z >>= shift
-
-	return
-}

lsm303agr/lsm303agr.go

@@ -36,7 +36,7 @@ type Configuration struct {
 	MagDataRate    uint8
 }
 
-var errNotConnected = errors.New("lsm303agr: failed to communicate with either accel or magnet sensor")
+var errNotConnected = errors.New("lsm303agr: failed to communicate with either acel or magnet sensor")
 
 // New creates a new LSM303AGR connection. The I2C bus must already be configured.
 //

lsm303dlhc/lsm303dlhc.go

@@ -1,214 +0,0 @@
-// Package lsm303dlhc implements a driver for the LSM303dlhc,
-// a 3 axis accelerometer/magnetic sensor typically available on breakout boards.
-//
-// Datasheet: https://www.st.com/resource/en/datasheet/lsm303dlhc.pdf
-
-package lsm303dlhc // import "tinygo.org/x/drivers/lsm303dlhc"
-
-import (
-	"math"
-
-	"tinygo.org/x/drivers"
-	"tinygo.org/x/drivers/internal/legacy"
-)
-
-// Device wraps an I2C connection to a LSM303dlhc device.
-type Device struct {
-	bus            drivers.I2C
-	AccelAddress   uint8
-	MagAddress     uint8
-	AccelPowerMode uint8
-	AccelRange     uint8
-	AccelDataRate  uint8
-	MagPowerMode   uint8
-	MagSystemMode  uint8
-	MagDataRate    uint8
-	buf            [6]uint8
-}
-
-// Configuration for LSM303dlhc device.
-type Configuration struct {
-	AccelPowerMode uint8
-	AccelRange     uint8
-	AccelDataRate  uint8
-	MagPowerMode   uint8
-	MagSystemMode  uint8
-	MagDataRate    uint8
-}
-
-// New creates a new LSM303DLHC connection. The I2C bus must already be configured.
-// This function only creates the Device object, it does not touch the device.
-func New(bus drivers.I2C) *Device {
-	return &Device{
-		bus:          bus,
-		AccelAddress: ACCEL_ADDRESS,
-		MagAddress:   MAG_ADDRESS,
-	}
-}
-
-// Configure sets up the LSM303dlhc device for communication.
-func (d *Device) Configure(cfg Configuration) (err error) {
-
-	if cfg.AccelDataRate != 0 {
-		d.AccelDataRate = cfg.AccelDataRate
-	} else {
-		d.AccelDataRate = ACCEL_DATARATE_100HZ
-	}
-
-	if cfg.AccelPowerMode != 0 {
-		d.AccelPowerMode = cfg.AccelPowerMode
-	} else {
-		d.AccelPowerMode = ACCEL_POWER_NORMAL
-	}
-
-	if cfg.AccelRange != 0 {
-		d.AccelRange = cfg.AccelRange
-	} else {
-		d.AccelRange = ACCEL_RANGE_2G
-	}
-
-	if cfg.MagPowerMode != 0 {
-		d.MagPowerMode = cfg.MagPowerMode
-	} else {
-		d.MagPowerMode = MAG_POWER_NORMAL
-	}
-
-	if cfg.MagDataRate != 0 {
-		d.MagDataRate = cfg.MagDataRate
-	} else {
-		d.MagDataRate = MAG_DATARATE_10HZ
-	}
-
-	if cfg.MagSystemMode != 0 {
-		d.MagSystemMode = cfg.MagSystemMode
-	} else {
-		d.MagSystemMode = MAG_SYSTEM_CONTINUOUS
-	}
-
-	data := d.buf[:1]
-
-	data[0] = byte(d.AccelDataRate<<4 | d.AccelPowerMode | 0x07)
-	err = legacy.WriteRegister(d.bus, uint8(d.AccelAddress), ACCEL_CTRL_REG1_A, data)
-	if err != nil {
-		return
-	}
-
-	data[0] = byte(0x80 | d.AccelRange<<4)
-	err = legacy.WriteRegister(d.bus, uint8(d.AccelAddress), ACCEL_CTRL_REG4_A, data)
-	if err != nil {
-		return
-	}
-
-	data[0] = byte(0xC0)
-	err = legacy.WriteRegister(d.bus, uint8(d.AccelAddress), CRA_REG_M, data)
-	if err != nil {
-		return
-	}
-
-	// Temperature compensation is on for magnetic sensor
-	data[0] = byte(0x80 | d.MagPowerMode<<4 | d.MagDataRate<<2 | d.MagSystemMode)
-	err = legacy.WriteRegister(d.bus, uint8(d.MagAddress), MAG_MR_REG_M, data)
-	if err != nil {
-		return
-	}
-
-	return nil
-}
-
-// ReadAcceleration reads the current acceleration from the device and returns
-// it in µg (micro-gravity). When one of the axes is pointing straight to Earth
-// and the sensor is not moving the returned value will be around 1000000 or
-// -1000000.
-func (d *Device) ReadAcceleration() (x, y, z int32, err error) {
-	data := d.buf[:6]
-	err = legacy.ReadRegister(d.bus, uint8(d.AccelAddress), ACCEL_OUT_AUTO_INC, data)
-	if err != nil {
-		return
-	}
-
-	rangeFactor := int16(0)
-	switch d.AccelRange {
-	case ACCEL_RANGE_2G:
-		rangeFactor = 1
-	case ACCEL_RANGE_4G:
-		rangeFactor = 2
-	case ACCEL_RANGE_8G:
-		rangeFactor = 4
-	case ACCEL_RANGE_16G:
-		rangeFactor = 12 // the readings in 16G are a bit lower
-	}
-
-	x = int32(int32(int16((uint16(data[1])<<8|uint16(data[0])))>>4*rangeFactor) * 1000000 / 1024)
-	y = int32(int32(int16((uint16(data[3])<<8|uint16(data[2])))>>4*rangeFactor) * 1000000 / 1024)
-	z = int32(int32(int16((uint16(data[5])<<8|uint16(data[4])))>>4*rangeFactor) * 1000000 / 1024)
-	return
-}
-
-// ReadPitchRoll reads the current pitch and roll angles from the device and
-// returns it in micro-degrees. When the z axis is pointing straight to Earth
-// the returned values of pitch and roll would be zero.
-func (d *Device) ReadPitchRoll() (pitch, roll int32, err error) {
-
-	x, y, z, err := d.ReadAcceleration()
-	if err != nil {
-		return
-	}
-	xf, yf, zf := float64(x), float64(y), float64(z)
-	pitch = int32((math.Round(math.Atan2(yf, math.Sqrt(math.Pow(xf, 2)+math.Pow(zf, 2)))*(180/math.Pi)*100) / 100) * 1000000)
-	roll = int32((math.Round(math.Atan2(xf, math.Sqrt(math.Pow(yf, 2)+math.Pow(zf, 2)))*(180/math.Pi)*100) / 100) * 1000000)
-	return
-
-}
-
-// ReadMagneticField reads the current magnetic field from the device and returns
-// it in mG (milligauss). 1 mG = 0.1 µT (microtesla).
-func (d *Device) ReadMagneticField() (x, y, z int32, err error) {
-
-	if d.MagSystemMode == MAG_SYSTEM_SINGLE {
-		cmd := d.buf[:1]
-		cmd[0] = byte(0x80 | d.MagPowerMode<<4 | d.MagDataRate<<2 | d.MagSystemMode)
-		err = legacy.WriteRegister(d.bus, uint8(d.MagAddress), MAG_MR_REG_M, cmd)
-		if err != nil {
-			return
-		}
-	}
-
-	data := d.buf[0:6]
-	legacy.ReadRegister(d.bus, uint8(d.MagAddress), MAG_OUT_AUTO_INC, data)
-
-	x = int32(int16((uint16(data[1])<<8 | uint16(data[0]))))
-	y = int32(int16((uint16(data[3])<<8 | uint16(data[2]))))
-	z = int32(int16((uint16(data[5])<<8 | uint16(data[4]))))
-	return
-}
-
-// ReadCompass reads the current compass heading from the device and returns
-// it in micro-degrees. When the z axis is pointing straight to Earth and
-// the y axis is pointing to North, the heading would be zero.
-//
-// However, the heading may be off due to electronic compasses would be effected
-// by strong magnetic fields and require constant calibration.
-func (d *Device) ReadCompass() (h int32, err error) {
-
-	x, y, _, err := d.ReadMagneticField()
-	if err != nil {
-		return
-	}
-	xf, yf := float64(x), float64(y)
-	h = int32(float32((180/math.Pi)*math.Atan2(yf, xf)) * 1000000)
-	return
-}
-
-// ReadTemperature returns the temperature in Celsius milli degrees (°C/1000)
-func (d *Device) ReadTemperature() (t int32, err error) {
-
-	data := d.buf[:2]
-	err = legacy.ReadRegister(d.bus, uint8(d.MagAddress), TEMP_OUT_AUTO_INC, data)
-	if err != nil {
-		return
-	}
-
-	r := int16((uint16(data[1])<<8 | uint16(data[0]))) >> 4 // temperature offset from 25 °C
-	t = 25000 + int32((float32(r)/8)*1000)
-	return
-}

lsm303dlhc/registers.go

@@ -1,75 +0,0 @@
-package lsm303dlhc
-
-const (
-
-	// Constants/addresses used for I2C.
-	ACCEL_ADDRESS = 0x19
-	MAG_ADDRESS   = 0x1E
-
-	// i2C 8-bit subaddress (SUB): the 7 LSb represent the actual register address
-	// while the MSB enables address auto increment.
-	// If the MSb of the SUB field is 1, the SUB (register address) is
-	// automatically increased to allow multiple data read/writes.
-	ADDR_AUTO_INC_MASK = 0x80
-
-	// accelerometer registers.
-	ACCEL_CTRL_REG1_A  = 0x20
-	ACCEL_CTRL_REG4_A  = 0x23
-	ACCEL_OUT_X_L_A    = 0x28
-	ACCEL_OUT_X_H_A    = 0x29
-	ACCEL_OUT_Y_L_A    = 0x2A
-	ACCEL_OUT_Y_H_A    = 0x2B
-	ACCEL_OUT_Z_L_A    = 0x2C
-	ACCEL_OUT_Z_H_A    = 0x2D
-	ACCEL_OUT_AUTO_INC = ACCEL_OUT_X_L_A | ADDR_AUTO_INC_MASK
-
-	// magnetic sensor registers.
-	MAG_MR_REG_M     = 0x02
-	MAG_OUT_X_L_M    = 0x68
-	MAG_OUT_X_H_M    = 0x69
-	MAG_OUT_Y_L_M    = 0x6A
-	MAG_OUT_Y_H_M    = 0x6B
-	MAG_OUT_Z_L_M    = 0x6C
-	MAG_OUT_Z_H_M    = 0x6D
-	MAG_OUT_AUTO_INC = MAG_OUT_X_L_M | ADDR_AUTO_INC_MASK
-
-	// temperature sensor registers.
-	CRA_REG_M         = 0x80
-	TEMP_OUT_L_M      = 0x32
-	TEMP_OUT_H_M      = 0x31
-	TEMP_OUT_AUTO_INC = TEMP_OUT_L_M | ADDR_AUTO_INC_MASK
-
-	// accelerometer power mode.
-	ACCEL_POWER_NORMAL = 0x00 // default
-	ACCEL_POWER_LOW    = 0x08
-
-	// accelerometer range.
-	ACCEL_RANGE_2G  = 0x00 // default
-	ACCEL_RANGE_4G  = 0x01
-	ACCEL_RANGE_8G  = 0x02
-	ACCEL_RANGE_16G = 0x03
-
-	// accelerometer data rate.
-	ACCEL_DATARATE_1HZ    = 0x01
-	ACCEL_DATARATE_10HZ   = 0x02
-	ACCEL_DATARATE_25HZ   = 0x03
-	ACCEL_DATARATE_50HZ   = 0x04
-	ACCEL_DATARATE_100HZ  = 0x05 // default
-	ACCEL_DATARATE_200HZ  = 0x06
-	ACCEL_DATARATE_400HZ  = 0x07
-	ACCEL_DATARATE_1344HZ = 0x09 // 5376Hz in low-power mode
-
-	// magnetic sensor power mode.
-	MAG_POWER_NORMAL = 0x00 // default
-	MAG_POWER_LOW    = 0x01
-
-	// magnetic sensor operate mode.
-	MAG_SYSTEM_CONTINUOUS = 0x00 // default
-	MAG_SYSTEM_SINGLE     = 0x01
-
-	// magnetic sensor data rate
-	MAG_DATARATE_10HZ  = 0x00 // default
-	MAG_DATARATE_20HZ  = 0x01
-	MAG_DATARATE_50HZ  = 0x02
-	MAG_DATARATE_100HZ = 0x03
-)

lsm6ds3tr/lsm6ds3tr.go

@@ -8,6 +8,7 @@ import (
 	"errors"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
 )
 
 type AccelRange uint8
@@ -25,7 +26,7 @@ type Device struct {
 	accelSampleRate AccelSampleRate
 	gyroRange       GyroRange
 	gyroSampleRate  GyroSampleRate
-	buf             [7]uint8 // up to 6 bytes for read + 1 byte for the register address
+	buf             [6]uint8
 }
 
 // Configuration for LSM6DS3TR device.
@@ -83,20 +84,30 @@ func (d *Device) doConfigure(cfg Configuration) (err error) {
 		d.gyroSampleRate = GYRO_SR_104
 	}
 
+	data := d.buf[:1]
+
 	// Configure accelerometer
-	err = d.writeByte(CTRL1_XL, uint8(d.accelRange)|uint8(d.accelSampleRate))
+	data[0] = uint8(d.accelRange) | uint8(d.accelSampleRate)
+	err = legacy.WriteRegister(d.bus, uint8(d.Address), CTRL1_XL, data)
 	if err != nil {
 		return
 	}
 
-	// Enable ODR scaling
-	err = d.setBits(CTRL4_C, BW_SCAL_ODR_ENABLED)
+	// Set ODR bit
+	err = legacy.ReadRegister(d.bus, uint8(d.Address), CTRL4_C, data)
+	if err != nil {
+		return
+	}
+	data[0] = data[0] &^ BW_SCAL_ODR_ENABLED
+	data[0] |= BW_SCAL_ODR_ENABLED
+	err = legacy.WriteRegister(d.bus, uint8(d.Address), CTRL4_C, data)
 	if err != nil {
 		return
 	}
 
 	// Configure gyroscope
-	err = d.writeByte(CTRL2_G, uint8(d.gyroRange)|uint8(d.gyroSampleRate))
+	data[0] = uint8(d.gyroRange) | uint8(d.gyroSampleRate)
+	err = legacy.WriteRegister(d.bus, uint8(d.Address), CTRL2_G, data)
 	if err != nil {
 		return
 	}
@@ -107,10 +118,8 @@ func (d *Device) doConfigure(cfg Configuration) (err error) {
 // Connected returns whether a LSM6DS3TR has been found.
 // It does a "who am I" request and checks the response.
 func (d *Device) Connected() bool {
-	data, err := d.readBytes(WHO_AM_I, 1)
-	if err != nil {
-		return false
-	}
+	data := d.buf[:1]
+	legacy.ReadRegister(d.bus, uint8(d.Address), WHO_AM_I, data)
 	return data[0] == 0x6A
 }
 
@@ -119,7 +128,8 @@ func (d *Device) Connected() bool {
 // and the sensor is not moving the returned value will be around 1000000 or
 // -1000000.
 func (d *Device) ReadAcceleration() (x, y, z int32, err error) {
-	data, err := d.readBytes(OUTX_L_XL, 6)
+	data := d.buf[:6]
+	err = legacy.ReadRegister(d.bus, uint8(d.Address), OUTX_L_XL, data)
 	if err != nil {
 		return
 	}
@@ -143,7 +153,8 @@ func (d *Device) ReadAcceleration() (x, y, z int32, err error) {
 // rotation along one axis and while doing so integrate all values over time,
 // you would get a value close to 360000000.
 func (d *Device) ReadRotation() (x, y, z int32, err error) {
-	data, err := d.readBytes(OUTX_L_G, 6)
+	data := d.buf[:6]
+	err = legacy.ReadRegister(d.bus, uint8(d.Address), OUTX_L_G, data)
 	if err != nil {
 		return
 	}
@@ -166,7 +177,8 @@ func (d *Device) ReadRotation() (x, y, z int32, err error) {
 
 // ReadTemperature returns the temperature in celsius milli degrees (°C/1000)
 func (d *Device) ReadTemperature() (t int32, err error) {
-	data, err := d.readBytes(OUT_TEMP_L, 2)
+	data := d.buf[:2]
+	err = legacy.ReadRegister(d.bus, uint8(d.Address), OUT_TEMP_L, data)
 	if err != nil {
 		return
 	}
@@ -175,26 +187,3 @@ func (d *Device) ReadTemperature() (t int32, err error) {
 	t = 25000 + (int32(int16((int16(data[1])<<8)|int16(data[0])))*125)/32
 	return
 }
-
-func (d *Device) readBytes(reg, size uint8) ([]byte, error) {
-	d.buf[0] = reg
-	err := d.bus.Tx(d.Address, d.buf[0:1], d.buf[1:size+1])
-	if err != nil {
-		return nil, err
-	}
-	return d.buf[1 : size+1], nil
-}
-
-func (d *Device) writeByte(reg, value uint8) error {
-	d.buf[0] = reg
-	d.buf[1] = value
-	return d.bus.Tx(d.Address, d.buf[0:2], nil)
-}
-
-func (d *Device) setBits(reg, bits uint8) error {
-	data, err := d.readBytes(reg, 1)
-	if err != nil {
-		return err
-	}
-	return d.writeByte(reg, (data[0]&^bits)|bits)
-}

lsm9ds1/lsm9ds1.go

@@ -7,6 +7,7 @@ import (
 	"errors"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
 )
 
 type AccelRange uint8
@@ -27,7 +28,7 @@ type Device struct {
 	accelMultiplier int32
 	gyroMultiplier  int32
 	magMultiplier   int32
-	buf             [7]uint8 // up to 6 bytes for read + 1 byte for the register address
+	buf             [6]uint8
 }
 
 // Configuration for LSM9DS1 device.
@@ -60,15 +61,10 @@ func New(bus drivers.I2C) *Device {
 // Case of boolean false and error nil means I2C is up,
 // but "who am I" responses have unexpected values.
 func (d *Device) Connected() bool {
-	data, err := d.readBytes(d.AccelAddress, WHO_AM_I, 1)
-	if err != nil || data[0] != 0x68 {
-		return false
-	}
-	data, err = d.readBytes(d.MagAddress, WHO_AM_I_M, 1)
-	if err != nil || data[0] != 0x3D {
-		return false
-	}
-	return true
+	data1, data2 := d.buf[:1], d.buf[1:2]
+	legacy.ReadRegister(d.bus, d.AccelAddress, WHO_AM_I, data1)
+	legacy.ReadRegister(d.bus, d.MagAddress, WHO_AM_I_M, data2)
+	return data1[0] == 0x68 && data2[0] == 0x3D
 }
 
 // ReadAcceleration reads the current acceleration from the device and returns
@@ -76,7 +72,8 @@ func (d *Device) Connected() bool {
 // and the sensor is not moving the returned value will be around 1000000 or
 // -1000000.
 func (d *Device) ReadAcceleration() (x, y, z int32, err error) {
-	data, err := d.readBytes(d.AccelAddress, OUT_X_L_XL, 6)
+	data := d.buf[:6]
+	err = legacy.ReadRegister(d.bus, uint8(d.AccelAddress), OUT_X_L_XL, data)
 	if err != nil {
 		return
 	}
@@ -91,7 +88,8 @@ func (d *Device) ReadAcceleration() (x, y, z int32, err error) {
 // rotation along one axis and while doing so integrate all values over time,
 // you would get a value close to 360000000.
 func (d *Device) ReadRotation() (x, y, z int32, err error) {
-	data, err := d.readBytes(d.AccelAddress, OUT_X_L_G, 6)
+	data := d.buf[:6]
+	err = legacy.ReadRegister(d.bus, uint8(d.AccelAddress), OUT_X_L_G, data)
 	if err != nil {
 		return
 	}
@@ -104,7 +102,8 @@ func (d *Device) ReadRotation() (x, y, z int32, err error) {
 // ReadMagneticField reads the current magnetic field from the device and returns
 // it in nT (nanotesla). 1 G (gauss) = 100_000 nT (nanotesla).
 func (d *Device) ReadMagneticField() (x, y, z int32, err error) {
-	data, err := d.readBytes(d.MagAddress, OUT_X_L_M, 6)
+	data := d.buf[:6]
+	err = legacy.ReadRegister(d.bus, uint8(d.MagAddress), OUT_X_L_M, data)
 	if err != nil {
 		return
 	}
@@ -116,7 +115,8 @@ func (d *Device) ReadMagneticField() (x, y, z int32, err error) {
 
 // ReadTemperature returns the temperature in Celsius milli degrees (°C/1000)
 func (d *Device) ReadTemperature() (t int32, err error) {
-	data, err := d.readBytes(d.AccelAddress, OUT_TEMP_L, 2)
+	data := d.buf[:2]
+	err = legacy.ReadRegister(d.bus, uint8(d.AccelAddress), OUT_TEMP_L, data)
 	if err != nil {
 		return
 	}
@@ -167,16 +167,20 @@ func (d *Device) doConfigure(cfg Configuration) (err error) {
 		d.magMultiplier = 58
 	}
 
+	data := d.buf[:1]
+
 	// Configure accelerometer
 	// Sample rate & measurement range
-	err = d.writeByte(d.AccelAddress, CTRL_REG6_XL, uint8(cfg.AccelSampleRate)<<5|uint8(cfg.AccelRange)<<3)
+	data[0] = uint8(cfg.AccelSampleRate)<<5 | uint8(cfg.AccelRange)<<3
+	err = legacy.WriteRegister(d.bus, d.AccelAddress, CTRL_REG6_XL, data)
 	if err != nil {
 		return
 	}
 
 	// Configure gyroscope
 	// Sample rate & measurement range
-	err = d.writeByte(d.AccelAddress, CTRL_REG1_G, uint8(cfg.GyroSampleRate)<<5|uint8(cfg.GyroRange)<<3)
+	data[0] = uint8(cfg.GyroSampleRate)<<5 | uint8(cfg.GyroRange)<<3
+	err = legacy.WriteRegister(d.bus, d.AccelAddress, CTRL_REG1_G, data)
 	if err != nil {
 		return
 	}
@@ -186,44 +190,33 @@ func (d *Device) doConfigure(cfg Configuration) (err error) {
 	// Temperature compensation enabled
 	// High-performance mode XY axis
 	// Sample rate
-	err = d.writeByte(d.MagAddress, CTRL_REG1_M, 0b10000000|0b01000000|uint8(cfg.MagSampleRate)<<2)
+	data[0] = 0b10000000 | 0b01000000 | uint8(cfg.MagSampleRate)<<2
+	err = legacy.WriteRegister(d.bus, d.MagAddress, CTRL_REG1_M, data)
 	if err != nil {
 		return
 	}
 
 	// Measurement range
-	err = d.writeByte(d.MagAddress, CTRL_REG2_M, uint8(cfg.MagRange)<<5)
+	data[0] = uint8(cfg.MagRange) << 5
+	err = legacy.WriteRegister(d.bus, d.MagAddress, CTRL_REG2_M, data)
 	if err != nil {
 		return
 	}
 
 	// Continuous-conversion mode
 	// https://electronics.stackexchange.com/questions/237397/continuous-conversion-vs-single-conversion-mode
-	err = d.writeByte(d.MagAddress, CTRL_REG3_M, 0b00000000)
+	data[0] = 0b00000000
+	err = legacy.WriteRegister(d.bus, d.MagAddress, CTRL_REG3_M, data)
 	if err != nil {
 		return
 	}
 
 	// High-performance mode Z axis
-	err = d.writeByte(d.MagAddress, CTRL_REG4_M, 0b00001000)
+	data[0] = 0b00001000
+	err = legacy.WriteRegister(d.bus, d.MagAddress, CTRL_REG4_M, data)
 	if err != nil {
 		return
 	}
 
 	return nil
 }
-
-func (d *Device) readBytes(addr, reg, size uint8) ([]byte, error) {
-	d.buf[0] = reg
-	err := d.bus.Tx(uint16(addr), d.buf[0:1], d.buf[1:size+1])
-	if err != nil {
-		return nil, err
-	}
-	return d.buf[1 : size+1], nil
-}
-
-func (d *Device) writeByte(addr, reg, value uint8) error {
-	d.buf[0] = reg
-	d.buf[1] = value
-	return d.bus.Tx(uint16(addr), d.buf[0:2], nil)
-}

max6675/max6675.go

@@ -3,9 +3,9 @@ package max6675
 
 import (
 	"errors"
-	"machine"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // ErrThermocoupleOpen is returned when the thermocouple input is open.
@@ -14,16 +14,16 @@ var ErrThermocoupleOpen = errors.New("thermocouple input open")
 
 type Device struct {
 	bus drivers.SPI
-	cs  machine.Pin
+	cs  drivers.PinOutput
 }
 
 // Create a new Device to read from a MAX6675 thermocouple.
 // Pins must be configured before use.  Frequency for SPI
 // should be 4.3MHz maximum.
-func NewDevice(bus drivers.SPI, cs machine.Pin) *Device {
+func NewDevice(bus drivers.SPI, cs pin.Output) *Device {
 	return &Device{
 		bus: bus,
-		cs:  cs,
+		cs:  cs.Set,
 	}
 }
 

max72xx/max72xx.go

@@ -3,31 +3,36 @@
 package max72xx
 
 import (
-	"machine"
-
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Device struct {
-	bus drivers.SPI
-	cs  machine.Pin
+	bus           drivers.SPI
+	cs            drivers.PinOutput
+	configurePins func()
 }
 
 // NewDriver creates a new max7219 connection. The SPI wire must already be configured
 // The SPI frequency must not be higher than 10MHz.
 // parameter cs: the datasheet also refers to this pin as "load" pin.
-func NewDevice(bus drivers.SPI, cs machine.Pin) *Device {
+func NewDevice(bus drivers.SPI, cs pin.Output) *Device {
 	return &Device{
 		bus: bus,
-		cs:  cs,
+		cs:  cs.Set,
+		configurePins: func() {
+			legacy.ConfigurePinOut(cs)
+		},
 	}
 }
 
 // Configure setups the pins.
 func (driver *Device) Configure() {
-	outPutConfig := machine.PinConfig{Mode: machine.PinOutput}
-
-	driver.cs.Configure(outPutConfig)
+	if driver.configurePins == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
+	driver.configurePins()
 }
 
 // SetScanLimit sets the scan limit. Maximum is 8.

mcp2515/mcp2515.go

@@ -8,18 +8,20 @@ package mcp2515 // import "tinygo.org/x/drivers/mcp2515"
 import (
 	"errors"
 	"fmt"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // Device wraps MCP2515 SPI CAN Module.
 type Device struct {
-	spi     SPI
-	cs      machine.Pin
-	msg     *CANMsg
-	mcpMode byte
+	spi           SPI
+	cs            drivers.PinOutput
+	msg           *CANMsg
+	mcpMode       byte
+	configurePins func()
 }
 
 // CANMsg stores CAN message fields.
@@ -36,15 +38,18 @@ const (
 )
 
 // New returns a new MCP2515 driver. Pass in a fully configured SPI bus.
-func New(b drivers.SPI, csPin machine.Pin) *Device {
+func New(b drivers.SPI, csPin pin.Output) *Device {
 	d := &Device{
 		spi: SPI{
 			bus: b,
 			tx:  make([]byte, 0, bufferSize),
 			rx:  make([]byte, 0, bufferSize),
 		},
-		cs:  csPin,
+		cs:  csPin.Set,
 		msg: &CANMsg{},
+		configurePins: func() {
+			legacy.ConfigurePinOut(csPin)
+		},
 	}
 
 	return d
@@ -52,7 +57,10 @@ func New(b drivers.SPI, csPin machine.Pin) *Device {
 
 // Configure sets up the device for communication.
 func (d *Device) Configure() {
-	d.cs.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	if d.configurePins == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
+	d.configurePins()
 }
 
 const beginTimeoutValue int = 10

onewire/onewire.go

@@ -5,8 +5,9 @@ package onewire // import "tinygo.org/x/drivers/onewire"
 
 import (
 	"errors"
-	"machine"
 	"time"
+
+	"tinygo.org/x/drivers"
 )
 
 // OneWire ROM commands
@@ -19,7 +20,8 @@ const (
 
 // Device wraps a connection to an 1-Wire devices.
 type Device struct {
-	p machine.Pin
+	set drivers.PinOutput
+	get drivers.PinInput
 }
 
 // Config wraps a configuration to an 1-Wire devices.
@@ -32,24 +34,30 @@ var (
 	errReadAddress    = errors.New("Error: OneWire. Read address error: CRC mismatch.")
 )
 
-// New creates a new GPIO 1-Wire connection.
-// The pin must be pulled up to the VCC via a resistor greater than 500 ohms (default 4.7k).
-func New(p machine.Pin) Device {
-	return Device{
-		p: p,
+// NewFromFuncs expects pin setter and getter for DQ line. Ideally this driver should receive
+// a one-wire bus HAL abstraction, but I was asked to show how this could be done using pin HAL so here goes.
+func NewFromFuncs(getPinLevel drivers.PinInput, setPinLevel drivers.PinOutput) *Device {
+	return &Device{
+		set: setPinLevel,
+		get: getPinLevel,
 	}
 }
 
 // Configure initializes the protocol.
 func (d *Device) Configure(config Config) {}
 
+// By setting the pin value one should configure as output and also expect a more
+// consistent behaviour across all tinygo hosts by pulling DQ line low consistently. Win-win.
+func (d *Device) cfgOut() { d.set.Low() }
+func (d *Device) cfgIn()  { d.get() }
+
 // Reset pull DQ line low, then up.
 func (d Device) Reset() error {
-	d.p.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	d.cfgOut()
 	time.Sleep(480 * time.Microsecond)
-	d.p.Configure(machine.PinConfig{Mode: machine.PinInputPullup})
+	d.cfgIn()
 	time.Sleep(70 * time.Microsecond)
-	precence := d.p.Get()
+	precence := d.get()
 	time.Sleep(410 * time.Microsecond)
 	if precence {
 		return errNoPresence
@@ -59,14 +67,14 @@ func (d Device) Reset() error {
 
 // WriteBit transmits a bit to 1-Wire bus.
 func (d Device) WriteBit(data uint8) {
-	d.p.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	d.cfgOut()
 	if data&1 == 1 { // Send '1'
 		time.Sleep(5 * time.Microsecond)
-		d.p.Configure(machine.PinConfig{Mode: machine.PinInputPullup})
+		d.cfgIn()
 		time.Sleep(60 * time.Microsecond)
 	} else { // Send '0'
 		time.Sleep(60 * time.Microsecond)
-		d.p.Configure(machine.PinConfig{Mode: machine.PinInputPullup})
+		d.cfgIn()
 		time.Sleep(5 * time.Microsecond)
 	}
 }
@@ -81,11 +89,11 @@ func (d Device) Write(data uint8) {
 
 // ReadBit receives a bit from 1-Wire bus.
 func (d Device) ReadBit() (data uint8) {
-	d.p.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	d.cfgOut()
 	time.Sleep(3 * time.Microsecond)
-	d.p.Configure(machine.PinConfig{Mode: machine.PinInputPullup})
+	d.cfgIn()
 	time.Sleep(8 * time.Microsecond)
-	if d.p.Get() {
+	if d.get() {
 		data = 1
 	}
 	time.Sleep(60 * time.Microsecond)

onewire/onewire_tinygo.go

@@ -0,0 +1,29 @@
+package onewire
+
+import (
+	"machine"
+
+	"tinygo.org/x/drivers/internal/legacy"
+)
+
+// New creates a new GPIO 1-Wire connection.
+// The pin must be pulled up to the VCC via a resistor greater than 500 ohms (default 4.7k).
+func New(p machine.Pin) Device {
+	isOut := false
+	return Device{
+		set: func(level bool) {
+			if !isOut {
+				legacy.ConfigurePinOut(p)
+				isOut = true
+			}
+			p.Set(level)
+		},
+		get: func() (level bool) {
+			if isOut {
+				legacy.ConfigurePinInputPullup(p)
+				isOut = false
+			}
+			return p.Get()
+		},
+	}
+}

pcd8544/pcd8544.go

@@ -6,18 +6,18 @@ package pcd8544 // import "tinygo.org/x/drivers/pcd8544"
 import (
 	"errors"
 	"image/color"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // Device wraps an SPI connection.
 type Device struct {
 	bus        drivers.SPI
-	dcPin      machine.Pin
-	rstPin     machine.Pin
-	scePin     machine.Pin
+	dcPin      drivers.PinOutput
+	rstPin     drivers.PinOutput
+	scePin     drivers.PinOutput
 	buffer     []byte
 	width      int16
 	height     int16
@@ -30,12 +30,12 @@ type Config struct {
 }
 
 // New creates a new PCD8544 connection. The SPI bus must already be configured.
-func New(bus drivers.SPI, dcPin, rstPin, scePin machine.Pin) *Device {
+func New(bus drivers.SPI, dcPin, rstPin, scePin pin.Output) *Device {
 	return &Device{
 		bus:    bus,
-		dcPin:  dcPin,
-		rstPin: rstPin,
-		scePin: scePin,
+		dcPin:  dcPin.Set,
+		rstPin: rstPin.Set,
+		scePin: scePin.Set,
 	}
 }
 

pin.go

@@ -0,0 +1,29 @@
+package drivers
+
+// PinOutput is hardware abstraction for a pin which outputs a
+// digital signal (high or low level).
+//
+//	// Code conversion demo: from machine.Pin to drivers.PinOutput
+//	led := machine.LED
+//	led.Configure(machine.PinConfig{Mode: machine.PinOutput})
+//	var pin drivers.PinOutput = led.Set // Going from a machine.Pin to a drivers.PinOutput
+type PinOutput func(level bool)
+
+// High sets the underlying pin's level to high. This is equivalent to calling PinOutput(true).
+func (po PinOutput) High() {
+	po(true)
+}
+
+// Low sets the underlying pin's level to low. This is equivalent to calling PinOutput(false).
+func (po PinOutput) Low() {
+	po(false)
+}
+
+// PinInput is hardware abstraction for a pin which receives a
+// digital signal and reads it (high or low level).
+//
+//	// Code conversion demo: from machine.Pin to drivers.PinInput
+//	input := machine.LED
+//	input.Configure(machine.PinConfig{Mode: machine.PinInputPulldown}) // or use machine.PinInputPullup or machine.PinInput
+//	var pin drivers.PinInput = input.Get // Going from a machine.Pin to a drivers.PinInput
+type PinInput func() (level bool)

seesaw/encoder.go

@@ -1,49 +0,0 @@
-package seesaw
-
-import (
-	"errors"
-)
-
-var errInvalidEncoderNumber = errors.New("invalid encoder choice, 0-15 are supported")
-
-// GetEncoderPosition returns the absolute position (or delta since the previous call) of the specified rotary encoder.
-func (d *Device) GetEncoderPosition(encoder uint, asDelta bool) (int32, error) {
-	if encoder >= 16 {
-		return 0, errInvalidEncoderNumber
-	}
-
-	// The function address' upper nibble is the function, the lower nibble selects which encoder to communicate with
-	fnAddr := FunctionAddress(encoder)
-	if asDelta {
-		fnAddr |= FunctionEncoderDelta
-	} else {
-		fnAddr |= FunctionEncoderPosition
-	}
-
-	var buf [4]byte
-	err := d.Read(ModuleEncoderBase, fnAddr, buf[:])
-	if err != nil {
-		return 0, err
-	}
-
-	return int32(buf[0])<<24 | int32(buf[1])<<16 | int32(buf[2])<<8 | int32(buf[3]), nil
-}
-
-// SetEncoderPosition calibrate's the encoder's current absolute position to be whatever the provided position is.
-func (d *Device) SetEncoderPosition(encoder uint, position int32) error {
-	if encoder >= 16 {
-		return errInvalidEncoderNumber
-	}
-
-	// The function address' upper nibble is the function, the lower nibble selects which encoder to communicate with
-	fnAddr := FunctionEncoderPosition | FunctionAddress(encoder)
-
-	buf := [4]byte{
-		byte(position >> 24),
-		byte(position >> 16),
-		byte(position >> 8),
-		byte(position),
-	}
-
-	return d.Write(ModuleEncoderBase, fnAddr, buf[:])
-}

seesaw/registers.go

@@ -98,13 +98,3 @@ const (
 	FunctionKeypadCount    FunctionAddress = 0x04
 	FunctionKeypadFifo     FunctionAddress = 0x10
 )
-
-// encoder module function address registers
-// these are the defaults for encoder 0, change the lower nibble to address other encoders
-// see the Device.GetEncoderPosition and SetEncoderPosition methods for examples.
-const (
-	FunctionEncoderIntenset FunctionAddress = 0x10
-	FunctionEncoderIntenclr FunctionAddress = 0x20
-	FunctionEncoderPosition FunctionAddress = 0x30
-	FunctionEncoderDelta    FunctionAddress = 0x40
-)

shiftregister/shiftregister.go

@@ -2,7 +2,9 @@
 package shiftregister
 
 import (
-	"machine"
+	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type NumberBit int8
@@ -16,9 +18,10 @@ const (
 
 // Device holds pin number
 type Device struct {
-	latch, clock, out machine.Pin // IC wiring
-	bits              NumberBit   // Pin number
-	mask              uint32      // keep all pins state
+	latch, clock, out drivers.PinOutput // IC wiring
+	config            func()
+	bits              NumberBit // Pin number
+	mask              uint32    // keep all pins state
 }
 
 // ShiftPin is the implementation of the ShiftPin interface.
@@ -29,20 +32,25 @@ type ShiftPin struct {
 }
 
 // New returns a new shift output register device
-func New(Bits NumberBit, Latch, Clock, Out machine.Pin) *Device {
+func New(Bits NumberBit, Latch, Clock, Out pin.Output) *Device {
 	return &Device{
-		latch: Latch,
-		clock: Clock,
-		out:   Out,
+		latch: Latch.Set,
+		clock: Clock.Set,
+		out:   Out.Set,
 		bits:  Bits,
+		config: func() {
+			legacy.ConfigurePinOut(Latch)
+			legacy.ConfigurePinOut(Clock)
+			legacy.ConfigurePinOut(Out)
+		},
 	}
 }
 
 // Configure set hardware configuration
 func (d *Device) Configure() {
-	d.latch.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.clock.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.out.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	if d.config == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
 	d.latch.High()
 }
 
@@ -53,7 +61,7 @@ func (d *Device) WriteMask(mask uint32) {
 	d.latch.Low()
 	for i := 0; i < int(d.bits); i++ {
 		d.clock.Low()
-		d.out.Set(mask&1 != 0)
+		d.out(mask&1 != 0)
 		mask = mask >> 1
 		d.clock.High()
 	}

smoketest.sh

@@ -44,7 +44,6 @@ tinygo build -size short -o ./build/test.hex -target=pyportal ./examples/ili9341
 tinygo build -size short -o ./build/test.hex -target=circuitplay-express ./examples/lis3dh/main.go
 tinygo build -size short -o ./build/test.hex -target=nano-33-ble ./examples/lps22hb/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/lsm303agr/main.go
-tinygo build -size short -o ./build/test.hex -target=feather-m4 ./examples/lsm303dlhc/main.go
 tinygo build -size short -o ./build/test.hex -target=arduino-nano33 ./examples/lsm6ds3/main.go
 tinygo build -size short -o ./build/test.hex -target=itsybitsy-m0 ./examples/mag3110/main.go
 tinygo build -size short -o ./build/test.hex -target=itsybitsy-m0 ./examples/mcp23017/main.go
@@ -59,17 +58,15 @@ tinygo build -size short -o ./build/test.hex -target=p1am-100 ./examples/p1am/ma
 tinygo build -size short -o ./build/test.hex -target=pico ./examples/pca9685/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/pcd8544/setbuffer/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/pcd8544/setpixel/main.go
-tinygo build -size short -o ./build/test.hex -target=feather-rp2040 ./examples/seesaw/soil-sensor
-tinygo build -size short -o ./build/test.hex -target=qtpy-rp2040 ./examples/seesaw/rotary-encoder
+tinygo build -size short -o ./build/test.hex -target=feather-rp2040 ./examples/seesaw
 tinygo build -size short -o ./build/test.hex -target=arduino ./examples/servo
 tinygo build -size short -o ./build/test.hex -target=pico     ./examples/sgp30
 tinygo build -size short -o ./build/test.hex -target=pybadge ./examples/shifter/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/sht3x/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/sht4x/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/shtc3/main.go
-tinygo build -size short -o ./build/test.hex -target=xiao-ble ./examples/ssd1306/
-tinygo build -size short -o ./build/test.hex -target=xiao-rp2040 ./examples/ssd1306/
-tinygo build -size short -o ./build/test.hex -target=thumby ./examples/ssd1306/
+tinygo build -size short -o ./build/test.hex -target=microbit ./examples/ssd1306/i2c_128x32/main.go
+tinygo build -size short -o ./build/test.hex -target=microbit ./examples/ssd1306/spi_128x64/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/ssd1331/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/st7735/main.go
 tinygo build -size short -o ./build/test.hex -target=microbit ./examples/st7789/main.go
@@ -143,7 +140,6 @@ tinygo build -size short -o ./build/test.uf2 -target=pico ./examples/tmc2209/mai
 tinygo build -size short -o ./build/test.hex -target=pico ./examples/tmc5160/main.go
 tinygo build -size short -o ./build/test.uf2 -target=nicenano ./examples/sharpmem/main.go
 tinygo build -size short -o ./build/test.hex -target=feather-nrf52840 ./examples/max6675/main.go
-tinygo build -size short -o ./build/test.hex -target=pico ./examples/ens160/main.go
 # network examples (espat)
 tinygo build -size short -o ./build/test.hex -target=challenger-rp2040 ./examples/net/ntpclient/
 # network examples (wifinina)

ssd1289/pinbus.go

@@ -1,3 +1,5 @@
+//go:build baremetal
+
 package ssd1289
 
 import "machine"

ssd1289/ssd1289.go

@@ -7,6 +7,9 @@ import (
 	"image/color"
 	"machine"
 	"time"
+
+	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
 )
 
 type Bus interface {
@@ -14,10 +17,10 @@ type Bus interface {
 }
 
 type Device struct {
-	rs  machine.Pin
-	wr  machine.Pin
-	cs  machine.Pin
-	rst machine.Pin
+	rs  drivers.PinOutput
+	wr  drivers.PinOutput
+	cs  drivers.PinOutput
+	rst drivers.PinOutput
 	bus Bus
 }
 
@@ -26,21 +29,20 @@ const height = int16(320)
 
 func New(rs machine.Pin, wr machine.Pin, cs machine.Pin, rst machine.Pin, bus Bus) Device {
 	d := Device{
-		rs:  rs,
-		wr:  wr,
-		cs:  cs,
-		rst: rst,
+		rs:  rs.Set,
+		wr:  wr.Set,
+		cs:  cs.Set,
+		rst: rst.Set,
 		bus: bus,
 	}
+	legacy.ConfigurePinOut(rs)
+	legacy.ConfigurePinOut(wr)
+	legacy.ConfigurePinOut(cs)
+	legacy.ConfigurePinOut(rst)
 
-	rs.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	wr.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	cs.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	rst.Configure(machine.PinConfig{Mode: machine.PinOutput})
-
-	cs.High()
-	rst.High()
-	wr.High()
+	cs.Set(true)
+	rst.Set(true)
+	wr.Set(true)
 
 	return d
 }

ssd1306/ssd1306.go

@@ -6,9 +6,11 @@ package ssd1306 // import "tinygo.org/x/drivers/ssd1306"
 import (
 	"errors"
 	"image/color"
+	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
 	"tinygo.org/x/drivers/pixel"
 )
 
@@ -21,15 +23,16 @@ type ResetValue [2]byte
 
 // Device wraps I2C or SPI connection.
 type Device struct {
-	bus       Buser
-	buffer    []byte
-	width     int16
-	height    int16
-	vccState  VccMode
-	canReset  bool
-	resetCol  ResetValue
-	resetPage ResetValue
-	rotation  drivers.Rotation
+	bus        Buser
+	buffer     []byte
+	width      int16
+	height     int16
+	bufferSize int16
+	vccState   VccMode
+	canReset   bool
+	resetCol   ResetValue
+	resetPage  ResetValue
+	rotation   drivers.Rotation
 }
 
 // Config is the configuration for the display
@@ -48,15 +51,51 @@ type Config struct {
 	Rotation  drivers.Rotation
 }
 
+type I2CBus struct {
+	wire    drivers.I2C
+	Address uint16
+}
+
+type SPIBus struct {
+	wire     drivers.SPI
+	dcPin    machine.Pin
+	resetPin machine.Pin
+	csPin    machine.Pin
+}
+
 type Buser interface {
-	configure(address uint16, size int16) []byte // configure the bus and return the image buffer to use
-	command(cmd uint8) error                     // send a command to the display
-	flush() error                                // send the image to the display, faster than "tx()" in i2c case since avoids slice copy
-	tx(data []byte, isCommand bool) error        // generic transmit function
+	configure() error
+	tx(data []byte, isCommand bool) error
+	setAddress(address uint16) error
 }
 
 type VccMode uint8
 
+// NewI2C creates a new SSD1306 connection. The I2C wire must already be configured.
+func NewI2C(bus drivers.I2C) Device {
+	return Device{
+		bus: &I2CBus{
+			wire:    bus,
+			Address: Address,
+		},
+	}
+}
+
+// NewSPI creates a new SSD1306 connection. The SPI wire must already be configured.
+func NewSPI(bus drivers.SPI, dcPin, resetPin, csPin machine.Pin) Device {
+	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	resetPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	return Device{
+		bus: &SPIBus{
+			wire:     bus,
+			dcPin:    dcPin,
+			resetPin: resetPin,
+			csPin:    csPin,
+		},
+	}
+}
+
 // Configure initializes the display with default configuration
 func (d *Device) Configure(cfg Config) {
 	var zeroReset ResetValue
@@ -70,6 +109,9 @@ func (d *Device) Configure(cfg Config) {
 	} else {
 		d.height = 64
 	}
+	if cfg.Address != 0 {
+		d.bus.setAddress(cfg.Address)
+	}
 	if cfg.VccState != 0 {
 		d.vccState = cfg.VccState
 	} else {
@@ -85,9 +127,11 @@ func (d *Device) Configure(cfg Config) {
 	} else {
 		d.resetPage = ResetValue{0, uint8(d.height/8) - 1}
 	}
+	d.bufferSize = d.width * d.height / 8
+	d.buffer = make([]byte, d.bufferSize)
 	d.canReset = cfg.Address != 0 || d.width != 128 || d.height != 64 // I2C or not 128x64
 
-	d.buffer = d.bus.configure(cfg.Address, d.width*d.height/8)
+	d.bus.configure()
 
 	time.Sleep(100 * time.Nanosecond)
 	d.Command(DISPLAYOFF)
@@ -149,22 +193,11 @@ func (d *Device) Configure(cfg Config) {
 	d.Command(NORMALDISPLAY)
 	d.Command(DEACTIVATE_SCROLL)
 	d.Command(DISPLAYON)
-
-}
-
-// Command sends a command to the display
-func (d *Device) Command(command uint8) {
-	d.bus.command(command)
-}
-
-// Tx sends data to the display; if isCommand is false, this also updates the image buffer.
-func (d *Device) Tx(data []byte, isCommand bool) error {
-	return d.bus.tx(data, isCommand)
 }
 
 // ClearBuffer clears the image buffer
 func (d *Device) ClearBuffer() {
-	for i := 0; i < len(d.buffer); i++ {
+	for i := int16(0); i < d.bufferSize; i++ {
 		d.buffer[i] = 0
 	}
 }
@@ -190,7 +223,7 @@ func (d *Device) Display() error {
 		d.Command(d.resetPage[1])
 	}
 
-	return d.bus.flush()
+	return d.Tx(d.buffer, false)
 }
 
 // SetPixel enables or disables a pixel in the buffer
@@ -219,10 +252,12 @@ func (d *Device) GetPixel(x int16, y int16) bool {
 
 // SetBuffer changes the whole buffer at once
 func (d *Device) SetBuffer(buffer []byte) error {
-	if len(buffer) != len(d.buffer) {
+	if int16(len(buffer)) != d.bufferSize {
 		return errBufferSize
 	}
-	copy(d.buffer, buffer)
+	for i := int16(0); i < d.bufferSize; i++ {
+		d.buffer[i] = buffer[i]
+	}
 	return nil
 }
 
@@ -231,6 +266,81 @@ func (d *Device) GetBuffer() []byte {
 	return d.buffer
 }
 
+// Command sends a command to the display
+func (d *Device) Command(command uint8) {
+	d.bus.tx([]byte{command}, true)
+}
+
+// setAddress sets the address to the I2C bus
+func (b *I2CBus) setAddress(address uint16) error {
+	b.Address = address
+	return nil
+}
+
+// setAddress does nothing, but it's required to avoid reflection
+func (b *SPIBus) setAddress(address uint16) error {
+	// do nothing
+	println("trying to Configure an address on a SPI device")
+	return nil
+}
+
+// configure does nothing, but it's required to avoid reflection
+func (b *I2CBus) configure() error { return nil }
+
+// configure configures some pins with the SPI bus
+func (b *SPIBus) configure() error {
+	b.csPin.Low()
+	b.dcPin.Low()
+	b.resetPin.Low()
+
+	b.resetPin.High()
+	time.Sleep(1 * time.Millisecond)
+	b.resetPin.Low()
+	time.Sleep(10 * time.Millisecond)
+	b.resetPin.High()
+
+	return nil
+}
+
+// Tx sends data to the display
+func (d *Device) Tx(data []byte, isCommand bool) error {
+	return d.bus.tx(data, isCommand)
+}
+
+// tx sends data to the display (I2CBus implementation)
+func (b *I2CBus) tx(data []byte, isCommand bool) error {
+	if isCommand {
+		return legacy.WriteRegister(b.wire, uint8(b.Address), 0x00, data)
+	} else {
+		return legacy.WriteRegister(b.wire, uint8(b.Address), 0x40, data)
+	}
+}
+
+// tx sends data to the display (SPIBus implementation)
+func (b *SPIBus) tx(data []byte, isCommand bool) error {
+	var err error
+
+	if isCommand {
+		b.csPin.High()
+		time.Sleep(1 * time.Millisecond)
+		b.dcPin.Low()
+		b.csPin.Low()
+
+		err = b.wire.Tx(data, nil)
+		b.csPin.High()
+	} else {
+		b.csPin.High()
+		time.Sleep(1 * time.Millisecond)
+		b.dcPin.High()
+		b.csPin.Low()
+
+		err = b.wire.Tx(data, nil)
+		b.csPin.High()
+	}
+
+	return err
+}
+
 // Size returns the current size of the display.
 func (d *Device) Size() (w, h int16) {
 	return d.width, d.height

ssd1306/ssd1306_i2c.go

@@ -1,52 +0,0 @@
-package ssd1306
-
-import (
-	"tinygo.org/x/drivers"
-)
-
-type I2CBus struct {
-	wire    drivers.I2C
-	address uint16
-	buffer  []byte // buffer to avoid heap allocations
-}
-
-// NewI2C creates a new SSD1306 connection. The I2C wire must already be configured.
-func NewI2C(bus drivers.I2C) *Device {
-	return &Device{
-		bus: &I2CBus{
-			wire:    bus,
-			address: Address,
-		},
-	}
-}
-
-// configure address for the I2C bus and allocate the buffer
-func (b *I2CBus) configure(address uint16, size int16) []byte {
-	if address != 0 {
-		b.address = address
-	}
-	b.buffer = make([]byte, size+2) // +1 for the mode and +1 for a command
-	return b.buffer[2:]             // return the image buffer
-}
-
-// command sends a command to the display
-func (b *I2CBus) command(cmd uint8) error {
-	b.buffer[0] = 0x00 // Command mode
-	b.buffer[1] = cmd
-	return b.wire.Tx(b.address, b.buffer[:2], nil)
-}
-
-// flush sends the image to the display
-func (b *I2CBus) flush() error {
-	b.buffer[1] = 0x40 // Data mode
-	return b.wire.Tx(b.address, b.buffer[1:], nil)
-}
-
-// tx sends data to the display
-func (b *I2CBus) tx(data []byte, isCommand bool) error {
-	if isCommand {
-		return b.command(data[0])
-	}
-	copy(b.buffer[2:], data)
-	return b.flush()
-}

ssd1306/ssd1306_spi.go

@@ -1,68 +0,0 @@
-package ssd1306
-
-import (
-	"machine"
-	"time"
-
-	"tinygo.org/x/drivers"
-)
-
-type SPIBus struct {
-	wire     drivers.SPI
-	dcPin    machine.Pin
-	resetPin machine.Pin
-	csPin    machine.Pin
-	buffer   []byte // buffer to avoid heap allocations
-}
-
-// NewSPI creates a new SSD1306 connection. The SPI wire must already be configured.
-func NewSPI(bus drivers.SPI, dcPin, resetPin, csPin machine.Pin) *Device {
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	resetPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	return &Device{
-		bus: &SPIBus{
-			wire:     bus,
-			dcPin:    dcPin,
-			resetPin: resetPin,
-			csPin:    csPin,
-		},
-	}
-}
-
-// configure pins with the SPI bus and allocate the buffer
-func (b *SPIBus) configure(address uint16, size int16) []byte {
-	b.csPin.Low()
-	b.dcPin.Low()
-	b.resetPin.Low()
-
-	b.resetPin.High()
-	time.Sleep(1 * time.Millisecond)
-	b.resetPin.Low()
-	time.Sleep(10 * time.Millisecond)
-	b.resetPin.High()
-
-	b.buffer = make([]byte, size+1) // +1 for a command
-	return b.buffer[1:]             // return the image buffer
-}
-
-// command sends a command to the display
-func (b *SPIBus) command(cmd uint8) error {
-	b.buffer[0] = cmd
-	return b.tx(b.buffer[:1], true)
-}
-
-// flush sends the image to the display
-func (b *SPIBus) flush() error {
-	return b.tx(b.buffer[1:], false)
-}
-
-// tx sends data to the display
-func (b *SPIBus) tx(data []byte, isCommand bool) error {
-	b.csPin.High()
-	b.dcPin.Set(!isCommand)
-	b.csPin.Low()
-	err := b.wire.Tx(data, nil)
-	b.csPin.High()
-	return err
-}

ssd1331/ssd1331.go

@@ -11,6 +11,7 @@ import (
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
 )
 
 type Model uint8
@@ -19,9 +20,9 @@ type Rotation uint8
 // Device wraps an SPI connection.
 type Device struct {
 	bus         drivers.SPI
-	dcPin       machine.Pin
-	resetPin    machine.Pin
-	csPin       machine.Pin
+	dcPin       drivers.PinOutput
+	resetPin    drivers.PinOutput
+	csPin       drivers.PinOutput
 	width       int16
 	height      int16
 	batchLength int16
@@ -37,14 +38,14 @@ type Config struct {
 
 // New creates a new SSD1331 connection. The SPI wire must already be configured.
 func New(bus drivers.SPI, resetPin, dcPin, csPin machine.Pin) Device {
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	resetPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(resetPin)
+	legacy.ConfigurePinOut(csPin)
 	return Device{
 		bus:      bus,
-		dcPin:    dcPin,
-		resetPin: resetPin,
-		csPin:    csPin,
+		dcPin:    dcPin.Set,
+		resetPin: resetPin.Set,
+		csPin:    csPin.Set,
 	}
 }
 
@@ -251,7 +252,7 @@ func (d *Device) Data(data uint8) {
 
 // Tx sends data to the display
 func (d *Device) Tx(data []byte, isCommand bool) {
-	d.dcPin.Set(!isCommand)
+	d.dcPin(!isCommand)
 	d.bus.Tx(data, nil)
 }
 

ssd1351/ssd1351.go

@@ -6,10 +6,11 @@ package ssd1351 // import "tinygo.org/x/drivers/ssd1351"
 import (
 	"errors"
 	"image/color"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 var (
@@ -19,17 +20,18 @@ var (
 
 // Device wraps an SPI connection.
 type Device struct {
-	bus          drivers.SPI
-	dcPin        machine.Pin
-	resetPin     machine.Pin
-	csPin        machine.Pin
-	enPin        machine.Pin
-	rwPin        machine.Pin
-	width        int16
-	height       int16
-	rowOffset    int16
-	columnOffset int16
-	bufferLength int16
+	bus           drivers.SPI
+	dcPin         drivers.PinOutput
+	resetPin      drivers.PinOutput
+	csPin         drivers.PinOutput
+	enPin         drivers.PinOutput
+	rwPin         drivers.PinOutput
+	configurePins func()
+	width         int16
+	height        int16
+	rowOffset     int16
+	columnOffset  int16
+	bufferLength  int16
 }
 
 // Config is the configuration for the display
@@ -41,19 +43,29 @@ type Config struct {
 }
 
 // New creates a new SSD1351 connection. The SPI wire must already be configured.
-func New(bus drivers.SPI, resetPin, dcPin, csPin, enPin, rwPin machine.Pin) Device {
+func New(bus drivers.SPI, resetPin, dcPin, csPin, enPin, rwPin pin.Output) Device {
 	return Device{
 		bus:      bus,
-		dcPin:    dcPin,
-		resetPin: resetPin,
-		csPin:    csPin,
-		enPin:    enPin,
-		rwPin:    rwPin,
+		dcPin:    dcPin.Set,
+		resetPin: resetPin.Set,
+		csPin:    csPin.Set,
+		enPin:    enPin.Set,
+		rwPin:    rwPin.Set,
+		configurePins: func() {
+			legacy.ConfigurePinOut(dcPin)
+			legacy.ConfigurePinOut(resetPin)
+			legacy.ConfigurePinOut(csPin)
+			legacy.ConfigurePinOut(enPin)
+			legacy.ConfigurePinOut(rwPin)
+		},
 	}
 }
 
 // Configure initializes the display with default configuration
 func (d *Device) Configure(cfg Config) {
+	if d.configurePins == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
 	if cfg.Width == 0 {
 		cfg.Width = 128
 	}
@@ -73,11 +85,7 @@ func (d *Device) Configure(cfg Config) {
 	}
 
 	// configure GPIO pins
-	d.dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.resetPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.enPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.rwPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	d.configurePins()
 
 	// reset the device
 	d.resetPin.High()
@@ -278,7 +286,7 @@ func (d *Device) Data(data uint8) {
 
 // Tx sends data to the display
 func (d *Device) Tx(data []byte, isCommand bool) {
-	d.dcPin.Set(!isCommand)
+	d.dcPin(!isCommand)
 	d.csPin.Low()
 	d.bus.Tx(data, nil)
 	d.csPin.High()

st7735/st7735.go

@@ -5,12 +5,13 @@ package st7735 // import "tinygo.org/x/drivers/st7735"
 
 import (
 	"image/color"
-	"machine"
 	"time"
 
 	"errors"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 	"tinygo.org/x/drivers/pixel"
 )
 
@@ -39,10 +40,10 @@ type Device = DeviceOf[pixel.RGB565BE]
 // formats.
 type DeviceOf[T Color] struct {
 	bus          drivers.SPI
-	dcPin        machine.Pin
-	resetPin     machine.Pin
-	csPin        machine.Pin
-	blPin        machine.Pin
+	dcPin        drivers.PinOutput
+	resetPin     drivers.PinOutput
+	csPin        drivers.PinOutput
+	blPin        drivers.PinOutput
 	width        int16
 	height       int16
 	columnOffset int16
@@ -65,23 +66,23 @@ type Config struct {
 }
 
 // New creates a new ST7735 connection. The SPI wire must already be configured.
-func New(bus drivers.SPI, resetPin, dcPin, csPin, blPin machine.Pin) Device {
+func New(bus drivers.SPI, resetPin, dcPin, csPin, blPin pin.Output) Device {
 	return NewOf[pixel.RGB565BE](bus, resetPin, dcPin, csPin, blPin)
 }
 
 // NewOf creates a new ST7735 connection with a particular pixel format. The SPI
 // wire must already be configured.
-func NewOf[T Color](bus drivers.SPI, resetPin, dcPin, csPin, blPin machine.Pin) DeviceOf[T] {
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	resetPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	blPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+func NewOf[T Color](bus drivers.SPI, resetPin, dcPin, csPin, blPin pin.Output) DeviceOf[T] {
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(resetPin)
+	legacy.ConfigurePinOut(csPin)
+	legacy.ConfigurePinOut(blPin)
 	return DeviceOf[T]{
 		bus:      bus,
-		dcPin:    dcPin,
-		resetPin: resetPin,
-		csPin:    csPin,
-		blPin:    blPin,
+		dcPin:    dcPin.Set,
+		resetPin: resetPin.Set,
+		csPin:    csPin.Set,
+		blPin:    blPin.Set,
 	}
 }
 
@@ -423,7 +424,7 @@ func (d *DeviceOf[T]) Data(data uint8) {
 
 // Tx sends data to the display
 func (d *DeviceOf[T]) Tx(data []byte, isCommand bool) {
-	d.dcPin.Set(!isCommand)
+	d.dcPin(!isCommand)
 	d.bus.Tx(data, nil)
 }
 

st7789/st7789.go

@@ -7,13 +7,14 @@ package st7789 // import "tinygo.org/x/drivers/st7789"
 
 import (
 	"image/color"
-	"machine"
 	"math"
 	"time"
 
 	"errors"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 	"tinygo.org/x/drivers/pixel"
 )
 
@@ -46,10 +47,10 @@ type Device = DeviceOf[pixel.RGB565BE]
 // formats.
 type DeviceOf[T Color] struct {
 	bus             drivers.SPI
-	dcPin           machine.Pin
-	resetPin        machine.Pin
-	csPin           machine.Pin
-	blPin           machine.Pin
+	dcPin           drivers.PinOutput
+	resetPin        drivers.PinOutput
+	csPin           drivers.PinOutput
+	blPin           drivers.PinOutput
 	width           int16
 	height          int16
 	columnOffsetCfg int16
@@ -83,23 +84,27 @@ type Config struct {
 }
 
 // New creates a new ST7789 connection. The SPI wire must already be configured.
-func New(bus drivers.SPI, resetPin, dcPin, csPin, blPin machine.Pin) Device {
+func New(bus drivers.SPI, resetPin, dcPin, csPin, blPin pin.Output) Device {
 	return NewOf[pixel.RGB565BE](bus, resetPin, dcPin, csPin, blPin)
 }
 
 // NewOf creates a new ST7789 connection with a particular pixel format. The SPI
 // wire must already be configured.
-func NewOf[T Color](bus drivers.SPI, resetPin, dcPin, csPin, blPin machine.Pin) DeviceOf[T] {
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	resetPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	blPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
+func NewOf[T Color](bus drivers.SPI, resetPin, dcPin, csPin, blPin pin.Output) DeviceOf[T] {
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(resetPin)
+	legacy.ConfigurePinOut(csPin)
+	legacy.ConfigurePinOut(blPin)
+	var cs drivers.PinOutput
+	if !legacy.PinIsNoPin(csPin) {
+		cs = csPin.Set
+	}
 	return DeviceOf[T]{
 		bus:      bus,
-		dcPin:    dcPin,
-		resetPin: resetPin,
-		csPin:    csPin,
-		blPin:    blPin,
+		dcPin:    dcPin.Set,
+		resetPin: resetPin.Set,
+		csPin:    cs,
+		blPin:    blPin.Set,
 	}
 }
 
@@ -229,7 +234,7 @@ func (d *DeviceOf[T]) sendCommand(command uint8, data []byte) error {
 // startWrite must be called at the beginning of all exported methods to set the
 // chip select pin low.
 func (d *DeviceOf[T]) startWrite() {
-	if d.csPin != machine.NoPin {
+	if d.csPin != nil {
 		d.csPin.Low()
 	}
 }
@@ -237,7 +242,7 @@ func (d *DeviceOf[T]) startWrite() {
 // endWrite must be called at the end of all exported methods to set the chip
 // select pin high.
 func (d *DeviceOf[T]) endWrite() {
-	if d.csPin != machine.NoPin {
+	if d.csPin != nil {
 		d.csPin.High()
 	}
 }

sx127x/sx127x.go

@@ -6,10 +6,10 @@ package sx127x
 
 import (
 	"errors"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 	"tinygo.org/x/drivers/lora"
 )
 
@@ -22,7 +22,7 @@ const (
 // Device wraps an SPI connection to a SX127x device.
 type Device struct {
 	spi            drivers.SPI          // SPI bus for module communication
-	rstPin         machine.Pin          // GPIO for reset
+	rstPin         drivers.PinOutput    // GPIO for reset
 	radioEventChan chan lora.RadioEvent // Channel for Receiving events
 	loraConf       lora.Config          // Current Lora configuration
 	controller     RadioController      // to manage interactions with the radio
@@ -43,10 +43,10 @@ func (d *Device) GetRadioEventChan() chan lora.RadioEvent {
 }
 
 // New creates a new SX127x connection. The SPI bus must already be configured.
-func New(spi drivers.SPI, rstPin machine.Pin) *Device {
+func New(spi drivers.SPI, rstPin pin.Output) *Device {
 	k := Device{
 		spi:            spi,
-		rstPin:         rstPin,
+		rstPin:         rstPin.Set,
 		radioEventChan: make(chan lora.RadioEvent, RADIOEVENTCHAN_SIZE),
 		spiTxBuf:       make([]byte, SPI_BUFFER_SIZE),
 		spiRxBuf:       make([]byte, SPI_BUFFER_SIZE),

tmc2209/uartcomm.go

@@ -57,10 +57,14 @@ func (comm *UARTComm) WriteRegister(register uint8, value uint32, driverIndex ui
 		byte((value >> 16) & 0xFF), // Middle byte
 		byte((value >> 8) & 0xFF),  // Next byte
 		byte(value & 0xFF),         // LSB of value
-		0,                          // CRC
 	}
 
-	buffer[7] = CalculateCRC(buffer[:7])
+	// Calculate checksum by XORing all bytes
+	checksum := byte(0)
+	for _, b := range buffer[:7] {
+		checksum ^= b
+	}
+	buffer[7] = checksum // Set checksum byte
 
 	// Write the data to the TMC2209
 	done := make(chan error, 1)
@@ -82,10 +86,10 @@ func (comm *UARTComm) WriteRegister(register uint8, value uint32, driverIndex ui
 // ReadRegister sends a register read command to the TMC2209 with a timeout.
 func (comm *UARTComm) ReadRegister(register uint8, driverIndex uint8) (uint32, error) {
 	var writeBuffer [4]byte
-	writeBuffer[0] = 0x05            // Sync byte
-	writeBuffer[1] = 0x00            // Slave address
-	writeBuffer[2] = register & 0x7F // Read command (MSB clear for read)
-	writeBuffer[3] = CalculateCRC(writeBuffer[:3])
+	writeBuffer[0] = 0x05                                             // Sync byte
+	writeBuffer[1] = 0x00                                             // Slave address
+	writeBuffer[2] = register & 0x7F                                  // Read command (MSB clear for read)
+	writeBuffer[3] = writeBuffer[0] ^ writeBuffer[1] ^ writeBuffer[2] // Checksum
 
 	// Send the read command
 	done := make(chan []byte, 1)
@@ -99,7 +103,11 @@ func (comm *UARTComm) ReadRegister(register uint8, driverIndex uint8) (uint32, e
 	// Implementing timeout using a 100ms timer
 	select {
 	case readBuffer := <-done:
-		checksum := CalculateCRC(readBuffer[:7])
+		// Validate checksum
+		checksum := byte(0)
+		for i := 0; i < 7; i++ {
+			checksum ^= readBuffer[i]
+		}
 		if checksum != readBuffer[7] {
 			return 0, CustomError("checksum error")
 		}

uc8151/uc8151.go

@@ -32,10 +32,10 @@ type Config struct {
 
 type Device struct {
 	bus                      drivers.SPI
-	cs                       pin.OutputFunc
-	dc                       pin.OutputFunc
-	rst                      pin.OutputFunc
-	isBusy                   pin.InputFunc
+	cs                       drivers.PinOutput
+	dc                       drivers.PinOutput
+	rst                      drivers.PinOutput
+	isBusy                   drivers.PinInput
 	width                    int16
 	height                   int16
 	buffer                   []uint8
@@ -50,12 +50,7 @@ type Device struct {
 type Speed uint8
 
 // New returns a new uc8151 driver. Pass in a fully configured SPI bus.
-// Pins passed in must be configured beforehand.
 func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
-	// For backwards compatibility.
-	// This driver used to configure pins,
-	// so leave in to not break users.
-	// May be removed in future so try not to depend on it!
 	legacy.ConfigurePinOut(csPin)
 	legacy.ConfigurePinOut(dcPin)
 	legacy.ConfigurePinOut(rstPin)

version.go

@@ -2,4 +2,4 @@ package drivers
 
 // Version returns a user-readable string showing the version of the drivers package for support purposes.
 // Update this value before release of new version of software.
-const Version = "0.33.0"
+const Version = "0.30.0"

waveshare-epd/epd1in54/epd1in54.go

@@ -12,6 +12,10 @@ import (
 	"image/color"
 	"machine"
 	"time"
+
+	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Config struct {
@@ -22,14 +26,14 @@ type Config struct {
 }
 
 type Device struct {
-	bus  *machine.SPI
-	cs   machine.Pin
-	dc   machine.Pin
-	rst  machine.Pin
-	busy machine.Pin
-
-	buffer   []uint8
-	rotation Rotation
+	bus           *machine.SPI
+	cs            drivers.PinOutput
+	dc            drivers.PinOutput
+	rst           drivers.PinOutput
+	isBusy        drivers.PinInput
+	configurePins func()
+	buffer        []uint8
+	rotation      Rotation
 }
 
 type Rotation uint8
@@ -79,22 +83,28 @@ var partialRefresh = [159]uint8{
 }
 
 // New returns a new epd1in54 driver. Pass in a fully configured SPI bus.
-func New(bus *machine.SPI, csPin, dcPin, rstPin, busyPin machine.Pin) Device {
+func New(bus *machine.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
 	return Device{
 		buffer: make([]uint8, (uint32(Width)*uint32(Height))/8),
 		bus:    bus,
-		cs:     csPin,
-		dc:     dcPin,
-		rst:    rstPin,
-		busy:   busyPin,
+		cs:     csPin.Set,
+		dc:     dcPin.Set,
+		rst:    rstPin.Set,
+		isBusy: busyPin.Get,
+		configurePins: func() {
+			legacy.ConfigurePinOut(csPin)
+			legacy.ConfigurePinOut(dcPin)
+			legacy.ConfigurePinOut(rstPin)
+			legacy.ConfigurePinInput(busyPin)
+		},
 	}
 }
 
 func (d *Device) LDirInit(cfg Config) {
-	d.cs.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.rst.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.dc.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.busy.Configure(machine.PinConfig{Mode: machine.PinInput})
+	if d.configurePins == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
+	d.configurePins()
 
 	d.bus.Configure(machine.SPIConfig{
 		Frequency: 2000000,
@@ -150,10 +160,10 @@ func (d *Device) LDirInit(cfg Config) {
 }
 
 func (d *Device) HDirInit(cfg Config) {
-	d.cs.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.rst.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.dc.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.busy.Configure(machine.PinConfig{Mode: machine.PinInput})
+	if d.configurePins == nil {
+		panic(legacy.ErrConfigBeforeInstantiated)
+	}
+	d.configurePins()
 
 	d.bus.Configure(machine.SPIConfig{
 		Frequency: 2000000,
@@ -369,7 +379,7 @@ func (d *Device) Clear() {
 
 // WaitUntilIdle waits until the display is ready
 func (d *Device) WaitUntilIdle() {
-	for d.busy.Get() {
+	for d.isBusy() {
 		time.Sleep(100 * time.Millisecond)
 	}
 	time.Sleep(200 * time.Millisecond)
@@ -377,7 +387,7 @@ func (d *Device) WaitUntilIdle() {
 
 // IsBusy returns the busy status of the display
 func (d *Device) IsBusy() bool {
-	return d.busy.Get()
+	return d.isBusy()
 }
 
 // ClearBuffer sets the buffer to 0xFF (white)

waveshare-epd/epd2in13/epd2in13.go

@@ -6,10 +6,11 @@ package epd2in13 // import "tinygo.org/x/drivers/waveshare-epd/epd2in13"
 import (
 	"errors"
 	"image/color"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Config struct {
@@ -21,10 +22,10 @@ type Config struct {
 
 type Device struct {
 	bus          drivers.SPI
-	cs           machine.Pin
-	dc           machine.Pin
-	rst          machine.Pin
-	busy         machine.Pin
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	logicalWidth int16
 	width        int16
 	height       int16
@@ -53,17 +54,17 @@ var lutPartialUpdate = [30]uint8{
 }
 
 // New returns a new epd2in13x driver. Pass in a fully configured SPI bus.
-func New(bus drivers.SPI, csPin, dcPin, rstPin, busyPin machine.Pin) Device {
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	rstPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	busyPin.Configure(machine.PinConfig{Mode: machine.PinInput})
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
+	legacy.ConfigurePinOut(csPin)
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(rstPin)
+	legacy.ConfigurePinInput(busyPin)
 	return Device{
-		bus:  bus,
-		cs:   csPin,
-		dc:   dcPin,
-		rst:  rstPin,
-		busy: busyPin,
+		bus:    bus,
+		cs:     csPin.Set,
+		dc:     dcPin.Set,
+		rst:    rstPin.Set,
+		isBusy: busyPin.Get,
 	}
 }
 
@@ -298,14 +299,14 @@ func (d *Device) setMemoryPointer(x int16, y int16) {
 
 // WaitUntilIdle waits until the display is ready
 func (d *Device) WaitUntilIdle() {
-	for d.busy.Get() {
+	for d.isBusy() {
 		time.Sleep(100 * time.Millisecond)
 	}
 }
 
 // IsBusy returns the busy status of the display
 func (d *Device) IsBusy() bool {
-	return d.busy.Get()
+	return d.isBusy()
 }
 
 // ClearBuffer sets the buffer to 0xFF (white)

waveshare-epd/epd2in13x/epd2in13x.go

@@ -6,10 +6,11 @@ package epd2in13x // import "tinygo.org/x/drivers/waveshare-epd/epd2in13x"
 import (
 	"errors"
 	"image/color"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Config struct {
@@ -20,10 +21,10 @@ type Config struct {
 
 type Device struct {
 	bus          drivers.SPI
-	cs           machine.Pin
-	dc           machine.Pin
-	rst          machine.Pin
-	busy         machine.Pin
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	width        int16
 	height       int16
 	buffer       [][]uint8
@@ -33,17 +34,17 @@ type Device struct {
 type Color uint8
 
 // New returns a new epd2in13x driver. Pass in a fully configured SPI bus.
-func New(bus drivers.SPI, csPin, dcPin, rstPin, busyPin machine.Pin) Device {
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	rstPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	busyPin.Configure(machine.PinConfig{Mode: machine.PinInput})
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
+	legacy.ConfigurePinOut(csPin)
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(rstPin)
+	legacy.ConfigurePinInput(busyPin)
 	return Device{
-		bus:  bus,
-		cs:   csPin,
-		dc:   dcPin,
-		rst:  rstPin,
-		busy: busyPin,
+		bus:    bus,
+		cs:     csPin.Set,
+		dc:     dcPin.Set,
+		rst:    rstPin.Set,
+		isBusy: busyPin.Get,
 	}
 }
 
@@ -277,14 +278,14 @@ func (d *Device) ClearDisplay() {
 
 // WaitUntilIdle waits until the display is ready
 func (d *Device) WaitUntilIdle() {
-	for !d.busy.Get() {
+	for !d.isBusy() {
 		time.Sleep(100 * time.Millisecond)
 	}
 }
 
 // IsBusy returns the busy status of the display
 func (d *Device) IsBusy() bool {
-	return d.busy.Get()
+	return d.isBusy()
 }
 
 // ClearBuffer sets the buffer to 0xFF (white)

waveshare-epd/epd2in66b/dev.go

@@ -5,7 +5,6 @@ package epd2in66b
 
 import (
 	"image/color"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
@@ -18,19 +17,12 @@ const (
 
 const Baudrate = 4_000_000 // 4 MHz
 
-type Config struct {
-	ResetPin      machine.Pin
-	DataPin       machine.Pin
-	ChipSelectPin machine.Pin
-	BusyPin       machine.Pin
-}
-
 type Device struct {
-	bus  drivers.SPI
-	cs   machine.Pin
-	dc   machine.Pin
-	rst  machine.Pin
-	busy machine.Pin
+	bus    drivers.SPI
+	cs     drivers.PinOutput
+	dc     drivers.PinOutput
+	rst    drivers.PinOutput
+	isBusy drivers.PinInput
 
 	blackBuffer []byte
 	redBuffer   []byte
@@ -50,21 +42,6 @@ func New(bus drivers.SPI) Device {
 	}
 }
 
-// Configure configures the device and its pins.
-func (d *Device) Configure(c Config) error {
-	d.cs = c.ChipSelectPin
-	d.dc = c.DataPin
-	d.rst = c.ResetPin
-	d.busy = c.BusyPin
-
-	d.cs.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.dc.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.rst.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	d.busy.Configure(machine.PinConfig{Mode: machine.PinInput})
-
-	return nil
-}
-
 func (d *Device) Size() (x, y int16) {
 	return displayWidth, displayHeight
 }
@@ -229,7 +206,7 @@ func (d *Device) WaitUntilIdle() {
 	// give it some time to get busy
 	time.Sleep(50 * time.Millisecond)
 
-	for d.busy.Get() { // high = busy
+	for d.isBusy() { // high = busy
 		time.Sleep(10 * time.Millisecond)
 	}
 

waveshare-epd/epd2in66b/dev_baremetal.go

@@ -0,0 +1,30 @@
+//go:build baremetal
+
+package epd2in66b
+
+import "machine"
+
+type Config struct {
+	ResetPin      machine.Pin
+	DataPin       machine.Pin
+	ChipSelectPin machine.Pin
+	BusyPin       machine.Pin
+}
+
+// Configure configures the device and its pins.
+func (d *Device) Configure(c Config) error {
+	cs := c.ChipSelectPin
+	dc := c.DataPin
+	rst := c.ResetPin
+	busy := c.BusyPin
+
+	cs.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	dc.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	rst.Configure(machine.PinConfig{Mode: machine.PinOutput})
+	busy.Configure(machine.PinConfig{Mode: machine.PinInput})
+	d.cs = cs.Set
+	d.dc = dc.Set
+	d.rst = rst.Set
+	d.isBusy = busy.Get
+	return nil
+}

waveshare-epd/epd2in9/epd2in9.go

@@ -13,10 +13,11 @@ package epd2in9 // import "tinygo.org/x/drivers/waveshare-epd/epd2in9"
 
 import (
 	"image/color"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Config struct {
@@ -28,10 +29,10 @@ type Config struct {
 
 type Device struct {
 	bus          drivers.SPI
-	cs           machine.Pin
-	dc           machine.Pin
-	rst          machine.Pin
-	busy         machine.Pin
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	logicalWidth int16
 	width        int16
 	height       int16
@@ -61,17 +62,17 @@ var lutPartialUpdate = [30]uint8{
 }
 
 // New returns a new epd2in9 driver. Pass in a fully configured SPI bus.
-func New(bus drivers.SPI, csPin, dcPin, rstPin, busyPin machine.Pin) Device {
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	rstPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	busyPin.Configure(machine.PinConfig{Mode: machine.PinInput})
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
+	legacy.ConfigurePinOut(csPin)
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(rstPin)
+	legacy.ConfigurePinInput(busyPin)
 	return Device{
-		bus:  bus,
-		cs:   csPin,
-		dc:   dcPin,
-		rst:  rstPin,
-		busy: busyPin,
+		bus:    bus,
+		cs:     csPin.Set,
+		dc:     dcPin.Set,
+		rst:    rstPin.Set,
+		isBusy: busyPin.Get,
 	}
 }
 
@@ -245,14 +246,14 @@ func (d *Device) setMemoryPointer(x int16, y int16) {
 
 // WaitUntilIdle waits until the display is ready
 func (d *Device) WaitUntilIdle() {
-	for d.busy.Get() {
+	for d.isBusy() {
 		time.Sleep(100 * time.Millisecond)
 	}
 }
 
 // IsBusy returns the busy status of the display
 func (d *Device) IsBusy() bool {
-	return d.busy.Get()
+	return d.isBusy()
 }
 
 // ClearBuffer sets the buffer to 0xFF (white)

waveshare-epd/epd4in2/epd4in2.go

@@ -10,10 +10,11 @@ package epd4in2
 
 import (
 	"image/color"
-	"machine"
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Config struct {
@@ -25,10 +26,10 @@ type Config struct {
 
 type Device struct {
 	bus          drivers.SPI
-	cs           machine.Pin
-	dc           machine.Pin
-	rst          machine.Pin
-	busy         machine.Pin
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	logicalWidth int16
 	width        int16
 	height       int16
@@ -40,17 +41,17 @@ type Device struct {
 type Rotation uint8
 
 // New returns a new epd4in2 driver. Pass in a fully configured SPI bus.
-func New(bus drivers.SPI, csPin, dcPin, rstPin, busyPin machine.Pin) Device {
-	csPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	dcPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	rstPin.Configure(machine.PinConfig{Mode: machine.PinOutput})
-	busyPin.Configure(machine.PinConfig{Mode: machine.PinInput})
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
+	legacy.ConfigurePinOut(csPin)
+	legacy.ConfigurePinOut(dcPin)
+	legacy.ConfigurePinOut(rstPin)
+	legacy.ConfigurePinInput(busyPin)
 	return Device{
-		bus:  bus,
-		cs:   csPin,
-		dc:   dcPin,
-		rst:  rstPin,
-		busy: busyPin,
+		bus:    bus,
+		cs:     csPin.Set,
+		dc:     dcPin.Set,
+		rst:    rstPin.Set,
+		isBusy: busyPin.Get,
 	}
 }
 
@@ -311,14 +312,14 @@ func (d *Device) ClearDisplay() {
 
 // WaitUntilIdle waits until the display is ready
 func (d *Device) WaitUntilIdle() {
-	for d.busy.Get() {
+	for d.isBusy() {
 		time.Sleep(100 * time.Millisecond)
 	}
 }
 
 // IsBusy returns the busy status of the display
 func (d *Device) IsBusy() bool {
-	return d.busy.Get()
+	return d.isBusy()
 }
 
 // ClearBuffer sets the buffer to 0xFF (white)

ws2812/ws2812-asm_cortexm.go

@@ -931,356 +931,6 @@ void ws2812_writeByte125(char c, uint32_t *portSet, uint32_t *portClear, uint32_
 	  [portClear]"m"(*portClear));
 }
 
-__attribute__((always_inline))
-void ws2812_writeByte150(char c, uint32_t *portSet, uint32_t *portClear, uint32_t maskSet, uint32_t maskClear) {
-	// Timings:
-	// T0H: 53 - 55 cycles or 353.3ns - 366.7ns
-	// T1H: 158 - 160 cycles or 1053.3ns - 1066.7ns
-	// TLD: 173 -    cycles or 1153.3ns -
-	uint32_t value = (uint32_t)c << 24;
-	char i = 8;
-	__asm__ __volatile__(
-		"1: @ send_bit\n"
-		"\t  str   %[maskSet], %[portSet]     @ [2]   T0H and T0L start here\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  lsls  %[value], #1               @ [1]\n"
-		"\t  bcs.n 2f                         @ [1/3] skip_store\n"
-		"\t  str   %[maskClear], %[portClear] @ [2]   T0H -> T0L transition\n"
-		"\t2: @ skip_store\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  str   %[maskClear], %[portClear] @ [2]   T1H -> T1L transition\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  subs  %[i], #1                   @ [1]\n"
-		"\t  beq.n 3f                         @ [1/3] end\n"
-		"\t  b     1b                         @ [1/3] send_bit\n"
-		"\t3: @ end\n"
-	: [value]"+r"(value),
-	  [i]"+r"(i)
-	: [maskSet]"r"(maskSet),
-	  [portSet]"m"(*portSet),
-	  [maskClear]"r"(maskClear),
-	  [portClear]"m"(*portClear));
-}
-
 __attribute__((always_inline))
 void ws2812_writeByte168(char c, uint32_t *portSet, uint32_t *portClear, uint32_t maskSet, uint32_t maskClear) {
 	// Timings:
@@ -1670,465 +1320,6 @@ void ws2812_writeByte168(char c, uint32_t *portSet, uint32_t *portClear, uint32_
 	  [maskClear]"r"(maskClear),
 	  [portClear]"m"(*portClear));
 }
-
-__attribute__((always_inline))
-void ws2812_writeByte200(char c, uint32_t *portSet, uint32_t *portClear, uint32_t maskSet, uint32_t maskClear) {
-	// Timings:
-	// T0H: 70 - 72 cycles or 350.0ns - 360.0ns
-	// T1H: 210 - 212 cycles or 1050.0ns - 1060.0ns
-	// TLD: 230 -    cycles or 1150.0ns -
-	uint32_t value = (uint32_t)c << 24;
-	char i = 8;
-	__asm__ __volatile__(
-		"1: @ send_bit\n"
-		"\t  str   %[maskSet], %[portSet]     @ [2]   T0H and T0L start here\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  lsls  %[value], #1               @ [1]\n"
-		"\t  bcs.n 2f                         @ [1/3] skip_store\n"
-		"\t  str   %[maskClear], %[portClear] @ [2]   T0H -> T0L transition\n"
-		"\t2: @ skip_store\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  str   %[maskClear], %[portClear] @ [2]   T1H -> T1L transition\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  nop\n"
-		"\t  subs  %[i], #1                   @ [1]\n"
-		"\t  beq.n 3f                         @ [1/3] end\n"
-		"\t  b     1b                         @ [1/3] send_bit\n"
-		"\t3: @ end\n"
-	: [value]"+r"(value),
-	  [i]"+r"(i)
-	: [maskSet]"r"(maskSet),
-	  [portSet]"m"(*portSet),
-	  [maskClear]"r"(maskClear),
-	  [portClear]"m"(*portClear));
-}
 */
 import "C"
 
@@ -2182,16 +1373,6 @@ func (d Device) writeByte125(c byte) {
 	interrupt.Restore(mask)
 }
 
-func (d Device) writeByte150(c byte) {
-	portSet, maskSet := d.Pin.PortMaskSet()
-	portClear, maskClear := d.Pin.PortMaskClear()
-
-	mask := interrupt.Disable()
-	C.ws2812_writeByte150(C.char(c), (*C.uint32_t)(unsafe.Pointer(portSet)), (*C.uint32_t)(unsafe.Pointer(portClear)), C.uint32_t(maskSet), C.uint32_t(maskClear))
-
-	interrupt.Restore(mask)
-}
-
 func (d Device) writeByte168(c byte) {
 	portSet, maskSet := d.Pin.PortMaskSet()
 	portClear, maskClear := d.Pin.PortMaskClear()
@@ -2201,13 +1382,3 @@ func (d Device) writeByte168(c byte) {
 
 	interrupt.Restore(mask)
 }
-
-func (d Device) writeByte200(c byte) {
-	portSet, maskSet := d.Pin.PortMaskSet()
-	portClear, maskClear := d.Pin.PortMaskClear()
-
-	mask := interrupt.Disable()
-	C.ws2812_writeByte200(C.char(c), (*C.uint32_t)(unsafe.Pointer(portSet)), (*C.uint32_t)(unsafe.Pointer(portClear)), C.uint32_t(maskSet), C.uint32_t(maskClear))
-
-	interrupt.Restore(mask)
-}

ws2812/ws2812.go

@@ -1,7 +1,7 @@
 // Package ws2812 implements a driver for WS2812 and SK6812 RGB LED strips.
 package ws2812 // import "tinygo.org/x/drivers/ws2812"
 
-//go:generate go run gen-ws2812.go -arch=cortexm 16 48 64 120 125 150 168 200
+//go:generate go run gen-ws2812.go -arch=cortexm 16 48 64 120 125 168
 //go:generate go run gen-ws2812.go -arch=tinygoriscv 160 320
 
 import (

ws2812/ws2812_cortexm.go

@@ -28,18 +28,12 @@ func (d Device) WriteByte(c byte) error {
 	case 120_000_000: // 120MHz
 		d.writeByte120(c)
 		return nil
-	case 125_000_000: // 125 MHz e.g. rp2040 originally
+	case 125_000_000: // 125 MHz e.g. rp2040
 		d.writeByte125(c)
 		return nil
-	case 150_000_000: // 150MHz, e.g. rp2350
-		d.writeByte150(c)
-		return nil
 	case 168_000_000: // 168MHz, e.g. stm32f405
 		d.writeByte168(c)
 		return nil
-	case 200_000_000: // 200MHz, e.g. rp2040 starting with TinyGo v0.37
-		d.writeByte200(c)
-		return nil
 	default:
 		return errUnknownClockSpeed
 	}