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

Update[1] external-ram.rst #670

Closed
wants to merge 1 commit into from
Closed

Update[1] external-ram.rst #670

wants to merge 1 commit into from

Conversation

ESP32DE
Copy link
Contributor

@ESP32DE ESP32DE commented Jun 3, 2017

see the Getting Starting Guide of Wrover Kit

wrover_get_started

"GPIO16 and GPIO17 are used as the CS and clock signal for PSRAM."

means
GPIO16 = CS
GPIO17 = CLK

btw:
PSRAM SIO[2] (pin 3) - flash WP
perhabs you can describe - for what we need this pin and how we handle it.
not sure why here is named WP
WP is usually used in SPI Flash - - do we need this in SPI RAM too?

see the Getting Starting Guide of Wrover Kit
http://espressif.com/sites/default/files/documentation/esp-wrover-kit_getting_started_guide_en.pdf

"GPIO16 and GPIO17 are used as the CS and clock signal for PSRAM."
 
means
GPIO16 = CS
GPIO17 = CLK

btw: 
PSRAM SIO[2] (pin 3) - flash WP
perhabs you can describe - for what we need this pin and how we handle it.
not sure why here is named WP
WP is usually used in SPI Flash - - do we need this in SPI RAM too?
@ESP32DE
Copy link
Contributor Author

ESP32DE commented Jun 4, 2017

and yes, if you see here
https://github.com/espressif/esp-idf/blob/feature/psram_malloc/components/esp32/psram.c#L50
the pins swapped to the describtion

@ESP32DE
Copy link
Contributor Author

ESP32DE commented Jun 10, 2017

you allready done it now without merge so this can be closed
new open PR is here #686

@ESP32DE ESP32DE closed this Jun 10, 2017
@ESP32DE ESP32DE changed the title Update external-ram.rst Update[1] external-ram.rst Jun 10, 2017
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.

None yet

1 participant