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

Compilation error with MMU2 enabled #23932

Closed
kursatu opened this issue Mar 21, 2022 · 4 comments
Closed

Compilation error with MMU2 enabled #23932

kursatu opened this issue Mar 21, 2022 · 4 comments

Comments

@kursatu
Copy link

kursatu commented Mar 21, 2022

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

I am getting compilation with MMU2 enabled.
I've fixed it. It was due to trivial mistake in mmu2.cpp
The fix was.
Commenting out
//inline void ATTN_BUZZ(const bool two=false) { BUZZ(200, 404); if (two) { BUZZ(10, 0); BUZZ(200, 404); } }
and removing the argument in this file
ATTN_BUZZ(true);
to
ATTN_BUZZ();

Bug Timeline

New bug

Expected behavior

The compilation would succeed.

Actual behavior

The compilation would fails.

Steps to Reproduce

No response

Version of Marlin Firmware

bugfix-2.0.x

Printer model

No response

Electronics

No response

Add-ons

No response

Bed Leveling

No response

Your Slicer

No response

Host Software

No response

Additional information & file uploads

No response

@kursatu kursatu changed the title Compilation erro with MMU2 enable Compilation error with MMU2 enabled Mar 21, 2022
@thisiskeithb
Copy link
Member

Looks like it was broken in 89a9c3a

@ellensp
Copy link
Contributor

ellensp commented Mar 21, 2022

yip, conflicts with #define ATTN_BUZZ() do{ BUZZ(198, 404); BUZZ(4, 0); BUZZ(198, 404); }while(0) in buzzer.h
Adding
#undef ATTN_BUZZ
above
inline void ATTN_BUZZ(const bool two=false) { BUZZ(200, 404); if (two) { BUZZ(10, 0); BUZZ(200, 404); } }
gets around it, but isn't very nice,,,

@ellensp
Copy link
Contributor

ellensp commented Mar 25, 2022

PR created, closing

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators May 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants