public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] watchdog/denali: Adding DesignWare watchdog driver support
Date: Fri, 21 Feb 2014 17:58:18 +0100	[thread overview]
Message-ID: <20140221175818.70f1ab70@lilith> (raw)
In-Reply-To: <1392998408.3602.8.camel@clsee-VirtualBox>

Hi Chin,

On Fri, 21 Feb 2014 10:00:08 -0600, Chin Liang See <clsee@altera.com>
wrote:

> Hi Albert,
> 
> On Thu, 2014-02-13 at 10:35 +0100, ZY - albert.u.boot wrote:
> > Hi Chin,
> > 
> > On Wed, 18 Dec 2013 16:23:35 -0600, Chin Liang See <clsee@altera.com>
> > wrote:
> > 
> > > To add the DesignWare watchdog driver support. It required
> > > information such as register base address and clock info from
> > > configuration header file  within include/configs folder.
> > > 
> > > Signed-off-by: Chin Liang See <clsee@altera.com>
> > > Cc: Anatolij Gustschin <agust@denx.de>
> > > Cc: Albert Aribaud <albert.u.boot@aribaud.net>
> > > Cc: Heiko Schocher <hs@denx.de>
> > > Cc: Tom Rini <trini@ti.com>
> > > ---
> > >  drivers/watchdog/Makefile         |    1 +
> > >  drivers/watchdog/designware_wdt.c |   75 +++++++++++++++++++++++++++++++++++++
> > >  2 files changed, 76 insertions(+)
> > >  create mode 100644 drivers/watchdog/designware_wdt.c
> > 
> > As such, this code is dead code in that no board uses this watchdog --
> > same as the NAND driver series assigned to scott Wood (cc:).
> > 
> > If there is a board which uses these drivers, please resubmit the
> > drivers and board patches in a single series where some board config
> > is made to use them.
> > 
> > Amicalement,
> 
> Sorry my bad as I would thought they would need 2 different patches. 

Actually it could have been two different patches, but in a single
series (i.e., patch 1/2 provides the driver, patch 2/2 provides the
configurations which use the driver).

> Chin Liang

Amicalement,
-- 
Albert.

  reply	other threads:[~2014-02-21 16:58 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-18 22:23 [U-Boot] [PATCH] watchdog/denali: Adding DesignWare watchdog driver support Chin Liang See
2014-01-09 14:47 ` Chin Liang See
2014-02-04 19:08 ` Chin Liang See
2014-02-13  9:35 ` Albert ARIBAUD
2014-02-21 16:00   ` Chin Liang See
2014-02-21 16:58     ` Albert ARIBAUD [this message]
2014-02-21 20:50       ` Chin Liang See

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=20140221175818.70f1ab70@lilith \
    --to=albert.u.boot@aribaud.net \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox