Skip to content

Pico - only one PWM pin is set #287

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
DocMoebiuz merged 4 commits into from
Nov 30, 2023
Merged

Pico - only one PWM pin is set #287

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
2432bf2
avoid double set of pin for Pico
elral Nov 26, 2023
ad8ef4f
keep old logic for AVR's
elral Nov 27, 2023
08558af
fixed missing ';'
elral Nov 27, 2023
2bee49e
use same set output logic for AVR's and Pico
elral Nov 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion CustomDevices
Open in desktop
Submodule CustomDevices updated from d7e3f3 to 6a2b28
13 changes: 10 additions & 3 deletions src/MF_Output/MFOutput.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,16 +14,23 @@ MFOutput::MFOutput()
void MFOutput::attach(uint8_t pin)
{
_pin = pin;
#if !defined(ARDUINO_ARCH_RP2040)
#if defined(ARDUINO_ARCH_RP2040)
pinMode(_pin, OUTPUT_12MA);
#else
pinMode(_pin, OUTPUT);
set(_value);
#endif
set(LOW);
}

void MFOutput::set(uint8_t value)
{
_value = value;
analogWrite(_pin, _value);
if (_value == 0xFF)
digitalWrite(_pin, HIGH);
else if (_value == 0x00)
digitalWrite(_pin, LOW);
else
analogWrite(_pin, _value);
}

void MFOutput::powerSavingMode(bool state)
Expand Down
13 changes: 7 additions & 6 deletions src/MF_Output/Output.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,6 @@ namespace Output
return;
outputs[outputsRegistered] = MFOutput();
outputs[outputsRegistered].attach(pin);
#if defined(ARDUINO_ARCH_RP2040)
pinMode(pin, OUTPUT_12MA);
analogWrite(pin, false);
#endif
outputsRegistered++;
#ifdef DEBUG2CMDMESSENGER
cmdMessenger.sendCmd(kDebug, F("Added output"));
Expand All @@ -52,9 +48,14 @@ namespace Output
// Read led state argument, interpret string as boolean
int pin = cmdMessenger.readInt16Arg();
int state = cmdMessenger.readInt16Arg();

// Set led
analogWrite(pin, state); // why does the UI sends the pin number and not the x.th output number like other devices?
// output[pin].set(state); // once this is changed uncomment this
if (state == 0xFF)
digitalWrite(pin, HIGH);
else if (state == 0x00)
digitalWrite(pin, LOW);
else
analogWrite(pin, state);
}

void PowerSave(bool state)
Expand Down