Skip to content

[feature] added flash support analogous to esp32c3 to esp32s3#5397

Open
Wikwoj0512 wants to merge 1 commit into
tinygo-org:devfrom
Wikwoj0512:esp32s3-flash
Open

[feature] added flash support analogous to esp32c3 to esp32s3#5397
Wikwoj0512 wants to merge 1 commit into
tinygo-org:devfrom
Wikwoj0512:esp32s3-flash

Conversation

@Wikwoj0512
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Member

@aykevl aykevl left a comment

Choose a reason for hiding this comment

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

Looks reasonable, though I didn't check the modifications to esp32s3.ld. Can someone else also take a look?

}

// WriteAt writes the given number of bytes to the block device.
// Only word (32 bits) length data can be programmed.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

// Only word (32 bits) length data can be programmed.

Did you check whether this is also true for the ESP32S3? I couldn't easily find it in the SDK: https://docs.espressif.com/projects/esp-idf/en/v6.0.1/esp32s3/api-reference/peripherals/spi_flash/index.html

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.

3 participants