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

[BUG?] get warning: 'EEPROM' defined but not used [-Wunused-variable] with SKR2.0 #22243

Closed
GER-HerrS opened this issue Jun 27, 2021 · 7 comments

Comments

@GER-HerrS
Copy link

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

Yes, and the problem still exists.

Bug Description

When compiling with SKR 2.0 Rev. B (newest Marlin 2.0.9.1) I get the following warning:
libraries\EEPROM\src/EEPROM.h:250:20: warning: 'EEPROM' defined but not used [-Wunused-variable]

As it is my first SKR I don't know if that is an important warning.
I never got this warning using Arduino 2560 Boards like RAMPS or stock Creality Boards.
So I wonder what is the reason for this and what does this means to my compiled firmware (as it completed compiling)?
Is there a need to change some config or is this some bug like the LCD problem I had as well (#22239)?

Bug Timeline

No response

Expected behavior

I expected no warning because I didn't do changes concerning EEPROM.

Actual behavior

I got the warning... .

Steps to Reproduce

Use SKR 2.0 Rev. B board.

Version of Marlin Firmware

2.0.1.9

Printer model

Creality CR-10 V2

Electronics

BTT SKR 2.0

Add-ons

No response

Your Slicer

No response

Host Software

No response

Additional information & file uploads

No response

@slowbro
Copy link
Member

slowbro commented Jun 27, 2021

I wasn't able to reproduce this. Can you upload your configuration files?

@GER-HerrS
Copy link
Author

Thanks for the quick response.
Here are the configs.
Config.zip

@slowbro
Copy link
Member

slowbro commented Jun 27, 2021

The warning can be ignored. It was introduced with PR #22054.

@GER-HerrS
Copy link
Author

Thanks again for your quick response!
Then I can flash my firmware even with this warning. Wasn't sure if I it would brick my board or so one.

@slowbro
Copy link
Member

slowbro commented Jun 27, 2021

Yep, it will be fine. The EEPROM.h header was added for USB support but not all parts of it are used, hence the warning.

@GER-HerrS
Copy link
Author

Then I close this issue now
Thanks again.

@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 Aug 27, 2021
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