Dear Prafulla Wadaskar,

In message <73173d32e9439e4abb5151606c3e19e202e1216...@sc-vexch1.marvell.com> 
you wrote:
> 
> I have gone through all the feedback that you have provided for this entire 
> patch series.
> Thanks a lot...

Thank you for your pat6ience. Sorry it's such a long story for such a
apparently small addition.

> I tried to maintain the flow in the series for easy understanding, but I 
> failed :-(

Actually you did not. I just did not agree with some implementation
details. And the changes need beter documentation.

> With these patches, new code looks differently compared to original code.
> So I feel instead of providing incremental patches- I should provide
> 1. basic mkimge framework patch,
> 2. mkimage: add: uimage support patch,
> 3. mkimage: add: fit image support patch,

Um... these three must never be split. I mean, we should not add any
patch that breaks the (existing) mkimage support.

> 4. mkimage: add: Kirkwood boot image support patch.

> I will do this in my next post.
> I hope you will like it.
> If you have any inputs in this regard or you think this is bad plan, pls 
> kindly let me know.

I'm not sure if this will be an improvement. I guess it will make it
even harder to understand the changes.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"Infidels in all ages have battled for the rights of man, and have at
all times been the fearless advocates of liberty and justice."
- Robert Green Ingersoll
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to