On Thu, 26 Jan 2017 18:08:49 +0100
Alexander Graf <ag...@suse.de> wrote:

> On 01/26/2017 05:46 PM, Emmanuel Vadot wrote:
> > Add a FreeBSD option that enables the API and the data cache command
> > as both are needed to boot the FreeBSD loader.
> >
> > Signed-off-by: Emmanuel Vadot <m...@bidouilliste.com>
> > ---
> >   common/Kconfig | 9 +++++++++
> >   1 file changed, 9 insertions(+)
> >
> > diff --git a/common/Kconfig b/common/Kconfig
> > index 9067d1e277..2fe19d354b 100644
> > --- a/common/Kconfig
> > +++ b/common/Kconfig
> > @@ -430,4 +430,13 @@ config BOARD_EARLY_INIT_F
> >   
> >   endmenu
> >   
> > +config FREEBSD
> > +   bool "Enable FreeBSD boot"
> > +   select API
> > +   select CMD_CACHE
> > +   default n
> > +   help
> > +     This options adds boot configuration that can run the FreeBSD
> 
> The help text should be more descriptive and actually tell users what 
> the option really does. 

 Ok will do that.

> Since the only thing CONFIG_FREEBSD does now is 
> to enable distro boot support for it, maybe even rename the the option 
> into CONFIG_DISTRO_FREEBSD.

 We might want to support uImage/FIT at one point (I'm currently
looking at this) so CONFIG_FREEBSD will make more sense. But we can
rename the option later if needed, I don't know which way is better.

-- 
Emmanuel Vadot <m...@bidouilliste.com> <m...@freebsd.org>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to