Make 'fill' optional with 'split' option enabled
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 64
- Forks
- 56
- PR merge metrics
- No merged PRs in 30d
Description
When generating a .bin boot image with the 'split' option enabled, it would be useful to allow the option to disable the 'fill' option. This allows the generation of smaller files and prevent erasing memory areas which are not to be erased.
This is useful if partitions are to be flashed separately (both in time and memory space) from each others or from the boot code / headers.
Currently the implementation ignores the "doFill" argument: https://github.com/Xilinx/bootgen/blame/1795f18361caf1536d76ffe40ae3cb66e212b020/binfile.cpp#L87
Implementation wise, this would involve a new command line option (e.g. -no-fill) which sets the doFill property https://github.com/Xilinx/bootgen/blob/master/options.h#L267 , maybe checking that the 'split' option is set.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with binfile.cpp around the ignored doFill argument and options.h around the doFill property. Trace how the split option is parsed and how command-line options reach image generation; done means a no-fill option can disable filling for split .bin images while the existing default behavior remains unchanged.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- cli, embedded-iot
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100