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

From: Thomas Zimmermann
Date: Mon Sep 28 2020 - 04:45:09 EST


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