Commit acdb1d90 authored by Stefan Wahren's avatar Stefan Wahren Committed by Greg Kroah-Hartman
Browse files

staging: vchiq_core: Simplify vchiq_send_remote_use* 



There is no need for variable status. So drop it and improve readability.

Signed-off-by: default avatarStefan Wahren <stefan.wahren@i2se.com>
Link: https://lore.kernel.org/r/1618164700-21150-2-git-send-email-stefan.wahren@i2se.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 8333e5a9

drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c

+10 −12
Original line number Diff line number Diff line
@@ -3702,24 +3702,22 @@ vchiq_loud_error_footer(void) 


enum vchiq_status vchiq_send_remote_use(struct vchiq_state *state)

{

	enum vchiq_status status = VCHIQ_RETRY;

	if (state->conn_state == VCHIQ_CONNSTATE_DISCONNECTED)

		return VCHIQ_RETRY;



	if (state->conn_state != VCHIQ_CONNSTATE_DISCONNECTED)

		status = queue_message(state, NULL,

	return queue_message(state, NULL,

			     VCHIQ_MAKE_MSG(VCHIQ_MSG_REMOTE_USE, 0, 0),

			     NULL, NULL, 0, 0);

	return status;

}



enum vchiq_status vchiq_send_remote_use_active(struct vchiq_state *state)

{

	enum vchiq_status status = VCHIQ_RETRY;

	if (state->conn_state == VCHIQ_CONNSTATE_DISCONNECTED)

		return VCHIQ_RETRY;



	if (state->conn_state != VCHIQ_CONNSTATE_DISCONNECTED)

		status = queue_message(state, NULL,

	return queue_message(state, NULL,

			     VCHIQ_MAKE_MSG(VCHIQ_MSG_REMOTE_USE_ACTIVE, 0, 0),

			     NULL, NULL, 0, 0);

	return status;

}



void vchiq_log_dump_mem(const char *label, u32 addr, const void *void_mem,