[PATCH 39/48] drm: omapdrm: dsi: Store the struct device pointer in struct dsi_data

Sebastian Reichel sre at kernel.org
Tue Oct 17 20:11:17 UTC 2017


Hi,

On Fri, Oct 13, 2017 at 05:59:35PM +0300, Laurent Pinchart wrote:
> The dsi_data structure stores a pointer to a struct platform_device. The
> driver only uses the dev member of the platform device structure. Store
> the struct device pointer instead and use it directly.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---

Reviewed-by: Sebastian Reichel <sebastian.reichel at collabora.co.uk>

-- Sebastian

>  drivers/gpu/drm/omapdrm/dss/dsi.c | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/drivers/gpu/drm/omapdrm/dss/dsi.c b/drivers/gpu/drm/omapdrm/dss/dsi.c
> index 4f67dd70f279..9be26eb262f3 100644
> --- a/drivers/gpu/drm/omapdrm/dss/dsi.c
> +++ b/drivers/gpu/drm/omapdrm/dss/dsi.c
> @@ -332,7 +332,7 @@ struct dsi_of_data {
>  };
>  
>  struct dsi_data {
> -	struct platform_device *pdev;
> +	struct device *dev;
>  	void __iomem *proto_base;
>  	void __iomem *phy_base;
>  	void __iomem *pll_base;
> @@ -1146,7 +1146,7 @@ static int dsi_runtime_get(struct dsi_data *dsi)
>  
>  	DSSDBG("dsi_runtime_get\n");
>  
> -	r = pm_runtime_get_sync(&dsi->pdev->dev);
> +	r = pm_runtime_get_sync(dsi->dev);
>  	WARN_ON(r < 0);
>  	return r < 0 ? r : 0;
>  }
> @@ -1157,7 +1157,7 @@ static void dsi_runtime_put(struct dsi_data *dsi)
>  
>  	DSSDBG("dsi_runtime_put\n");
>  
> -	r = pm_runtime_put_sync(&dsi->pdev->dev);
> +	r = pm_runtime_put_sync(dsi->dev);
>  	WARN_ON(r < 0 && r != -ENOSYS);
>  }
>  
> @@ -1168,7 +1168,7 @@ static int dsi_regulator_init(struct dsi_data *dsi)
>  	if (dsi->vdds_dsi_reg != NULL)
>  		return 0;
>  
> -	vdds_dsi = devm_regulator_get(&dsi->pdev->dev, "vdd");
> +	vdds_dsi = devm_regulator_get(dsi->dev, "vdd");
>  
>  	if (IS_ERR(vdds_dsi)) {
>  		if (PTR_ERR(vdds_dsi) != -EPROBE_DEFER)
> @@ -4953,7 +4953,7 @@ static int dsi_get_clocks(struct dsi_data *dsi)
>  {
>  	struct clk *clk;
>  
> -	clk = devm_clk_get(&dsi->pdev->dev, "fck");
> +	clk = devm_clk_get(dsi->dev, "fck");
>  	if (IS_ERR(clk)) {
>  		DSSERR("can't get fck\n");
>  		return PTR_ERR(clk);
> @@ -5048,7 +5048,7 @@ static void dsi_init_output(struct dsi_data *dsi)
>  {
>  	struct omap_dss_device *out = &dsi->output;
>  
> -	out->dev = &dsi->pdev->dev;
> +	out->dev = dsi->dev;
>  	out->id = dsi->module_id == 0 ?
>  			OMAP_DSS_OUTPUT_DSI1 : OMAP_DSS_OUTPUT_DSI2;
>  
> @@ -5070,7 +5070,7 @@ static void dsi_uninit_output(struct dsi_data *dsi)
>  
>  static int dsi_probe_of(struct dsi_data *dsi)
>  {
> -	struct device_node *node = dsi->pdev->dev.of_node;
> +	struct device_node *node = dsi->dev->of_node;
>  	struct property *prop;
>  	u32 lane_arr[10];
>  	int len, num_pins;
> @@ -5084,7 +5084,7 @@ static int dsi_probe_of(struct dsi_data *dsi)
>  
>  	prop = of_find_property(ep, "lanes", &len);
>  	if (prop == NULL) {
> -		dev_err(&dsi->pdev->dev, "failed to find lane data\n");
> +		dev_err(dsi->dev, "failed to find lane data\n");
>  		r = -EINVAL;
>  		goto err;
>  	}
> @@ -5093,14 +5093,14 @@ static int dsi_probe_of(struct dsi_data *dsi)
>  
>  	if (num_pins < 4 || num_pins % 2 != 0 ||
>  		num_pins > dsi->num_lanes_supported * 2) {
> -		dev_err(&dsi->pdev->dev, "bad number of lanes\n");
> +		dev_err(dsi->dev, "bad number of lanes\n");
>  		r = -EINVAL;
>  		goto err;
>  	}
>  
>  	r = of_property_read_u32_array(ep, "lanes", lane_arr, num_pins);
>  	if (r) {
> -		dev_err(&dsi->pdev->dev, "failed to read lane data\n");
> +		dev_err(dsi->dev, "failed to read lane data\n");
>  		goto err;
>  	}
>  
> @@ -5110,7 +5110,7 @@ static int dsi_probe_of(struct dsi_data *dsi)
>  
>  	r = dsi_configure_pins(&dsi->output, &pin_cfg);
>  	if (r) {
> -		dev_err(&dsi->pdev->dev, "failed to configure pins");
> +		dev_err(dsi->dev, "failed to configure pins");
>  		goto err;
>  	}
>  
> @@ -5216,7 +5216,7 @@ static int dsi_init_pll_data(struct dss_device *dss, struct dsi_data *dsi)
>  	struct clk *clk;
>  	int r;
>  
> -	clk = devm_clk_get(&dsi->pdev->dev, "sys_clk");
> +	clk = devm_clk_get(dsi->dev, "sys_clk");
>  	if (IS_ERR(clk)) {
>  		DSSERR("can't get sys_clk\n");
>  		return PTR_ERR(clk);
> @@ -5320,7 +5320,7 @@ static int dsi_bind(struct device *dev, struct device *master, void *data)
>  		return -ENOMEM;
>  
>  	dsi->dss = dss;
> -	dsi->pdev = pdev;
> +	dsi->dev = dev;
>  	dev_set_drvdata(dev, dsi);
>  
>  	spin_lock_init(&dsi->irq_lock);
> -- 
> Regards,
> 
> Laurent Pinchart
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20171017/ca4d2bcf/attachment-0001.sig>


More information about the dri-devel mailing list