Commit 877e50b3 authored by Lee Jones's avatar Lee Jones Committed by Mika Westerberg
Browse files

thunderbolt: tb: Kernel-doc function headers should document their parameters 



Fixes the following W=1 kernel build warning(s):

 drivers/thunderbolt/tb.c:535: warning: Function parameter or member 'sw' not described in 'tb_scan_switch'
 drivers/thunderbolt/tb.c:551: warning: Function parameter or member 'port' not described in 'tb_scan_port'
 drivers/thunderbolt/tb.c:711: warning: Function parameter or member 'tb' not described in 'tb_free_invalid_tunnels'
 drivers/thunderbolt/tb.c:726: warning: Function parameter or member 'sw' not described in 'tb_free_unplugged_children'
 drivers/thunderbolt/tb.c:1129: warning: Function parameter or member 'work' not described in 'tb_handle_hotplug'
 drivers/thunderbolt/tb.c:1239: warning: Function parameter or member 'tb' not described in 'tb_handle_event'
 drivers/thunderbolt/tb.c:1239: warning: Function parameter or member 'type' not described in 'tb_handle_event'
 drivers/thunderbolt/tb.c:1239: warning: Function parameter or member 'buf' not described in 'tb_handle_event'
 drivers/thunderbolt/tb.c:1239: warning: Function parameter or member 'size' not described in 'tb_handle_event'
 drivers/thunderbolt/tb.c:1239: warning: expecting prototype for tb_schedule_hotplug_handler(). Prototype was for tb_handle_event() instead

Cc: Andreas Noever <andreas.noever@gmail.com>
Cc: Michael Jamet <michael.jamet@intel.com>
Cc: Yehezkel Bernat <YehezkelShB@gmail.com>
Cc: linux-usb@vger.kernel.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarMika Westerberg <mika.westerberg@linux.intel.com>
parent a7bfb27b

drivers/thunderbolt/tb.c

+6 −6
Original line number Diff line number Diff line
@@ -528,7 +528,7 @@ static int tb_create_usb3_tunnels(struct tb_switch *sw) 


static void tb_scan_port(struct tb_port *port);



/**

/*

 * tb_scan_switch() - scan for and initialize downstream switches

 */

static void tb_scan_switch(struct tb_switch *sw)
@@ -544,7 +544,7 @@ static void tb_scan_switch(struct tb_switch *sw) 
	pm_runtime_put_autosuspend(&sw->dev);

}



/**

/*

 * tb_scan_port() - check for and initialize switches below port

 */

static void tb_scan_port(struct tb_port *port)
@@ -704,7 +704,7 @@ static void tb_deactivate_and_free_tunnel(struct tb_tunnel *tunnel) 
	tb_tunnel_free(tunnel);

}



/**

/*

 * tb_free_invalid_tunnels() - destroy tunnels of devices that have gone away

 */

static void tb_free_invalid_tunnels(struct tb *tb)
@@ -719,7 +719,7 @@ static void tb_free_invalid_tunnels(struct tb *tb) 
	}

}



/**

/*

 * tb_free_unplugged_children() - traverse hierarchy and free unplugged switches

 */

static void tb_free_unplugged_children(struct tb_switch *sw)
@@ -1120,7 +1120,7 @@ static int tb_disconnect_xdomain_paths(struct tb *tb, struct tb_xdomain *xd) 


/* hotplug handling */



/**

/*

 * tb_handle_hotplug() - handle hotplug event

 *

 * Executes on tb->wq.
@@ -1229,7 +1229,7 @@ static void tb_handle_hotplug(struct work_struct *work) 
	kfree(ev);

}



/**

/*

 * tb_schedule_hotplug_handler() - callback function for the control channel

 *

 * Delegates to tb_handle_hotplug.