lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <de14c352-8474-46cb-858b-27d5c22e67b2@arm.com>
Date: Thu, 24 Oct 2024 15:55:21 +0100
From: Steven Price <steven.price@....com>
To: Adrián Larumbe <adrian.larumbe@...labora.com>,
 Boris Brezillon <boris.brezillon@...labora.com>,
 Rob Herring <robh@...nel.org>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
 David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
 Philipp Zabel <p.zabel@...gutronix.de>
Cc: kernel@...labora.com, dri-devel@...ts.freedesktop.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/9] drm/panfrost: Replace DRM driver allocation method
 with newer one

On 15/10/2024 00:31, Adrián Larumbe wrote:
> Drop the deprecated DRM driver allocation method in favour of
> devm_drm_dev_alloc(). Overall just make it the same as in Panthor.
> Also discard now superfluous generic and platform device pointers inside
> the main panfrost device structure.
> 
> Some ancient checkpatch issues unearthed as a result of these changes
> were also fixed, like lines too long or double assignment in one line.
> 
> Signed-off-by: Adrián Larumbe <adrian.larumbe@...labora.com>

Reviewed-by: Steven Price <steven.price@....com>

> ---
>  drivers/gpu/drm/panfrost/panfrost_devfreq.c   |  4 +-
>  drivers/gpu/drm/panfrost/panfrost_device.c    | 49 ++++++-------
>  drivers/gpu/drm/panfrost/panfrost_device.h    |  6 +-
>  drivers/gpu/drm/panfrost/panfrost_drv.c       | 68 +++++++------------
>  drivers/gpu/drm/panfrost/panfrost_dump.c      |  8 +--
>  drivers/gpu/drm/panfrost/panfrost_gem.c       |  6 +-
>  .../gpu/drm/panfrost/panfrost_gem_shrinker.c  |  4 +-
>  drivers/gpu/drm/panfrost/panfrost_gpu.c       | 49 ++++++-------
>  drivers/gpu/drm/panfrost/panfrost_job.c       | 37 +++++-----
>  drivers/gpu/drm/panfrost/panfrost_mmu.c       | 40 ++++++-----
>  drivers/gpu/drm/panfrost/panfrost_perfcnt.c   | 22 +++---
>  11 files changed, 141 insertions(+), 152 deletions(-)
> 
> diff --git a/drivers/gpu/drm/panfrost/panfrost_devfreq.c b/drivers/gpu/drm/panfrost/panfrost_devfreq.c
> index 2d30da38c2c3..093910cdf6a1 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_devfreq.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_devfreq.c
> @@ -74,7 +74,7 @@ static int panfrost_devfreq_get_dev_status(struct device *dev,
>  
>  	spin_unlock_irqrestore(&pfdevfreq->lock, irqflags);
>  
> -	dev_dbg(pfdev->dev, "busy %lu total %lu %lu %% freq %lu MHz\n",
> +	dev_dbg(pfdev->base.dev, "busy %lu total %lu %lu %% freq %lu MHz\n",
>  		status->busy_time, status->total_time,
>  		status->busy_time / (status->total_time / 100),
>  		status->current_frequency / 1000 / 1000);
> @@ -119,7 +119,7 @@ int panfrost_devfreq_init(struct panfrost_device *pfdev)
>  	int ret;
>  	struct dev_pm_opp *opp;
>  	unsigned long cur_freq;
> -	struct device *dev = &pfdev->pdev->dev;
> +	struct device *dev = pfdev->base.dev;
>  	struct devfreq *devfreq;
>  	struct thermal_cooling_device *cooling;
>  	struct panfrost_devfreq *pfdevfreq = &pfdev->pfdevfreq;
> diff --git a/drivers/gpu/drm/panfrost/panfrost_device.c b/drivers/gpu/drm/panfrost/panfrost_device.c
> index a45e4addcc19..4fe29286bbe3 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_device.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_device.c
> @@ -20,9 +20,9 @@
>  
>  static int panfrost_reset_init(struct panfrost_device *pfdev)
>  {
> -	pfdev->rstc = devm_reset_control_array_get_optional_exclusive(pfdev->dev);
> +	pfdev->rstc = devm_reset_control_array_get_optional_exclusive(pfdev->base.dev);
>  	if (IS_ERR(pfdev->rstc)) {
> -		dev_err(pfdev->dev, "get reset failed %ld\n", PTR_ERR(pfdev->rstc));
> +		dev_err(pfdev->base.dev, "get reset failed %ld\n", PTR_ERR(pfdev->rstc));
>  		return PTR_ERR(pfdev->rstc);
>  	}
>  
> @@ -39,22 +39,22 @@ static int panfrost_clk_init(struct panfrost_device *pfdev)
>  	int err;
>  	unsigned long rate;
>  
> -	pfdev->clock = devm_clk_get(pfdev->dev, NULL);
> +	pfdev->clock = devm_clk_get(pfdev->base.dev, NULL);
>  	if (IS_ERR(pfdev->clock)) {
> -		dev_err(pfdev->dev, "get clock failed %ld\n", PTR_ERR(pfdev->clock));
> +		dev_err(pfdev->base.dev, "get clock failed %ld\n", PTR_ERR(pfdev->clock));
>  		return PTR_ERR(pfdev->clock);
>  	}
>  
>  	rate = clk_get_rate(pfdev->clock);
> -	dev_info(pfdev->dev, "clock rate = %lu\n", rate);
> +	dev_info(pfdev->base.dev, "clock rate = %lu\n", rate);
>  
>  	err = clk_prepare_enable(pfdev->clock);
>  	if (err)
>  		return err;
>  
> -	pfdev->bus_clock = devm_clk_get_optional(pfdev->dev, "bus");
> +	pfdev->bus_clock = devm_clk_get_optional(pfdev->base.dev, "bus");
>  	if (IS_ERR(pfdev->bus_clock)) {
> -		dev_err(pfdev->dev, "get bus_clock failed %ld\n",
> +		dev_err(pfdev->base.dev, "get bus_clock failed %ld\n",
>  			PTR_ERR(pfdev->bus_clock));
>  		err = PTR_ERR(pfdev->bus_clock);
>  		goto disable_clock;
> @@ -62,7 +62,7 @@ static int panfrost_clk_init(struct panfrost_device *pfdev)
>  
>  	if (pfdev->bus_clock) {
>  		rate = clk_get_rate(pfdev->bus_clock);
> -		dev_info(pfdev->dev, "bus_clock rate = %lu\n", rate);
> +		dev_info(pfdev->base.dev, "bus_clock rate = %lu\n", rate);
>  
>  		err = clk_prepare_enable(pfdev->bus_clock);
>  		if (err)
> @@ -87,7 +87,7 @@ static int panfrost_regulator_init(struct panfrost_device *pfdev)
>  {
>  	int ret, i;
>  
> -	pfdev->regulators = devm_kcalloc(pfdev->dev, pfdev->comp->num_supplies,
> +	pfdev->regulators = devm_kcalloc(pfdev->base.dev, pfdev->comp->num_supplies,
>  					 sizeof(*pfdev->regulators),
>  					 GFP_KERNEL);
>  	if (!pfdev->regulators)
> @@ -96,12 +96,12 @@ static int panfrost_regulator_init(struct panfrost_device *pfdev)
>  	for (i = 0; i < pfdev->comp->num_supplies; i++)
>  		pfdev->regulators[i].supply = pfdev->comp->supply_names[i];
>  
> -	ret = devm_regulator_bulk_get(pfdev->dev,
> +	ret = devm_regulator_bulk_get(pfdev->base.dev,
>  				      pfdev->comp->num_supplies,
>  				      pfdev->regulators);
>  	if (ret < 0) {
>  		if (ret != -EPROBE_DEFER)
> -			dev_err(pfdev->dev, "failed to get regulators: %d\n",
> +			dev_err(pfdev->base.dev, "failed to get regulators: %d\n",
>  				ret);
>  		return ret;
>  	}
> @@ -109,7 +109,7 @@ static int panfrost_regulator_init(struct panfrost_device *pfdev)
>  	ret = regulator_bulk_enable(pfdev->comp->num_supplies,
>  				    pfdev->regulators);
>  	if (ret < 0) {
> -		dev_err(pfdev->dev, "failed to enable regulators: %d\n", ret);
> +		dev_err(pfdev->base.dev, "failed to enable regulators: %d\n", ret);
>  		return ret;
>  	}
>  
> @@ -144,7 +144,7 @@ static int panfrost_pm_domain_init(struct panfrost_device *pfdev)
>  	int err;
>  	int i, num_domains;
>  
> -	num_domains = of_count_phandle_with_args(pfdev->dev->of_node,
> +	num_domains = of_count_phandle_with_args(pfdev->base.dev->of_node,
>  						 "power-domains",
>  						 "#power-domain-cells");
>  
> @@ -156,7 +156,7 @@ static int panfrost_pm_domain_init(struct panfrost_device *pfdev)
>  		return 0;
>  
>  	if (num_domains != pfdev->comp->num_pm_domains) {
> -		dev_err(pfdev->dev,
> +		dev_err(pfdev->base.dev,
>  			"Incorrect number of power domains: %d provided, %d needed\n",
>  			num_domains, pfdev->comp->num_pm_domains);
>  		return -EINVAL;
> @@ -168,20 +168,21 @@ static int panfrost_pm_domain_init(struct panfrost_device *pfdev)
>  
>  	for (i = 0; i < num_domains; i++) {
>  		pfdev->pm_domain_devs[i] =
> -			dev_pm_domain_attach_by_name(pfdev->dev,
> -					pfdev->comp->pm_domain_names[i]);
> +			dev_pm_domain_attach_by_name(pfdev->base.dev,
> +						     pfdev->comp->pm_domain_names[i]);
>  		if (IS_ERR_OR_NULL(pfdev->pm_domain_devs[i])) {
>  			err = PTR_ERR(pfdev->pm_domain_devs[i]) ? : -ENODATA;
>  			pfdev->pm_domain_devs[i] = NULL;
> -			dev_err(pfdev->dev,
> +			dev_err(pfdev->base.dev,
>  				"failed to get pm-domain %s(%d): %d\n",
>  				pfdev->comp->pm_domain_names[i], i, err);
>  			goto err;
>  		}
>  
> -		pfdev->pm_domain_links[i] = device_link_add(pfdev->dev,
> -				pfdev->pm_domain_devs[i], DL_FLAG_PM_RUNTIME |
> -				DL_FLAG_STATELESS | DL_FLAG_RPM_ACTIVE);
> +		pfdev->pm_domain_links[i] =
> +			device_link_add(pfdev->base.dev,
> +					pfdev->pm_domain_devs[i], DL_FLAG_PM_RUNTIME |
> +					DL_FLAG_STATELESS | DL_FLAG_RPM_ACTIVE);
>  		if (!pfdev->pm_domain_links[i]) {
>  			dev_err(pfdev->pm_domain_devs[i],
>  				"adding device link failed!\n");
> @@ -211,14 +212,14 @@ int panfrost_device_init(struct panfrost_device *pfdev)
>  
>  	err = panfrost_clk_init(pfdev);
>  	if (err) {
> -		dev_err(pfdev->dev, "clk init failed %d\n", err);
> +		dev_err(pfdev->base.dev, "clk init failed %d\n", err);
>  		return err;
>  	}
>  
>  	err = panfrost_devfreq_init(pfdev);
>  	if (err) {
>  		if (err != -EPROBE_DEFER)
> -			dev_err(pfdev->dev, "devfreq init failed %d\n", err);
> +			dev_err(pfdev->base.dev, "devfreq init failed %d\n", err);
>  		goto out_clk;
>  	}
>  
> @@ -231,7 +232,7 @@ int panfrost_device_init(struct panfrost_device *pfdev)
>  
>  	err = panfrost_reset_init(pfdev);
>  	if (err) {
> -		dev_err(pfdev->dev, "reset init failed %d\n", err);
> +		dev_err(pfdev->base.dev, "reset init failed %d\n", err);
>  		goto out_regulator;
>  	}
>  
> @@ -239,7 +240,7 @@ int panfrost_device_init(struct panfrost_device *pfdev)
>  	if (err)
>  		goto out_reset;
>  
> -	pfdev->iomem = devm_platform_ioremap_resource(pfdev->pdev, 0);
> +	pfdev->iomem = devm_platform_ioremap_resource(to_platform_device(pfdev->base.dev), 0);
>  	if (IS_ERR(pfdev->iomem)) {
>  		err = PTR_ERR(pfdev->iomem);
>  		goto out_pm_domain;
> diff --git a/drivers/gpu/drm/panfrost/panfrost_device.h b/drivers/gpu/drm/panfrost/panfrost_device.h
> index cffcb0ac7c11..d9aea2c2cbe5 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_device.h
> +++ b/drivers/gpu/drm/panfrost/panfrost_device.h
> @@ -98,9 +98,7 @@ struct panfrost_compatible {
>  };
>  
>  struct panfrost_device {
> -	struct device *dev;
> -	struct drm_device *ddev;
> -	struct platform_device *pdev;
> +	struct drm_device base;
>  	int gpu_irq;
>  	int mmu_irq;
>  
> @@ -181,7 +179,7 @@ struct panfrost_file_priv {
>  
>  static inline struct panfrost_device *to_panfrost_device(struct drm_device *ddev)
>  {
> -	return ddev->dev_private;
> +	return container_of(ddev, struct panfrost_device, base);
>  }
>  
>  static inline int panfrost_model_cmp(struct panfrost_device *pfdev, s32 id)
> diff --git a/drivers/gpu/drm/panfrost/panfrost_drv.c b/drivers/gpu/drm/panfrost/panfrost_drv.c
> index 04d615df5259..6284397a7030 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_drv.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_drv.c
> @@ -35,7 +35,7 @@ static int panfrost_ioctl_query_timestamp(struct panfrost_device *pfdev,
>  {
>  	int ret;
>  
> -	ret = pm_runtime_resume_and_get(pfdev->dev);
> +	ret = pm_runtime_resume_and_get(pfdev->base.dev);
>  	if (ret)
>  		return ret;
>  
> @@ -43,14 +43,14 @@ static int panfrost_ioctl_query_timestamp(struct panfrost_device *pfdev,
>  	*arg = panfrost_timestamp_read(pfdev);
>  	panfrost_cycle_counter_put(pfdev);
>  
> -	pm_runtime_put(pfdev->dev);
> +	pm_runtime_put(pfdev->base.dev);
>  	return 0;
>  }
>  
>  static int panfrost_ioctl_get_param(struct drm_device *ddev, void *data, struct drm_file *file)
>  {
>  	struct drm_panfrost_get_param *param = data;
> -	struct panfrost_device *pfdev = ddev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(ddev);
>  	int ret;
>  
>  	if (param->pad != 0)
> @@ -274,7 +274,7 @@ panfrost_copy_in_sync(struct drm_device *dev,
>  static int panfrost_ioctl_submit(struct drm_device *dev, void *data,
>  		struct drm_file *file)
>  {
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  	struct panfrost_file_priv *file_priv = file->driver_priv;
>  	struct drm_panfrost_submit *args = data;
>  	struct drm_syncobj *sync_out = NULL;
> @@ -435,7 +435,7 @@ static int panfrost_ioctl_madvise(struct drm_device *dev, void *data,
>  {
>  	struct panfrost_file_priv *priv = file_priv->driver_priv;
>  	struct drm_panfrost_madvise *args = data;
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  	struct drm_gem_object *gem_obj;
>  	struct panfrost_gem_object *bo;
>  	int ret = 0;
> @@ -507,7 +507,7 @@ static int
>  panfrost_open(struct drm_device *dev, struct drm_file *file)
>  {
>  	int ret;
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  	struct panfrost_file_priv *panfrost_priv;
>  
>  	panfrost_priv = kzalloc(sizeof(*panfrost_priv), GFP_KERNEL);
> @@ -604,8 +604,7 @@ static void panfrost_gpu_show_fdinfo(struct panfrost_device *pfdev,
>  
>  static void panfrost_show_fdinfo(struct drm_printer *p, struct drm_file *file)
>  {
> -	struct drm_device *dev = file->minor->dev;
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(file->minor->dev);
>  
>  	panfrost_gpu_show_fdinfo(pfdev, file->driver_priv, p);
>  
> @@ -647,15 +646,12 @@ static const struct drm_driver panfrost_drm_driver = {
>  static int panfrost_probe(struct platform_device *pdev)
>  {
>  	struct panfrost_device *pfdev;
> -	struct drm_device *ddev;
>  	int err;
>  
> -	pfdev = devm_kzalloc(&pdev->dev, sizeof(*pfdev), GFP_KERNEL);
> -	if (!pfdev)
> -		return -ENOMEM;
> -
> -	pfdev->pdev = pdev;
> -	pfdev->dev = &pdev->dev;
> +	pfdev = devm_drm_dev_alloc(&pdev->dev, &panfrost_drm_driver,
> +				   struct panfrost_device, base);
> +	if (IS_ERR(pfdev))
> +		return PTR_ERR(pfdev);
>  
>  	platform_set_drvdata(pdev, pfdev);
>  
> @@ -665,14 +661,6 @@ static int panfrost_probe(struct platform_device *pdev)
>  
>  	pfdev->coherent = device_get_dma_attr(&pdev->dev) == DEV_DMA_COHERENT;
>  
> -	/* Allocate and initialize the DRM device. */
> -	ddev = drm_dev_alloc(&panfrost_drm_driver, &pdev->dev);
> -	if (IS_ERR(ddev))
> -		return PTR_ERR(ddev);
> -
> -	ddev->dev_private = pfdev;
> -	pfdev->ddev = ddev;
> -
>  	mutex_init(&pfdev->shrinker_lock);
>  	INIT_LIST_HEAD(&pfdev->shrinker_list);
>  
> @@ -683,51 +671,47 @@ static int panfrost_probe(struct platform_device *pdev)
>  		goto err_out0;
>  	}
>  
> -	pm_runtime_set_active(pfdev->dev);
> -	pm_runtime_mark_last_busy(pfdev->dev);
> -	pm_runtime_enable(pfdev->dev);
> -	pm_runtime_set_autosuspend_delay(pfdev->dev, 50); /* ~3 frames */
> -	pm_runtime_use_autosuspend(pfdev->dev);
> +	pm_runtime_set_active(pfdev->base.dev);
> +	pm_runtime_mark_last_busy(pfdev->base.dev);
> +	pm_runtime_enable(pfdev->base.dev);
> +	pm_runtime_set_autosuspend_delay(pfdev->base.dev, 50); /* ~3 frames */
> +	pm_runtime_use_autosuspend(pfdev->base.dev);
>  
>  	/*
>  	 * Register the DRM device with the core and the connectors with
>  	 * sysfs
>  	 */
> -	err = drm_dev_register(ddev, 0);
> +	err = drm_dev_register(&pfdev->base, 0);
>  	if (err < 0)
>  		goto err_out1;
>  
> -	err = panfrost_gem_shrinker_init(ddev);
> +	err = panfrost_gem_shrinker_init(&pfdev->base);
>  	if (err)
>  		goto err_out2;
>  
>  	return 0;
>  
>  err_out2:
> -	drm_dev_unregister(ddev);
> +	drm_dev_unregister(&pfdev->base);
>  err_out1:
> -	pm_runtime_disable(pfdev->dev);
> +	pm_runtime_disable(pfdev->base.dev);
>  	panfrost_device_fini(pfdev);
> -	pm_runtime_set_suspended(pfdev->dev);
> +	pm_runtime_set_suspended(pfdev->base.dev);
>  err_out0:
> -	drm_dev_put(ddev);
>  	return err;
>  }
>  
>  static void panfrost_remove(struct platform_device *pdev)
>  {
>  	struct panfrost_device *pfdev = platform_get_drvdata(pdev);
> -	struct drm_device *ddev = pfdev->ddev;
>  
> -	drm_dev_unregister(ddev);
> -	panfrost_gem_shrinker_cleanup(ddev);
> +	drm_dev_unregister(&pfdev->base);
> +	panfrost_gem_shrinker_cleanup(&pfdev->base);
>  
> -	pm_runtime_get_sync(pfdev->dev);
> -	pm_runtime_disable(pfdev->dev);
> +	pm_runtime_get_sync(pfdev->base.dev);
> +	pm_runtime_disable(pfdev->base.dev);
>  	panfrost_device_fini(pfdev);
> -	pm_runtime_set_suspended(pfdev->dev);
> -
> -	drm_dev_put(ddev);
> +	pm_runtime_set_suspended(pfdev->base.dev);
>  }
>  
>  static ssize_t profiling_show(struct device *dev,
> diff --git a/drivers/gpu/drm/panfrost/panfrost_dump.c b/drivers/gpu/drm/panfrost/panfrost_dump.c
> index 47751302f1bc..ce704d077fa7 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_dump.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_dump.c
> @@ -163,7 +163,7 @@ void panfrost_core_dump(struct panfrost_job *job)
>  	iter.start = __vmalloc(file_size, GFP_KERNEL | __GFP_NOWARN |
>  			__GFP_NORETRY);
>  	if (!iter.start) {
> -		dev_warn(pfdev->dev, "failed to allocate devcoredump file\n");
> +		dev_warn(pfdev->base.dev, "failed to allocate devcoredump file\n");
>  		return;
>  	}
>  
> @@ -204,14 +204,14 @@ void panfrost_core_dump(struct panfrost_job *job)
>  		mapping = job->mappings[i];
>  
>  		if (!bo->base.sgt) {
> -			dev_err(pfdev->dev, "Panfrost Dump: BO has no sgt, cannot dump\n");
> +			dev_err(pfdev->base.dev, "Panfrost Dump: BO has no sgt, cannot dump\n");
>  			iter.hdr->bomap.valid = 0;
>  			goto dump_header;
>  		}
>  
>  		ret = drm_gem_vmap_unlocked(&bo->base.base, &map);
>  		if (ret) {
> -			dev_err(pfdev->dev, "Panfrost Dump: couldn't map Buffer Object\n");
> +			dev_err(pfdev->base.dev, "Panfrost Dump: couldn't map Buffer Object\n");
>  			iter.hdr->bomap.valid = 0;
>  			goto dump_header;
>  		}
> @@ -237,5 +237,5 @@ dump_header:	panfrost_core_dump_header(&iter, PANFROSTDUMP_BUF_BO, iter.data +
>  	}
>  	panfrost_core_dump_header(&iter, PANFROSTDUMP_BUF_TRAILER, iter.data);
>  
> -	dev_coredumpv(pfdev->dev, iter.start, iter.data - iter.start, GFP_KERNEL);
> +	dev_coredumpv(pfdev->base.dev, iter.start, iter.data - iter.start, GFP_KERNEL);
>  }
> diff --git a/drivers/gpu/drm/panfrost/panfrost_gem.c b/drivers/gpu/drm/panfrost/panfrost_gem.c
> index 8e0ff3efede7..768fed21c985 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_gem.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_gem.c
> @@ -17,7 +17,7 @@
>  static void panfrost_gem_free_object(struct drm_gem_object *obj)
>  {
>  	struct panfrost_gem_object *bo = to_panfrost_bo(obj);
> -	struct panfrost_device *pfdev = obj->dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(obj->dev);
>  
>  	/*
>  	 * Make sure the BO is no longer inserted in the shrinker list before
> @@ -41,7 +41,7 @@ static void panfrost_gem_free_object(struct drm_gem_object *obj)
>  
>  		for (i = 0; i < n_sgt; i++) {
>  			if (bo->sgts[i].sgl) {
> -				dma_unmap_sgtable(pfdev->dev, &bo->sgts[i],
> +				dma_unmap_sgtable(pfdev->base.dev, &bo->sgts[i],
>  						  DMA_BIDIRECTIONAL, 0);
>  				sg_free_table(&bo->sgts[i]);
>  			}
> @@ -249,7 +249,7 @@ static const struct drm_gem_object_funcs panfrost_gem_funcs = {
>   */
>  struct drm_gem_object *panfrost_gem_create_object(struct drm_device *dev, size_t size)
>  {
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  	struct panfrost_gem_object *obj;
>  
>  	obj = kzalloc(sizeof(*obj), GFP_KERNEL);
> diff --git a/drivers/gpu/drm/panfrost/panfrost_gem_shrinker.c b/drivers/gpu/drm/panfrost/panfrost_gem_shrinker.c
> index 3d9f51bd48b6..ee22777d06c8 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_gem_shrinker.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_gem_shrinker.c
> @@ -97,7 +97,7 @@ panfrost_gem_shrinker_scan(struct shrinker *shrinker, struct shrink_control *sc)
>   */
>  int panfrost_gem_shrinker_init(struct drm_device *dev)
>  {
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  
>  	pfdev->shrinker = shrinker_alloc(0, "drm-panfrost");
>  	if (!pfdev->shrinker)
> @@ -120,7 +120,7 @@ int panfrost_gem_shrinker_init(struct drm_device *dev)
>   */
>  void panfrost_gem_shrinker_cleanup(struct drm_device *dev)
>  {
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  
>  	if (pfdev->shrinker)
>  		shrinker_free(pfdev->shrinker);
> diff --git a/drivers/gpu/drm/panfrost/panfrost_gpu.c b/drivers/gpu/drm/panfrost/panfrost_gpu.c
> index f19f918e2330..5a33919fa213 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_gpu.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_gpu.c
> @@ -36,12 +36,12 @@ static irqreturn_t panfrost_gpu_irq_handler(int irq, void *data)
>  		u64 address = (u64) gpu_read(pfdev, GPU_FAULT_ADDRESS_HI) << 32;
>  		address |= gpu_read(pfdev, GPU_FAULT_ADDRESS_LO);
>  
> -		dev_warn(pfdev->dev, "GPU Fault 0x%08x (%s) at 0x%016llx\n",
> +		dev_warn(pfdev->base.dev, "GPU Fault 0x%08x (%s) at 0x%016llx\n",
>  			 fault_status, panfrost_exception_name(fault_status & 0xFF),
>  			 address);
>  
>  		if (state & GPU_IRQ_MULTIPLE_FAULT)
> -			dev_warn(pfdev->dev, "There were multiple GPU faults - some have not been reported\n");
> +			dev_warn(pfdev->base.dev, "There were multiple GPU faults - some have not been reported\n");
>  
>  		gpu_write(pfdev, GPU_INT_MASK, 0);
>  	}
> @@ -72,13 +72,13 @@ int panfrost_gpu_soft_reset(struct panfrost_device *pfdev)
>  		val, val & GPU_IRQ_RESET_COMPLETED, 10, 10000);
>  
>  	if (ret) {
> -		dev_err(pfdev->dev, "gpu soft reset timed out, attempting hard reset\n");
> +		dev_err(pfdev->base.dev, "gpu soft reset timed out, attempting hard reset\n");
>  
>  		gpu_write(pfdev, GPU_CMD, GPU_CMD_HARD_RESET);
>  		ret = readl_relaxed_poll_timeout(pfdev->iomem + GPU_INT_RAWSTAT, val,
>  						 val & GPU_IRQ_RESET_COMPLETED, 100, 10000);
>  		if (ret) {
> -			dev_err(pfdev->dev, "gpu hard reset timed out\n");
> +			dev_err(pfdev->base.dev, "gpu hard reset timed out\n");
>  			return ret;
>  		}
>  	}
> @@ -95,7 +95,7 @@ int panfrost_gpu_soft_reset(struct panfrost_device *pfdev)
>  	 * All in-flight jobs should have released their cycle
>  	 * counter references upon reset, but let us make sure
>  	 */
> -	if (drm_WARN_ON(pfdev->ddev, atomic_read(&pfdev->cycle_counter.use_count) != 0))
> +	if (drm_WARN_ON(&pfdev->base, atomic_read(&pfdev->cycle_counter.use_count) != 0))
>  		atomic_set(&pfdev->cycle_counter.use_count, 0);
>  
>  	return 0;
> @@ -327,13 +327,13 @@ static void panfrost_gpu_init_features(struct panfrost_device *pfdev)
>  	bitmap_from_u64(pfdev->features.hw_features, hw_feat);
>  	bitmap_from_u64(pfdev->features.hw_issues, hw_issues);
>  
> -	dev_info(pfdev->dev, "mali-%s id 0x%x major 0x%x minor 0x%x status 0x%x",
> +	dev_info(pfdev->base.dev, "mali-%s id 0x%x major 0x%x minor 0x%x status 0x%x",
>  		 name, gpu_id, major, minor, status);
> -	dev_info(pfdev->dev, "features: %64pb, issues: %64pb",
> +	dev_info(pfdev->base.dev, "features: %64pb, issues: %64pb",
>  		 pfdev->features.hw_features,
>  		 pfdev->features.hw_issues);
>  
> -	dev_info(pfdev->dev, "Features: L2:0x%08x Shader:0x%08x Tiler:0x%08x Mem:0x%0x MMU:0x%08x AS:0x%x JS:0x%x",
> +	dev_info(pfdev->base.dev, "Features: L2:0x%08x Shader:0x%08x Tiler:0x%08x Mem:0x%0x MMU:0x%08x AS:0x%x JS:0x%x",
>  		 pfdev->features.l2_features,
>  		 pfdev->features.core_features,
>  		 pfdev->features.tiler_features,
> @@ -342,7 +342,7 @@ static void panfrost_gpu_init_features(struct panfrost_device *pfdev)
>  		 pfdev->features.as_present,
>  		 pfdev->features.js_present);
>  
> -	dev_info(pfdev->dev, "shader_present=0x%0llx l2_present=0x%0llx",
> +	dev_info(pfdev->base.dev, "shader_present=0x%0llx l2_present=0x%0llx",
>  		 pfdev->features.shader_present, pfdev->features.l2_present);
>  }
>  
> @@ -408,7 +408,7 @@ static u64 panfrost_get_core_mask(struct panfrost_device *pfdev)
>  	 */
>  	core_mask = ~(pfdev->features.l2_present - 1) &
>  		     (pfdev->features.l2_present - 2);
> -	dev_info_once(pfdev->dev, "using only 1st core group (%lu cores from %lu)\n",
> +	dev_info_once(pfdev->base.dev, "using only 1st core group (%lu cores from %lu)\n",
>  		      hweight64(core_mask),
>  		      hweight64(pfdev->features.shader_present));
>  
> @@ -429,7 +429,7 @@ void panfrost_gpu_power_on(struct panfrost_device *pfdev)
>  		val, val == (pfdev->features.l2_present & core_mask),
>  		10, 20000);
>  	if (ret)
> -		dev_err(pfdev->dev, "error powering up gpu L2");
> +		dev_err(pfdev->base.dev, "error powering up gpu L2");
>  
>  	gpu_write(pfdev, SHADER_PWRON_LO,
>  		  pfdev->features.shader_present & core_mask);
> @@ -437,13 +437,13 @@ void panfrost_gpu_power_on(struct panfrost_device *pfdev)
>  		val, val == (pfdev->features.shader_present & core_mask),
>  		10, 20000);
>  	if (ret)
> -		dev_err(pfdev->dev, "error powering up gpu shader");
> +		dev_err(pfdev->base.dev, "error powering up gpu shader");
>  
>  	gpu_write(pfdev, TILER_PWRON_LO, pfdev->features.tiler_present);
>  	ret = readl_relaxed_poll_timeout(pfdev->iomem + TILER_READY_LO,
>  		val, val == pfdev->features.tiler_present, 10, 1000);
>  	if (ret)
> -		dev_err(pfdev->dev, "error powering up gpu tiler");
> +		dev_err(pfdev->base.dev, "error powering up gpu tiler");
>  }
>  
>  void panfrost_gpu_power_off(struct panfrost_device *pfdev)
> @@ -455,19 +455,19 @@ void panfrost_gpu_power_off(struct panfrost_device *pfdev)
>  	ret = readl_relaxed_poll_timeout(pfdev->iomem + SHADER_PWRTRANS_LO,
>  					 val, !val, 1, 2000);
>  	if (ret)
> -		dev_err(pfdev->dev, "shader power transition timeout");
> +		dev_err(pfdev->base.dev, "shader power transition timeout");
>  
>  	gpu_write(pfdev, TILER_PWROFF_LO, pfdev->features.tiler_present);
>  	ret = readl_relaxed_poll_timeout(pfdev->iomem + TILER_PWRTRANS_LO,
>  					 val, !val, 1, 2000);
>  	if (ret)
> -		dev_err(pfdev->dev, "tiler power transition timeout");
> +		dev_err(pfdev->base.dev, "tiler power transition timeout");
>  
>  	gpu_write(pfdev, L2_PWROFF_LO, pfdev->features.l2_present);
>  	ret = readl_poll_timeout(pfdev->iomem + L2_PWRTRANS_LO,
>  				 val, !val, 0, 2000);
>  	if (ret)
> -		dev_err(pfdev->dev, "l2 power transition timeout");
> +		dev_err(pfdev->base.dev, "l2 power transition timeout");
>  }
>  
>  void panfrost_gpu_suspend_irq(struct panfrost_device *pfdev)
> @@ -488,21 +488,22 @@ int panfrost_gpu_init(struct panfrost_device *pfdev)
>  
>  	panfrost_gpu_init_features(pfdev);
>  
> -	err = dma_set_mask_and_coherent(pfdev->dev,
> -		DMA_BIT_MASK(FIELD_GET(0xff00, pfdev->features.mmu_features)));
> +	err = dma_set_mask_and_coherent(pfdev->base.dev,
> +					DMA_BIT_MASK(FIELD_GET(0xff00,
> +							       pfdev->features.mmu_features)));
>  	if (err)
>  		return err;
>  
> -	dma_set_max_seg_size(pfdev->dev, UINT_MAX);
> +	dma_set_max_seg_size(pfdev->base.dev, UINT_MAX);
>  
> -	pfdev->gpu_irq = platform_get_irq_byname(to_platform_device(pfdev->dev), "gpu");
> +	pfdev->gpu_irq = platform_get_irq_byname(to_platform_device(pfdev->base.dev), "gpu");
>  	if (pfdev->gpu_irq < 0)
>  		return pfdev->gpu_irq;
>  
> -	err = devm_request_irq(pfdev->dev, pfdev->gpu_irq, panfrost_gpu_irq_handler,
> +	err = devm_request_irq(pfdev->base.dev, pfdev->gpu_irq, panfrost_gpu_irq_handler,
>  			       IRQF_SHARED, KBUILD_MODNAME "-gpu", pfdev);
>  	if (err) {
> -		dev_err(pfdev->dev, "failed to request gpu irq");
> +		dev_err(pfdev->base.dev, "failed to request gpu irq");
>  		return err;
>  	}
>  
> @@ -522,9 +523,9 @@ u32 panfrost_gpu_get_latest_flush_id(struct panfrost_device *pfdev)
>  
>  	if (panfrost_has_hw_feature(pfdev, HW_FEATURE_FLUSH_REDUCTION)) {
>  		/* Flush reduction only makes sense when the GPU is kept powered on between jobs */
> -		if (pm_runtime_get_if_in_use(pfdev->dev)) {
> +		if (pm_runtime_get_if_in_use(pfdev->base.dev)) {
>  			flush_id = gpu_read(pfdev, GPU_LATEST_FLUSH_ID);
> -			pm_runtime_put(pfdev->dev);
> +			pm_runtime_put(pfdev->base.dev);
>  			return flush_id;
>  		}
>  	}
> diff --git a/drivers/gpu/drm/panfrost/panfrost_job.c b/drivers/gpu/drm/panfrost/panfrost_job.c
> index 9b8e82fb8bc4..f640d211cc3a 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_job.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_job.c
> @@ -94,7 +94,7 @@ static struct dma_fence *panfrost_fence_create(struct panfrost_device *pfdev, in
>  	if (!fence)
>  		return ERR_PTR(-ENOMEM);
>  
> -	fence->dev = pfdev->ddev;
> +	fence->dev = &pfdev->base;
>  	fence->queue = js_num;
>  	fence->seqno = ++js->queue[js_num].emit_seqno;
>  	dma_fence_init(&fence->base, &panfrost_fence_ops, &js->job_lock,
> @@ -205,7 +205,7 @@ static void panfrost_job_hw_submit(struct panfrost_job *job, int js)
>  
>  	panfrost_devfreq_record_busy(&pfdev->pfdevfreq);
>  
> -	ret = pm_runtime_get_sync(pfdev->dev);
> +	ret = pm_runtime_get_sync(pfdev->base.dev);
>  	if (ret < 0)
>  		return;
>  
> @@ -256,7 +256,7 @@ static void panfrost_job_hw_submit(struct panfrost_job *job, int js)
>  		}
>  
>  		job_write(pfdev, JS_COMMAND_NEXT(js), JS_COMMAND_START);
> -		dev_dbg(pfdev->dev,
> +		dev_dbg(pfdev->base.dev,
>  			"JS: Submitting atom %p to js[%d][%d] with head=0x%llx AS %d",
>  			job, js, subslot, jc_head, cfg & 0xf);
>  	}
> @@ -437,12 +437,12 @@ static void panfrost_job_handle_err(struct panfrost_device *pfdev,
>  	bool signal_fence = true;
>  
>  	if (!panfrost_exception_is_fault(js_status)) {
> -		dev_dbg(pfdev->dev, "js event, js=%d, status=%s, head=0x%x, tail=0x%x",
> +		dev_dbg(pfdev->base.dev, "js event, js=%d, status=%s, head=0x%x, tail=0x%x",
>  			js, exception_name,
>  			job_read(pfdev, JS_HEAD_LO(js)),
>  			job_read(pfdev, JS_TAIL_LO(js)));
>  	} else {
> -		dev_err(pfdev->dev, "js fault, js=%d, status=%s, head=0x%x, tail=0x%x",
> +		dev_err(pfdev->base.dev, "js fault, js=%d, status=%s, head=0x%x, tail=0x%x",
>  			js, exception_name,
>  			job_read(pfdev, JS_HEAD_LO(js)),
>  			job_read(pfdev, JS_TAIL_LO(js)));
> @@ -474,7 +474,7 @@ static void panfrost_job_handle_err(struct panfrost_device *pfdev,
>  	if (signal_fence)
>  		dma_fence_signal_locked(job->done_fence);
>  
> -	pm_runtime_put_autosuspend(pfdev->dev);
> +	pm_runtime_put_autosuspend(pfdev->base.dev);
>  
>  	if (panfrost_exception_needs_reset(pfdev, js_status)) {
>  		atomic_set(&pfdev->reset.pending, 1);
> @@ -493,7 +493,7 @@ static void panfrost_job_handle_done(struct panfrost_device *pfdev,
>  	panfrost_devfreq_record_idle(&pfdev->pfdevfreq);
>  
>  	dma_fence_signal_locked(job->done_fence);
> -	pm_runtime_put_autosuspend(pfdev->dev);
> +	pm_runtime_put_autosuspend(pfdev->base.dev);
>  }
>  
>  static void panfrost_job_handle_irq(struct panfrost_device *pfdev, u32 status)
> @@ -602,7 +602,7 @@ static void panfrost_job_handle_irqs(struct panfrost_device *pfdev)
>  	u32 status = job_read(pfdev, JOB_INT_RAWSTAT);
>  
>  	while (status) {
> -		pm_runtime_mark_last_busy(pfdev->dev);
> +		pm_runtime_mark_last_busy(pfdev->base.dev);
>  
>  		spin_lock(&pfdev->js->job_lock);
>  		panfrost_job_handle_irq(pfdev, status);
> @@ -683,7 +683,7 @@ panfrost_reset(struct panfrost_device *pfdev,
>  				 10, 10000);
>  
>  	if (ret)
> -		dev_err(pfdev->dev, "Soft-stop failed\n");
> +		dev_err(pfdev->base.dev, "Soft-stop failed\n");
>  
>  	/* Handle the remaining interrupts before we reset. */
>  	panfrost_job_handle_irqs(pfdev);
> @@ -701,7 +701,7 @@ panfrost_reset(struct panfrost_device *pfdev,
>  			if (pfdev->jobs[i][j]->requirements & PANFROST_JD_REQ_CYCLE_COUNT ||
>  			    pfdev->jobs[i][j]->is_profiled)
>  				panfrost_cycle_counter_put(pfdev->jobs[i][j]->pfdev);
> -			pm_runtime_put_noidle(pfdev->dev);
> +			pm_runtime_put_noidle(pfdev->base.dev);
>  			panfrost_devfreq_record_idle(&pfdev->pfdevfreq);
>  		}
>  	}
> @@ -769,11 +769,11 @@ static enum drm_gpu_sched_stat panfrost_job_timedout(struct drm_sched_job
>  	synchronize_irq(pfdev->js->irq);
>  
>  	if (dma_fence_is_signaled(job->done_fence)) {
> -		dev_warn(pfdev->dev, "unexpectedly high interrupt latency\n");
> +		dev_warn(pfdev->base.dev, "unexpectedly high interrupt latency\n");
>  		return DRM_GPU_SCHED_STAT_NOMINAL;
>  	}
>  
> -	dev_err(pfdev->dev, "gpu sched timeout, js=%d, config=0x%x, status=0x%x, head=0x%x, tail=0x%x, sched_job=%p",
> +	dev_err(pfdev->base.dev, "gpu sched timeout, js=%d, config=0x%x, status=0x%x, head=0x%x, tail=0x%x, sched_job=%p",
>  		js,
>  		job_read(pfdev, JS_CONFIG(js)),
>  		job_read(pfdev, JS_STATUS(js)),
> @@ -847,24 +847,25 @@ int panfrost_job_init(struct panfrost_device *pfdev)
>  	if (!panfrost_has_hw_feature(pfdev, HW_FEATURE_JOBCHAIN_DISAMBIGUATION))
>  		nentries = 1;
>  
> -	pfdev->js = js = devm_kzalloc(pfdev->dev, sizeof(*js), GFP_KERNEL);
> +	js = devm_kzalloc(pfdev->base.dev, sizeof(*js), GFP_KERNEL);
>  	if (!js)
>  		return -ENOMEM;
> +	pfdev->js = js;
>  
>  	INIT_WORK(&pfdev->reset.work, panfrost_reset_work);
>  	spin_lock_init(&js->job_lock);
>  
> -	js->irq = platform_get_irq_byname(to_platform_device(pfdev->dev), "job");
> +	js->irq = platform_get_irq_byname(to_platform_device(pfdev->base.dev), "job");
>  	if (js->irq < 0)
>  		return js->irq;
>  
> -	ret = devm_request_threaded_irq(pfdev->dev, js->irq,
> +	ret = devm_request_threaded_irq(pfdev->base.dev, js->irq,
>  					panfrost_job_irq_handler,
>  					panfrost_job_irq_handler_thread,
>  					IRQF_SHARED, KBUILD_MODNAME "-job",
>  					pfdev);
>  	if (ret) {
> -		dev_err(pfdev->dev, "failed to request job irq");
> +		dev_err(pfdev->base.dev, "failed to request job irq");
>  		return ret;
>  	}
>  
> @@ -881,9 +882,9 @@ int panfrost_job_init(struct panfrost_device *pfdev)
>  				     nentries, 0,
>  				     msecs_to_jiffies(JOB_TIMEOUT_MS),
>  				     pfdev->reset.wq,
> -				     NULL, "pan_js", pfdev->dev);
> +				     NULL, "pan_js", pfdev->base.dev);
>  		if (ret) {
> -			dev_err(pfdev->dev, "Failed to create scheduler: %d.", ret);
> +			dev_err(pfdev->base.dev, "Failed to create scheduler: %d.", ret);
>  			goto err_sched;
>  		}
>  	}
> diff --git a/drivers/gpu/drm/panfrost/panfrost_mmu.c b/drivers/gpu/drm/panfrost/panfrost_mmu.c
> index b91019cd5acb..2189e42d2bfa 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_mmu.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_mmu.c
> @@ -39,7 +39,7 @@ static int wait_ready(struct panfrost_device *pfdev, u32 as_nr)
>  	if (ret) {
>  		/* The GPU hung, let's trigger a reset */
>  		panfrost_device_schedule_reset(pfdev);
> -		dev_err(pfdev->dev, "AS_ACTIVE bit stuck\n");
> +		dev_err(pfdev->base.dev, "AS_ACTIVE bit stuck\n");
>  	}
>  
>  	return ret;
> @@ -212,7 +212,9 @@ u32 panfrost_mmu_as_get(struct panfrost_device *pfdev, struct panfrost_mmu *mmu)
>  	atomic_set(&mmu->as_count, 1);
>  	list_add(&mmu->list, &pfdev->as_lru_list);
>  
> -	dev_dbg(pfdev->dev, "Assigned AS%d to mmu %p, alloc_mask=%lx", as, mmu, pfdev->as_alloc_mask);
> +	dev_dbg(pfdev->base.dev,
> +		"Assigned AS%d to mmu %p, alloc_mask=%lx",
> +		as, mmu, pfdev->as_alloc_mask);
>  
>  	panfrost_mmu_enable(pfdev, mmu);
>  
> @@ -278,13 +280,13 @@ static void panfrost_mmu_flush_range(struct panfrost_device *pfdev,
>  	if (mmu->as < 0)
>  		return;
>  
> -	pm_runtime_get_noresume(pfdev->dev);
> +	pm_runtime_get_noresume(pfdev->base.dev);
>  
>  	/* Flush the PTs only if we're already awake */
> -	if (pm_runtime_active(pfdev->dev))
> +	if (pm_runtime_active(pfdev->base.dev))
>  		mmu_hw_do_operation(pfdev, mmu, iova, size, AS_COMMAND_FLUSH_PT);
>  
> -	pm_runtime_put_autosuspend(pfdev->dev);
> +	pm_runtime_put_autosuspend(pfdev->base.dev);
>  }
>  
>  static int mmu_map_sg(struct panfrost_device *pfdev, struct panfrost_mmu *mmu,
> @@ -299,7 +301,9 @@ static int mmu_map_sg(struct panfrost_device *pfdev, struct panfrost_mmu *mmu,
>  		unsigned long paddr = sg_dma_address(sgl);
>  		size_t len = sg_dma_len(sgl);
>  
> -		dev_dbg(pfdev->dev, "map: as=%d, iova=%llx, paddr=%lx, len=%zx", mmu->as, iova, paddr, len);
> +		dev_dbg(pfdev->base.dev,
> +			"map: as=%d, iova=%llx, paddr=%lx, len=%zx",
> +			mmu->as, iova, paddr, len);
>  
>  		while (len) {
>  			size_t pgcount, mapped = 0;
> @@ -359,7 +363,7 @@ void panfrost_mmu_unmap(struct panfrost_gem_mapping *mapping)
>  	if (WARN_ON(!mapping->active))
>  		return;
>  
> -	dev_dbg(pfdev->dev, "unmap: as=%d, iova=%llx, len=%zx",
> +	dev_dbg(pfdev->base.dev, "unmap: as=%d, iova=%llx, len=%zx",
>  		mapping->mmu->as, iova, len);
>  
>  	while (unmapped_len < len) {
> @@ -456,7 +460,7 @@ static int panfrost_mmu_map_fault_addr(struct panfrost_device *pfdev, int as,
>  
>  	bo = bomapping->obj;
>  	if (!bo->is_heap) {
> -		dev_WARN(pfdev->dev, "matching BO is not heap type (GPU VA = %llx)",
> +		dev_WARN(pfdev->base.dev, "matching BO is not heap type (GPU VA = %llx)",
>  			 bomapping->mmnode.start << PAGE_SHIFT);
>  		ret = -EINVAL;
>  		goto err_bo;
> @@ -523,7 +527,7 @@ static int panfrost_mmu_map_fault_addr(struct panfrost_device *pfdev, int as,
>  	if (ret)
>  		goto err_unlock;
>  
> -	ret = dma_map_sgtable(pfdev->dev, sgt, DMA_BIDIRECTIONAL, 0);
> +	ret = dma_map_sgtable(pfdev->base.dev, sgt, DMA_BIDIRECTIONAL, 0);
>  	if (ret)
>  		goto err_map;
>  
> @@ -533,7 +537,7 @@ static int panfrost_mmu_map_fault_addr(struct panfrost_device *pfdev, int as,
>  	bomapping->active = true;
>  	bo->heap_rss_size += SZ_2M;
>  
> -	dev_dbg(pfdev->dev, "mapped page fault @ AS%d %llx", as, addr);
> +	dev_dbg(pfdev->base.dev, "mapped page fault @ AS%d %llx", as, addr);
>  
>  out:
>  	dma_resv_unlock(obj->resv);
> @@ -559,10 +563,10 @@ static void panfrost_mmu_release_ctx(struct kref *kref)
>  
>  	spin_lock(&pfdev->as_lock);
>  	if (mmu->as >= 0) {
> -		pm_runtime_get_noresume(pfdev->dev);
> -		if (pm_runtime_active(pfdev->dev))
> +		pm_runtime_get_noresume(pfdev->base.dev);
> +		if (pm_runtime_active(pfdev->base.dev))
>  			panfrost_mmu_disable(pfdev, mmu->as);
> -		pm_runtime_put_autosuspend(pfdev->dev);
> +		pm_runtime_put_autosuspend(pfdev->base.dev);
>  
>  		clear_bit(mmu->as, &pfdev->as_alloc_mask);
>  		clear_bit(mmu->as, &pfdev->as_in_use_mask);
> @@ -637,7 +641,7 @@ struct panfrost_mmu *panfrost_mmu_ctx_create(struct panfrost_device *pfdev)
>  		.oas		= FIELD_GET(0xff00, pfdev->features.mmu_features),
>  		.coherent_walk	= pfdev->coherent,
>  		.tlb		= &mmu_tlb_ops,
> -		.iommu_dev	= pfdev->dev,
> +		.iommu_dev	= pfdev->base.dev,
>  	};
>  
>  	mmu->pgtbl_ops = alloc_io_pgtable_ops(ARM_MALI_LPAE, &mmu->pgtbl_cfg,
> @@ -720,7 +724,7 @@ static irqreturn_t panfrost_mmu_irq_handler_thread(int irq, void *data)
>  
>  		if (ret) {
>  			/* terminal fault, print info about the fault */
> -			dev_err(pfdev->dev,
> +			dev_err(pfdev->base.dev,
>  				"Unhandled Page fault in AS%d at VA 0x%016llX\n"
>  				"Reason: %s\n"
>  				"raw fault status: 0x%X\n"
> @@ -768,18 +772,18 @@ int panfrost_mmu_init(struct panfrost_device *pfdev)
>  {
>  	int err;
>  
> -	pfdev->mmu_irq = platform_get_irq_byname(to_platform_device(pfdev->dev), "mmu");
> +	pfdev->mmu_irq = platform_get_irq_byname(to_platform_device(pfdev->base.dev), "mmu");
>  	if (pfdev->mmu_irq < 0)
>  		return pfdev->mmu_irq;
>  
> -	err = devm_request_threaded_irq(pfdev->dev, pfdev->mmu_irq,
> +	err = devm_request_threaded_irq(pfdev->base.dev, pfdev->mmu_irq,
>  					panfrost_mmu_irq_handler,
>  					panfrost_mmu_irq_handler_thread,
>  					IRQF_SHARED, KBUILD_MODNAME "-mmu",
>  					pfdev);
>  
>  	if (err) {
> -		dev_err(pfdev->dev, "failed to request mmu irq");
> +		dev_err(pfdev->base.dev, "failed to request mmu irq");
>  		return err;
>  	}
>  
> diff --git a/drivers/gpu/drm/panfrost/panfrost_perfcnt.c b/drivers/gpu/drm/panfrost/panfrost_perfcnt.c
> index ba9b6e2b2636..f30817bcf8ba 100644
> --- a/drivers/gpu/drm/panfrost/panfrost_perfcnt.c
> +++ b/drivers/gpu/drm/panfrost/panfrost_perfcnt.c
> @@ -84,11 +84,11 @@ static int panfrost_perfcnt_enable_locked(struct panfrost_device *pfdev,
>  	else if (perfcnt->user)
>  		return -EBUSY;
>  
> -	ret = pm_runtime_get_sync(pfdev->dev);
> +	ret = pm_runtime_get_sync(pfdev->base.dev);
>  	if (ret < 0)
>  		goto err_put_pm;
>  
> -	bo = drm_gem_shmem_create(pfdev->ddev, perfcnt->bosize);
> +	bo = drm_gem_shmem_create(&pfdev->base, perfcnt->bosize);
>  	if (IS_ERR(bo)) {
>  		ret = PTR_ERR(bo);
>  		goto err_put_pm;
> @@ -173,7 +173,7 @@ static int panfrost_perfcnt_enable_locked(struct panfrost_device *pfdev,
>  err_put_bo:
>  	drm_gem_object_put(&bo->base);
>  err_put_pm:
> -	pm_runtime_put(pfdev->dev);
> +	pm_runtime_put(pfdev->base.dev);
>  	return ret;
>  }
>  
> @@ -201,8 +201,8 @@ static int panfrost_perfcnt_disable_locked(struct panfrost_device *pfdev,
>  	panfrost_mmu_as_put(pfdev, perfcnt->mapping->mmu);
>  	panfrost_gem_mapping_put(perfcnt->mapping);
>  	perfcnt->mapping = NULL;
> -	pm_runtime_mark_last_busy(pfdev->dev);
> -	pm_runtime_put_autosuspend(pfdev->dev);
> +	pm_runtime_mark_last_busy(pfdev->base.dev);
> +	pm_runtime_put_autosuspend(pfdev->base.dev);
>  
>  	return 0;
>  }
> @@ -210,7 +210,7 @@ static int panfrost_perfcnt_disable_locked(struct panfrost_device *pfdev,
>  int panfrost_ioctl_perfcnt_enable(struct drm_device *dev, void *data,
>  				  struct drm_file *file_priv)
>  {
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  	struct panfrost_perfcnt *perfcnt = pfdev->perfcnt;
>  	struct drm_panfrost_perfcnt_enable *req = data;
>  	int ret;
> @@ -237,7 +237,7 @@ int panfrost_ioctl_perfcnt_enable(struct drm_device *dev, void *data,
>  int panfrost_ioctl_perfcnt_dump(struct drm_device *dev, void *data,
>  				struct drm_file *file_priv)
>  {
> -	struct panfrost_device *pfdev = dev->dev_private;
> +	struct panfrost_device *pfdev = to_panfrost_device(dev);
>  	struct panfrost_perfcnt *perfcnt = pfdev->perfcnt;
>  	struct drm_panfrost_perfcnt_dump *req = data;
>  	void __user *user_ptr = (void __user *)(uintptr_t)req->buf_ptr;
> @@ -272,13 +272,13 @@ void panfrost_perfcnt_close(struct drm_file *file_priv)
>  	struct panfrost_device *pfdev = pfile->pfdev;
>  	struct panfrost_perfcnt *perfcnt = pfdev->perfcnt;
>  
> -	pm_runtime_get_sync(pfdev->dev);
> +	pm_runtime_get_sync(pfdev->base.dev);
>  	mutex_lock(&perfcnt->lock);
>  	if (perfcnt->user == pfile)
>  		panfrost_perfcnt_disable_locked(pfdev, file_priv);
>  	mutex_unlock(&perfcnt->lock);
> -	pm_runtime_mark_last_busy(pfdev->dev);
> -	pm_runtime_put_autosuspend(pfdev->dev);
> +	pm_runtime_mark_last_busy(pfdev->base.dev);
> +	pm_runtime_put_autosuspend(pfdev->base.dev);
>  }
>  
>  int panfrost_perfcnt_init(struct panfrost_device *pfdev)
> @@ -316,7 +316,7 @@ int panfrost_perfcnt_init(struct panfrost_device *pfdev)
>  		       COUNTERS_PER_BLOCK * BYTES_PER_COUNTER;
>  	}
>  
> -	perfcnt = devm_kzalloc(pfdev->dev, sizeof(*perfcnt), GFP_KERNEL);
> +	perfcnt = devm_kzalloc(pfdev->base.dev, sizeof(*perfcnt), GFP_KERNEL);
>  	if (!perfcnt)
>  		return -ENOMEM;
>  


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ