Hi Peter,
On 4/16/20 8:29 AM, Peter Maydell wrote:
> On Sun, 22 Mar 2020 at 21:19, Guenter Roeck <li...@roeck-us.net> wrote:
>>
>> i.MX7 supports watchdog pretimeout interupts. With this commit,
>> the watchdog in mcimx7d-sabre is fully operational, including
>> pretimeout support.
>>
>> Signed-off-by: Guenter Roeck <li...@roeck-us.net>
>
>> diff --git a/include/hw/arm/fsl-imx7.h b/include/hw/arm/fsl-imx7.h
>> index 47826da2b7..da977f9ffb 100644
>> --- a/include/hw/arm/fsl-imx7.h
>> +++ b/include/hw/arm/fsl-imx7.h
>> @@ -228,6 +228,11 @@ enum FslIMX7IRQs {
>> FSL_IMX7_USB2_IRQ = 42,
>> FSL_IMX7_USB3_IRQ = 40,
>>
>> + FSL_IMX7_WDOG1_IRQ = 78,
>> + FSL_IMX7_WDOG2_IRQ = 79,
>> + FSL_IMX7_WDOG3_IRQ = 10,
>> + FSL_IMX7_WDOG4_IRQ = 109,
>
> irq 10 for wdog3 seems to match the kernel's dts, but it's
> a bit weird that it's way out of the range of the others.
> Did you sanity check it against the imx7 data sheet and/or
> real h/w behaviour that it's not a typo for
> one-hundred-and-something? (108 would be the obvious guess...)
>
I actually did check, for that very same reason. To be sure I looked
again. 10 is correct per datasheet. 108 is TZASC1 (TZASC (PL380)
interrupt).
> Otherwise
> Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
>
Thanks,
Guenter
