[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1364134092.2906.4.camel@hobbes.kokotovo>
Date: Sun, 24 Mar 2013 15:08:12 +0100
From: Lubomir Rintel <lkundrak@...sk>
To: Arend van Spriel <arend@...adcom.com>
Cc: linux-kernel@...r.kernel.org,
Stephen Warren <swarren@...dotorg.org>,
Wim Van Sebroeck <wim@...ana.be>,
linux-rpi-kernel@...ts.infradead.org,
linux-watchdog@...r.kernel.org
Subject: Re: [PATCH] watchdog: Add Broadcom BCM2708 watchdog timer driver
On Fri, 2013-03-22 at 19:46 +0100, Arend van Spriel wrote:
Thanks for your response!
> On 03/22/2013 01:55 PM, Lubomir Rintel wrote:
> > Signed-off-by: Lubomir Rintel <lkundrak@...sk>
> > Cc: Stephen Warren <swarren@...dotorg.org>
> > Cc: Wim Van Sebroeck <wim@...ana.be>
> > Cc: linux-rpi-kernel@...ts.infradead.org
> > Cc: linux-watchdog@...r.kernel.org
> > ---
> > arch/arm/configs/bcm2835_defconfig | 4 +
> > drivers/watchdog/Kconfig | 11 +++
> > drivers/watchdog/Makefile | 1 +
> > drivers/watchdog/bcm2835_wdt.c | 158 ++++++++++++++++++++++++++++++++++++
> > 4 files changed, 174 insertions(+), 0 deletions(-)
> > create mode 100644 drivers/watchdog/bcm2835_wdt.c
>
> Seems like the subject line of the patch is wrong. Otherwise the commit
> message should be a little, tiny bit longer explainer how bcm2708 is
> related to bcm2835.
Good catch -- it indeed is wrong. It's probably still a good idea to
have a more descriptive commit message. I'll fix that up in next
revision.
--
Lubomir Rintel <lkundrak@...sk>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists