Messages in this thread | | | Subject | Re: [PATCH] watchdog: Add Broadcom BCM2708 watchdog timer driver | From | Lubomir Rintel <> | Date | Sun, 24 Mar 2013 15:08:12 +0100 |
| |
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@v3.sk> > > Cc: Stephen Warren <swarren@wwwdotorg.org> > > Cc: Wim Van Sebroeck <wim@iguana.be> > > Cc: linux-rpi-kernel@lists.infradead.org > > Cc: linux-watchdog@vger.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@v3.sk>
| |