Haavard Skinnemoen wrote: > On Thu, 12 Jun 2008 19:19:47 +0200 > "Ulf Samuelsson" <[EMAIL PROTECTED]> wrote: > >> Haavard Skinnemoen wrote: >>> It's a bit hard to see what your proposal is all about when you >>> create a new file instead of modifying the exising one... >>> >> >> If you want to see changes right now, >> then just replace the existing file with the Diopsis file and do a >> diff. > > The whole idea about e-mail review is that someone posts a patch and > someone else reviews it. >>> So how about we start by introducing a new drivers/mmc directory and >>> move the existing AVR32 driver there? After that, you can apply your >>> changes to it and send a patch which clearly shows the differences >>> from the old code. Don't worry about breaking AVR32 -- I'll help you >>> test it before it gets merged upstream. >>> >> >> Why not get the Diopsis support in first, and then do the merge >> afterwards. I do agree that they should be merged, but that does not >> mean >> that delaying the availability of Diopsis support in U-Boot is a >> good idea. > > I disagree. Why do a half-assed job when you can do it properly?
Some times half-assed jobs, are good enough, and if you concentrate all your efforts on one parts, then everything else suffers. Currently the Diopsis configuration does not support environment variables, and I much rather have Antonio spend time on fixing that problem, than merging the MCI support. Either by using the onboard parallel flash (which is complicated since it is 2 x 16 bit AT45BV64x chips in a by 32 configuration). I am really unsure U-boot supports this... Or figures out a way to read/write the environment from/to the SD-Card. While the duplication is unfortunate, the end user will not suffer too much compared to not having environment variables. > Besides, the merge window is closed now, isn't it? So we have lots of > time to review and test things before the next merge window. > > Haavard Best Regards Ulf Samuelsson ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://sourceforge.net/services/buy/index.php _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users