Commit cfb1693c authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Mauro Carvalho Chehab
Browse files

media: staging: atomisp: Remove redundant PCI code 



There is no need to keep a reference to PCI root bridge.

Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent e583bfbf

drivers/staging/media/atomisp/pci/atomisp2/atomisp_internal.h

+0 −1
Original line number Diff line number Diff line
@@ -227,7 +227,6 @@ struct atomisp_device { 
	struct media_device media_dev;

	struct atomisp_platform_data *pdata;

	void *mmu_l1_base;

	struct pci_dev *pci_root;

	const struct firmware *firmware;



	struct pm_qos_request pm_qos;

drivers/staging/media/atomisp/pci/atomisp2/atomisp_v4l2.c

+0 −8
Original line number Diff line number Diff line
@@ -1210,11 +1210,6 @@ static int atomisp_pci_probe(struct pci_dev *dev, 
	isp->pdev = dev;

	isp->dev = &dev->dev;

	isp->sw_contex.power_state = ATOM_ISP_POWER_UP;

	isp->pci_root = pci_get_domain_bus_and_slot(0, 0, 0);

	if (!isp->pci_root) {

		dev_err(&dev->dev, "Unable to find PCI host\n");

		return -ENODEV;

	}

	isp->saved_regs.ispmmadr = start;



	rt_mutex_init(&isp->mutex);
@@ -1494,7 +1489,6 @@ load_fw_fail: 
	/* Address later when we worry about the ...field chips */

	if (IS_ENABLED(CONFIG_PM) && atomisp_mrfld_power_down(isp))

		dev_err(&dev->dev, "Failed to switch off ISP\n");

	pci_dev_put(isp->pci_root);

	return err;

}
@@ -1515,8 +1509,6 @@ static void atomisp_pci_remove(struct pci_dev *dev) 
	pm_qos_remove_request(&isp->pm_qos);



	atomisp_msi_irq_uninit(isp, dev);

	pci_dev_put(isp->pci_root);



	atomisp_unregister_entities(isp);



	destroy_workqueue(isp->wdt_work_queue);

CRA Git | Maintained and supported by SUSTech CRA and CCSE