Re: [PATCH v9 0/9] mtd:nand:omap2: clean-up of supported ECC schemes

2013-10-16 Thread Brian Norris
Hi,

On Tue, Oct 15, 2013 at 11:19:48AM +0530, Pekon Gupta wrote:
> 
> *changes v8 -> v9*
> [PATCH 1/9] 
> [PATCH 2/9] 
>  As per feedbacks from Brian Norris  previous
>  revision [PATCH v8 3/6] and [PATCH 4/6] are split into following sub-patches:
> - [PATCH 3/9]  replaces local reference with generic names (mtd, 
> nand_chip)
> - [PATCH 4/9]  enables auto-detection of bus-width
> - [PATCH 5/9]  removes omap3_init_bch: populates ecc-scheme data
> - [PATCH 6/9]  removes omap3_init_bch_tail: populates ecc-layout
> - [PATCH 7/9]  replaces lib/bch.c with nand_bch.c wrapper
> [PATCH 8/9]  [PATCH 9/9] removed devm_free_xx functions

[...]

I had a few comments on patches 4 and 7, but everything else is looking
a lot better.

Thanks,
Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH v9 0/9] mtd:nand:omap2: clean-up of supported ECC schemes

2013-10-16 Thread Gupta, Pekon
Hi Brian,

> 
> *changes v8 -> v9*
> [PATCH 1/9] 
> [PATCH 2/9] 
>  As per feedbacks from Brian Norris 
> previous
>  revision [PATCH v8 3/6] and [PATCH 4/6] are split into following sub-patches:
> - [PATCH 3/9]  replaces local reference with generic names (mtd,
> nand_chip)
> - [PATCH 4/9]  enables auto-detection of bus-width
> - [PATCH 5/9]  removes omap3_init_bch: populates ecc-scheme data
> - [PATCH 6/9]  removes omap3_init_bch_tail: populates ecc-layout
> - [PATCH 7/9]  replaces lib/bch.c with nand_bch.c wrapper
> [PATCH 8/9]  [PATCH 9/9] removed devm_free_xx functions
> 

Request you to have a look at v9.
If this is acceptable, then I would work to rebase the other pending
series on top of this one.


with regards, pekon
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html