Commit fa15d6d3 authored by Tim Sell's avatar Tim Sell Committed by Greg Kroah-Hartman
Browse files

staging: unisys: visornic: correct obvious double-allocation of workqueues 



Looks like an errant patch fitting caused us to redundantly allocate the
workqueues at both the beginning and end of visornic_init().  This was
corrected by removing the allocations at the beginning.

Signed-off-by: default avatarTim Sell <Timothy.Sell@unisys.com>
Signed-off-by: default avatarBenjamin Romer <benjamin.romer@unisys.com>
Fixes: 68905a14 ('staging: unisys: Add s-Par visornic ethernet driver')
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 00748b0c

drivers/staging/unisys/visornic/visornic_main.c

+0 −12
Original line number Diff line number Diff line
@@ -2127,18 +2127,6 @@ static int visornic_init(void) 
	struct dentry *ret;

	int err = -ENOMEM;



	/* create workqueue for serverdown completion */

	visornic_serverdown_workqueue =

		create_singlethread_workqueue("visornic_serverdown");

	if (!visornic_serverdown_workqueue)

		return -ENOMEM;



	/* create workqueue for tx timeout reset */

	visornic_timeout_reset_workqueue =

		create_singlethread_workqueue("visornic_timeout_reset");

	if (!visornic_timeout_reset_workqueue)

		return -ENOMEM;



	visornic_debugfs_dir = debugfs_create_dir("visornic", NULL);

	if (!visornic_debugfs_dir)

		return err;