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

-c arduino keep-alive feature similar to -c urclock. #1229

Closed
mcuee opened this issue Dec 19, 2022 · 7 comments
Closed

-c arduino keep-alive feature similar to -c urclock. #1229

mcuee opened this issue Dec 19, 2022 · 7 comments
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@mcuee
Copy link
Collaborator

mcuee commented Dec 19, 2022

As of now -c urclock can keep bootloaders in terminal mode -t` without the bootloader resetting itself through the watchdog timer.

This is a nice feature which would be nice to bring to -c arduino as well.

@mcuee mcuee added the enhancement New feature or request label Dec 19, 2022
@stefanrueger
Copy link
Collaborator

Why? There is -c urclock that can be used instead. Duplicating code for -c arduino costs effort and increases the maintenance surface. Our energy is better spent on bug fixes, or implementing keep-alive for -c wiring, -c xbee etc. And before you rush raising issues for that just a note that I for one don't know what NOPs look like for these bootloaders, so this needs people using these bootloaders to chip in with their knowledge.

@mcuee
Copy link
Collaborator Author

mcuee commented Dec 19, 2022

I think this request came from @MCUdude. Let's see if he is still interested in this improvement on -c arduino or not. If not we can close this issue as not planned.

@mcuee
Copy link
Collaborator Author

mcuee commented Dec 25, 2022

@MCUdude

Do you still think this feature is useful? If not I will close this issue. Thanks,

@MCUdude
Copy link
Collaborator

MCUdude commented Dec 27, 2022

Even though Urboot is vastly superior, there are still millions(?) of microcontrollers out there that run Optiboot. I'd argue that this would be a nice feature for those who are stuck with Optiboot or can't/don't want to switch to a new and improved bootloader.

@mcuee
Copy link
Collaborator Author

mcuee commented Dec 28, 2022

From here as mentioned by @stefanrueger

Being able to use the pgm->term_keep_alive() function relies on the following:

  1. AVRDUDE is compiled with libreadline
  2. The programmer provides a function term_keep_alive() to keep the bootloader happy
  3. The timing of reading characters and draining serial input does not get in the way
  4. AVRDUDE is able to figure out whether or not the next read from stdin would block

@stefanrueger
Copy link
Collaborator

there are still millions(?) of microcontrollers out there that run Optiboot.

And they can use -c urclock if they want to use the keep-alive feature, can't they? So, I'd rather spend time on other things.

@stefanrueger stefanrueger added the wontfix This will not be worked on label Dec 28, 2022
@MCUdude
Copy link
Collaborator

MCUdude commented Dec 29, 2022

And they can use -c urclock if they want to use the keep-alive feature, can't they?

Ah, of course. Totally forgot this, sorry

@MCUdude MCUdude closed this as completed Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants