[U-Boot] [PATCH v7 1/5] mtd: nand: omap: enable BCH ECC scheme using ELM for generic platform

Gupta, Pekon pekon at ti.com
Tue Oct 8 21:06:05 CEST 2013


Hi,

> On Tue, 2013-10-08 at 11:52 -0500, Scott Wood wrote:
> > On Tue, 2013-10-08 at 05:30 +0000, Gupta, Pekon wrote:
> > Anyways I would take the changes if you wish so..
> > But request you to please provide comments on all the patches, before
> > I send next revision. This would help me consolidate all changes.
>
> I'm working on it, but I also have a lot of other patches to review, and
> doing nothing but patch reviews all day long can be very draining (not
> to mention starving other tasks that need to be done), and reviewing
> patches for hardware I'm not familiar with is even worse.  I realize
> that the delays can be frustrating from your end, but I only have so
> much time and energy to expend.
> 
Firstly my apologies for being bit aggressive in getting patches
reviewed. Yes, I was bit irritated as patch re-submissions were
taking much of my energy and time as well, while lot of other things
are still pending on me to cleanup this driver and add functionality to it.

And I understand the patch reviewing is tough because reviewer
needs to first understand the driver & hardware context, and then
reply from a broader point of view of generic frame-work.


>   Is there nobody else who's familiar
> with this driver that can help review?  Looking at the commit history no
> single person stands out as a logical maintainer for this driver.  It
> would be nice to designate someone.
> 
Currently I'm the only supporting NAND driver in both kernel and
u-boot from omap side. And therefore, I plan to get both u-boot and 
kernel nand drivers cleaned-up and optimized, so that it becomes
scalable for future enhancements.
Aim is to get most of end-users to boot from mainline, so that they
are self dependent and up-to-date to new frameworks.

Once I'm done with my current commitments, I can help you in getting
some enhancements for generic framework.
For example, following patch adds checks for automatic bus-width
detection in generic NAND framework.
http://lists.denx.de/pipermail/u-boot/2013-September/163882.html

Hope such things help reduce some of your load.
And thanks again for the reviews, I'll submit next revision of patch soon.


with regards, pekon


More information about the U-Boot mailing list