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

Save bytes for custom chars (Hitachi LCD) #4553

Merged

Conversation

thinkyhead
Copy link
Member

@thinkyhead thinkyhead commented Aug 6, 2016

By adding static to the custom characters they take only half as much space in the binary.

There's also an issue where the LCD fails to clear when switching screens. This PR attempts to address that by explicitly clearing the screen rather than relying on the draw flag.

@thinkyhead thinkyhead merged commit cffe77c into MarlinFirmware:RCBugFix Aug 7, 2016
@thinkyhead thinkyhead deleted the rc_lcd_chars_save_ram branch August 7, 2016 01:29
@ghost
Copy link

ghost commented Aug 8, 2016

I've confirmed that debris of previous content has been solved.

But I found that new bug? introduced by this PR?.
When printer is booted/reset without host software, "Card removed" or "Card inserted" is displayed always, thus "xxxxxx ready." isn't displayed.
When printer is connected with host software(Repetier-Host), this phenomenon isn't occured.

@thinkyhead
Copy link
Member Author

thinkyhead commented Aug 9, 2016

I've confirmed that debris of previous content has been solved.

Great!

"Card removed" or "Card inserted" is displayed

Aha, yes, I did make a change that's causing "Card removed/inserted" to appear at boot. I'll have a patch for that shortly.

@jbrazio jbrazio modified the milestone: 1.1.0 Aug 11, 2016
thinkyhead added a commit that referenced this pull request Oct 22, 2016
Fix for the PR #4553 (Save bytes for custom chars (Hitachi LCD))
@thinkyhead thinkyhead mentioned this pull request Oct 24, 2016
drewmoseley pushed a commit to drewmoseley/Marlin that referenced this pull request May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants