On Fri, 2015-12-11 at 15:28 +0000, Julien Grall wrote:
> Rather than getting dabt every time through info->dabt, introduce a
> local variable and use it.
>
> Also fix a coding style error in the if condition.
>
> Signed-off-by: Julien Grall <julien.gr...@citrix.com>
Acked-by: Ian Campbell <ian.campb...@citrix.com>
> ---
> xen/arch/arm/io.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/xen/arch/arm/io.c b/xen/arch/arm/io.c
> index de5765a..7e29943 100644
> --- a/xen/arch/arm/io.c
> +++ b/xen/arch/arm/io.c
> @@ -26,7 +26,8 @@
> static int handle_read(const struct mmio_handler *handler, struct vcpu
> *v,
> mmio_info_t *info, register_t *r)
> {
> - uint8_t size = (1 << info->dabt.size) * 8;
> + const struct hsr_dabt dabt = info->dabt;
> + uint8_t size = (1 << dabt.size) * 8;
>
> if ( !handler->ops->read(v, info, r, handler->priv) )
> return 0;
> @@ -36,7 +37,7 @@ static int handle_read(const struct mmio_handler
> *handler, struct vcpu *v,
> * Note that we expect the read handler to have zeroed the bits
> * outside the requested access size.
> */
> - if ( info->dabt.sign && (*r & (1UL << (size - 1)) ))
> + if ( dabt.sign && (*r & (1UL << (size - 1))) )
> {
> /*
> * We are relying on register_t using the same as
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
