Commit 34197588 authored by Gaston Gonzalez's avatar Gaston Gonzalez Committed by Greg Kroah-Hartman
Browse files

staging: vchiq: add braces to if block



The rule of not using braces in single statement does not apply if only
one branch of the conditional statement is a single statement. Add
braces to fix this.

While at it, remove extra blank space after a comment inside the if
block.

Reported by checkpatch.pl

Signed-off-by: default avatarGaston Gonzalez <gascoar@gmail.com>
Link: https://lore.kernel.org/r/20210915202916.413914-2-gascoar@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5b3087ef
Loading
Loading
Loading
Loading
+4 −5
Original line number Diff line number Diff line
@@ -34,16 +34,15 @@ void vchiq_add_connected_callback(void (*callback)(void))
	if (mutex_lock_killable(&g_connected_mutex))
		return;

	if (g_connected)
	if (g_connected) {
		/* We're already connected. Call the callback immediately. */

		callback();
	else {
		if (g_num_deferred_callbacks >= MAX_CALLBACKS)
	} else {
		if (g_num_deferred_callbacks >= MAX_CALLBACKS) {
			vchiq_log_error(vchiq_core_log_level,
				"There already %d callback registered - please increase MAX_CALLBACKS",
				g_num_deferred_callbacks);
		else {
		} else {
			g_deferred_callback[g_num_deferred_callbacks] =
				callback;
			g_num_deferred_callbacks++;