Messages in this thread | ![/](/images/icornerl.gif) | | Date | Thu, 15 Apr 2010 07:57:15 +0200 | From | Wim Van Sebroeck <> | Subject | Re: [PATCH] fixed book E watchdog period register mask. |
| |
Hi Luuk,
will add this one today or tomorrow, when I finished translating an audit report.
Kind regards, Wim.
> A previous fix changed the WDTP function to use the period directly, > rather than subtracting from 63. However the mask generation was > not changed, so the mask was coming out as 0. This patch fixes it. > > Signed-off-by: Luuk Paulussen <luuk.paulussen@alliedtelesis.co.nz> > --- > drivers/watchdog/booke_wdt.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/watchdog/booke_wdt.c b/drivers/watchdog/booke_wdt.c > index 8b724aa..500d383 100644 > --- a/drivers/watchdog/booke_wdt.c > +++ b/drivers/watchdog/booke_wdt.c > @@ -44,7 +44,7 @@ u32 booke_wdt_period = WDT_PERIOD_DEFAULT; > > #ifdef CONFIG_FSL_BOOKE > #define WDTP(x) ((((x)&0x3)<<30)|(((x)&0x3c)<<15)) > -#define WDTP_MASK (WDTP(0)) > +#define WDTP_MASK (WDTP(0x3f)) > #else > #define WDTP(x) (TCR_WP(x)) > #define WDTP_MASK (TCR_WP_MASK) > -- > 1.6.3.4
| ![\](/images/icornerr.gif) |