Re: [Qemu-devel] [PATCH 0/7] timer: move APIs together according to their category

[Alex Bligh]

> 
> This patchset includes below changes:
> - put QEMUTimerList related functions together and make them private
> - put QEMUClockType related functions together
> - move QEMUTimerListGroup function to be below QEMUClockType
> - put QEMUTimer related functions together
> - put general utility functions together
> - clean unnecessary #include and use minimal required #include
> 
> Xuebing Wang (7):
>  timer: move QEMUTimerList functions together
>  timer: make QEMUTimerList functions private (remove from APIs)
>  timer: move QEMUClockType related functions together
>  timer: move QEMUTimerListGroup function to be below QEMUClockType
>  timer: move QEMUTimer related functions together
>  timer: move general utility functions together
>  timer: clean unnecessary #include and use minimal required #include


I'm generally in favour of this patch set or something like it. It's
nicely constrained to 2 include files. It appears to touch no code.
And it appears to makes things tidier.

Stefan / Paolo? If you agree I will review in greater detail.

-- 
Alex Bligh