Messages in this thread | | | Date | Tue, 29 Jul 2014 03:39:00 -0700 | From | Tony Lindgren <> | Subject | Re: [RFC PATCH 00/10] OMAP: GPMC: NAND: Introduce GPMC APIs for OMAP NAND |
| |
* Roger Quadros <rogerq@ti.com> [140711 03:25]: > On 07/11/2014 12:42 PM, Gupta, Pekon wrote: > > > > Therefore, I suggest you to move following code from NAND driver to > > GPMC driver, nothing other than. And this exported function does not > > need any NAND framework information, except number_of_sectors. > > Well I you have still not pointed out what is the benefit of that approach or what are the problems with the approach I am suggesting. > > I am suggesting that the API just return the ECC/BCH result registers as it is. > You are suggesting that the API do the calculations and return the NAND ready ECC buffer. > > To do the calculations it needs to know about the ECC scheme which is NAND controller specific and this is unnecessary in the GPMC driver. > > Tony, what is your opinion?
As long as the GPMC registers are exposed to drivers in a controlled way and we have only one driver using the ECC registers, I don't care.
Regards,
Tony
| |