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

PFW-1184:Basic Printer name in EEPROM #4457

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

3d-gussner
Copy link
Collaborator

@3d-gussner 3d-gussner commented Oct 20, 2023

  • Output with M115 works ✔️
  • New printer name has been limited to 16 chars and can be modified in EEPROM using D3 command.

@3d-gussner 3d-gussner added the Prusa-Link_Connect Prusa Link and Prusa Connect related label Oct 20, 2023
@3d-gussner 3d-gussner added this to the FW 3.14.0 milestone Oct 20, 2023
@github-actions
Copy link

github-actions bot commented Oct 20, 2023

All values in bytes. Δ Delta to base

Target ΔFlash ΔSRAM Used Flash Used SRAM Free Flash Free SRAM
MK3S_MULTILANG 44 14 246164 5643 7788 2549
MK3_MULTILANG 42 12 245448 5650 8504 2542

@3d-gussner 3d-gussner added the in progress We are working on solving this one label Oct 20, 2023
@3d-gussner 3d-gussner changed the title 🚧 PFW-1184:Basic Printer name in EEPROM PFW-1184:Basic Printer name in EEPROM Nov 17, 2023
@3d-gussner 3d-gussner marked this pull request as ready for review November 17, 2023 13:49
- Output with M115 works
- ToDo
  - LCD screen
  - M550 gcode
  - Detect length of EEPROM CUSTOM_MENDLE_NAME
@3d-gussner
Copy link
Collaborator Author

@gudnimg ready for review

@3d-gussner 3d-gussner removed the in progress We are working on solving this one label Nov 20, 2023
@3d-gussner 3d-gussner merged commit b39b83e into prusa3d:MK3 Nov 20, 2023
5 checks passed
@3d-gussner 3d-gussner deleted the MK3_PrinterName branch November 20, 2023 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prusa-Link_Connect Prusa Link and Prusa Connect related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants