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

spi_nor: do not DPD in idle on HW3 #322

Merged
merged 1 commit into from
Jul 21, 2024
Merged

spi_nor: do not DPD in idle on HW3 #322

merged 1 commit into from
Jul 21, 2024

Conversation

ldab
Copy link
Collaborator

@ldab ldab commented Jul 20, 2024

As CI will show, by changing this the overall performance increases significantly

As CI will show, by changing this the overall performance increases significantly
@Kampi
Copy link
Collaborator

Kampi commented Jul 20, 2024

Is there a way to manually set the flash in DPD? Otherwise we will have an increased power consumption

@jakkra
Copy link
Owner

jakkra commented Jul 20, 2024

Is there a way to manually set the flash in DPD? Otherwise we will have an increased power consumption

I'm not sure exactly what this config does, but if it means never dpd, the cut is from 2uA to 10uA which is not that bad overall. This also only affects v3 pcb.

Manual enter and exit of dpd I'll investigate on v5, as doing it while we are active will just slow down.

@Kampi
Copy link
Collaborator

Kampi commented Jul 20, 2024

mmh let´s test it :)
I´m not sure about the current consumption yet, because the current consumption isn´t good (with my rev4), so I would avoid adding more.

@jakkra
Copy link
Owner

jakkra commented Jul 20, 2024

mmh let´s test it :) I´m not sure about the current consumption yet, because the current consumption isn´t good (with my rev4), so I would avoid adding more.

Does does not affect v4, it's v3 only because this is for spi driver and not the QSPI driver

@Kampi
Copy link
Collaborator

Kampi commented Jul 20, 2024

mmh let´s test it :) I´m not sure about the current consumption yet, because the current consumption isn´t good (with my rev4), so I would avoid adding more.

Does does not affect v4, it's v3 only because this is for spi driver and not the QSPI driver

Ah okay. Guess I understand it wrong :)

@ldab ldab merged commit 2d26ac6 into main Jul 21, 2024
12 of 13 checks passed
@ldab ldab deleted the ldab-patch-1 branch July 21, 2024 05:27
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