Commit 8f66090b authored by Thomas Zimmermann's avatar Thomas Zimmermann

drm/amdgpu: Remove references to struct drm_device.pdev

Using struct drm_device.pdev is deprecated. Convert amdgpu to struct
drm_device.dev. No functional changes.

v3:
	* rebased
Signed-off-by: default avatarThomas Zimmermann <tzimmermann@suse.de>
Acked-by: default avatarChristian König <christian.koenig@amd.com>
Acked-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Acked-by: default avatarSam Ravnborg <sam@ravnborg.org>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: Christian König <christian.koenig@amd.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210107080748.4768-3-tzimmermann@suse.de
parent 8a11d283
...@@ -1427,9 +1427,9 @@ static void amdgpu_switcheroo_set_state(struct pci_dev *pdev, ...@@ -1427,9 +1427,9 @@ static void amdgpu_switcheroo_set_state(struct pci_dev *pdev,
/* don't suspend or resume card normally */ /* don't suspend or resume card normally */
dev->switch_power_state = DRM_SWITCH_POWER_CHANGING; dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
pci_set_power_state(dev->pdev, PCI_D0); pci_set_power_state(pdev, PCI_D0);
amdgpu_device_load_pci_state(dev->pdev); amdgpu_device_load_pci_state(pdev);
r = pci_enable_device(dev->pdev); r = pci_enable_device(pdev);
if (r) if (r)
DRM_WARN("pci_enable_device failed (%d)\n", r); DRM_WARN("pci_enable_device failed (%d)\n", r);
amdgpu_device_resume(dev, true); amdgpu_device_resume(dev, true);
...@@ -1441,10 +1441,10 @@ static void amdgpu_switcheroo_set_state(struct pci_dev *pdev, ...@@ -1441,10 +1441,10 @@ static void amdgpu_switcheroo_set_state(struct pci_dev *pdev,
drm_kms_helper_poll_disable(dev); drm_kms_helper_poll_disable(dev);
dev->switch_power_state = DRM_SWITCH_POWER_CHANGING; dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
amdgpu_device_suspend(dev, true); amdgpu_device_suspend(dev, true);
amdgpu_device_cache_pci_state(dev->pdev); amdgpu_device_cache_pci_state(pdev);
/* Shut down the device */ /* Shut down the device */
pci_disable_device(dev->pdev); pci_disable_device(pdev);
pci_set_power_state(dev->pdev, PCI_D3cold); pci_set_power_state(pdev, PCI_D3cold);
dev->switch_power_state = DRM_SWITCH_POWER_OFF; dev->switch_power_state = DRM_SWITCH_POWER_OFF;
} }
} }
...@@ -1707,8 +1707,7 @@ static void amdgpu_device_enable_virtual_display(struct amdgpu_device *adev) ...@@ -1707,8 +1707,7 @@ static void amdgpu_device_enable_virtual_display(struct amdgpu_device *adev)
adev->enable_virtual_display = false; adev->enable_virtual_display = false;
if (amdgpu_virtual_display) { if (amdgpu_virtual_display) {
struct drm_device *ddev = adev_to_drm(adev); const char *pci_address_name = pci_name(adev->pdev);
const char *pci_address_name = pci_name(ddev->pdev);
char *pciaddstr, *pciaddstr_tmp, *pciaddname_tmp, *pciaddname; char *pciaddstr, *pciaddstr_tmp, *pciaddname_tmp, *pciaddname;
pciaddstr = kstrdup(amdgpu_virtual_display, GFP_KERNEL); pciaddstr = kstrdup(amdgpu_virtual_display, GFP_KERNEL);
...@@ -3401,7 +3400,7 @@ int amdgpu_device_init(struct amdgpu_device *adev, ...@@ -3401,7 +3400,7 @@ int amdgpu_device_init(struct amdgpu_device *adev,
} }
} }
pci_enable_pcie_error_reporting(adev->ddev.pdev); pci_enable_pcie_error_reporting(adev->pdev);
/* Post card if necessary */ /* Post card if necessary */
if (amdgpu_device_need_post(adev)) { if (amdgpu_device_need_post(adev)) {
......
...@@ -926,6 +926,7 @@ amdgpu_display_user_framebuffer_create(struct drm_device *dev, ...@@ -926,6 +926,7 @@ amdgpu_display_user_framebuffer_create(struct drm_device *dev,
struct drm_file *file_priv, struct drm_file *file_priv,
const struct drm_mode_fb_cmd2 *mode_cmd) const struct drm_mode_fb_cmd2 *mode_cmd)
{ {
struct amdgpu_device *adev = drm_to_adev(dev);
struct drm_gem_object *obj; struct drm_gem_object *obj;
struct amdgpu_framebuffer *amdgpu_fb; struct amdgpu_framebuffer *amdgpu_fb;
int ret; int ret;
......
...@@ -1204,7 +1204,6 @@ static int amdgpu_pci_probe(struct pci_dev *pdev, ...@@ -1204,7 +1204,6 @@ static int amdgpu_pci_probe(struct pci_dev *pdev,
if (ret) if (ret)
return ret; return ret;
ddev->pdev = pdev;
pci_set_drvdata(pdev, ddev); pci_set_drvdata(pdev, ddev);
ret = amdgpu_driver_load_kms(adev, ent->driver_data); ret = amdgpu_driver_load_kms(adev, ent->driver_data);
......
...@@ -271,7 +271,7 @@ static int amdgpufb_create(struct drm_fb_helper *helper, ...@@ -271,7 +271,7 @@ static int amdgpufb_create(struct drm_fb_helper *helper,
DRM_INFO("fb depth is %d\n", fb->format->depth); DRM_INFO("fb depth is %d\n", fb->format->depth);
DRM_INFO(" pitch is %d\n", fb->pitches[0]); DRM_INFO(" pitch is %d\n", fb->pitches[0]);
vga_switcheroo_client_fb_set(adev_to_drm(adev)->pdev, info); vga_switcheroo_client_fb_set(adev->pdev, info);
return 0; return 0;
out: out:
......
...@@ -619,7 +619,7 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data, ...@@ -619,7 +619,7 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data,
int r = 0; int r = 0;
if (args->va_address < AMDGPU_VA_RESERVED_SIZE) { if (args->va_address < AMDGPU_VA_RESERVED_SIZE) {
dev_dbg(&dev->pdev->dev, dev_dbg(dev->dev,
"va_address 0x%LX is in reserved area 0x%LX\n", "va_address 0x%LX is in reserved area 0x%LX\n",
args->va_address, AMDGPU_VA_RESERVED_SIZE); args->va_address, AMDGPU_VA_RESERVED_SIZE);
return -EINVAL; return -EINVAL;
...@@ -627,7 +627,7 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data, ...@@ -627,7 +627,7 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data,
if (args->va_address >= AMDGPU_GMC_HOLE_START && if (args->va_address >= AMDGPU_GMC_HOLE_START &&
args->va_address < AMDGPU_GMC_HOLE_END) { args->va_address < AMDGPU_GMC_HOLE_END) {
dev_dbg(&dev->pdev->dev, dev_dbg(dev->dev,
"va_address 0x%LX is in VA hole 0x%LX-0x%LX\n", "va_address 0x%LX is in VA hole 0x%LX-0x%LX\n",
args->va_address, AMDGPU_GMC_HOLE_START, args->va_address, AMDGPU_GMC_HOLE_START,
AMDGPU_GMC_HOLE_END); AMDGPU_GMC_HOLE_END);
...@@ -639,14 +639,14 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data, ...@@ -639,14 +639,14 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data,
vm_size = adev->vm_manager.max_pfn * AMDGPU_GPU_PAGE_SIZE; vm_size = adev->vm_manager.max_pfn * AMDGPU_GPU_PAGE_SIZE;
vm_size -= AMDGPU_VA_RESERVED_SIZE; vm_size -= AMDGPU_VA_RESERVED_SIZE;
if (args->va_address + args->map_size > vm_size) { if (args->va_address + args->map_size > vm_size) {
dev_dbg(&dev->pdev->dev, dev_dbg(dev->dev,
"va_address 0x%llx is in top reserved area 0x%llx\n", "va_address 0x%llx is in top reserved area 0x%llx\n",
args->va_address + args->map_size, vm_size); args->va_address + args->map_size, vm_size);
return -EINVAL; return -EINVAL;
} }
if ((args->flags & ~valid_flags) && (args->flags & ~prt_flags)) { if ((args->flags & ~valid_flags) && (args->flags & ~prt_flags)) {
dev_dbg(&dev->pdev->dev, "invalid flags combination 0x%08X\n", dev_dbg(dev->dev, "invalid flags combination 0x%08X\n",
args->flags); args->flags);
return -EINVAL; return -EINVAL;
} }
...@@ -658,7 +658,7 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data, ...@@ -658,7 +658,7 @@ int amdgpu_gem_va_ioctl(struct drm_device *dev, void *data,
case AMDGPU_VA_OP_REPLACE: case AMDGPU_VA_OP_REPLACE:
break; break;
default: default:
dev_dbg(&dev->pdev->dev, "unsupported operation %d\n", dev_dbg(dev->dev, "unsupported operation %d\n",
args->operation); args->operation);
return -EINVAL; return -EINVAL;
} }
......
...@@ -176,7 +176,7 @@ struct amdgpu_i2c_chan *amdgpu_i2c_create(struct drm_device *dev, ...@@ -176,7 +176,7 @@ struct amdgpu_i2c_chan *amdgpu_i2c_create(struct drm_device *dev,
i2c->rec = *rec; i2c->rec = *rec;
i2c->adapter.owner = THIS_MODULE; i2c->adapter.owner = THIS_MODULE;
i2c->adapter.class = I2C_CLASS_DDC; i2c->adapter.class = I2C_CLASS_DDC;
i2c->adapter.dev.parent = &dev->pdev->dev; i2c->adapter.dev.parent = dev->dev;
i2c->dev = dev; i2c->dev = dev;
i2c_set_adapdata(&i2c->adapter, i2c); i2c_set_adapdata(&i2c->adapter, i2c);
mutex_init(&i2c->mutex); mutex_init(&i2c->mutex);
......
...@@ -142,7 +142,7 @@ int amdgpu_driver_load_kms(struct amdgpu_device *adev, unsigned long flags) ...@@ -142,7 +142,7 @@ int amdgpu_driver_load_kms(struct amdgpu_device *adev, unsigned long flags)
(amdgpu_is_atpx_hybrid() || (amdgpu_is_atpx_hybrid() ||
amdgpu_has_atpx_dgpu_power_cntl()) && amdgpu_has_atpx_dgpu_power_cntl()) &&
((flags & AMD_IS_APU) == 0) && ((flags & AMD_IS_APU) == 0) &&
!pci_is_thunderbolt_attached(dev->pdev)) !pci_is_thunderbolt_attached(to_pci_dev(dev->dev)))
flags |= AMD_IS_PX; flags |= AMD_IS_PX;
parent = pci_upstream_bridge(adev->pdev); parent = pci_upstream_bridge(adev->pdev);
...@@ -156,7 +156,7 @@ int amdgpu_driver_load_kms(struct amdgpu_device *adev, unsigned long flags) ...@@ -156,7 +156,7 @@ int amdgpu_driver_load_kms(struct amdgpu_device *adev, unsigned long flags)
*/ */
r = amdgpu_device_init(adev, flags); r = amdgpu_device_init(adev, flags);
if (r) { if (r) {
dev_err(&dev->pdev->dev, "Fatal error during GPU init\n"); dev_err(dev->dev, "Fatal error during GPU init\n");
goto out; goto out;
} }
...@@ -199,7 +199,7 @@ int amdgpu_driver_load_kms(struct amdgpu_device *adev, unsigned long flags) ...@@ -199,7 +199,7 @@ int amdgpu_driver_load_kms(struct amdgpu_device *adev, unsigned long flags)
acpi_status = amdgpu_acpi_init(adev); acpi_status = amdgpu_acpi_init(adev);
if (acpi_status) if (acpi_status)
dev_dbg(&dev->pdev->dev, "Error during ACPI methods call\n"); dev_dbg(dev->dev, "Error during ACPI methods call\n");
if (adev->runpm) { if (adev->runpm) {
/* only need to skip on ATPX */ /* only need to skip on ATPX */
...@@ -735,10 +735,10 @@ int amdgpu_info_ioctl(struct drm_device *dev, void *data, struct drm_file *filp) ...@@ -735,10 +735,10 @@ int amdgpu_info_ioctl(struct drm_device *dev, void *data, struct drm_file *filp)
if (!dev_info) if (!dev_info)
return -ENOMEM; return -ENOMEM;
dev_info->device_id = dev->pdev->device; dev_info->device_id = adev->pdev->device;
dev_info->chip_rev = adev->rev_id; dev_info->chip_rev = adev->rev_id;
dev_info->external_rev = adev->external_rev_id; dev_info->external_rev = adev->external_rev_id;
dev_info->pci_rev = dev->pdev->revision; dev_info->pci_rev = adev->pdev->revision;
dev_info->family = adev->family; dev_info->family = adev->family;
dev_info->num_shader_engines = adev->gfx.config.max_shader_engines; dev_info->num_shader_engines = adev->gfx.config.max_shader_engines;
dev_info->num_shader_arrays_per_engine = adev->gfx.config.max_sh_per_se; dev_info->num_shader_arrays_per_engine = adev->gfx.config.max_sh_per_se;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment