init_common_resources always return 0 and it is redundant to check the return value, let's make it void to simplify the code. Cc: Christoffer Dall <christoffer.dall@xxxxxxx> Cc: Marc Zyngier <marc.zyngier@xxxxxxx> Signed-off-by: Shaokun Zhang <zhangshaokun@xxxxxxxxxxxxx> --- virt/kvm/arm/arm.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/virt/kvm/arm/arm.c b/virt/kvm/arm/arm.c index c92053b..66a29c9 100644 --- a/virt/kvm/arm/arm.c +++ b/virt/kvm/arm/arm.c @@ -1406,13 +1406,11 @@ static inline void hyp_cpu_pm_exit(void) } #endif -static int init_common_resources(void) +static void init_common_resources(void) { /* set size of VMID supported by CPU */ kvm_vmid_bits = kvm_get_vmid_bits(); kvm_info("%d-bit VMID\n", kvm_vmid_bits); - - return 0; } static int init_subsystems(void) @@ -1655,9 +1653,7 @@ int kvm_arch_init(void *opaque) } } - err = init_common_resources(); - if (err) - return err; + init_common_resources(); in_hyp_mode = is_kernel_in_hyp_mode(); -- 2.7.4 _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/mailman/listinfo/kvmarm