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

subsis/disk/disk_access_spi_sdhc: response data eaten by idle byte consumption #17657

Closed
pabigot opened this issue Jul 18, 2019 · 3 comments
Closed
Labels
area: Disk Access bug The issue is a bug, or the PR is fixing a bug

Comments

@pabigot
Copy link
Collaborator

pabigot commented Jul 18, 2019

I'm working with samples/subsys/fs which uses the SPI SDHC driver. I've got a uSD breakout board on the Particle Xenon (nRF52840 using SPI not SPIM).

The driver has several locations, starting in sdhc_spi_cmd_r1_raw, where the following is present:

	/* Ensure there's a idle byte between commands */
	sdhc_spi_rx_u8(data);

In my system what this does is eat the first byte of the response, causing a Storage init ERROR!. If I remove that read the sample runs and displays the files present on the card. I can also take out the other places where this is done, and it continues to work.

I've got to assume that this works for somebody as-is and so might be necessary, so I don't want to just take it out. I'd be happy to coordinate with somebody who understands that driver to help figure out why it's there and the conditions under which it's necessary.

@pabigot pabigot added bug The issue is a bug, or the PR is fixing a bug area: Disk Access labels Jul 18, 2019
@mrfuchs
Copy link
Contributor

mrfuchs commented Jul 18, 2019

Does #15790 help as well?

@pabigot
Copy link
Collaborator Author

pabigot commented Jul 18, 2019

#15790, when rebased, fixes it. Thanks.

@pabigot
Copy link
Collaborator Author

pabigot commented Jul 18, 2019

Duplicate of #15444

@pabigot pabigot marked this as a duplicate of #15444 Jul 18, 2019
@pabigot pabigot closed this as completed Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Disk Access bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Development

No branches or pull requests

2 participants