EEPROM lib : debug support for 328D variant #129
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8 tasks
strangely! |
|
Bizarre behaviour again ... Is it uploaded with a CH340 again ? Into void test_writing( unsigned int COLS )
{
Serial.print(F("Writing using EEPROM[0.."));
Serial.print( EEPROM.length() );
Serial.println(F("] = xx : "));
rngSeed(0);
if ( EEPROM.length() >= 1024 )
{
Serial.println(F("ERROR : max address overflowing !"));
Serial.println( EEPROM.length() );
return;
}
for( uint16_t a=0; a< EEPROM.length(); a++ )
{
if ( a >= 1024 )
{
Serial.println(F("ERROR : address overflowing !"));
Serial.println( EEPROM.length() );
return;
}
EEPROM[a] = rng() & 0xff;
print_hex( EEPROM[a] );
Serial.print(F("."));
if ( ( a % COLS ) == ( COLS-1) ) Serial.println();
}
} |
|
this time is much better! most time are correct (by reset),
|
|
Thank you a lot for testing so many times. I have a last request : if possible, could you give a link to a picture of the board and of the USB-serial adapter ? |
|
#109 (comment) here is the speed I use oscilloscope to test: and I see a test for ch340 |
|
I had too much instability issues with ch340E. Uploading tiny sketches was ok, but uploading large sketches was always unstable and corrupted. Also, stability was depending too much on room and chip temperature. I just replaced it with CP2040 adapter, and everything is fine now. |
|
@SuperUserNameMan This request ready to merge. |
|
I tested your EEPROM library on LGT8F88DS20 with SWDISP Please add "#if defined" for LGT8F88 support. I can develop my project with EEPROM library, Thank you |
|
I'm no more active on this project. Regarding the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
If github is going to do what i hope it is going to do, this is a PR to patch the new EEPROM library to debug and improve 328D support.