# Re: \[PATCH V2\] powerpc: Fix Book-E watchdog timer interval setting

- From: Timur Tabi \<hidden\>
- Date: 2008-09-23 16:31:58 
- List: linuxppc-dev
- Page: https://ratatoskr.run/linuxppc-dev/2008/09/9445698
- Thread: https://ratatoskr.run/linuxppc-dev/2008/08/9444301/t.md

```text
On Tue, Sep 23, 2008 at 10:04 AM, Matthias Fuchs
[ref] wrote:
>  #ifdef CONFIG_FSL_BOOKE
>  #define WDTP(x)                ((((63-x)&0x3)<<30)|(((63-x)&0x3c)<<15))
> +#define WDTP_MASK      (WDTP(63))

WDTP(63) is "((((63-63)&0x3)<<30)|(((63-63)&0x3c)<<15))", which is
equal to 0.  Shouldn't WDTP_MASK be equal to "(3 << 30) | (15 << 15)"?

-- 
Timur Tabi
Linux kernel developer at Freescale
```