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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
