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

@@ -8,6 +8,7 @@ import (
 
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 const (
@@ -37,7 +38,7 @@ 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 drivers.PinOutput, delay uint32) *Device {
+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)

apa102/softspi.go

@@ -11,8 +11,8 @@ import (
 // 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           drivers.PinOutputFunc
-	SDO           drivers.PinOutputFunc
+	SCK           drivers.PinOutput
+	SDO           drivers.PinOutput
 	Delay         uint32
 	configurePins func()
 }

bmi160/bmi160.go

@@ -5,13 +5,14 @@ import (
 
 	"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 drivers.PinOutputFunc
+	csb drivers.PinOutput
 
 	buf [7]byte
 
@@ -23,7 +24,7 @@ type DeviceSPI struct {
 // 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 drivers.PinOutput, spi drivers.SPI) *DeviceSPI {
+func NewSPI(csb pin.Output, spi drivers.SPI) *DeviceSPI {
 	return &DeviceSPI{
 		csb: csb.Set, // chip select
 		bus: spi,

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

@@ -5,17 +5,18 @@ import (
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // Device wraps a GPIO connection to a buzzer.
 type Device struct {
-	pin  drivers.PinOutputFunc
+	pin  drivers.PinOutput
 	High bool
 	BPM  float64
 }
 
 // New returns a new buzzer driver given which pin to use
-func New(pin drivers.PinOutput) Device {
+func New(pin pin.Output) Device {
 	return Device{
 		pin:  pin.Set,
 		High: false,

easystepper/easystepper.go

@@ -32,7 +32,7 @@ func (sm StepMode) stepCount() uint {
 
 // Device holds the pins and the delay between steps
 type Device struct {
-	pins       [4]drivers.PinOutputFunc
+	pins       [4]drivers.PinOutput
 	config     func()
 	stepDelay  time.Duration
 	stepNumber uint8

easystepper/easystepper_go.go

@@ -7,7 +7,7 @@ import (
 	"tinygo.org/x/drivers"
 )
 
-func NewCrossPlatform(stepcount, rpm uint, mode StepMode, pins [4]drivers.PinOutputFunc) (*Device, error) {
+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")
 	}

easystepper/easystepper_tinygo.go

@@ -17,7 +17,7 @@ func New(config DeviceConfig) (*Device, error) {
 		return nil, errors.New("config.StepCount and config.RPM must be > 0")
 	}
 	return &Device{
-		pins:      [4]drivers.PinOutputFunc{config.Pin1.Set, config.Pin2.Set, config.Pin3.Set, config.Pin4.Set},
+		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() {

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/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

@@ -7,6 +7,7 @@ package ft6336
 import (
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 	"tinygo.org/x/drivers/touch"
 )
 
@@ -19,7 +20,7 @@ type Device struct {
 }
 
 // New returns FT6336 device for the provided I2C bus using default address.
-func New(i2c drivers.I2C, intPin drivers.PinInput) *Device {
+func New(i2c drivers.I2C, intPin pin.Input) *Device {
 	return &Device{
 		bus:     i2c,
 		buf:     make([]byte, 11),

gc9a01/gc9a01.go

@@ -11,6 +11,7 @@ import (
 
 	"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           drivers.PinOutputFunc
-	resetPin        drivers.PinOutputFunc
-	csPin           drivers.PinOutputFunc
-	blPin           drivers.PinOutputFunc
+	dcPin           drivers.PinOutput
+	resetPin        drivers.PinOutput
+	csPin           drivers.PinOutput
+	blPin           drivers.PinOutput
 	width           int16
 	height          int16
 	columnOffsetCfg int16
@@ -52,7 +53,7 @@ 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 drivers.PinOutput) Device {
+func New(bus drivers.SPI, resetPin, dcPin, csPin, blPin pin.Output) Device {
 	legacy.ConfigurePinOut(resetPin)
 	legacy.ConfigurePinOut(dcPin)
 	legacy.ConfigurePinOut(csPin)

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

@@ -9,19 +9,20 @@ import (
 
 	"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       drivers.PinOutputFunc
-	echo          drivers.PinInputFunc
+	trigger       drivers.PinOutput
+	echo          drivers.PinInput
 	configurePins func()
 }
 
 // New returns a new ultrasonic driver given 2 pins
-func New(trigger drivers.PinOutput, echo drivers.PinInput) Device {
+func New(trigger pin.Output, echo pin.Input) Device {
 	return Device{
 		trigger: trigger.Set,
 		echo:    echo.Get,

internal/legacy/pinhal.go

@@ -3,7 +3,7 @@ package legacy
 import (
 	"errors"
 
-	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // ConfigurePinOut is a legacy function used to configure pins as outputs.
@@ -11,7 +11,7 @@ import (
 // Deprecated: Do not configure pins in drivers.
 // This is a legacy feature and should only be used by drivers that
 // previously configured pins in initialization to avoid breaking users.
-func ConfigurePinOut(po drivers.PinOutput) {
+func ConfigurePinOut(po pin.Output) {
 	configurePinOut(po)
 }
 
@@ -20,7 +20,7 @@ func ConfigurePinOut(po drivers.PinOutput) {
 // Deprecated: Do not configure pins in drivers.
 // This is a legacy feature and should only be used by drivers that
 // previously configured pins in initialization to avoid breaking users.
-func ConfigurePinInputPulldown(pi drivers.PinInput) {
+func ConfigurePinInputPulldown(pi pin.Input) {
 	configurePinInputPulldown(pi)
 }
 
@@ -29,7 +29,7 @@ func ConfigurePinInputPulldown(pi drivers.PinInput) {
 // Deprecated: Do not configure pins in drivers.
 // This is a legacy feature and should only be used by drivers that
 // previously configured pins in initialization to avoid breaking users.
-func ConfigurePinInput(pi drivers.PinInput) {
+func ConfigurePinInput(pi pin.Input) {
 	configurePinInput(pi)
 }
 
@@ -38,7 +38,7 @@ func ConfigurePinInput(pi drivers.PinInput) {
 // Deprecated: Do not configure pins in drivers.
 // This is a legacy feature and should only be used by drivers that
 // previously configured pins in initialization to avoid breaking users.
-func ConfigurePinInputPullup(pi drivers.PinInput) {
+func ConfigurePinInputPullup(pi pin.Input) {
 	configurePinInputPullup(pi)
 }
 

internal/legacy/pinhal_go.go

@@ -2,12 +2,10 @@
 
 package legacy
 
-import (
-	"tinygo.org/x/drivers"
-)
+import "tinygo.org/x/drivers/internal/pin"
 
-func configurePinOut(p drivers.PinOutput)          {}
-func configurePinInput(p drivers.PinInput)         {}
-func configurePinInputPulldown(p drivers.PinInput) {}
-func configurePinInputPullup(p drivers.PinInput)   {}
-func pinIsNoPin(a any) bool                        { return false }
+func configurePinOut(p pin.Output)          {}
+func configurePinInput(p pin.Input)         {}
+func configurePinInputPulldown(p pin.Input) {}
+func configurePinInputPullup(p pin.Input)   {}
+func pinIsNoPin(a any) bool                 { return false }

internal/legacy/pinhal_tinygo.go

@@ -5,22 +5,22 @@ package legacy
 import (
 	"machine"
 
-	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
-func configurePinOut(po drivers.PinOutput) {
+func configurePinOut(po pin.Output) {
 	configurePin(po, machine.PinOutput)
 }
 
-func configurePinInputPulldown(pi drivers.PinInput) {
+func configurePinInputPulldown(pi pin.Input) {
 	configurePin(pi, pulldown) // some chips do not have pull down, in which case pulldown==machine.PinInput.
 }
 
-func configurePinInput(pi drivers.PinInput) {
+func configurePinInput(pi pin.Input) {
 	configurePin(pi, machine.PinInput)
 }
 
-func configurePinInputPullup(pi drivers.PinInput) {
+func configurePinInputPullup(pi pin.Input) {
 	configurePin(pi, pullup) // some chips do not have pull up, in which case pullup==machine.PinInput.
 }
 

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/devices.go

@@ -1,121 +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.
-type Device8 struct {
-	buf [10]byte
-}
-
-// clear zeroes Device8's buffers.
-func (d *Device8) clear() {
-	d.buf = [10]byte{}
-}
-
-// I2C methods.
-
-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
-}
-
-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
-}
-
-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
-}
-
-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)
-}
-
-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)
-}
-
-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)
-}
-
-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.
-
-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
-}
-
-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
-}
-
-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
-}
-
-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)
-}
-
-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)
-}
-
-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)
-}

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)
-}

max6675/max6675.go

@@ -5,6 +5,7 @@ import (
 	"errors"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // ErrThermocoupleOpen is returned when the thermocouple input is open.
@@ -13,13 +14,13 @@ var ErrThermocoupleOpen = errors.New("thermocouple input open")
 
 type Device struct {
 	bus drivers.SPI
-	cs  drivers.PinOutputFunc
+	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 drivers.PinOutput) *Device {
+func NewDevice(bus drivers.SPI, cs pin.Output) *Device {
 	return &Device{
 		bus: bus,
 		cs:  cs.Set,

max72xx/max72xx.go

@@ -5,18 +5,19 @@ package max72xx
 import (
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Device struct {
 	bus           drivers.SPI
-	cs            drivers.PinOutputFunc
+	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 drivers.PinOutput) *Device {
+func NewDevice(bus drivers.SPI, cs pin.Output) *Device {
 	return &Device{
 		bus: bus,
 		cs:  cs.Set,

mcp2515/mcp2515.go

@@ -12,12 +12,13 @@ import (
 
 	"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            drivers.PinOutputFunc
+	cs            drivers.PinOutput
 	msg           *CANMsg
 	mcpMode       byte
 	configurePins func()
@@ -37,7 +38,7 @@ const (
 )
 
 // New returns a new MCP2515 driver. Pass in a fully configured SPI bus.
-func New(b drivers.SPI, csPin drivers.PinOutput) *Device {
+func New(b drivers.SPI, csPin pin.Output) *Device {
 	d := &Device{
 		spi: SPI{
 			bus: b,

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

@@ -9,14 +9,15 @@ import (
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 // Device wraps an SPI connection.
 type Device struct {
 	bus        drivers.SPI
-	dcPin      drivers.PinOutputFunc
-	rstPin     drivers.PinOutputFunc
-	scePin     drivers.PinOutputFunc
+	dcPin      drivers.PinOutput
+	rstPin     drivers.PinOutput
+	scePin     drivers.PinOutput
 	buffer     []byte
 	width      int16
 	height     int16
@@ -29,7 +30,7 @@ type Config struct {
 }
 
 // New creates a new PCD8544 connection. The SPI bus must already be configured.
-func New(bus drivers.SPI, dcPin, rstPin, scePin drivers.PinOutput) *Device {
+func New(bus drivers.SPI, dcPin, rstPin, scePin pin.Output) *Device {
 	return &Device{
 		bus:    bus,
 		dcPin:  dcPin.Set,

pin.go

@@ -1,39 +1,29 @@
 package drivers
 
-// PinOutput represents a pin hardware abstraction layer for a pin that can output a digital signal.
-type PinOutput interface {
-	Set(level bool)
-}
-
-// PinInput represents a pin hardware abstraction layer.
-type PinInput interface {
-	Get() (level bool)
-}
-
-// PinOutputFunc is hardware abstraction for a function that causes pin to output a
+// PinOutput is hardware abstraction for a pin which outputs a
 // digital signal (high or low level).
 //
-//	// Code conversion demo: from machine.Pin to drivers.PinOutputFunc
+//	// Code conversion demo: from machine.Pin to drivers.PinOutput
 //	led := machine.LED
 //	led.Configure(machine.PinConfig{Mode: machine.PinOutput})
-//	var pin drivers.PinOutputFunc = led.Set // Going from a machine.Pin to a drivers.PinOutputFunc
-type PinOutputFunc func(level bool)
+//	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 PinOutputFunc) High() {
+func (po PinOutput) High() {
 	po(true)
 }
 
 // Low sets the underlying pin's level to low. This is equivalent to calling PinOutput(false).
-func (po PinOutputFunc) Low() {
+func (po PinOutput) Low() {
 	po(false)
 }
 
-// PinInputFunc is hardware abstraction for a pin which receives a
+// 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.PinInputFunc
+//	// 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.PinInputFunc = input.Get // Going from a machine.Pin to a drivers.PinInputFunc
-type PinInputFunc func() (level bool)
+//	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

@@ -4,6 +4,7 @@ package shiftregister
 import (
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type NumberBit int8
@@ -17,7 +18,7 @@ const (
 
 // Device holds pin number
 type Device struct {
-	latch, clock, out drivers.PinOutputFunc // IC wiring
+	latch, clock, out drivers.PinOutput // IC wiring
 	config            func()
 	bits              NumberBit // Pin number
 	mask              uint32    // keep all pins state
@@ -31,7 +32,7 @@ type ShiftPin struct {
 }
 
 // New returns a new shift output register device
-func New(Bits NumberBit, Latch, Clock, Out drivers.PinOutput) *Device {
+func New(Bits NumberBit, Latch, Clock, Out pin.Output) *Device {
 	return &Device{
 		latch: Latch.Set,
 		clock: Clock.Set,

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/ssd1289.go

@@ -17,10 +17,10 @@ type Bus interface {
 }
 
 type Device struct {
-	rs  drivers.PinOutputFunc
-	wr  drivers.PinOutputFunc
-	cs  drivers.PinOutputFunc
-	rst drivers.PinOutputFunc
+	rs  drivers.PinOutput
+	wr  drivers.PinOutput
+	cs  drivers.PinOutput
+	rst drivers.PinOutput
 	bus Bus
 }
 

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

@@ -20,9 +20,9 @@ type Rotation uint8
 // Device wraps an SPI connection.
 type Device struct {
 	bus         drivers.SPI
-	dcPin       drivers.PinOutputFunc
-	resetPin    drivers.PinOutputFunc
-	csPin       drivers.PinOutputFunc
+	dcPin       drivers.PinOutput
+	resetPin    drivers.PinOutput
+	csPin       drivers.PinOutput
 	width       int16
 	height      int16
 	batchLength int16

ssd1351/ssd1351.go

@@ -10,6 +10,7 @@ import (
 
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 var (
@@ -20,11 +21,11 @@ var (
 // Device wraps an SPI connection.
 type Device struct {
 	bus           drivers.SPI
-	dcPin         drivers.PinOutputFunc
-	resetPin      drivers.PinOutputFunc
-	csPin         drivers.PinOutputFunc
-	enPin         drivers.PinOutputFunc
-	rwPin         drivers.PinOutputFunc
+	dcPin         drivers.PinOutput
+	resetPin      drivers.PinOutput
+	csPin         drivers.PinOutput
+	enPin         drivers.PinOutput
+	rwPin         drivers.PinOutput
 	configurePins func()
 	width         int16
 	height        int16
@@ -42,7 +43,7 @@ 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 drivers.PinOutput) Device {
+func New(bus drivers.SPI, resetPin, dcPin, csPin, enPin, rwPin pin.Output) Device {
 	return Device{
 		bus:      bus,
 		dcPin:    dcPin.Set,

st7735/st7735.go

@@ -11,6 +11,7 @@ import (
 
 	"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        drivers.PinOutputFunc
-	resetPin     drivers.PinOutputFunc
-	csPin        drivers.PinOutputFunc
-	blPin        drivers.PinOutputFunc
+	dcPin        drivers.PinOutput
+	resetPin     drivers.PinOutput
+	csPin        drivers.PinOutput
+	blPin        drivers.PinOutput
 	width        int16
 	height       int16
 	columnOffset int16
@@ -65,13 +66,13 @@ 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 drivers.PinOutput) 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 drivers.PinOutput) DeviceOf[T] {
+func NewOf[T Color](bus drivers.SPI, resetPin, dcPin, csPin, blPin pin.Output) DeviceOf[T] {
 	legacy.ConfigurePinOut(dcPin)
 	legacy.ConfigurePinOut(resetPin)
 	legacy.ConfigurePinOut(csPin)

st7789/st7789.go

@@ -14,6 +14,7 @@ import (
 
 	"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           drivers.PinOutputFunc
-	resetPin        drivers.PinOutputFunc
-	csPin           drivers.PinOutputFunc
-	blPin           drivers.PinOutputFunc
+	dcPin           drivers.PinOutput
+	resetPin        drivers.PinOutput
+	csPin           drivers.PinOutput
+	blPin           drivers.PinOutput
 	width           int16
 	height          int16
 	columnOffsetCfg int16
@@ -83,18 +84,18 @@ 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 drivers.PinOutput) 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 drivers.PinOutput) DeviceOf[T] {
+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.PinOutputFunc
+	var cs drivers.PinOutput
 	if !legacy.PinIsNoPin(csPin) {
 		cs = csPin.Set
 	}

sx127x/sx127x.go

@@ -9,6 +9,7 @@ import (
 	"time"
 
 	"tinygo.org/x/drivers"
+	"tinygo.org/x/drivers/internal/pin"
 	"tinygo.org/x/drivers/lora"
 )
 
@@ -20,15 +21,15 @@ const (
 
 // Device wraps an SPI connection to a SX127x device.
 type Device struct {
-	spi            drivers.SPI           // SPI bus for module communication
-	rstPin         drivers.PinOutputFunc // 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
-	deepSleep      bool                  // Internal Sleep state
-	deviceType     int                   // sx1261,sx1262,sx1268 (defaults sx1261)
-	spiTxBuf       []byte                // global Tx buffer to avoid heap allocations in interrupt
-	spiRxBuf       []byte                // global Rx buffer to avoid heap allocations in interrupt
+	spi            drivers.SPI          // SPI bus for module communication
+	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
+	deepSleep      bool                 // Internal Sleep state
+	deviceType     int                  // sx1261,sx1262,sx1268 (defaults sx1261)
+	spiTxBuf       []byte               // global Tx buffer to avoid heap allocations in interrupt
+	spiRxBuf       []byte               // global Rx buffer to avoid heap allocations in interrupt
 }
 
 // --------------------------------------------------
@@ -42,7 +43,7 @@ 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 drivers.PinOutput) *Device {
+func New(spi drivers.SPI, rstPin pin.Output) *Device {
 	k := Device{
 		spi:            spi,
 		rstPin:         rstPin.Set,

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

@@ -12,6 +12,7 @@ import (
 
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 	"tinygo.org/x/drivers/pixel"
 )
 
@@ -31,10 +32,10 @@ type Config struct {
 
 type Device struct {
 	bus                      drivers.SPI
-	cs                       drivers.PinOutputFunc
-	dc                       drivers.PinOutputFunc
-	rst                      drivers.PinOutputFunc
-	isBusy                   drivers.PinInputFunc
+	cs                       drivers.PinOutput
+	dc                       drivers.PinOutput
+	rst                      drivers.PinOutput
+	isBusy                   drivers.PinInput
 	width                    int16
 	height                   int16
 	buffer                   []uint8
@@ -49,7 +50,7 @@ type Device struct {
 type Speed uint8
 
 // New returns a new uc8151 driver. Pass in a fully configured SPI bus.
-func New(bus drivers.SPI, csPin, dcPin, rstPin drivers.PinOutput, busyPin drivers.PinInput) Device {
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
 	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

@@ -15,6 +15,7 @@ import (
 
 	"tinygo.org/x/drivers"
 	"tinygo.org/x/drivers/internal/legacy"
+	"tinygo.org/x/drivers/internal/pin"
 )
 
 type Config struct {
@@ -26,10 +27,10 @@ type Config struct {
 
 type Device struct {
 	bus           *machine.SPI
-	cs            drivers.PinOutputFunc
-	dc            drivers.PinOutputFunc
-	rst           drivers.PinOutputFunc
-	isBusy        drivers.PinInputFunc
+	cs            drivers.PinOutput
+	dc            drivers.PinOutput
+	rst           drivers.PinOutput
+	isBusy        drivers.PinInput
 	configurePins func()
 	buffer        []uint8
 	rotation      Rotation
@@ -82,7 +83,7 @@ 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 drivers.PinOutput, busyPin drivers.PinInput) 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,

waveshare-epd/epd2in13/epd2in13.go

@@ -10,6 +10,7 @@ import (
 
 	"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           drivers.PinOutputFunc
-	dc           drivers.PinOutputFunc
-	rst          drivers.PinOutputFunc
-	isBusy       drivers.PinInputFunc
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	logicalWidth int16
 	width        int16
 	height       int16
@@ -53,7 +54,7 @@ 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 drivers.PinOutput, busyPin drivers.PinInput) Device {
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
 	legacy.ConfigurePinOut(csPin)
 	legacy.ConfigurePinOut(dcPin)
 	legacy.ConfigurePinOut(rstPin)

waveshare-epd/epd2in13x/epd2in13x.go

@@ -10,6 +10,7 @@ import (
 
 	"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           drivers.PinOutputFunc
-	dc           drivers.PinOutputFunc
-	rst          drivers.PinOutputFunc
-	isBusy       drivers.PinInputFunc
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	width        int16
 	height       int16
 	buffer       [][]uint8
@@ -33,7 +34,7 @@ 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 drivers.PinOutput, busyPin drivers.PinInput) Device {
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
 	legacy.ConfigurePinOut(csPin)
 	legacy.ConfigurePinOut(dcPin)
 	legacy.ConfigurePinOut(rstPin)

waveshare-epd/epd2in66b/dev.go

@@ -19,10 +19,10 @@ const Baudrate = 4_000_000 // 4 MHz
 
 type Device struct {
 	bus    drivers.SPI
-	cs     drivers.PinOutputFunc
-	dc     drivers.PinOutputFunc
-	rst    drivers.PinOutputFunc
-	isBusy drivers.PinInputFunc
+	cs     drivers.PinOutput
+	dc     drivers.PinOutput
+	rst    drivers.PinOutput
+	isBusy drivers.PinInput
 
 	blackBuffer []byte
 	redBuffer   []byte

waveshare-epd/epd2in9/epd2in9.go

@@ -17,6 +17,7 @@ import (
 
 	"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           drivers.PinOutputFunc
-	dc           drivers.PinOutputFunc
-	rst          drivers.PinOutputFunc
-	isBusy       drivers.PinInputFunc
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	logicalWidth int16
 	width        int16
 	height       int16
@@ -61,7 +62,7 @@ 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 drivers.PinOutput, busyPin drivers.PinInput) Device {
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
 	legacy.ConfigurePinOut(csPin)
 	legacy.ConfigurePinOut(dcPin)
 	legacy.ConfigurePinOut(rstPin)

waveshare-epd/epd4in2/epd4in2.go

@@ -14,6 +14,7 @@ import (
 
 	"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           drivers.PinOutputFunc
-	dc           drivers.PinOutputFunc
-	rst          drivers.PinOutputFunc
-	isBusy       drivers.PinInputFunc
+	cs           drivers.PinOutput
+	dc           drivers.PinOutput
+	rst          drivers.PinOutput
+	isBusy       drivers.PinInput
 	logicalWidth int16
 	width        int16
 	height       int16
@@ -40,7 +41,7 @@ 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 drivers.PinOutput, busyPin drivers.PinInput) Device {
+func New(bus drivers.SPI, csPin, dcPin, rstPin pin.Output, busyPin pin.Input) Device {
 	legacy.ConfigurePinOut(csPin)
 	legacy.ConfigurePinOut(dcPin)
 	legacy.ConfigurePinOut(rstPin)

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
 	}