Hi Michal,


On Wed, 6 Aug 2014 13:27:27 +0200
Michal Simek <michal.si...@xilinx.com> wrote:


> > 
> > In this case, savedefconfig was done based on
> > configs/sandbox_defconfig which is empty for now,
> > but which will have more options in the future.
> 
> No problem with that.
> The question and my concern is that every .config
> will contain this line and this is just one usage
> which is questionable. IMHO if there is no .config
> savedefconfig should failed to let user to know
> that something is wrong. But that's just my opinion.


Finally, I have posted a patch.

http://patchwork.ozlabs.org/patch/380400/



> > 
> > Whereas, this case, savedefconfig failed,
> > that is why the file is empty.
> 
> It doesn't look at it is failing - just don't use that default one.

You are right.  It seems to exit successfully with doing nothing.




Best Regards
Masahiro Yamada
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to