From 796105805769527776ca88636a4db392703698c2 Mon Sep 17 00:00:00 2001
From: caternuson <caternuson@gmail.com>
Date: Mon, 24 Jan 2022 16:56:58 -0800
Subject: [PATCH] refinements for no CS configs

---
 Adafruit_SPIDevice.cpp | 31 +++++++++++++++++++------------
 Adafruit_SPIDevice.h   |  1 +
 2 files changed, 20 insertions(+), 12 deletions(-)

diff --git a/Adafruit_SPIDevice.cpp b/Adafruit_SPIDevice.cpp
index a17111b..fdd50c0 100644
--- a/Adafruit_SPIDevice.cpp
+++ b/Adafruit_SPIDevice.cpp
@@ -88,8 +88,10 @@ Adafruit_SPIDevice::~Adafruit_SPIDevice() {
  * init
  */
 bool Adafruit_SPIDevice::begin(void) {
-  pinMode(_cs, OUTPUT);
-  digitalWrite(_cs, HIGH);
+  if (_cs != -1) {
+    pinMode(_cs, OUTPUT);
+    digitalWrite(_cs, HIGH);
+  }
 
   if (_spi) { // hardware SPI
     _spi->begin();
@@ -299,7 +301,7 @@ bool Adafruit_SPIDevice::write(uint8_t *buffer, size_t len,
     _spi->beginTransaction(*_spiSetting);
   }
 
-  digitalWrite(_cs, LOW);
+  setChipSelect(LOW);
   // do the writing
 #if defined(ARDUINO_ARCH_ESP32)
   if (_spi) {
@@ -319,7 +321,7 @@ bool Adafruit_SPIDevice::write(uint8_t *buffer, size_t len,
       transfer(buffer[i]);
     }
   }
-  digitalWrite(_cs, HIGH);
+  setChipSelect(HIGH);
 
   if (_spi) {
     _spi->endTransaction();
@@ -362,9 +364,10 @@ bool Adafruit_SPIDevice::read(uint8_t *buffer, size_t len, uint8_t sendvalue) {
   if (_spi) {
     _spi->beginTransaction(*_spiSetting);
   }
-  digitalWrite(_cs, LOW);
+
+  setChipSelect(LOW);
   transfer(buffer, len);
-  digitalWrite(_cs, HIGH);
+  setChipSelect(HIGH);
 
   if (_spi) {
     _spi->endTransaction();
@@ -406,7 +409,7 @@ bool Adafruit_SPIDevice::write_then_read(uint8_t *write_buffer,
     _spi->beginTransaction(*_spiSetting);
   }
 
-  digitalWrite(_cs, LOW);
+  setChipSelect(LOW);
   // do the writing
 #if defined(ARDUINO_ARCH_ESP32)
   if (_spi) {
@@ -452,7 +455,7 @@ bool Adafruit_SPIDevice::write_then_read(uint8_t *write_buffer,
   DEBUG_SERIAL.println();
 #endif
 
-  digitalWrite(_cs, HIGH);
+  setChipSelect(HIGH);
 
   if (_spi) {
     _spi->endTransaction();
@@ -476,11 +479,9 @@ bool Adafruit_SPIDevice::write_and_read(uint8_t *buffer, size_t len) {
     _spi->beginTransaction(*_spiSetting);
   }
 
-  digitalWrite(_cs, LOW);
-
+  setChipSelect(LOW);
   transfer(buffer, len);
-
-  digitalWrite(_cs, HIGH);
+  setChipSelect(HIGH);
 
   if (_spi) {
     _spi->endTransaction();
@@ -489,4 +490,10 @@ bool Adafruit_SPIDevice::write_and_read(uint8_t *buffer, size_t len) {
   return true;
 }
 
+void Adafruit_SPIDevice::setChipSelect(int value) {
+  if (_cs == -1)
+    return;
+  digitalWrite(_cs, value);
+}
+
 #endif // SPI exists
diff --git a/Adafruit_SPIDevice.h b/Adafruit_SPIDevice.h
index 6d2dc13..8c4d162 100644
--- a/Adafruit_SPIDevice.h
+++ b/Adafruit_SPIDevice.h
@@ -95,6 +95,7 @@ private:
   uint32_t _freq;
   BusIOBitOrder _dataOrder;
   uint8_t _dataMode;
+  void setChipSelect(int value);
 
   int8_t _cs, _sck, _mosi, _miso;
 #ifdef BUSIO_USE_FAST_PINIO