Fixed PUYA flash write buffer alignment by drzony · Pull Request #7491 · esp8266/Arduino · GitHub
Skip to content

Fixed PUYA flash write buffer alignment#7491

Merged
devyte merged 3 commits into
esp8266:masterfrom
drzony:master
Jul 30, 2020
Merged

Fixed PUYA flash write buffer alignment#7491
devyte merged 3 commits into
esp8266:masterfrom
drzony:master

Conversation

@drzony

@drzony drzony commented Jul 29, 2020

Copy link
Copy Markdown
Contributor

This merge fixes flash writing on PUYA flashes when write size is not a multiple of 4.
This bug prevents updating via Arduino OTA when using gzipped image.

@earlephilhower earlephilhower left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, thanks! The rounding up to x4 is a little odd but I think it works fine.

Comment thread cores/esp8266/Esp.cpp Outdated
@devyte devyte merged commit 85ea47e into esp8266:master Jul 30, 2020
@devyte devyte added this to the 3.0.0 milestone Jul 30, 2020
d-a-v added a commit that referenced this pull request Aug 2, 2020
@yumkam

yumkam commented Aug 7, 2020

Copy link
Copy Markdown

@drzony

drzony commented Aug 7, 2020

Copy link
Copy Markdown
Contributor Author

Thanks for the observation @yumkam , must have missed this somehow. Another pull request is on the way.

@drzony

drzony commented Aug 7, 2020

Copy link
Copy Markdown
Contributor Author

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.

5 participants