[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 1/4] s390x/css: introduce css data stream
From: |
Dong Jia Shi |
Subject: |
Re: [Qemu-devel] [PATCH v2 1/4] s390x/css: introduce css data stream |
Date: |
Tue, 19 Sep 2017 10:21:42 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
* Halil Pasic <address@hidden> [2017-09-13 13:50:26 +0200]:
[...]
> diff --git a/include/hw/s390x/css.h b/include/hw/s390x/css.h
> index 0653d3c9be..79acaf99b7 100644
> --- a/include/hw/s390x/css.h
> +++ b/include/hw/s390x/css.h
> @@ -75,6 +75,29 @@ typedef struct CMBE {
> uint32_t reserved[7];
> } QEMU_PACKED CMBE;
>
> +typedef enum CcwDataStreamOp {
> + CDS_OP_R = 0,
> + CDS_OP_W = 1,
> + CDS_OP_A = 2
Nit:
typedef enum CcwDataStreamOp {
CDS_OP_R, /* read */
CDS_OP_W, /* write */
CDS_OP_A /* advance */
} CcwDataStreamOp;
(I just keep translating 'A' as append in my mind...)
> +} CcwDataStreamOp;
> +
[...]
> +static inline uint16_t ccw_dstream_avail(CcwDataStream *cds)
> +{
> + return ccw_dstream_good(cds) ? ccw_dstream_residual_count(cds) : 0;
^^
Nit.
> +}
[...]
With or w/o responses the nit picks:
Reviewed-by: Dong Jia Shi <address@hidden>
--
Dong Jia Shi
[Qemu-devel] [PATCH v2 2/4] s390x/css: use ccw data stream, Halil Pasic, 2017/09/13
[Qemu-devel] [PATCH v2 1/4] s390x/css: introduce css data stream, Halil Pasic, 2017/09/13
[Qemu-devel] [PATCH v2 3/4] virtio-ccw: use ccw data stream, Halil Pasic, 2017/09/13