On Fri, Sep 17, 2010 at 7:37 PM, Josh Boyer <jwbo...@gmail.com> wrote:
>>  config BOOKE_WDT
>> -       bool "PowerPC Book-E Watchdog Timer"
>> +       tristate "PowerPC Book-E Watchdog Timer"
>>        depends on BOOKE || 4xx
>>        ---help---
>> +         Watchdog driver for PowerPC e500 chips, such as the Freescale
>> +         MPC85xx SOCs.
>> +
>
> Again, used for more than e500.  That || 4xx in the depends statement
> right above your addition isn't there for fun :).

Does this mean that this comment which is already in the driver is
wrong?  It alludes that "Book-E" is synonymous with "e500".

/* If the kernel parameter wdt=1, the watchdog will be enabled at boot.
 * Also, the wdt_period sets the watchdog timer period timeout.
 * For E500 cpus the wdt_period sets which bit changing from 0->1 will
 * trigger a watchog timeout. This watchdog timeout will occur 3 times, the
 * first time nothing will happen, the second time a watchdog exception will
 * occur, and the final time the board will reset.
 */

#ifdef  CONFIG_FSL_BOOKE
#define WDT_PERIOD_DEFAULT 38   /* Ex. wdt_period=28 bus=333Mhz,reset=~40sec */
#else
#define WDT_PERIOD_DEFAULT 3    /* Refer to the PPC40x and PPC4xx manuals */
#endif                          /* for timing information */

-- 
Timur Tabi
Linux kernel developer at Freescale
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to