Timur, I missed you posting. But you are right. My patch is ok for 4xx CPUs and touching the CONFIG_FSL_BOOKE path was not my intention.
So for CONFIG_FSL_BOOKE WDTP_MASK should be WDTP(0). There is still a slightly difference between WDTP(0)="(3 << 30) | (0x3c << 15)" and "(3 << 30) | (15 << 15)". Can you check that please and I will resend my patch. Thanks for pointing that out. Matthias On Tuesday 23 September 2008 18:31, Timur Tabi wrote: > On Tue, Sep 23, 2008 at 10:04 AM, Matthias Fuchs > <[EMAIL PROTECTED]> 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)"? > _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
