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

SDMMC from SPI error - KeyError('mosi') #5

Closed
phodina opened this issue Aug 17, 2022 · 4 comments
Closed

SDMMC from SPI error - KeyError('mosi') #5

phodina opened this issue Aug 17, 2022 · 4 comments

Comments

@phodina
Copy link

phodina commented Aug 17, 2022

Hi,

I'm trying to run the protocol analyzer but I get this weird error in Logic2 (version 2.3.58).

  File "/home/pethod/.config/Logic/Marketplace/24/sdmmc_from_spi.py", line 374, in decode
    data.data["mosi"], data.start_time, data.end_time

Could you help?

@timkostka
Copy link
Owner

timkostka commented Aug 17, 2022

Does your board have separate DAT0 and CMD signals? Can you provide your capture file?

@phodina
Copy link
Author

phodina commented Aug 17, 2022

The issue was I used the extension provided on the Marketplace.

When I checked out this repo 0cf9422 and followed the instructions it worked correctly without any issue.

The difference was I had to create an empty extension as there was no possibility to load one. After that I moved the files from this repo there and renamed it to recognize it in the protocol analyzers.

Not sure if there was some change in the data structure in Logic2 between the versions.

Thanks for this extension as it helps debugging MMC which does not respond to commands.

@timkostka
Copy link
Owner

Well that's weird, and unfortunate.

I can't replicate it sadly. I uninstalled my extensions, added the marketplace one, and was able to use it just fine.

Shrug.

There have been some minor changes to the data structure since this was released, but they don't affect functionality too much. Mostly just the "mytype()" showing in results.

Thanks for reporting what worked for you.

@phodina
Copy link
Author

phodina commented Nov 8, 2022

Thanks. I unfortunately don't have access anymore to Salae so I switched to Keysight osciloscope and simply dump the data and load it into Pulseview. I applied patches that add eMMC decoding and it solved the issues we had with the memory.

sigrokproject/libsigrokdecode#93

@phodina phodina closed this as completed Nov 8, 2022
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

No branches or pull requests

2 participants