Re: [PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-23 Thread Fu Wei
Hi Mark, On 19 November 2016 at 04:12, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:49:04PM +0800, fu@linaro.org wrote: > >> +#define for_each_platform_timer(_g) for (; _g; _g = next_platform_timer(_g)) > > This doesn't fit the usual for_each_* pattern, since _g

Re: [PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-23 Thread Fu Wei
Hi Mark, On 19 November 2016 at 04:12, Mark Rutland wrote: > On Wed, Nov 16, 2016 at 09:49:04PM +0800, fu@linaro.org wrote: > >> +#define for_each_platform_timer(_g) for (; _g; _g = next_platform_timer(_g)) > > This doesn't fit the usual for_each_* pattern, since _g has to be > manually

Re: [PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-18 Thread Mark Rutland
On Wed, Nov 16, 2016 at 09:49:04PM +0800, fu@linaro.org wrote: > +#define for_each_platform_timer(_g) for (; _g; _g = next_platform_timer(_g)) This doesn't fit the usual for_each_* pattern, since _g has to be manually initialised first. Either come up with a way of maknig this fit the usual

Re: [PATCH v16 11/15] acpi/arm64: Add GTDT table parse driver

2016-11-18 Thread Mark Rutland
On Wed, Nov 16, 2016 at 09:49:04PM +0800, fu@linaro.org wrote: > +#define for_each_platform_timer(_g) for (; _g; _g = next_platform_timer(_g)) This doesn't fit the usual for_each_* pattern, since _g has to be manually initialised first. Either come up with a way of maknig this fit the usual