Hi Stefano,

On 09/29/2017 09:10 PM, Stefano Stabellini wrote:
On Wed, 27 Sep 2017, Bhupinder Thakur wrote:
DEFINE_XEN_FLEX_RING(xencons) defines common helper functions such as
xencons_queued() to tell the current size of the ring buffer,
xencons_mask() to mask off the index, which are useful helper functions.
pl011 emulation code will use these helper functions.

io/console.h includes io/ring.h which defines DEFINE_XEN_FLEX_RING.

In console/daemon/io.c, string.h had to be included before io/console.h
because ring.h uses string functions.

Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org>
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
Acked-by: Wei Liu <wei.l...@citrix.com>
Acked-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>

With the changes you should have probably removed my old reviewed-by.
But that's OK:

Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>

If it's OK for everybody, I'll go ahead and commit the series on Monday.
Jan, shout if you disagree on something.

I'd like to have a look at patch #26 and #27. They can be committed separately from the rest of the series.

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Reply via email to