diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2018-05-01 15:43:58 +1000 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2018-05-04 15:00:37 +1000 |
commit | cf116ad4703a37b66122d97f139afb2321b9c40e (patch) | |
tree | 863cee740258bf5fd4d2bd41b99ebd9161249fd5 /hw/timer/ds1338.c | |
parent | 5ad553154d1137fbee9ec92b991e52123fa9bcd8 (diff) |
spapr: Clean up rtas_start_cpu() & rtas_stop_self()
This makes several minor cleanups to these functions:
* Follow usual convention of an early exit on error, rather than having
most of the body in an if
* Clearer naming of cpu and cpu_. Now callcpu is the cpu from which the
RTAS call is invoked, newcpu is the cpu which we're starting
* Use cpu_synchronize_state() instead of kvm_cpu_synchronize_state()
directly
* Remove pointless comment describing what cpu_synchronize_state() does
* Use ppc_store_lpcr() instead of directly writing the register field
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Tested-by: Cédric Le Goater <clg@kaod.org>
Reviewed-by: Greg Kurz <groug@kaod.org>
Diffstat (limited to 'hw/timer/ds1338.c')
0 files changed, 0 insertions, 0 deletions