[U-Boot] [PATCH v3 2/4] kconfig: Add a FREEBSD option

Emmanuel Vadot manu at bidouilliste.com
Thu Jan 26 18:44:08 CET 2017


On Thu, 26 Jan 2017 18:08:49 +0100
Alexander Graf <agraf at 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 <manu at 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 <manu at bidouilliste.com> <manu at freebsd.org>


More information about the U-Boot mailing list