Pad all but last sector in flash binary to write whole of sector #800
No reviewers
Labels
No labels
bazel
bluetooth
board settings
boot_stage2
bug
build
c++
can't reproduce
clang
clib-interface
cmsis
code cleanliness
code size
compiler version issues
cyw43_driver
debug
discussion
documentation
duplicate
dynamic_clock
elf2uf2
enhancement
errata
filesystem
flash layout
floating point
FreeRTOS
hardware_adc
hardware_claim
hardware_clocks
hardware_dma
hardware_exception
hardware_flash
hardware_gpio
hardware_i2c
hardware_irq
hardware_pio
hardware_pwm
hardware_rcp
hardware_rtc
hardware_spi
hardware_sync
hardware_uart
hardware_vreg
hardware_watchdog
hardware_xosc
help wanted
host
IAR
installation
invalid
linker scripts
lto
lwip
mbedtls
missing API
needs reporter detail
networking
newlib
newlib-multicore
non-c
non-cmake
non-gcc
openocd
orphan PR
OS integration
pico_async_context
pico_base
pico_binary_info
pico_flash
picolibc
pico_lwip
pico_malloc
pico_multicore
pico_printf
pico_runtime
pico_stdio
pico_stdio_usb
pico_sync
pico_time
pico_util
pioasm
port
question
review
review2
rtos interop
runtime init
static library
svd
test
tinyusb
vscode
wifi integration
wontfix
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
gh-raspberrypi/pico-sdk!800
Loading…
Reference in a new issue
No description provided.
Delete branch "sparse_uf2"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Additional changes on top of #763
@ -296,0 +315,4 @@// Create a dummy page, if it does not exist yet. note that all present pages are first// zeroed before they are filled with any contents, so a dummy page will be all zeros.auto &dummy = pages[page];}For this last sector, it makes sense that we don't want/need to add any zero-padding after the data in that sector; but do any holes before the data in the last sector still need to be filled with zeroes?
@ -296,0 +315,4 @@// Create a dummy page, if it does not exist yet. note that all present pages are first// zeroed before they are filled with any contents, so a dummy page will be all zeros.auto &dummy = pages[page];}good point :-)
@ -296,0 +315,4 @@// Create a dummy page, if it does not exist yet. note that all present pages are first// zeroed before they are filled with any contents, so a dummy page will be all zeros.auto &dummy = pages[page];}actually it doesn't matter from a correctness point of view (the actual data of the last sector will be written correctly), however as it stands a gap in the last sector will not be filled with zeroes.
fixes #762