[U-Boot] [PATCH v4] ftwdt010_wdt: support faraday ftwdt010 watchdog

Macpaul Lin macpaul at gmail.com
Tue Apr 12 06:56:56 CEST 2011


Hi Wolfgang

2011/4/12 Wolfgang Denk <wd at denx.de>:
> Dear Macpaul Lin,
>
> In message <1296038788-12702-1-git-send-email-macpaul at andestech.com> you wrote:
>> Faraday ftwdt010 watchdog is an architecture independant
>> watchdog. It is usaually used in SoC chip design.
>>
>> Signed-off-by: Macpaul Lin <macpaul at andestech.com>
>> ---
>>  Change v2:
>>    Fix the wrong title of ftwdt010_wdt patch.
>>  Change v3:
>>    Patch v2 was encoded in utf-8. Fixed to ASCII.
>>  Change v4:
>>    Replace the copyright statement "origin: Linux kernel" because the Linux
>>    driver haven't commit into kernel yet.

Thanks for your patience for reviewing and fixing up the code.

However, the latest version of this watchdog ftwdt010_wdt is v6.
http://patchwork.ozlabs.org/patch/88164/

Since you have already applied and fix patch v4, I'll mark patch v6 as
superseded and
send the diff between patch v4 and patch v6 as an "enhancement" patch
of ftwdt010_wdt.c later.

The differ between v4 and v6 are
    1. Move header to include/faraday
    2. Fix include path in ftwdt010_wdt.c
    3. Fix function prototype and declaration to
      - ftwdt010_wdt_settimeout
      - ftwdt010_wdt_reset
      - ftwdt010_wdt_disable
    4. Add "#if definde (CONFIG_HW_WATCHDOG)" let user have flexibilty
      to choose which better to his product.

Sorry for the bothering.

Thanks.


-- 
Best regards,
Macpaul Lin


More information about the U-Boot mailing list