[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 12/14] target-arm: Drop cpu_reset_model_id()
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v2 12/14] target-arm: Drop cpu_reset_model_id() |
Date: |
Fri, 20 Apr 2012 17:16:46 +0100 |
On 20 April 2012 17:14, Andreas Färber <address@hidden> wrote:
> Am 14.04.2012 18:42, schrieb Peter Maydell:
>> /* TODO Move contents into arm_cpu_reset() in cpu.c,
>> * once cpu_reset_model_id() is eliminated,
>
> ...if you were to drop this comment line it would be cleaner. There is
> no cpu_reset_model_id() any more.
...the whole comment disappears in the next patch.
> Also, can't some CPUID defines already be dropped in this patch, now
> that the switch is gone?
It would be possible, but I didn't really like the approach of
dropping some but not all of them.
-- PMM
- [Qemu-devel] [PATCH v2 09/14] target-arm: Move feature register setup to per-CPU init fns, (continued)
- [Qemu-devel] [PATCH v2 09/14] target-arm: Move feature register setup to per-CPU init fns, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 14/14] target-arm: Move A9 config_base_address reset value to ARMCPU, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 10/14] target-arm: Move OMAP cp15_i_{max, min} reset to cpu_state_reset, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 07/14] target-arm: Drop JTAG_ID documentation, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 12/14] target-arm: Drop cpu_reset_model_id(), Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 06/14] target-arm: Move SCTLR reset value setup to per cpu init fns, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 00/14] target-arm: Create QOM subclass per CPU implementation, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 02/14] target-arm: Move feature bit settings to CPU init fns, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 03/14] target-arm: Move FPSID config to cpu init fns, Peter Maydell, 2012/04/14
- [Qemu-devel] [PATCH v2 01/14] target-arm: Add QOM subclasses for each ARM cpu implementation, Peter Maydell, 2012/04/14