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.

Sign up for GitHub

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

Jump to bottom

[BUG] Robin Nano 1.2 boot issue #26577

Closed
1 task done
ahmetcemturan opened this issue Dec 27, 2023 · 24 comments
Closed
1 task done

[BUG] Robin Nano 1.2 boot issue #26577

ahmetcemturan opened this issue Dec 27, 2023 · 24 comments
Labels
Bug: Confirmed !

Comments

@ahmetcemturan
Copy link

ahmetcemturan commented Dec 27, 2023 •
edited by thinkyhead
Loading

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

Yes, and the problem still exists.

Bug Description

When trying to compile for the Robin NAno 1.2 I get the following message:

Bug Timeline

It is existent in the latest bugfix

Expected behavior

Compile correctly

Actual behavior

Fails and gives following error:

In file included from Marlin\src\lcd/dogm/u8g_fontutf8.h:11,
                 from Marlin\src\lcd/lcdprint.h:53,
                 from Marlin\src\lcd/marlinui.h:74,
                 from Marlin\src\MarlinCore.cpp:64:
.pio\libdeps\mks_robin_nano_v1v2\U8glib-HAL\src/U8glib-HAL.h: In member function 'void U8GLIB_SSD1309_128X64_F::init(uint8_t, uint8_t, uint8_t, uint8_t, uint8_t)':
.pio\libdeps\mks_robin_nano_v1v2\U8glib-HAL\src/U8glib-HAL.h:944:110: error: 'u8g_dev_ssd1309_128x64_f_sw_spi' was not declared in this scope; did you mean 'u8g_dev_ssd1309_128x64_sw_spi'?
  944 |   void init(uint8_t sck, uint8_t mosi, uint8_t cs, uint8_t a0, uint8_t reset = U8G_PIN_NONE) { U8GLIB::init(&u8g_dev_ssd1309_128x64_f_sw_spi, sck, mosi, cs, a0, reset); }

Steps to Reproduce

I have attached my config files.

Version of Marlin Firmware

2.1.2 or 2.1.bugfix

Printer model

Sapphire Pro

Electronics

Robin Nano 1.2

LCD/Controller

No response

Other add-ons

No response

Bed Leveling

None

Your Slicer

None

Host Software

None

Don't forget to include

  • A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

Marlin.zip

I tried to compile by commenting out the whole section that throws the errors and it compiled but then the printer got stuck at "updating 100%" . After reboot it hangs at "booting".
@ellensp
Copy link
Contributor

ellensp commented Dec 27, 2023

recent changes to U8glib-HAL have broken things MarlinFirmware/U8glib-HAL@c1ec1a4

@ellensp
Copy link
Contributor

ellensp commented Dec 27, 2023 •
edited
Loading

fixing your max temp to #define HEATER_0_MAXTEMP 260 and forcing marlin to use older U8glib-HAL

Change features.ini line
HAS_MARLINUI_U8GLIB = marlinfirmware/U8glib-HAL@~0.5.2
to
HAS_MARLINUI_U8GLIB = marlinfirmware/[email protected]

Will allow you to build right now, till this issue is fixed properly

@thisiskeithb thisiskeithb mentioned this issue Dec 27, 2023
Merged
@ellensp
Copy link
Contributor

ellensp commented Dec 27, 2023

Can also be fixed by editing .pio/libdeps/mks_robin_nano_v1v2/U8glib-HAL/src/clib/u8g.h

And adding the block

/* OLED 128x64 Display with SSD1309 Controller - Fast Single Page */
extern u8g_dev_t u8g_dev_ssd1309_128x64_f_sw_spi;
extern u8g_dev_t u8g_dev_ssd1309_128x64_f_hw_spi;
extern u8g_dev_t u8g_dev_ssd1309_128x64_f_i2c;

after the block

/* OLED 128x64 Display with SSD1309 Controller */
extern u8g_dev_t u8g_dev_ssd1309_128x64_sw_spi;
extern u8g_dev_t u8g_dev_ssd1309_128x64_hw_spi;
extern u8g_dev_t u8g_dev_ssd1309_128x64_i2c;

@ellensp
Copy link
Contributor

ellensp commented Dec 27, 2023

Issue opened on U8glib-HAL MarlinFirmware/U8glib-HAL#32

@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 27, 2023 via email

@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 27, 2023 via email

@thisiskeithb
Copy link
Member

The same happens with this solution stuck at 100% :(

In addition to the HAS_MARLINUI_U8GLIB = marlinfirmware/[email protected] change, you may also need to roll back your fork of bugfix-2.1.x to c485f51 which is right before "🧑‍💻 Move U8G defines to HALs" / bd36644

@thinkyhead
Copy link
Member

If it is confirmed that rolling back that commit fixes an issue then I need another set of eyes on bd36644. I don't see how that commit changes anything that would affect a Robin Nano 1.2 (or indeed anything at all, since it just moves the u8g assignments to the HALs but does not change them).

@thinkyhead
Copy link
Member

Once U8G 0.5.4 propagates to the PIO registry the build issue will be fixed and this can be closed.

It compiles but screen freezes at: Updating 100%

That may or may not be related to the U8G changes. When was the last time you re-flashed Marlin before today?

@thisiskeithb
Copy link
Member

Once U8G 0.5.4 propagates to the PIO registry the build issue will be fixed and this can be closed.

For completeness, I forced Marlin to use 0.5.4 with HAS_MARLINUI_U8GLIB = U8glib-HAL=https://github.com/MarlinFirmware/U8glib-HAL/archive/refs/tags/0.5.4.zip with current bugfix-2.1.x and tested changes on a BTT Mini 12864 V2 (with NeoPixels) and it works fine.

Not sure about ahmetcemturan's issue though.

@thisiskeithb thisiskeithb mentioned this issue Dec 28, 2023
Closed
1 task
@thisiskeithb thisiskeithb changed the title [BUG] LAtest Bugfix Fails to compile for Robin Nano v1.2 ) [BUG] U8glib compile errors Dec 28, 2023
@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 28, 2023 via email

@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 28, 2023 via email •
edited by thisiskeithb
Loading

@ellensp
Copy link
Contributor

ellensp commented Dec 28, 2023

On board drivers? url to the board? as thats not standard and I wonder if other things have been changed,

@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 28, 2023 via email •
edited by thisiskeithb
Loading

@thisiskeithb thisiskeithb mentioned this issue Dec 28, 2023
Merged
@thisiskeithb
Copy link
Member

Have you successfully ran a prior version of Marlin on this board or is it brand new?

@thisiskeithb thisiskeithb changed the title [BUG] U8glib compile errors [BUG] Robin Nano 1.2 boot issue Dec 28, 2023
@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 28, 2023 via email

@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 28, 2023 via email •
edited by thisiskeithb
Loading

@thisiskeithb
Copy link
Member

thisiskeithb commented Dec 28, 2023 •
edited
Loading

#define STRING_DISTRIBUTION_DATE "2023-10-12"

So you were running this version prior to updating? I think you just happened to get caught in that u8glib error like everyone else and it's not related.

Would you be able to test some bugfix-2.1.x commits to narrow this down?

The strategy is to find a commit from some point in the "distant" past where the feature works (2023-10-12 in your case). Then, test a commit from halfway between that date and today… And then you keep going to the commit halfway in between your "known to work" commit and your "known to be broken" commit until you find the exact day where it broke.

If you started from a point 256 commits in the past, it would take no more than 8 tests to find the exact commit that broke it.

You can do this manually from the bugfix-2.1.x commit history if that's easier than using git bisect:

Manually Test Commits

  1. To manually test commits, browse bugfix-2.1.x's commit history using the < > button on the right:

    image

  2. Click on the green < > Code ▼ button and then "Download ZIP":

    Download ZIP

  3. Apply your minimal config changes, build & flash Marlin to your printer, and observe if the reported issue can be reproduced.

@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 28, 2023 via email

@ahmetcemturan
Copy link
Author

ahmetcemturan commented Dec 28, 2023 via email

@thisiskeithb
Copy link
Member

thisiskeithb commented Dec 28, 2023 •
edited
Loading

U8Glib version 0.5.4 is now available in PIO's registry, so it'll be used when compiling new firmware (and I opened a PR to update our features.ini to ensure it is used), so this is resolved.

@ahmetcemturan
Copy link
Author

Y Endstop is unresponsive. Not inverted.
It Works when I disable //#define FT_MOTION in FW

@thinkyhead
Copy link
Member

thinkyhead commented Dec 29, 2023 •
edited
Loading

Y Endstop is unresponsive. Not inverted. when I disable FT_MOTION

If you can't find a resolution post a new issue about it and there we can work towards a solution. In the meantime, you might enable PINS_DEBUGGING and examine the output of M43 to make sure nothing is conflicting with the Y endstop pin. The FT_MOTION feature is pretty new, and it might not be 100% compatible with every type of endstop yet, so we may want to look at that as well.

@github-actions 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 Feb 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Bug: Confirmed !
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ellensp @thinkyhead @ahmetcemturan @thisiskeithb