Philippe Mathieu-Daudé <phi...@redhat.com> writes:
> Hi Thiago, > > On 7/22/20 5:50 AM, Thiago Jung Bauermann wrote: >> There are other platforms which also have CPUs that start powered off, so >> generalize the start-powered-off property so that it can be used by them. >> >> Note that ARMv7MState also has a property of the same name but this patch >> doesn't change it because that class isn't a subclass of CPUState so it >> wouldn't be a trivial change. >> >> This change should not cause any change in behavior. >> >> Suggested-by: Eduardo Habkost <ehabk...@redhat.com> >> Signed-off-by: Thiago Jung Bauermann <bauer...@linux.ibm.com> > > As I participated in reviewing your v1, I'd have appreciated > being Cc'ed for v2. I'm sorry about this. I fixed the Cc list for the next version. > Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com> Thank you very much for your prompt review, and suggestions. I will post a new version addressing your comments shortly. >> --- >> exec.c | 1 + >> include/hw/core/cpu.h | 4 ++++ >> target/arm/cpu.c | 5 ++--- >> target/arm/cpu.h | 3 --- >> target/arm/kvm32.c | 2 +- >> target/arm/kvm64.c | 2 +- >> 6 files changed, 9 insertions(+), 8 deletions(-) -- Thiago Jung Bauermann IBM Linux Technology Center
