From: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
To: Jason Cooper <jason@lakedaemon.net>
Cc: Wim Van Sebroeck <wim@iguana.be>,
Andrew Morton <akpm@linux-foundation.org>,
linux-watchdog@vger.kernel.org,
Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
Gregory Clement <gregory.clement@free-electrons.com>,
Lior Amsalem <alior@marvell.com>,
Tawfik Bayouk <tawfik@marvell.com>
Subject: Re: [RESEND/PATCH v7 00/18] Armada 370/XP SoC watchdog support
Date: Mon, 17 Feb 2014 16:49:19 -0300 [thread overview]
Message-ID: <20140217194918.GA23046@localhost> (raw)
In-Reply-To: <20140217193325.GX7862@titan.lakedaemon.net>
On Mon, Feb 17, 2014 at 02:33:25PM -0500, Jason Cooper wrote:
> On Mon, Feb 17, 2014 at 08:29:48PM +0100, Wim Van Sebroeck wrote:
> > Hi All,
> >
> > > Hi Jason, Andrew,
> > >
> > > This patchset adds watchdog support for the Armada 370/XP ARM SoCs.
> > > The series was sent seven times:
> > >
> > > [v1] http://archive.arm.linux.org.uk/lurker/message/20130827.143424.4e961f2b.ca.html
> > > [v2] http://www.spinics.net/lists/arm-kernel/msg302104.html
> > > [v3] http://www.spinics.net/lists/arm-kernel/msg302245.html
> > > [v4] http://www.spinics.net/lists/arm-kernel/msg302602.html
> > > [v5] http://www.spinics.net/lists/arm-kernel/msg303349.html
> > > [v6] http://www.spinics.net/lists/arm-kernel/msg305547.html
> > > [v7] http://www.spinics.net/lists/arm-kernel/msg306534.html
> > >
> > > As you can see, it received numerous feedback, testing and reviews;
> > > we've finally reached agreement and so we'd like to merge this series for
> > > v3.15, if at all possible.
> > >
> > > However, the watchdog subsystem maintainer remained completely silent through
> > > the *entire* life of this patchset, and so we've no idea what are his plans
> > > about it.
> > >
> > > Do you think this can be merged through Andrew's tree? Or maybe through
> > > Jason Cooper's so the irqchip dependency is easily handled?
> > >
> > > Thanks a lot!
> > >
> > > Original cover letter
> > > =====================
> > >
> > > Yet another round. This time, addressing Guenter's comment about the nowayout
> > > parameter. So, to allow for proper nowayout behavior, we now restrict the
> > > initial watchdog stop; which is only done if the watchdog is not properly
> > > enabled.
> > >
> > > I've dropped the clocksource patch which was previously part of the series.
> > > The patch is independent and will be merged through the subsystem maintainer,
> > > Daniel Lezcano.
> > >
> > > Also, I've updated the multi_v7_defconfig, as per Jason Cooper request.
> > >
> > > This series depends on Sebastian's irqchip fixes to clear the bridge irqs
> > > cause, which Jason picked here:
> > >
> > > http://git.infradead.org/linux-mvebu.git mvebu-next/irqchip-fixes
> > >
> > > Wim: Can we take all the watchdog changes through mvebu, with your Ack?
> > > This way, we can sort out the irqchip dependency.
> >
> > I reviewed all patches and you have an Acked-by from me on all of them.
>
> Thanks, Wim!
>
Thanks a lot Wim!
>
> Jason.
Jason: I'm leaving this in your hands then, take good care of it ;-)
--
Ezequiel García, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com
prev parent reply other threads:[~2014-02-17 19:49 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-14 22:24 [RESEND/PATCH v7 00/18] Armada 370/XP SoC watchdog support Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 01/18] watchdog: orion: Add clock error handling Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 02/18] watchdog: orion: Use atomic access for shared registers Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 03/18] watchdog: orion: Remove unused macros Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 04/18] watchdog: orion: Make sure the watchdog is initially stopped Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 05/18] watchdog: orion: Handle the interrupt so it's properly acked Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 06/18] watchdog: orion: Make RSTOUT register a separate resource Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 07/18] watchdog: orion: Remove unneeded BRIDGE_CAUSE clear Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 08/18] watchdog: orion: Introduce an orion_watchdog device structure Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 09/18] watchdog: orion: Introduce per-compatible of_device_id data Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 10/18] watchdog: orion: Add per-compatible clock initialization Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 11/18] watchdog: orion: Add per-compatible watchdog start implementation Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 12/18] watchdog: orion: Add support for Armada 370 and Armada XP SoC Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 13/18] ARM: mvebu: Enable Armada 370/XP watchdog in the devicetree Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 14/18] ARM: kirkwood: Add RSTOUT 'reg' entry to devicetree Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 15/18] ARM: dove: Enable Dove watchdog in the devicetree Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 16/18] watchdog: orion: Enable the build on ARCH_MVEBU Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 17/18] ARM: mvebu: Enable watchdog support in defconfig Ezequiel Garcia
2014-02-14 22:24 ` [RESEND/PATCH v7 18/18] ARM: dove: Enable watchdog support in the defconfig Ezequiel Garcia
2014-02-17 19:29 ` [RESEND/PATCH v7 00/18] Armada 370/XP SoC watchdog support Wim Van Sebroeck
2014-02-17 19:33 ` Jason Cooper
2014-02-17 19:49 ` Ezequiel Garcia [this message]
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=20140217194918.GA23046@localhost \
--to=ezequiel.garcia@free-electrons.com \
--cc=akpm@linux-foundation.org \
--cc=alior@marvell.com \
--cc=gregory.clement@free-electrons.com \
--cc=jason@lakedaemon.net \
--cc=linux-watchdog@vger.kernel.org \
--cc=tawfik@marvell.com \
--cc=thomas.petazzoni@free-electrons.com \
--cc=wim@iguana.be \
/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