ArduinoCore-sam icon indicating copy to clipboard operation
ArduinoCore-sam copied to clipboard

added weak attribute to UART handlers

Open anklimov opened this issue 4 years ago • 10 comments

added weak attribute to UART handlers to allow custom handlers override default UART routine That is vital changes to implement DMX512 protocol over UART (for example) without manual changes of CORE libraries

anklimov avatar Oct 17 '19 10:10 anklimov

Hi! is some progress with PR review? That is a real problem now to ask everybody who compiling every project with low-level UART access to fix Arduino libs manually.

anklimov avatar Nov 03 '19 18:11 anklimov

hi anklimov, I am having a similar problem for implementing an efficient LIN slave. Would be great if the team merged this (for all Serials)! Georg

gicking avatar Nov 06 '19 05:11 gicking

Is somebody still alive here? What reason do not accept PR? That is really difficult for users to modify core libraries every time then needed to re-compile project

anklimov avatar May 12 '20 18:05 anklimov

overwriting the UART ISRs seems to be a common wish as a similar proposal was already made in 2017 (see https://github.com/arduino/ArduinoCore-sam/pull/1, and recently by me (see https://github.com/arduino/ArduinoCore-sam/pull/95. And at least for the latter I have given up and implemented a LIN master handler differently. What a shame!

gicking avatar May 16 '20 04:05 gicking

Hi @gicking It looks you're right - the DUE project is discarded I will appreciated if you share with everybody another option to override default USART handler (i didn't find another option but correct core source again and again to compile my code on DUE)

anklimov avatar May 23 '20 17:05 anklimov

hi anklimov, actually I am hoping that the Due team is just overwhelmed with other tasks - after all the ARM-based Arduinos are still young, compared to the AVR 8-bitters... Anyway, the effect is the same: pull requests are ignored for a long time at least.

Regarding your question: as discussed in my pull-request #95, the elegant solution would to separate the UART-ISRs such that they can be overwritten by a library to support e.g. attaching user functions via NeoHWSerial. But that would require support by the Due team again... Alternatively, pull request #95 implements this functionality, namely attach an optional user routine to the UART Rx-ISR. Is that what you're looking for?

gicking avatar May 24 '20 12:05 gicking

overwriting the UART ISRs seems to be a common wish as a similar proposal was already made in 2017 (see #1, and recently by me (see #95. And at least for the latter I have given up and implemented a LIN master handler differently. What a shame!

Actually, the MR was open back in 2015, see https://github.com/arduino/Arduino/pull/3023. They really should seriously consider adding more maintainers to this project.

odurc avatar Dec 05 '20 10:12 odurc

I give up. There is simply no point in contributing to Due core libraries if there is no response within >1 year - or in your example even 5 years! This way the community cannot be kept alive. CU in another project. Over and out...

gicking avatar Dec 28 '20 18:12 gicking

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Apr 09 '21 13:04 CLAassistant

seriously? It takes 16 months to automatically send a legal form...???

gicking avatar Apr 10 '21 06:04 gicking