All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 2/2] watchdog: omap_wdt: Disable DM watchdog support in SPL
Date: Tue, 13 Aug 2019 12:50:52 -0400	[thread overview]
Message-ID: <20190813165052.GG2718@bill-the-cat> (raw)
In-Reply-To: <1564590247-22406-2-git-send-email-sunil.m@techveda.org>

On Wed, Jul 31, 2019 at 09:54:07PM +0530, sunil.m at techveda.org wrote:

> From: Suniel Mahesh <sunil.m@techveda.org>
> 
> This patch disables DM watchdog support for SPL builds and uses
> the legacy omap watchdog driver on TI AM335x chipsets.
> 
> The following build error is reported if DM watchdog support was
> enabled in SPL:
> 
>   CC      spl/drivers/usb/gadget/rndis.o
>   LD      spl/drivers/usb/gadget/built-in.o
>   LD      spl/drivers/usb/musb-new/built-in.o
>   LD      spl/drivers/built-in.o
>   LD      spl/u-boot-spl
> arm-linux-ld.bfd: u-boot-spl section .u_boot_list will not fit in region .sram
> arm-linux-ld.bfd: region .sram overflowed by 440 bytes
> make[1]: *** [spl/u-boot-spl] Error 1
> make: *** [spl/u-boot-spl] Error 2
> 
> Adjusted WATCHDOG_RESET macro accordingly. Earlier it was pointing
> to hw_watchdog_reset. Since CONFIG_WATCHDOG replaces CONFIG_HW_WATCHDOG,
> now WATCHDOG_RESET macro points to watchdog_reset. This watchdog_reset
> is not defined anywhere for am33xx/omap2 and needs to be defined. Fixed
> this by simply calling hw_watchdog_reset in watchdog_reset.
> 
> Built and tested on AM335x device (BeagleboneBlack), compile tested for
> all other AM33xx/omap2 based boards.
> 
> Signed-off-by: Suniel Mahesh <sunil.m@techveda.org>

Applied to u-boot/master, thanks!

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

  reply	other threads:[~2019-08-13 16:50 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-12  9:23 [U-Boot] [PATCH 2/2] watchdog: omap_wdt: Disable DM watchdog support in SPL sunil.m at techveda.org
2019-07-17 17:28 ` Tom Rini
2019-07-18  6:07   ` Suniel Mahesh
2019-07-31 16:24   ` [U-Boot] [PATCH v2 1/2] watchdog: omap_wdt: Convert watchdog driver to use DT and DM sunil.m at techveda.org
2019-07-31 16:24     ` [U-Boot] [PATCH v2 2/2] watchdog: omap_wdt: Disable DM watchdog support in SPL sunil.m at techveda.org
2019-08-13 16:50       ` Tom Rini [this message]
2019-08-13 16:50     ` [U-Boot] [PATCH v2 1/2] watchdog: omap_wdt: Convert watchdog driver to use DT and DM Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190813165052.GG2718@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.