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

ArduinoISP: Add an option to enable a 4 MHz clock signal on pin 3 #17

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Sep 26, 2020

Moved from arduino/Arduino#8757 by @procsynth

This clock signal can be used to reprogram the fuses of an AVR that
has been set wrong (ie. when an external clock has been configured but
the board/chip does not have one).

This is disabled by default. We check that we compile for known chips
before enabling the clock generation. Maybe there is other chips that
support this code.

This clock signal can be used to reprogram the fuses of an AVR that
has been set wrong (ie. when an external clock has been configured but
the board/chip does not have one).

This is disabled by default. We check that we compile for known chips
before enabling the clock generation. Maybe there is other chips that
support this code.
@per1234 per1234 added the type: enhancement Proposed improvement label Sep 26, 2020
@per1234
Copy link
Contributor Author

per1234 commented Sep 26, 2020

From facchinm on 2019-04-18

Hi @procsynth ,
thanks for your PR.

ArduinoISP sketch is meant to run on all Arduinos (also the ones with ARM chips), so it would be nice to keep the sketch functionalities in sync between all the targets.

The functionality is interesting and should be expanded a bit, like having a couple of options for the pin and the frequency to use, and less magic numbers 😉


#ifdef ENABLE_CLOCK_GEN

#if defined(__AVR_ATmega328__) || defined(__AVR_ATmega328P__) || defined(__AVR_ATmega168__) || defined(__AVR_ATmega88__)
Copy link
Contributor Author

@per1234 per1234 Sep 26, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@facchinm requested changes on 2019-04-18

ArduinoISP sketch is meant to run on all Arduinos (also the ones with ARM chips), so it would be nice to keep the sketch functionalities in sync between all the targets.

@per1234
Copy link
Contributor Author

per1234 commented Sep 26, 2020

From @procsynth on 2019-04-23

Hi @facchinm, thanks for your reply. I did not think in the scope of the whole Arduino project.
I'll make the changes requested, explain and test the code a little better when I have some time :)

@arduino arduino deleted a comment Dec 27, 2020
@arduino arduino deleted a comment Dec 27, 2020
@arduino arduino deleted a comment Dec 27, 2020
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ per1234
❌ pr1csynth
You have signed the CLA already but the status is still pending? Let us recheck it.

@per1234 per1234 changed the title Add an option on Uno boards to enable a 4 MHz clock signal on pin 3 ArduinoISP: Add an option to enable a 4 MHz clock signal on pin 3 Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants