[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/22] hw/timer/cmsdk-apb-dualtimer: Implement C
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 03/22] hw/timer/cmsdk-apb-dualtimer: Implement CMSDK dual timer module |
Date: |
Thu, 23 Aug 2018 07:08:54 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 08/20/2018 07:10 AM, Peter Maydell wrote:
> + if (!(m->control & R_CONTROL_SIZE_MASK)) {
> + load = deposit32(load, 16, 16, extract32(m->load, 16, 16));
> + }
The same result, but clearer as
load = deposit32(m->load, 0, 16, load);
Several instances of this same pattern.
Otherwise,
Reviewed-by: Richard Henderson <address@hidden>
r~
- [Qemu-devel] [PATCH 04/22] hw/arm/iotkit: Wire up the dualtimer, (continued)
- [Qemu-devel] [PATCH 04/22] hw/arm/iotkit: Wire up the dualtimer, Peter Maydell, 2018/08/20
- [Qemu-devel] [PATCH 06/22] hw/arm/iotkit: Wire up the watchdogs, Peter Maydell, 2018/08/20
- [Qemu-devel] [PATCH 01/22] hw/misc/mps2-fpgaio: Implement 1Hz and 100Hz counters, Peter Maydell, 2018/08/20
- [Qemu-devel] [PATCH 07/22] hw/arm/iotkit: Wire up the S32KTIMER, Peter Maydell, 2018/08/20
- [Qemu-devel] [PATCH 03/22] hw/timer/cmsdk-apb-dualtimer: Implement CMSDK dual timer module, Peter Maydell, 2018/08/20
- Re: [Qemu-devel] [PATCH 03/22] hw/timer/cmsdk-apb-dualtimer: Implement CMSDK dual timer module,
Richard Henderson <=
- [Qemu-devel] [PATCH 09/22] hw/misc/iotkit-sysinfo: Implement IoTKit system information block, Peter Maydell, 2018/08/20
- [Qemu-devel] [PATCH 10/22] hw/misc/iotkit: Wire up the sysctl and sysinfo register blocks, Peter Maydell, 2018/08/20
- [Qemu-devel] [PATCH 08/22] hw/misc/iotkit-sysctl: Implement IoTKit system control element, Peter Maydell, 2018/08/20
- [Qemu-devel] [PATCH 12/22] hw/misc/iotkit-secctl: Wire up registers for controlling MSCs, Peter Maydell, 2018/08/20