Re: [PATCH] drm/hisilicon: Using the to_hibmc_drm_private to convert

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

Am 28.09.20 um 10:32 schrieb Tian Tao:
> Using the to_hibmc_drm_private to convert over all uses of dev_private
> over to the function, and fix a little formatting issue.

Sounds strange to me. Maybe remove "over to the function" and it should
be better.

> 
> Signed-off-by: Tian Tao <tiantao6@xxxxxxxxxxxxx>

Reviewed-by: Thomas Zimmermann <tzimmermann@xxxxxxx>

> ---
>  drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c   | 22 +++++++++++-----------
>  drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c  |  5 ++---
>  drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c |  4 ++--
>  3 files changed, 15 insertions(+), 16 deletions(-)
> 
> diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c
> index 4d57ec6..a98f993 100644
> --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c
> +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c
> @@ -105,7 +105,7 @@ static void hibmc_plane_atomic_update(struct drm_plane *plane,
>  	u32 reg;
>  	s64 gpu_addr = 0;
>  	unsigned int line_l;
> -	struct hibmc_drm_private *priv = plane->dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(plane->dev);
>  	struct drm_gem_vram_object *gbo;
>  
>  	if (!state->fb)
> @@ -159,7 +159,7 @@ static const struct drm_plane_helper_funcs hibmc_plane_helper_funcs = {
>  
>  static void hibmc_crtc_dpms(struct drm_crtc *crtc, int dpms)
>  {
> -	struct hibmc_drm_private *priv = crtc->dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev);
>  	unsigned int reg;
>  
>  	reg = readl(priv->mmio + HIBMC_CRT_DISP_CTL);
> @@ -175,7 +175,7 @@ static void hibmc_crtc_atomic_enable(struct drm_crtc *crtc,
>  				     struct drm_crtc_state *old_state)
>  {
>  	unsigned int reg;
> -	struct hibmc_drm_private *priv = crtc->dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev);
>  
>  	hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0);
>  
> @@ -194,7 +194,7 @@ static void hibmc_crtc_atomic_disable(struct drm_crtc *crtc,
>  				      struct drm_crtc_state *old_state)
>  {
>  	unsigned int reg;
> -	struct hibmc_drm_private *priv = crtc->dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev);
>  
>  	hibmc_crtc_dpms(crtc, HIBMC_CRT_DPMS_OFF);
>  	drm_crtc_vblank_off(crtc);
> @@ -254,7 +254,7 @@ static unsigned int format_pll_reg(void)
>  static void set_vclock_hisilicon(struct drm_device *dev, unsigned long pll)
>  {
>  	u32 val;
> -	struct hibmc_drm_private *priv = dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(dev);
>  
>  	val = readl(priv->mmio + CRT_PLL1_HS);
>  	val &= ~(CRT_PLL1_HS_OUTER_BYPASS(1));
> @@ -315,7 +315,7 @@ static unsigned int display_ctrl_adjust(struct drm_device *dev,
>  	unsigned long x, y;
>  	u32 pll1; /* bit[31:0] of PLL */
>  	u32 pll2; /* bit[63:32] of PLL */
> -	struct hibmc_drm_private *priv = dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(dev);
>  
>  	x = mode->hdisplay;
>  	y = mode->vdisplay;
> @@ -363,7 +363,7 @@ static void hibmc_crtc_mode_set_nofb(struct drm_crtc *crtc)
>  	unsigned int val;
>  	struct drm_display_mode *mode = &crtc->state->mode;
>  	struct drm_device *dev = crtc->dev;
> -	struct hibmc_drm_private *priv = dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(dev);
>  	int width = mode->hsync_end - mode->hsync_start;
>  	int height = mode->vsync_end - mode->vsync_start;
>  
> @@ -397,7 +397,7 @@ static void hibmc_crtc_atomic_begin(struct drm_crtc *crtc,
>  {
>  	unsigned int reg;
>  	struct drm_device *dev = crtc->dev;
> -	struct hibmc_drm_private *priv = dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(dev);
>  
>  	hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0);
>  
> @@ -427,7 +427,7 @@ static void hibmc_crtc_atomic_flush(struct drm_crtc *crtc,
>  
>  static int hibmc_crtc_enable_vblank(struct drm_crtc *crtc)
>  {
> -	struct hibmc_drm_private *priv = crtc->dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev);
>  
>  	writel(HIBMC_RAW_INTERRUPT_EN_VBLANK(1),
>  	       priv->mmio + HIBMC_RAW_INTERRUPT_EN);
> @@ -437,7 +437,7 @@ static int hibmc_crtc_enable_vblank(struct drm_crtc *crtc)
>  
>  static void hibmc_crtc_disable_vblank(struct drm_crtc *crtc)
>  {
> -	struct hibmc_drm_private *priv = crtc->dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev);
>  
>  	writel(HIBMC_RAW_INTERRUPT_EN_VBLANK(0),
>  	       priv->mmio + HIBMC_RAW_INTERRUPT_EN);
> @@ -445,7 +445,7 @@ static void hibmc_crtc_disable_vblank(struct drm_crtc *crtc)
>  
>  static void hibmc_crtc_load_lut(struct drm_crtc *crtc)
>  {
> -	struct hibmc_drm_private *priv = crtc->dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev);
>  	void __iomem   *mmio = priv->mmio;
>  	u16 *r, *g, *b;
>  	unsigned int reg;
> diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c
> index 085d1b2..5632bce 100644
> --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c
> +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c
> @@ -29,8 +29,7 @@ DEFINE_DRM_GEM_FOPS(hibmc_fops);
>  static irqreturn_t hibmc_drm_interrupt(int irq, void *arg)
>  {
>  	struct drm_device *dev = (struct drm_device *)arg;
> -	struct hibmc_drm_private *priv =
> -		(struct hibmc_drm_private *)dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(dev);
>  	u32 status;
>  
>  	status = readl(priv->mmio + HIBMC_RAW_INTERRUPT);
> @@ -244,7 +243,7 @@ static int hibmc_hw_init(struct hibmc_drm_private *priv)
>  
>  static int hibmc_unload(struct drm_device *dev)
>  {
> -	struct hibmc_drm_private *priv = dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(dev);
>  
>  	drm_atomic_helper_shutdown(dev);
>  
> diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
> index c6999ed..74e26c2 100644
> --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
> +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
> @@ -43,7 +43,7 @@ static int hibmc_connector_get_modes(struct drm_connector *connector)
>  }
>  
>  static enum drm_mode_status hibmc_connector_mode_valid(struct drm_connector *connector,
> -				      struct drm_display_mode *mode)
> +						       struct drm_display_mode *mode)
>  {
>  	return MODE_OK;
>  }
> @@ -76,7 +76,7 @@ static void hibmc_encoder_mode_set(struct drm_encoder *encoder,
>  {
>  	u32 reg;
>  	struct drm_device *dev = encoder->dev;
> -	struct hibmc_drm_private *priv = dev->dev_private;
> +	struct hibmc_drm_private *priv = to_hibmc_drm_private(dev);
>  
>  	reg = readl(priv->mmio + HIBMC_DISPLAY_CONTROL_HISILE);
>  	reg |= HIBMC_DISPLAY_CONTROL_FPVDDEN(1);
> 

-- 
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux