From 22e241fab5ba41f3600ff766d34a706eeea26edb Mon Sep 17 00:00:00 2001
From: George Sexton <georges@mhsoftware.com>
Date: Wed, 12 Feb 2025 13:57:01 -0700
Subject: [PATCH] Address review comments

---
 matrixorbital/lk2047t.go | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/matrixorbital/lk2047t.go b/matrixorbital/lk2047t.go
index 8ba2adc..ac99727 100644
--- a/matrixorbital/lk2047t.go
+++ b/matrixorbital/lk2047t.go
@@ -12,8 +12,6 @@ import (
 	"io"
 	"sync"
 
-	//	"time"
-
 	"periph.io/x/conn/v3"
 	"periph.io/x/conn/v3/display"
 	"periph.io/x/conn/v3/gpio"
@@ -39,12 +37,13 @@ type LK2047T struct {
 	// Pins represents the set of gpio.PinOut pins exposed by the device. For
 	// units with LEDS, the pins are used to manipulate them. For the Adafruit
 	// USB/LCD backpack, 4 pins are exposed.
-	Pins       []gpio.PinOut
+	Pins []gpio.PinOut
+	rows int
+	cols int
+
+	mu         sync.Mutex
 	d          conn.Conn
 	writer     io.Writer
-	mu         sync.Mutex
-	rows       int
-	cols       int
 	chKeyboard chan byte
 	shutdown   chan struct{}
 }
@@ -336,8 +335,7 @@ func (dev *LK2047T) LED(number int, color LEDColor) error {
 	if err != nil {
 		return err
 	}
-	err = dev.Pins[number*2+1].Out(gpio.Level(color&Green == Green))
-	return err
+	return dev.Pins[number*2+1].Out(gpio.Level(color&Green == Green))
 }
 
 func (dev *LK2047T) String() string {