Hi Rafael, Great thanks for your feedback! On 18 March 2016 at 08:41, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote: > On Thursday, March 17, 2016 10:07:30 AM Timur Tabi wrote: >> fu.wei@xxxxxxxxxx wrote: >> > From: Fu Wei <fu.wei@xxxxxxxxxx> >> > >> > This patchset: >> > (1)Move some enums and marcos to header file for arm_arch_timer, >> > improve the pr_* code by defining "pr_fmt(fmt)" in arm_arch_timer.c >> > >> > (2)Introduce ACPI GTDT parser: drivers/acpi/gtdt.c >> > Parse all kinds of timer in GTDT table of ACPI:arch timer, >> > memory-mapped timer and SBSA Generic Watchdog timer. >> > This driver can help to simplify all the relevant timer drivers, >> > and separate all the ACPI GTDT knowledge from them. >> > >> > (3)Simplify ACPI code for arch timer in arm_arch_timer.c >> > >> > (4)Add memory-mapped timer support in arm_arch_timer.c >> >> The SBSA watchdog driver is already in watchdog-next and will be merged >> into 4.6. These five patches are necessary for the watchdog driver to >> probe on an ACPI system. >> >> I'm guessing since that v4 was posted in the middle of the 4.6 merge >> window, that these patches won't make it and we'll have to wait until >> 4.7 (at the earliest)? > > First off, the last version I've seen is the v3. I must have overlooked > the v4 then, but I don't think it was CCed to linux-acpi (which should > have happened). Sorry for this, That is my fault. I updated the script of git-send-email for sending the v4 patchset, but lost a space between two "--cc". :-( > > Second, these patches require review from multiple maintainers and (or including) > people who actually know how the GTDT is supposed to be used. I haven't > seen any responses from any of them yet, so the answer to your question is > "yes". > > I will have a couple of comments for patch [1/6] (in the v3), but it's > not been a priority due to the apparent overall lack of interest in this > series. Since this device is a generic watchdog for ARM server, it is(will be) used in a lot of chips. So I think many people are interested in it. They haven't provided any feedback, maybe due to my "--cc" mistake, or busy on something else now. :-( I would like to learn your comments for patch [1/6] , then improve my v4, post v5 ASAP. This patchset includes GTDT and memory-mapped timer patches, because my memory-mapped timer patches depends on GTDT. But maybe we can upstream GTDT separately? any suggestion? Thanks for your help! > > Thanks, > Rafael > -- 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 linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
