Hi Arnd, Great thanks for your review! :-) On 21 May 2015 at 16:46, Arnd Bergmann <arnd@xxxxxxxx> wrote: > On Thursday 21 May 2015 16:32:29 fu.wei@xxxxxxxxxx wrote: >> >> This patchset: >> >> (1)Export "arch_timer_get_rate" in arm_arch_timer.c for the >> other drivers, like SBSA watchdog driver >> >> (2)Introduce Documentation/devicetree/bindings/watchdog/sbsa-gwdt.txt >> for FDT info of SBSA Generic Watchdog, and give two examples of >> adding SBSA Generic Watchdog device node into the dts files: >> foundation-v8.dts and amd-seattle-soc.dtsi. >> >> (3)Introduce "pretimeout" into the watchdog framework, and update >> Documentation/watchdog/watchdog-kernel-api.txt to introduce: >> (1)the new elements in the watchdog_device and watchdog_ops struct; >> (2)the new API "watchdog_init_timeouts". >> >> (4)Introduce ARM SBSA watchdog driver: >> a.Use linux kernel watchdog framework; >> b.Work with FDT on ARM64; >> c.Use "pretimeout" in watchdog framework; >> d.In first timeout(WS0), do panic to save system context; >> e.Support geting timeout and pretimeout from parameter and FDT >> at the driver init stage. > > Looks all good to me, please add > > Acked-by: Arnd Bergmann <arnd@xxxxxxxx> > > for patches 1-6. Great! happy to do so :-) > >> (5)Parse SBSA Generic Watchdog Structure in GTDT table of ACPI, >> and create a platform device with that information. >> This platform device can be used by This Watchdog driver. > > This needs to be reviewed by someone who understands ACPI. yes, maybe Hanjun and Al can help :-) > > Arnd -- 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html