Skip to content

Preserve reg_info_*.bin during make clobber/distclean - #11

Merged
widgetii merged 1 commit into
masterfrom
fix/preserve-reg-info-on-clobber
May 5, 2026
Merged

Preserve reg_info_*.bin during make clobber/distclean#11
widgetii merged 1 commit into
masterfrom
fix/preserve-reg-info-on-clobber

Conversation

@widgetii

@widgetii widgetii commented May 5, 2026

Copy link
Copy Markdown
Member

Summary

  • make clobber (and therefore make distclean / make mrproper) was deleting the committed reg_info_hi3516av200.bin / reg_info_hi3519v101.bin source blobs at the repo root, because its find $(OBJTREE) ... -name '*.bin' swept them up alongside build artifacts.
  • Add a single ! -name 'reg_info_*.bin' filter to the find so the register-init source files survive a deep clean while everything else (mini-boot.bin, u-boot.bin, etc.) is still removed.
  • make clean was never the culprit — its find pattern doesn't include *.bin. Only clobber/distclean/mrproper triggered the deletion.

Test plan

  • ls reg_info_*.bin shows both files present.
  • After a full build (so mini-boot.bin and u-boot.bin exist at root): make distclean leaves both reg_info_*.bin in place and removes mini-boot.bin / u-boot.bin.
  • ./build.sh builds both SoCs end-to-end and reg_info_*.bin are still present afterwards.

The clobber target's recursive find matched every *.bin under OBJTREE,
including the committed reg_info_hi3516av200.bin and reg_info_hi3519v101.bin
source blobs at the repo root, forcing a git checkout to recover before
each rebuild. Exclude them from the find so distclean still cleans
mini-boot.bin, u-boot.bin and friends but leaves source data alone.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@widgetii
widgetii merged commit 0216599 into master May 5, 2026
2 checks passed
@widgetii
widgetii deleted the fix/preserve-reg-info-on-clobber branch May 5, 2026 16:45
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