[U-Boot] [PATCH v2 02/15] zynq: kconfig: move board select menu and commonsettings

Masahiro Yamada yamada.m at jp.panasonic.com
Mon Aug 18 04:55:42 CEST 2014


Hi Michal,



On Wed, 6 Aug 2014 13:27:27 +0200
Michal Simek <michal.simek at 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


More information about the U-Boot mailing list