Re: [U-Boot] [PATCH 0/4] config: Add FreeBSD kconfig options

2016-11-17 Thread Tom Rini 
On Thu, Nov 17, 2016 at 03:12:02PM +0100, Emmanuel Vadot wrote:
> On Fri, 11 Nov 2016 11:07:34 -0500
> Tom Rini  wrote:
> 
> > On Mon, Nov 07, 2016 at 09:34:27AM +0100, Emmanuel Vadot wrote:
> > 
> > > This series of patches add the needed bits for booting the FreeBSD
> > > loader.
> > > FreeBSD loader needs the U-Boot API and dache disabled for it to run so
> > > add kconfig options for them.
> > > Also add some some boot command that locate and run the FreeBSD loader
> > > if found.
> > > 
> > > Emmanuel Vadot (4):
> > >   kconfig: Add API kconfig file
> > >   kconfig: arm: Add SYS_DCACHE_OFF option
> > >   kconfig: Add a FREEBSD option
> > >   distro_bootcmd: Add command to run FreeBSD
> > > 
> > >  Kconfig |  2 ++
> > >  api/Kconfig |  9 +
> > >  arch/arm/Kconfig|  6 ++
> > >  common/Kconfig  |  9 +
> > >  include/config_distro_bootcmd.h | 32 
> > 
> > This is a good first start.  But I think there's a few things that need
> > tweaking.  First, we don't want to globally turn off dcache, but it does
> > need to be migrated to Kconfig (so we know if it's enabled or not).
> > Second, we should instead use CMD_CACHE and the 'dcache' command to
> > disable dcache prior to running the FreeBSD loader.  Then we make sure
> > that the generic distro feature has CMD_CACHE if !SYS_DCACHE_OFF (and
> > !SYS_ICACHE_OFF).  Thanks!
> > 
> > -- 
> > Tom
> 
>  Hi Tom,
> 
>  I will make the modification, should I sent the patch for
> migration of SYS_DCACHE_OFF to Kconfig as a separate patch ?

I'll grab (and re-run the moveconfig.py part) for the first two parts of
your series.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 0/4] config: Add FreeBSD kconfig options

2016-11-17 Thread Emmanuel Vadot 
On Fri, 11 Nov 2016 11:07:34 -0500
Tom Rini  wrote:

> On Mon, Nov 07, 2016 at 09:34:27AM +0100, Emmanuel Vadot wrote:
> 
> > This series of patches add the needed bits for booting the FreeBSD
> > loader.
> > FreeBSD loader needs the U-Boot API and dache disabled for it to run so
> > add kconfig options for them.
> > Also add some some boot command that locate and run the FreeBSD loader
> > if found.
> > 
> > Emmanuel Vadot (4):
> >   kconfig: Add API kconfig file
> >   kconfig: arm: Add SYS_DCACHE_OFF option
> >   kconfig: Add a FREEBSD option
> >   distro_bootcmd: Add command to run FreeBSD
> > 
> >  Kconfig |  2 ++
> >  api/Kconfig |  9 +
> >  arch/arm/Kconfig|  6 ++
> >  common/Kconfig  |  9 +
> >  include/config_distro_bootcmd.h | 32 
> 
> This is a good first start.  But I think there's a few things that need
> tweaking.  First, we don't want to globally turn off dcache, but it does
> need to be migrated to Kconfig (so we know if it's enabled or not).
> Second, we should instead use CMD_CACHE and the 'dcache' command to
> disable dcache prior to running the FreeBSD loader.  Then we make sure
> that the generic distro feature has CMD_CACHE if !SYS_DCACHE_OFF (and
> !SYS_ICACHE_OFF).  Thanks!
> 
> -- 
> Tom

 Hi Tom,

 I will make the modification, should I sent the patch for
migration of SYS_DCACHE_OFF to Kconfig as a separate patch ?


-- 
Emmanuel Vadot  
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 0/4] config: Add FreeBSD kconfig options

2016-11-11 Thread Tom Rini 
On Mon, Nov 07, 2016 at 09:34:27AM +0100, Emmanuel Vadot wrote:

> This series of patches add the needed bits for booting the FreeBSD
> loader.
> FreeBSD loader needs the U-Boot API and dache disabled for it to run so
> add kconfig options for them.
> Also add some some boot command that locate and run the FreeBSD loader
> if found.
> 
> Emmanuel Vadot (4):
>   kconfig: Add API kconfig file
>   kconfig: arm: Add SYS_DCACHE_OFF option
>   kconfig: Add a FREEBSD option
>   distro_bootcmd: Add command to run FreeBSD
> 
>  Kconfig |  2 ++
>  api/Kconfig |  9 +
>  arch/arm/Kconfig|  6 ++
>  common/Kconfig  |  9 +
>  include/config_distro_bootcmd.h | 32 

This is a good first start.  But I think there's a few things that need
tweaking.  First, we don't want to globally turn off dcache, but it does
need to be migrated to Kconfig (so we know if it's enabled or not).
Second, we should instead use CMD_CACHE and the 'dcache' command to
disable dcache prior to running the FreeBSD loader.  Then we make sure
that the generic distro feature has CMD_CACHE if !SYS_DCACHE_OFF (and
!SYS_ICACHE_OFF).  Thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 0/4] config: Add FreeBSD kconfig options

2016-11-07 Thread Emmanuel Vadot 
This series of patches add the needed bits for booting the FreeBSD
loader.
FreeBSD loader needs the U-Boot API and dache disabled for it to run so
add kconfig options for them.
Also add some some boot command that locate and run the FreeBSD loader
if found.

Emmanuel Vadot (4):
  kconfig: Add API kconfig file
  kconfig: arm: Add SYS_DCACHE_OFF option
  kconfig: Add a FREEBSD option
  distro_bootcmd: Add command to run FreeBSD

 Kconfig |  2 ++
 api/Kconfig |  9 +
 arch/arm/Kconfig|  6 ++
 common/Kconfig  |  9 +
 include/config_distro_bootcmd.h | 32 
 5 files changed, 58 insertions(+)
 create mode 100644 api/Kconfig

-- 
2.9.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot