[PATCH v8 0/4] mtd: devices: elm: add checks ELM H/W constrains, driver code cleanup
Ezequiel Garcia
ezequiel.garcia at free-electrons.com
Thu Mar 20 09:48:28 EDT 2014
On Mar 20, Pekon Gupta wrote:
> *changes v7 -> v8*
> Incorporated feedbacks from Brian Norris <computersforpeace at gmail.com>
> - renamed ecc_step_bytes -> ecc_syndrome_size
>
>
> *changes v6 -> v7*
> Incorporated feedbacks from Ezequiel Garcia <ezequiel.garcia at free-electrons.com>
> - using dev_err() instead of pr_err()
> - moved un-related addition of info->ecc_steps, info->ecc_step_bytes,
> info->ecc_step_size from [PATCH v6 1/4] into subsequent patches
> - dropped pr_fmt() change
>
>
> *changes v5 -> v6*
> [PATCH 02/04] minor cleanup
>
> *changes v4 -> v5*
> This patch series is split version from earlier series [1].
> This series refactors and cleans ELM driver which is used by Hardware
> based BCHx ecc-schemes.
> - Undo: introduction of 'struct mtd_info' and 'struct nand_chip'. Instead
> keep ELM driver independent of mtd_info and nand_chip structs and
> pass only required ECC configurations as elm_config() arguments
> elm_config(..., int ecc_steps, int ecc_step_size, int ecc_step_bytes)
> - Undo: re-writing of elm_load_syndrome() ECC register configurations.
>
> *changes v3 -> v4 [1]*
> - in-corporated feedbacks from Brian Norris <computersforpeace at gmail.com>
> - updated: use 'pr_fmt(fmt)' to suffix DRIVER_NAME
> - removed: local 'eccsteps' in ELM driver, instead using nand_chip->ecc.steps
> - undo: irrelavant white-space changes
>
> [1] http://lists.infradead.org/pipermail/linux-mtd/2013-November/050242.html
>
> Pekon Gupta (4):
> mtd: devices: elm: check for hardware engine's design constraints
> mtd: devices: elm: clean elm_load_syndrome
> mtd: devices: elm: configure parallel channels based on ecc_steps
> mtd: devices: elm: update DRIVER_NAME as "omap-elm"
>
For the whole series:
Reviewed-by: Ezequiel Garcia <ezequiel.garcia at free-electrons.com>
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
More information about the linux-mtd
mailing list