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.

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

Fix for SREG error - with this flow sensor is working #225

Closed
wants to merge 1 commit into from
Closed

Conversation

goran-mahovlic
Copy link

http://www.stm32duino.com/viewtopic.php?f=27&t=1167&sid=aba6dee79a24c8ad667381a999c4c08c

https://github.com/sekdiy/FlowMeter

Other sensors that are using SREG will probably work to.
When tested more you can add it to F3 and F4 boards.

@rogerclarkmelbourne
Copy link
Owner

Looks OK. But I need to check blink etc compiles OK

@rogerclarkmelbourne
Copy link
Owner

Wont compile if you include SPI.h :-(

@goran-mahovlic
Copy link
Author

ok, I will check

tnx

On Thu, Nov 10, 2016 at 11:34 AM, Roger Clark notifications@github.com
wrote:

Wont compile if you include SPI.h :-(


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#225 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEDUoKy5jqI5fJoeOMLpTimAkv5AhwOBks5q8vM1gaJpZM4KqOXt
.

@victorpv
Copy link
Contributor

victorpv commented May 9, 2017

There is another pull including this pinToInterrupt function, but does it a different way.
#225

I think would be positive if one of you post in the forum in stm32duino a new thread about this to collect all the input and submit a single solution.

@rogerclarkmelbourne
Copy link
Owner

@rogerclarkmelbourne
Copy link
Owner

Whats the status of this?

Looks like no one uses it etc so perhaps I should close this due to lack of interest ???

@goran-mahovlic
Copy link
Author

I did try now and will close it SREG error does not exist any more.
And to fix sei() cli() error we just need to add
#define sei() interrupts();
#define cli() noInterrupts();
at the end of wirish.h

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants