refactor: remove out parameter from `allocate_share_state`

`allocate_share_state` returned a `struct ffa_memory_share_state *` via
an out parameter and indicated success by returning a `bool`. This patch
removes the out parameter and returns the `struct ffa_memory_share_state
*` directly.

Change-Id: I3d0bd0952d39332282b9303c0cb60850317cd8e3
Signed-off-by: Karl Meakin <karl.meakin@arm.com>
diff --git a/inc/hf/ffa_memory_internal.h b/inc/hf/ffa_memory_internal.h
index 7e14765..6ac69c6 100644
--- a/inc/hf/ffa_memory_internal.h
+++ b/inc/hf/ffa_memory_internal.h
@@ -118,11 +118,10 @@
 	struct ffa_memory_share_state *share_states;
 };
 
-bool allocate_share_state(struct share_states_locked share_states,
-			  uint32_t share_func,
-			  struct ffa_memory_region *memory_region,
-			  uint32_t fragment_length, ffa_memory_handle_t handle,
-			  struct ffa_memory_share_state **share_state_ret);
+struct ffa_memory_share_state *allocate_share_state(
+	struct share_states_locked share_states, uint32_t share_func,
+	struct ffa_memory_region *memory_region, uint32_t fragment_length,
+	ffa_memory_handle_t handle);
 struct share_states_locked share_states_lock(void);
 void share_states_unlock(struct share_states_locked *share_states);
 void share_state_free(struct share_states_locked share_states,