Default Hypnos pins are incorrect for V3.3

[colinhalebrown]

Describe the bug
The V2.5.1 of loom with the hypnos V3.3 board revision so default pins no longer work for sleep or for SD read and write.

Hardware in Use
Feather M0
Hypnos V3.3

To Reproduce
Set config settings to default then try reading or writing data to SD.

Expected behavior
The SD card cannot be configured correctly.

Code
///////////////////////////////////////////////////////////////////////////////

// This is a basic example that demonstrates usage of the Hypnos board
// Deep sleep functionality.

// The Hypnos board includes
// - SD
// - DS3231 RTC
// - Ability to power of peripherals

// Further details about the Hypnos board can be found here:
// https://github.com/OPEnSLab-OSU/OPEnS-Lab-Home/wiki/Hypnos

///////////////////////////////////////////////////////////////////////////////

#include <Loom.h>
// int Flow,Turbidity;
// Include configuration
const char* json_config =
#include "config.h"
;

// Set enabled modules
LoomFactory<
Enable::Internet::Disabled,
Enable::Sensors::Enabled,
Enable::Radios::Enabled,
Enable::Actuators::Enabled,
Enable::Max::Enabled

ModuleFactory{};

LoomManager Loom{ &ModuleFactory };

volatile bool rtc_flag = false;

void wakeISR_RTC() {
// disable the interrupt
detachInterrupt(12);
rtc_flag = true;
}

void setup()
{
// Needs to be done for Hypno Board
pinMode(5, OUTPUT); // Enable control of 3.3V rail
pinMode(6, OUTPUT); // Enable control of 5V rail
pinMode(12, INPUT_PULLUP); // Enable waiting for RTC interrupt, MUST use a pullup since signal is active low
pinMode(13, OUTPUT);

//See Above
digitalWrite(5, LOW);	// Enable 3.3V rail
digitalWrite(6, HIGH);	// Enable 5V rail

digitalWrite(13, LOW);

Loom.begin_serial(true);
Loom.parse_config(json_config);
Loom.print_config();

// Register an interrupt on the RTC alarm pin
Loom.InterruptManager().register_ISR(12, wakeISR_RTC, LOW, ISR_Type::IMMEDIATE);

LPrintln("\n ** Setup Complete ** ");
Serial.flush();

}

void loop()
{
digitalWrite(5, LOW); // Disable 3.3V rail
digitalWrite(6, HIGH); // Disable 5V rail
digitalWrite(13, HIGH);
digitalWrite(LED_BUILTIN, HIGH);
// As it turns out, if the SD card is initialized and you change
// the states of the pins to ANY VALUE, the SD card will fail to
// write. As a result, we ensure that the board has been turned
// off at least once before we make any changes to the pin states
if (rtc_flag) {
pinMode(23, OUTPUT);
pinMode(24, OUTPUT);
pinMode(10, OUTPUT);

// delay(1000);

	Loom.power_up();
}

delay(10000);
Loom.measure();
Loom.package();
//Flow = Loom.get_data_as(LoomModule(),ADS1115);
//Turbidity = Loom.get_data_as(ADS1115,analog2);
Loom.display_data();

Loom.SDCARD().log();

// set the RTC alarm to a duration of 10 seconds with TimeSpan
Loom.InterruptManager().RTC_alarm_duration(TimeSpan(0,0,0,30)); // (days, hours, minutes, seconds)
Loom.InterruptManager().reconnect_interrupt(12);

digitalWrite(13, LOW);
digitalWrite(5, HIGH); // Enable 3.3V rail
digitalWrite(6, LOW); // Enable 5V rail
pinMode(23, INPUT);
pinMode(24, INPUT);
pinMode(10, INPUT);
digitalWrite(LED_BUILTIN, LOW);
// Sleep Manager autmatically calls power_down on every sensor before sleeping
// And power_up after waking.

rtc_flag = false;
Loom.SleepManager().sleep();
while (!rtc_flag);
}

Config
"{
'general':
{
'name':'Device',
'instance':1,
'interval':5000,
'print_verbosity':2
},
'components':[
{
'name':'ADS1115',
'params':'default'
},
{
'name':'MAX31856',
'params':'default'
},
{
'name':'SD',
'params': 'default'
},
{
'name':'DS3231',
'params':'default'
},
{
'name':'Interrupt_Manager',
'params':[0]
},
{
'name':'Sleep_Manager',
'params':[true,false,1]
}
]
}"

Additional context
Add any other context about the problem here.

[WL-Richards]

{\ 'name':'SD',\ 'params': [true, 1000, 11, "test", true]\ },\

Use that to set the chip select pin to 11 instead of 10 in the config

Same thing for pinMode(10, OUTPUT); this needs to be pinMode(11, OUTPUT);
And pinMode(10, INPUT); needs to be pinMode(11, INPUT);

Please close this issue if the problem is fixed

Thanks,
Will

[colinhalebrown]

Yeah we got it fixed a couple days ago just wanted to tell loom that the default had changed for future hypnos.