On Thu, Apr 27, 2023 at 7:24 AM Heinrich Schuchardt <xypron.g...@gmx.de> wrote:
> On 4/26/23 13:05, Shirokov Alexander wrote: > > On Tue, Apr 25, 2023 at 8:01 PM Simon Glass <s...@chromium.org> wrote: > > > >>> --- a/doc/usage/cmd/mmc.rst > >>> +++ b/doc/usage/cmd/mmc.rst > >>> @@ -216,7 +216,7 @@ The raw data can be read/written via 'mmc > >> read/write' command: > >>> => mmc read 0x40000000 0x5000 0x100 > >>> MMC read: dev # 0, block # 20480, count 256 ... 256 blocks read: > OK > >>> > >>> - => mmc write 0x40000000 0x5000 0x10 > >>> + => mmc write 0x40000000 0x5000 100 > >> > >> Drop the other 0x things too? > >> > >>> MMC write: dev # 0, block # 20480, count 256 ... 256 blocks > >> written: OK > >>> > >> > >> Regards, > >> Simon > >> > > > > Hi Simon, > > Sure. I'll do that. But one clarification before. Should I change 0x > only in > > "mmc write" line or change "mmc read" too, to keep the example > consistent? > > > > Please, keep the document consistent. do_mmc_read() uses hextoul(). So > it is safe to remove the 0x prefixes in the mmc read example too. > > Best regards > > Heinrich > > Best regards > > Heinrich > Hello Heinrich, Thank you for your response. The new version of the patch is here: http://patchwork.ozlabs.org/project/uboot/patch/20230427101207.119814-1-shirokoval...@gmail.com/ -- Regards, Alexander