From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752005Ab1GFKGF (ORCPT ); Wed, 6 Jul 2011 06:06:05 -0400 Received: from mail-yw0-f46.google.com ([209.85.213.46]:33809 "EHLO mail-yw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750955Ab1GFKGD (ORCPT ); Wed, 6 Jul 2011 06:06:03 -0400 Reply-To: From: Tommy Lin To: "'Anton Vorontsov'" , "'Tommy Lin'" Cc: "'Wim Van Sebroeck'" , , , "Lin, Mac" References: <1309867548-7842-1-git-send-email-tommy.lin@caviumnetworks.com> <20110705140559.GA15771@oksana.dev.rtsoft.ru> Subject: RE: [PATCH 2/2] watchdog: mpcore_wdt Add reload value setting for CNS3xxx hardware Date: Wed, 6 Jul 2011 18:05:37 +0800 Organization: CHOG Message-ID: <281B1F8BAB4D4BDABE7568809A0AEC1A@starsemi.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Office Outlook 11 Thread-Index: Acw7HK1U/mWKu7amTqKKsN5M6HLtGgApnN2w In-Reply-To: <20110705140559.GA15771@oksana.dev.rtsoft.ru> X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.6109 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, I just found another patch that can solve the watchdog counter reload issue. Please ignore this patch, since it should not acceptable to use #ifdefs. From Vitaly Kuzmichev Subject [PATCH V2 2/6] arm_smp_twd: mpcore_wdt: Fix watchdog counter loading Date Tue, 5 Jul 2011 23:00:36 +0400 -- Tommy Lin -----Original Message----- From: Anton Vorontsov [mailto:cbouatmailru@gmail.com] Sent: Tuesday, July 05, 2011 10:06 PM To: Tommy Lin Cc: Wim Van Sebroeck; linux-watchdog@vger.kernel.org; linux-kernel@vger.kernel.org; mac.lin@caviumnetworks.com; Tommy Lin Subject: Re: [PATCH 2/2] watchdog: mpcore_wdt Add reload value setting for CNS3xxx hardware Hello, thanks for the patches. On Tue, Jul 05, 2011 at 08:05:48PM +0800, Tommy Lin wrote: > Original MPcore watchdog setting about load register (offset 0x20) is different > from CNS3xxx data sheet. The CNS3xxx data sheet says watchdog has following > features: > 1. The Watchdog Counter Register (offset 0x24) is a down counter. > 2. The timer interval is calculated using following equation: > (PRESCALER_value+1) X (Load_value+1) X 2 / CPU CLK_frequency > Thus the watchdog load register control in CNS3xxx way is add to MPcore watchdog > source. The original control method is also kept if the CPU architecture is not > CNS3xxx. > > Signed-off-by: Tommy Lin > --- > drivers/watchdog/mpcore_wdt.c | 12 ++++++++++++ > 1 files changed, 12 insertions(+), 0 deletions(-) > > diff --git a/drivers/watchdog/mpcore_wdt.c b/drivers/watchdog/mpcore_wdt.c > index 2b4af22..816e0c5 100644 > --- a/drivers/watchdog/mpcore_wdt.c > +++ b/drivers/watchdog/mpcore_wdt.c > @@ -42,6 +42,9 @@ struct mpcore_wdt { > int irq; > unsigned int perturb; > char expect_close; > +#ifdef CONFIG_ARCH_CNS3XXX > + unsigned long reload_unit; /* ticks per second */ > +#endif Nope, these #ifdefs are not acceptible. Kernel should support multi-arch images soon. [...] > +#ifdef CONFIG_ARCH_CNS3XXX > + /* Assume prescale is set to 256 */ > + wdt->reload_unit = cns3xxx_cpu_clock() * 1000000 / 256 / 2; > +#endif You probably want to pass reload_unit via platform_data, and also check (in runtime) that the watchdog is CNS3xxx. Or if reload_unit != 0. Thanks, -- Anton Vorontsov Email: cbouatmailru@gmail.com