This series mostly addresses the following points from the TODO list: - Get rid of custom function return values - Reformat core code with more sane indentations Additionally this addresses some issues reported by checkpatch.pl Changes in V2: - move unrelated changes in patch 2 into separate one (found by Fabio) - change default error to EINVAL for vchiq_set_service_option (found by Nicolas) Stefan Wahren (11): staging: vchiq_arm: avoid crashing the kernel staging: vchiq_core: break early in vchiq_close_service_internal staging: vchiq_core: return early in do_abort_bulks staging: vchiq_core: introduce get_bulk_reason staging: vchiq_core: Drop unnecessary check in notify_bulks staging: vchiq_arm: drop return value of vchiq_arm_init_state staging: vchiq_2835_arm: drop enum vchiq_status staging: vchiq_arm: drop enum vchiq_status from vchiq_*_internal staging: vchiq_core: drop vchiq_status from vchiq_set_service_option staging: vchiq_core: drop vchiq_status from vchiq_initialise staging: vchiq_core: drop vchiq_status from vchiq_init_state .../interface/vchiq_arm/vchiq_2835_arm.c | 23 ++- .../vc04_services/interface/vchiq_arm/vchiq_arm.c | 75 ++++---- .../vc04_services/interface/vchiq_arm/vchiq_arm.h | 2 +- .../vc04_services/interface/vchiq_arm/vchiq_core.c | 191 +++++++++++---------- .../vc04_services/interface/vchiq_arm/vchiq_core.h | 12 +- 5 files changed, 158 insertions(+), 145 deletions(-) -- 2.7.4
