Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 4b4b535e authored by Daeseok Youn's avatar Daeseok Youn Committed by Greg Kroah-Hartman
Browse files

staging: unisys: remove incorrect error handling after queue_delayed_work 



The queue_delayed_work() return false if the work is
already on the queue, true otherwise.
So return value cannot be less than zero.

Signed-off-by: default avatarDaeseok Youn <daeseok.youn@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 93c76c9c

drivers/staging/unisys/visorchipset/visorchipset_main.c

+4 −10
Original line number Diff line number Diff line
@@ -2054,11 +2054,8 @@ controlvm_periodic_work(struct work_struct *work) 
		}

	}



	if (queue_delayed_work(Periodic_controlvm_workqueue,

			       &Periodic_controlvm_work, Poll_jiffies) < 0) {

		LOGERR("queue_delayed_work failed!");

		POSTCODE_LINUX_2(QUEUE_DELAYED_WORK_PC, POSTCODE_SEVERITY_ERR);

	}

	queue_delayed_work(Periodic_controlvm_workqueue,

			   &Periodic_controlvm_work, Poll_jiffies);

}



static void
@@ -2192,11 +2189,8 @@ setup_crash_devices_work_queue(struct work_struct *work) 


	Poll_jiffies = POLLJIFFIES_CONTROLVMCHANNEL_SLOW;



	if (queue_delayed_work(Periodic_controlvm_workqueue,

			       &Periodic_controlvm_work, Poll_jiffies) < 0) {

		LOGERR("queue_delayed_work failed!");

		POSTCODE_LINUX_2(QUEUE_DELAYED_WORK_PC, POSTCODE_SEVERITY_ERR);

	}

	queue_delayed_work(Periodic_controlvm_workqueue,

			   &Periodic_controlvm_work, Poll_jiffies);

}



static void