Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Partition table size is documented as 0xC00, but is set to 0x1000 (IDFGH-8377) #9846

Closed
3 tasks done
axos88 opened this issue Sep 24, 2022 · 2 comments
Closed
3 tasks done
Assignees
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally Type: Bug bugs in IDF

Comments

@axos88
Copy link
Contributor

axos88 commented Sep 24, 2022

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.

master

Operating System used.

Linux

How did you build your project?

Command line with idf.py

If you are using Windows, please specify command line type.

No response

What is the expected behavior?

With the default partition table offset at 0x8000, and the partition table length of 0xC00, the offset of the first partition should be able to start at 0x8C00, as documented here: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-guides/partition-tables.html

What is the actual behavior?

PARTITION_TABLE_SIZE = 0x1000 # Size of partition table

Steps to reproduce.

Using any example project, modify the partition table so that the first partition starts at offset 0x8C00, and attempt to build

Build or installation Logs.

CSV Error: First partition offset 0x8c00 overlaps end of partition table 0x9000

More Information.

Either the documentation needs to be updated, or the

PARTITION_TABLE_SIZE = 0x1000 # Size of partition table
needs to be updated according to the actual size.

1K space is lost this way.

@axos88 axos88 added the Type: Bug bugs in IDF label Sep 24, 2022
@espressif-bot espressif-bot added the Status: Opened Issue is new label Sep 24, 2022
@github-actions github-actions bot changed the title Bootloader size is documented as 0xC00, but is set to 0x1000 Bootloader size is documented as 0xC00, but is set to 0x1000 (IDFGH-8377) Sep 24, 2022
@igrr
Copy link
Member

igrr commented Sep 25, 2022

@axos88 the error message is misleading here, the actual reason why the partition can't start at 0x8c00 is that we don't support partitions which aren't aligned to SPI Flash erase sector size (0x1000).
We'll take a look at the tool and make changes so that the correct error is reported.

@igrr igrr changed the title Bootloader size is documented as 0xC00, but is set to 0x1000 (IDFGH-8377) Partition table size is documented as 0xC00, but is set to 0x1000 (IDFGH-8377) Sep 25, 2022
@axos88
Copy link
Contributor Author

axos88 commented Sep 26, 2022

Ah thanks for updating the title. Yeah I can see why that would be a problem, but it indeed needs to throw a better error message, and it probably needs mention in the partition table documentation

@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development and removed Status: Opened Issue is new labels Oct 19, 2022
@espressif-bot espressif-bot added Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Resolution: Done Issue is done internally and removed Status: Selected for Development Issue is selected for development Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed Resolution: NA Issue resolution is unavailable labels Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally Type: Bug bugs in IDF
Projects
None yet
Development

No branches or pull requests

4 participants