parisc: remove driver_data direct access of struct device
PATCH 1/1 parisc: remove driver_data direct access of struct device
From: Greg Kroah-Hartman <[email protected]>
In the near future, the driver core is going to not allow direct access to the driver_data pointer in struct device. Instead, the functions dev_get_drvdata() and dev_set_drvdata() should be used. These functions have been around since the beginning, so are backwards compatible with all older kernel versions.
--- drivers/parisc/eisa.c | 2 +- drivers/parisc/sba_iommu.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-)
drivers/parisc/eisa.c +1 -1
@@ -373,7 +373,7 @@ static int __init eisa_probe(struct pariif (result >=0 ) {/* FIXME : Don't enumerate the bus twice. */ eisa_dev.root.dev = &dev->dev; - dev->dev.driver_data = &eisa_dev.root; +dev_set_drvdata (&dev->dev, &eisa_dev.root); eisa_dev.root.bus_base_addr =0 ; eisa_dev.root.res = &eisa_dev.hba.io_space; eisa_dev.root.slots = result;
drivers/parisc/sba_iommu.c +3 -3
@@ -2010,7 +2010,7 @@ void __init sba_init(void)void *sba_get_iommu (struct parisc_device *pci_hba) {struct parisc_device *sba_dev =parisc_parent (pci_hba); -struct sba_device *sba = sba_dev->dev.driver_data; +struct sba_device *sba =dev_get_drvdata (&sba_dev->dev);char t = sba_dev->id.hw_type;int iocnum = (pci_hba->hw_path >>3 );/* rope # */
@@ -2031,7 +2031,7 @@ void * sba_get_iommu(struct parisc_devicvoid sba_directed_lmmio (struct parisc_device *pci_hba,struct resource *r) {struct parisc_device *sba_dev =parisc_parent (pci_hba); -struct sba_device *sba = sba_dev->dev.driver_data; +struct sba_device *sba =dev_get_drvdata (&sba_dev->dev);char t = sba_dev->id.hw_type;int i;int rope = (pci_hba->hw_path & (ROPES_PER_IOC -1 ));/* rope # */
@@ -2073,7 +2073,7 @@ void sba_directed_lmmio(struct parisc_devoid sba_distributed_lmmio (struct parisc_device *pci_hba,struct resource *r ) {struct parisc_device *sba_dev =parisc_parent (pci_hba); -struct sba_device *sba = sba_dev->dev.driver_data; +struct sba_device *sba =dev_get_drvdata (&sba_dev->dev);char t = sba_dev->id.hw_type;int base, size;int rope = (pci_hba->hw_path & (ROPES_PER_IOC -1 ));/* rope # */
Quoted 8 lines · Greg Kroah-Hartman
From: Greg Kroah-Hartman <[email protected]>
In the near future, the driver core is going to not allow direct access to the
driver_datapointer in struct device. Instead, the functionsdev_get_drvdata()anddev_set_drvdata()should be used. These functions have been around since the beginning, so are backwards compatible with all older kernel versions.
Cool, applied and will be sent along tonight unless you'd rather merge it yourself in which case:
Quoted 50 lines
Cc Helge Deller <[email protected]>Cc Kyle McMartin <[email protected]>Signed-off-by Greg Kroah-Hartman <[email protected]>--- drivers/parisc/eisa.c | 2 +- drivers/parisc/sba_iommu.c | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-)drivers/parisc/eisa.c +1
-1@@ -373,7 +373,7 @@ static int __init eisa_probe(struct pariif (result >=0 ) {/* FIXME : Don't enumerate the bus twice. */ eisa_dev.root.dev = &dev->dev; - dev->dev.driver_data = &eisa_dev.root; +dev_set_drvdata (&dev->dev, &eisa_dev.root); eisa_dev.root.bus_base_addr =0 ; eisa_dev.root.res = &eisa_dev.hba.io_space; eisa_dev.root.slots = result;drivers/parisc/sba_iommu.c +3
-3@@ -2010,7 +2010,7 @@ void __init sba_init(void)void *sba_get_iommu (struct parisc_device *pci_hba) {struct parisc_device *sba_dev =parisc_parent (pci_hba); -struct sba_device *sba = sba_dev->dev.driver_data; +struct sba_device *sba =dev_get_drvdata (&sba_dev->dev);char t = sba_dev->id.hw_type;int iocnum = (pci_hba->hw_path >>3 );/* rope # */ @@ -2031,7 +2031,7 @@ void * sba_get_iommu(struct parisc_devicvoid sba_directed_lmmio (struct parisc_device *pci_hba,struct resource *r) {struct parisc_device *sba_dev =parisc_parent (pci_hba); -struct sba_device *sba = sba_dev->dev.driver_data; +struct sba_device *sba =dev_get_drvdata (&sba_dev->dev);char t = sba_dev->id.hw_type;int i;int rope = (pci_hba->hw_path & (ROPES_PER_IOC -1 ));/* rope # */ @@ -2073,7 +2073,7 @@ void sba_directed_lmmio(struct parisc_devoid sba_distributed_lmmio (struct parisc_device *pci_hba,struct resource *r ) {struct parisc_device *sba_dev =parisc_parent (pci_hba); -struct sba_device *sba = sba_dev->dev.driver_data; +struct sba_device *sba =dev_get_drvdata (&sba_dev->dev);char t = sba_dev->id.hw_type;int base, size;int rope = (pci_hba->hw_path & (ROPES_PER_IOC -1 ));/* rope # */
Quoted 10 lines · Kyle McMartin
Quoted 8 lines · Greg Kroah-Hartman
From: Greg Kroah-Hartman <[email protected]>
In the near future, the driver core is going to not allow direct access to the
driver_datapointer in struct device. Instead, the functionsdev_get_drvdata()anddev_set_drvdata()should be used. These functions have been around since the beginning, so are backwards compatible with all older kernel versions.
Quoted 4 lines
Cool, applied and will be sent along tonight unless you'd rather merge it yourself in which case:
Acked-by Kyle McMartin <[email protected]>
Nope, feel free to send it yourself, no objection from me at all :)
thanks,
greg k-h