Skip to content
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

Jump to bottom

Read Vcc pulls 200uA with LowPower sleep #7

Open
skjolddesign opened this issue Feb 15, 2020 · 3 comments · May be fixed by #8
Open

Read Vcc pulls 200uA with LowPower sleep #7

skjolddesign opened this issue Feb 15, 2020 · 3 comments · May be fixed by #8

Comments

@skjolddesign
Copy link

skjolddesign commented Feb 15, 2020 •
edited
Loading

Your example VccSleep.ino, makes arduino pull 200 micro Amps in LowPower.
Commenting out Vcc.h code, gives correct LowPower of 4.5 micro Amps.
Tested on Pro mini 8Mhz.
It's not only your Vcc code that does this, but all readVcc code I found online does this.
I would be glad to see any solution to this.

4.5uA code:

#include <Vcc.h>
#include <LowPower.h>

const float VccMin        = 2.0*0.6;  // Minimum expected Vcc level, in Volts. Example for 2xAA Alkaline.
const float VccMax        = 2.0*1.5;  // Maximum expected Vcc level, in Volts. Example for 2xAA Alkaline.
const float VccCorrection = 1.0/1.0;  // Measured Vcc by multimeter divided by reported Vcc

Vcc vcc(VccCorrection);

void setup()
{
  Serial.begin(9600);
}

void loop()
{  
//  float v = vcc.Read_Volts();
//  Serial.print("VCC = ");
//  Serial.print(v);
//  Serial.println(" Volts");
//
//  float p = vcc.Read_Perc(VccMin, VccMax);
//  Serial.print("VCC = ");
//  Serial.print(p);
//  Serial.println(" %");
//
//  delay(200); //delay to allow serial to fully print before sleep

  LowPower.powerDown(SLEEP_8S, ADC_OFF, BOD_OFF);
}
@skjolddesign
Copy link
Author

I managed to fix the issue adding ADMUX = _BV(REFS1)
new readVcc code:

long readVccFixedLowPower() {
  long result;
  // Read 1.1V reference against AVcc
  ADMUX = _BV(REFS0) | _BV(MUX3) | _BV(MUX2) | _BV(MUX1);
  delay(1); // Wait for Vref to settle
  ADCSRA |= _BV(ADSC); // Convert
  while (bit_is_set(ADCSRA,ADSC));
  result = ADCL;
  result |= ADCH<<8;
  result = ( 1023L * 1100L) / result; // Back-calculate AVcc in mV. 1100 is volt on vRef.
  //to fix lowpower drain, ADMUX is changed again.
  ADMUX = _BV(REFS1); //fixes readVcc current drain in lowPower
  Serial.print("Vcc = " );
  Serial.println(result); 
  return result;
}

@Yveaux
Copy link
Owner

Yveaux commented Feb 16, 2020

Can you prepare a PR?

danielrheinbay added a commit to danielrheinbay/Arduino_Vcc that referenced this issue Jun 2, 2020
@danielrheinbay
Fix LowPower sleep by setting ADMUX to _BV(REFS1) after reading Vcc. F…
c034076 
…ixes Yveaux#7.
@danielrheinbay danielrheinbay linked a pull request Jun 2, 2020 that will close this issue
Open
@danielrheinbay
Copy link

Can you prepare a PR?

See #8 :)

@LaZsolt LaZsolt mentioned this issue May 10, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix LowPower sleep by setting ADMUX to _BV(REFS1) after reading Vcc. … danielrheinbay/Arduino_Vcc
3 participants
@skjolddesign @Yveaux @danielrheinbay