Commit c7d3df35 authored by Tomas Winkler's avatar Tomas Winkler Committed by Greg Kroah-Hartman
Browse files

mei: use internal watchdog device registration tracking 



remove bool wd_interface_reg as watchdog device already
keeps track of its registration

Signed-off-by: default avatarTomas Winkler <tomas.winkler@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 15ea1910

drivers/misc/mei/init.c

+0 −2
Original line number Diff line number Diff line
@@ -111,8 +111,6 @@ struct mei_device *mei_device_init(struct pci_dev *pdev) 
	init_waitqueue_head(&dev->wait_stop_wd);

	dev->dev_state = MEI_DEV_INITIALIZING;

	dev->iamthif_state = MEI_IAMTHIF_IDLE;

	dev->wd_interface_reg = false;





	mei_io_list_init(&dev->read_list);

	mei_io_list_init(&dev->write_list);

drivers/misc/mei/mei_dev.h

+0 −1
Original line number Diff line number Diff line
@@ -249,7 +249,6 @@ struct mei_device { 


	struct mei_cl wd_cl;

	enum mei_wd_states wd_state;

	bool wd_interface_reg;

	bool wd_pending;

	u16 wd_timeout;

	unsigned char wd_data[MEI_WD_START_MSG_SIZE];

drivers/misc/mei/wd.c

+1 −4
Original line number Diff line number Diff line
@@ -360,23 +360,20 @@ void mei_watchdog_register(struct mei_device *dev) 
	if (watchdog_register_device(&amt_wd_dev)) {

		dev_err(&dev->pdev->dev,

			"wd: unable to register watchdog device.\n");

		dev->wd_interface_reg = false;

		return;

	}



	dev_dbg(&dev->pdev->dev,

		"wd: successfully register watchdog interface.\n");

	dev->wd_interface_reg = true;

	watchdog_set_drvdata(&amt_wd_dev, dev);

}



void mei_watchdog_unregister(struct mei_device *dev)

{

	if (!dev->wd_interface_reg)

	if (test_bit(WDOG_UNREGISTERED, &amt_wd_dev.status))

		return;



	watchdog_set_drvdata(&amt_wd_dev, NULL);

	watchdog_unregister_device(&amt_wd_dev);

	dev->wd_interface_reg = false;

}