[U-Boot] [PATCH v3 0/3] spi, sf: add mtdparts feature to spi and sf commands

[Heiko Schocher <hs@denx.de>]

Hello Jagannadha Sutradharudu Teki,

Am 05.09.2014 07:38, schrieb Heiko Schocher:
> This patchserie add the popssibility to define mtd partitions on
> spi nor flash, and use this settings with the sf commands.
>
> steps:
>
> - add MTD layer driver for spi, original patch from:
> http://git.denx.de/?p=u-boot/u-boot-mips.git;a=commitdiff;h=bb246819cdc90493dd7089eaa51b9e639765cced
>
>    and addapted it to current mainline.
>
> - move common functions to get offset and size from
>    cmdline nand command to extract offset and size from
>    a mtd partition to common place "drivers/mtd/mtd_uboot.c"
>    maybe another place is better?
>
> - add to the sf command the possibility to use offset and size from
>    the settings in mtdparts
>
> With this patchset, the sf command looks now:
>
> =>  sf
> sf - SPI flash sub-system
>
> Usage:
> sf probe [[bus:]cs] [hz] [mode] - init flash device on given SPI bus
>                                    and chip select
> sf read addr offset|partition len       - read `len' bytes starting at
>                                            `offset' to memory at `addr'
> sf write addr offset|partition len      - write `len' bytes from memory
>                                            at `addr' to flash at `offset'
> sf erase offset|partition [+]len        - erase `len' bytes from `offset'
>                                            `+len' round up `len' to block size
> sf update addr offset|partition len     - erase and write `len' bytes from memory
>                                            at `addr' to flash at `offset'
> =>
> for example "env" is defined in mtdparts:
>
> =>  sf read 13000000 env
> device 0 offset 0xd0000, size 0x10000
> SF: 65536 bytes @ 0xd0000 Read: OK
> =>


>
> There are the followings checkpatch warnings:
>
> CHECK: Alignment should match open parenthesis
> #153: FILE: common/cmd_nand.c:217:
> +               if (arg_off(argv[2],&idx,&addr,&maxsize,&maxsize,
> +                   MTD_DEV_TYPE_NAND, nand_info[idx].size)) {
>
> CHECK: Alignment should match open parenthesis
> #179: FILE: common/cmd_nand.c:557:
> +                       if (arg_off(argv[3],&dev,&off,&size,&maxsize,
> +                           MTD_DEV_TYPE_NAND, nand_info[dev].size))
>
> CHECK: Alignment should match open parenthesis
> #193: FILE: common/cmd_nand.c:576:
> +                       if (arg_off_size(argc - 3, argv + 3,&dev,&off,&size,
> +&maxsize, MTD_DEV_TYPE_NAND,
>
> total: 0 errors, 0 warnings, 3 checks, 361 lines checked
>
> NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX MULTISTATEMENT_MACRO_USE_DO_WHILE NETWORKING_BLOCK_COMMENT_STYLE USLEEP_RANGE
>
> 20140714_ml_mtdparts/0002-mtd-nand-move-common-functions-from-cmd_nand.c-to-co.patch has style problems, please review.
>
> I see not, why this warning pops up ...
>
> - changes for v2:
>    - mtd-spi-add-MTD-layer-driver.patch
>      - add comment from Daniel Schwierzeck:
>        fix compile error from original patch with
>        "static inline" rather than "static __maybe_unused"
> - changes for v3:
>    - rebase with d6c1ffc7d23f4fe4ae8c91101861055b8e1501b6
>    - add comments from scott wood:
>      - align MTD_DEV_TYPE_NAND correct
>      - remove unnecessary inline
>      - rework "jffs2 header" problem later
>
> Cc: Scott Wood<scottwood@freescale.com>
> Cc: Tom Rini<trini@ti.com>
> Cc: Daniel Schwierzeck<daniel.schwierzeck@gmail.com>
> Cc: Jagannadha Sutradharudu Teki<jagannadh.teki@gmail.com>

ping? Do you have any objections against this patchseries, or do you
think it is ok?

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany