All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH V2 00/11] staging: vchiq_arm: address TODO list
@ 2021-04-25 10:50 Stefan Wahren
  2021-04-25 10:50 ` [PATCH V2 01/11] staging: vchiq_arm: avoid crashing the kernel Stefan Wahren
                   ` (10 more replies)
  0 siblings, 11 replies; 16+ messages in thread
From: Stefan Wahren @ 2021-04-25 10:50 UTC (permalink / raw)
  To: Greg Kroah-Hartman, Nicolas Saenz Julienne
  Cc: Fabio Aiuto, linux-staging, Stefan Wahren

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


^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2021-06-03  8:03 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-25 10:50 [PATCH V2 00/11] staging: vchiq_arm: address TODO list Stefan Wahren
2021-04-25 10:50 ` [PATCH V2 01/11] staging: vchiq_arm: avoid crashing the kernel Stefan Wahren
2021-04-25 10:50 ` [PATCH V2 02/11] staging: vchiq_core: break early in vchiq_close_service_internal Stefan Wahren
2021-04-25 10:50 ` [PATCH V2 03/11] staging: vchiq_core: return early in do_abort_bulks Stefan Wahren
2021-04-26  8:37   ` nicolas saenz julienne
2021-04-25 10:50 ` [PATCH V2 04/11] staging: vchiq_core: introduce get_bulk_reason Stefan Wahren
2021-04-26  8:38   ` nicolas saenz julienne
2021-04-25 10:50 ` [PATCH V2 05/11] staging: vchiq_core: Drop unnecessary check in notify_bulks Stefan Wahren
2021-04-25 10:50 ` [PATCH V2 06/11] staging: vchiq_arm: drop return value of vchiq_arm_init_state Stefan Wahren
2021-04-25 10:50 ` [PATCH V2 07/11] staging: vchiq_2835_arm: drop enum vchiq_status Stefan Wahren
2021-04-25 10:51 ` [PATCH V2 08/11] staging: vchiq_arm: drop enum vchiq_status from vchiq_*_internal Stefan Wahren
2021-04-25 10:51 ` [PATCH V2 09/11] staging: vchiq_core: drop vchiq_status from vchiq_set_service_option Stefan Wahren
2021-04-26  8:39   ` nicolas saenz julienne
2021-06-03  8:03   ` nicolas saenz julienne
2021-04-25 10:51 ` [PATCH V2 10/11] staging: vchiq_core: drop vchiq_status from vchiq_initialise Stefan Wahren
2021-04-25 10:51 ` [PATCH V2 11/11] staging: vchiq_core: drop vchiq_status from vchiq_init_state Stefan Wahren

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.