Skip to content

fix(cardputer-ci): Fix CI build of cardputer which consumes more flash on older ESP-IDF#670

Merged
finger563 merged 1 commit into
mainfrom
fix/ci-cardputer-biuld
Jul 15, 2026
Merged

fix(cardputer-ci): Fix CI build of cardputer which consumes more flash on older ESP-IDF#670
finger563 merged 1 commit into
mainfrom
fix/ci-cardputer-biuld

Conversation

@finger563

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings July 15, 2026 18:39
@github-actions

Copy link
Copy Markdown

✅Static analysis result - no issues found! ✅

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts the m5stack-cardputer example’s flash partitioning so the CI build has a larger/appropriate memory layout on older ESP-IDF versions that produce bigger binaries, aligning this example with other repo examples that use a custom partition table at 0x9000.

Changes:

  • Enables a custom partition table for the cardputer example (CONFIG_PARTITION_TABLE_CUSTOM*) and sets the partition table offset to 0x9000.
  • Adds an example-local partitions.csv defining NVS/PHY/app/littlefs partitions sized to fit within 8MB flash.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
components/m5stack-cardputer/example/sdkconfig.defaults Enables custom partition table usage and points to partitions.csv with offset 0x9000.
components/m5stack-cardputer/example/partitions.csv Defines the custom partition layout (NVS/PHY/factory app/littlefs) for the example.

@finger563
finger563 merged commit 2a241da into main Jul 15, 2026
128 checks passed
@finger563
finger563 deleted the fix/ci-cardputer-biuld branch July 15, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants