Hi Otavio, On 28/10/2013 12:19, Otavio Salvador wrote: > Hello, > > I am not sure I agree ... > > On Sun, Oct 27, 2013 at 6:47 PM, Marek Vasut <ma...@denx.de> wrote: >> Dear Otavio Salvador, >> >>> On Sun, Oct 27, 2013 at 2:16 PM, Marek Vasut <ma...@denx.de> wrote: >>>> Dear Otavio Salvador, >>>> >>>>> This reads the kernel, ftd and boot into ubifs filesystem. While on >>>>> that, the SD firmware filename definition has been moved next to the >>>>> other SD related commands. >>>>> >>>>> Signed-off-by: Otavio Salvador <ota...@ossystems.com.br> >>>> >>>> Any changes to built-in environment are now NAKd I believe. >>> >>> I don't see a reason to not merged it for now; I will work in porting >>> the environments for the external file but this could go now, anyway. >> >> Read [1], in particular what Stefano said. >> >> http://comments.gmane.org/gmane.comp.boot-loaders.u-boot/169042 >> >> Thus this patch shall be rejected as well, sorry. > > I agree with the goal but until we have a way to insert the /default/ > environment into the binary of U-Boot for use (not adding an extra > environment image) this should be accepted.
Indeed. However, why cannot we do the right thing soon ? It is very interesting if Simon's patches will flow into mainline soon - as feeling, I see some comments by Wolfgang, but no evident signal that they will be stopped. Octavio, I will wait for your patches for a while and if Simon's patches will be accepted soon, I will kindly ask you to reformat your patches, moving the default environment into an .env file. Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
