RIOT icon indicating copy to clipboard operation
RIOT copied to clipboard

sys/shell/cmds: add MTD command from `mtd_raw` test

Open fabian18 opened this issue 7 months ago • 3 comments

Contribution description

This PR moves the MTD shell command from tests/drivers/mtd_raw to the general reusable shell commands, because I find it useful and want to reuse it.

Additionally the optional parameter -b can be used to write binary data..

Testing procedure

make -C tests/drivers/mtd_raw/ flash

2025-05-14 15:43:57,877 # mtd 0 read 0 1024
2025-05-14 15:43:57,973 # 00000000  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00
...
2025-05-14 15:43:58,084 # 00000100  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00
...
2025-05-14 15:43:58,181 # 00000200  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00
...
2025-05-14 15:43:58,277 # 00000300  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00
...
2025-05-14 15:43:58,373 # 000003F0  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00
2025-05-14 15:44:18,947 # mtd 0 write -b 0 abcd
> mtd 0 read 0 1024
2025-05-14 15:44:21,747 # mtd 0 read 0 1024
2025-05-14 15:44:21,860 # 00000000  AB  CD  00  00  00  00  00  00  00  00  00  00  00  00  00  00
2025-05-14 15:45:04,321 # mtd 0 write 256 abcd
> mtd read 0 1024cd
2025-05-14 15:45:05,905 # mtd 0 read 0 1024
2025-05-14 15:45:06,002 # 00000000  AB  CD  00  00  00  00  00  00  00  00  00  00  00  00  00  00
....
2025-05-14 15:45:06,099 # 00000100  61  62  63  64  00  00  00  00  00  00  00  00  00  00  00  00

Issues/PRs references

fabian18 avatar May 14 '25 14:05 fabian18

Murdock results

:heavy_check_mark: PASSED

1d1b3547aa2d213746cb22afe04014ca402fad13 sys/shell/cmds: add mtd.doc.md

Success Failures Total Runtime
10559 0 10560 17m:37s

Artifacts

riot-ci avatar May 14 '25 14:05 riot-ci

image

Hello empty page bug, my old friend...

This is a Doxygen thing though. @mguetschow any ideas how to hide that? 👀

crasbe avatar May 19 '25 14:05 crasbe

Actually, https://ci.riot-os.org/results/4a5ff24f6b76447193af650c2ff1a0f2/doc-preview/group__sys__shell__commands.html is also not what I would expect. Are we sure we're using the right doxygen commands?

mguetschow avatar May 19 '25 16:05 mguetschow