[U-Boot] [PATCH 2/5] am335x_evm: Switch to using imply keyword

[Tom Rini]

On Sun, Mar 05, 2017 at 11:53:43PM +0100, Lukasz Majewski wrote:
> Hi Tom,
> 
> > These particular SPL options are part of what the ROM provides, but
> > for compatibility with how we have previously used them, move them to
> > being implied by the board being selected.
> > 
> > Signed-off-by: Tom Rini <trini at konsulko.com>
> > ---
> >  arch/arm/mach-omap2/am33xx/Kconfig | 3 +++
> >  board/ti/am335x/Kconfig            | 9 ---------
> >  2 files changed, 3 insertions(+), 9 deletions(-)
> > 
> > diff --git a/arch/arm/mach-omap2/am33xx/Kconfig
> > b/arch/arm/mach-omap2/am33xx/Kconfig index 5b5d3f8cec29..5adcead185de
> > 100644 --- a/arch/arm/mach-omap2/am33xx/Kconfig
> > +++ b/arch/arm/mach-omap2/am33xx/Kconfig
> > @@ -15,6 +15,9 @@ config TARGET_AM335X_EVM
> >  	select DM_SERIAL
> >  	select DM_GPIO
> >  	select TI_I2C_BOARD_DETECT
> > +	imply SPL_YMODEM_SUPPORT
> > +	imply SPL_ENV_SUPPORT
> > +	imply SPL_WATCHDOG_SUPPORT
> 
> Are you sure that WDT is enabled by ROM by default?

OK, good point, my wording was slightly unclear.  What I meant was that
these are SPL-usable features that are part of the SoC, rather than
something board-centric.  That said, yes, a quick re-read of the
initialization section of the AM335x TRM says that ROM sets up WDT1,
which matches up with my recollections from TI days and board bring up.
It's likely this should become a 'select' as if we don't twiddle it in
time, we will trigger it.

> IIRC we must enable it explicitly in the SPL.

We can reconfigure it to be more "precise" perhaps is the answer.  I
have a recollection of a bring up issue where in the problem was "Oh, we
got somewhat far, but hadn't touched the WDT and reset".

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170305/db79cade/attachment.sig>