[riot-notifications] [RIOT-OS/RIOT] Bugfix/sfr04 avr overflow (#15494)

Wouter Symons notifications at github.com
Thu Nov 26 10:59:49 CET 2020


@fjmolinas Yes, I saw the output of his test. But I was curious if anyone else (for example benpicco) would have also tested it to verify the results. 
Because maybe he had also changed other things that were suggested in that thread that ultimately did not end up in the commit. Who knows? (For example changing the stacksize, which was also a suggestion)

That's why I asked. Because I tried the exact same thing as this commit does a few months ago, and for me it did not resolve the issue. But maybe things have changed elsewhere in RIOT that influence this, or... there were other things wrong with my setup back when I tried it. I'm not ruling out that possibility. 

In any case, I'm not trying to block the merging of this PR with my questions. Like I said: the changes make a lot of sense and they definitely fix a mistake that needed correcting in the driver code. 

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/15494#issuecomment-734199124
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201126/72e835e7/attachment.htm>


More information about the notifications mailing list