Re: [PATCH v11 4/4] Watchdog: introduce ARM SBSA watchdog driver

From: Fu Wei
Date: Tue Feb 09 2016 - 12:47:34 EST


Hi Timur,

On 10 February 2016 at 01:39, Timur Tabi <timur@xxxxxxxxxxxxxx> wrote:
> fu.wei@xxxxxxxxxx wrote:
>
>> + if (!action)
>> + dev_warn(dev, "fall back to signle stage
>> mode.\n");
>
>
> "falling back to single stage mode"

Thanks for your correction. :-)
Will fixed it

>
> Acked-by: Timur Tabi <timur@xxxxxxxxxxxxxx>
>

Thanks, will add this

> The whole "calculate the timeouts different if 'action' is enabled" seems
> clunky to me, but I can't think of a better way to handle it.
>
> --
> Qualcomm Innovation Center, Inc.
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora
> Forum, a Linux Foundation collaborative project.



--
Best regards,

Fu Wei
Software Engineer
Red Hat Software (Beijing) Co.,Ltd.Shanghai Branch
Ph: +86 21 61221326(direct)
Ph: +86 186 2020 4684 (mobile)
Room 1512, Regus One Corporate Avenue,Level 15,
One Corporate Avenue,222 Hubin Road,Huangpu District,
Shanghai,China 200021