Skip to content

13 pic16f13145 partially programmed#24

Merged
edras merged 7 commits intomainfrom
13-pic16f13145-partially-programmed
Mar 29, 2025
Merged

13 pic16f13145 partially programmed#24
edras merged 7 commits intomainfrom
13-pic16f13145-partially-programmed

Conversation

@edras
Copy link
Copy Markdown
Collaborator

@edras edras commented Mar 29, 2025

This PR handles some modifications in code:

Changed inheritance model

  • Different PIC families have their own class that should register itself to the GenericPIC class.
  • When retrieving devID bei the GenericPIC, a new instance of a PIC device will be obtained regarding the respective PIC family.
  • Common methods moved to GenericPIC
  • Inserted timings for bulk_erase, row_erase, flash_erase, config_word_erase for different families.
  • The bug of partially programming appears to be solved.

Inserted verification flag

  • Memory is checked and get a flag false if verification failed
  • Log and Message are showed when verification failed.
  • The failed memory page is NOT YET reprogrammed.
  • Further tests are needed to check if it is necessary. According to the documentation, only check is necessary.

common methods of write and verification were unified and optimized - DRY

  • the specific programming areas may be optimized as well. We will refactor this after testing.

@edras edras linked an issue Mar 29, 2025 that may be closed by this pull request
@edras edras merged commit 2519842 into main Mar 29, 2025
@edras edras deleted the 13-pic16f13145-partially-programmed branch March 29, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sometimes PIC16F13145 only partially programmed

1 participant