On 7/25/19 11:43 PM, tony.ngu...@bt.com wrote: > } MemOp; > > +/* No-op while memory_region_dispatch_[read|write] is converted to MemOp */ > +#define MEMOP_SIZE(op) (op) /* MemOp to size. */ > +#define SIZE_MEMOP(ul) (ul) /* Size to MemOp. */ > +
This doesn't thrill me, because for 9 patches these macros don't do what they say on the tin, but I'll accept it. I would prefer lower-case and that the real implementation in patch 10 be inline functions with proper types instead of typeless macros. In particular, "unsigned" not "unsigned long" as you imply from "ul" here, since that's what was used ... > MemTxResult memory_region_dispatch_read(MemoryRegion *mr, > hwaddr addr, > uint64_t *pval, > - unsigned size, > + MemOp op, > MemTxAttrs attrs); ... here. With the name case change, Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~
