Skip to content

move SPI_FLASH_BACKUP to Conditionals-5 again#27936

Merged
thinkyhead merged 3 commits intoMarlinFirmware:bugfix-2.1.xfrom
ellensp:SPI_FLASH_BACKUP-update-again
Jun 22, 2025
Merged

move SPI_FLASH_BACKUP to Conditionals-5 again#27936
thinkyhead merged 3 commits intoMarlinFirmware:bugfix-2.1.xfrom
ellensp:SPI_FLASH_BACKUP-update-again

Conversation

@ellensp
Copy link
Contributor

@ellensp ellensp commented Jun 22, 2025

Description

The test for SPI_FLASH_BACKUP is in Marlin/src/inc/Conditionals-4-adv.h
#if ALL(SPI_FLASH, HAS_MEDIA, MARLIN_DEV_MODE)

SPI_FLASH is set in the boards pins.h file

From Conditionals-4-adv.h
* Conditionals set before pins.h and which depend on Configuration_adv.h.
This is accurate. defines in the pins files are not used here.

Moved the test to Marlin/src/inc/Conditionals-5-post.h again.

Requirements

A board that has SPI flash (such as BOARD_MKS_ROBIN_NANO_V3_1
MARLIN_DEV_MODE to add M993 aand M994
SDSUPPORT required for these gcodes

Benefits

SPI_FLASH_BACKUP is set as expected

Configurations

Configuration.zip

Related Issues

  • [BUG] Compilation Issues as Conditionals not correctly calculated/processed - M993/M994 #27922
  • Spi flash backup update #27934
  • @ellensp ellensp added this to the Version 2.1.3 milestone Jun 22, 2025
    @thinkyhead thinkyhead merged commit 55c15f9 into MarlinFirmware:bugfix-2.1.x Jun 22, 2025
    66 checks passed
    @ellensp ellensp deleted the SPI_FLASH_BACKUP-update-again branch June 22, 2025 21:43
    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.

    [BUG] Compilation Issues as Conditionals not correctly calculated/processed - M993/M994

    2 participants