* [PATCH drm/hisilicon 0/2] Use the same style of variable type @ 2020-09-30 7:13 ` Tian Tao 0 siblings, 0 replies; 10+ messages in thread From: Tian Tao @ 2020-09-30 7:13 UTC (permalink / raw) To: airlied, daniel, tzimmermann, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel Cc: linuxarm patch #1 and #2 Use the same style of variable type in hisilicon drm driver and both are clean up, no actual functional changes. Tian Tao (2): drm/hisilicon: Use the same style of variable type in hibmc_drm_de drm/hisilicon: Use the same style of variable type in hibmc_drm_drv drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 59 ++++++++++++------------- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 13 +++--- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 8 ++-- 3 files changed, 39 insertions(+), 41 deletions(-) -- 2.7.4 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH drm/hisilicon 0/2] Use the same style of variable type @ 2020-09-30 7:13 ` Tian Tao 0 siblings, 0 replies; 10+ messages in thread From: Tian Tao @ 2020-09-30 7:13 UTC (permalink / raw) To: airlied, daniel, tzimmermann, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel Cc: linuxarm patch #1 and #2 Use the same style of variable type in hisilicon drm driver and both are clean up, no actual functional changes. Tian Tao (2): drm/hisilicon: Use the same style of variable type in hibmc_drm_de drm/hisilicon: Use the same style of variable type in hibmc_drm_drv drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 59 ++++++++++++------------- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 13 +++--- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 8 ++-- 3 files changed, 39 insertions(+), 41 deletions(-) -- 2.7.4 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH drm/hisilicon 1/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_de 2020-09-30 7:13 ` Tian Tao @ 2020-09-30 7:13 ` Tian Tao -1 siblings, 0 replies; 10+ messages in thread From: Tian Tao @ 2020-09-30 7:13 UTC (permalink / raw) To: airlied, daniel, tzimmermann, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel Cc: linuxarm Consistently Use the same style of variable type in hibmc_drm_de.c. Signed-off-by: Tian Tao <tiantao6@hisilicon.com> --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 59 +++++++++++++------------- 1 file changed, 29 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c index a3a9e0a..c54f93d 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c @@ -23,15 +23,15 @@ #include "hibmc_drm_regs.h" struct hibmc_display_panel_pll { - unsigned long M; - unsigned long N; - unsigned long OD; - unsigned long POD; + u64 M; + u64 N; + u64 OD; + u64 POD; }; struct hibmc_dislay_pll_config { - unsigned long hdisplay; - unsigned long vdisplay; + u64 hdisplay; + u64 vdisplay; u32 pll1_config_value; u32 pll2_config_value; }; @@ -102,7 +102,7 @@ static void hibmc_plane_atomic_update(struct drm_plane *plane, struct drm_plane_state *state = plane->state; u32 reg; s64 gpu_addr = 0; - unsigned int line_l; + u32 line_l; struct hibmc_drm_private *priv = to_hibmc_drm_private(plane->dev); struct drm_gem_vram_object *gbo; @@ -155,10 +155,10 @@ static const struct drm_plane_helper_funcs hibmc_plane_helper_funcs = { .atomic_update = hibmc_plane_atomic_update, }; -static void hibmc_crtc_dpms(struct drm_crtc *crtc, int dpms) +static void hibmc_crtc_dpms(struct drm_crtc *crtc, u32 dpms) { struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); - unsigned int reg; + u32 reg; reg = readl(priv->mmio + HIBMC_CRT_DISP_CTL); reg &= ~HIBMC_CRT_DISP_CTL_DPMS_MASK; @@ -172,7 +172,7 @@ static void hibmc_crtc_dpms(struct drm_crtc *crtc, int dpms) static void hibmc_crtc_atomic_enable(struct drm_crtc *crtc, struct drm_crtc_state *old_state) { - unsigned int reg; + u32 reg; struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); @@ -191,7 +191,7 @@ static void hibmc_crtc_atomic_enable(struct drm_crtc *crtc, static void hibmc_crtc_atomic_disable(struct drm_crtc *crtc, struct drm_crtc_state *old_state) { - unsigned int reg; + u32 reg; struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); hibmc_crtc_dpms(crtc, HIBMC_CRT_DPMS_OFF); @@ -212,7 +212,7 @@ static enum drm_mode_status hibmc_crtc_mode_valid(struct drm_crtc *crtc, const struct drm_display_mode *mode) { - int i = 0; + u32 i = 0; int vrefresh = drm_mode_vrefresh(mode); if (vrefresh < 59 || vrefresh > 61) @@ -227,9 +227,9 @@ hibmc_crtc_mode_valid(struct drm_crtc *crtc, return MODE_BAD; } -static unsigned int format_pll_reg(void) +static u32 format_pll_reg(void) { - unsigned int pllreg = 0; + u32 pllreg = 0; struct hibmc_display_panel_pll pll = {0}; /* @@ -249,7 +249,7 @@ static unsigned int format_pll_reg(void) return pllreg; } -static void set_vclock_hisilicon(struct drm_device *dev, unsigned long pll) +static void set_vclock_hisilicon(struct drm_device *dev, u64 pll) { u32 val; struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); @@ -279,11 +279,10 @@ static void set_vclock_hisilicon(struct drm_device *dev, unsigned long pll) writel(val, priv->mmio + CRT_PLL1_HS); } -static void get_pll_config(unsigned long x, unsigned long y, - u32 *pll1, u32 *pll2) +static void get_pll_config(u64 x, u64 y, u32 *pll1, u32 *pll2) { - int i; - int count = ARRAY_SIZE(hibmc_pll_table); + u32 i; + u32 count = ARRAY_SIZE(hibmc_pll_table); for (i = 0; i < count; i++) { if (hibmc_pll_table[i].hdisplay == x && @@ -306,11 +305,11 @@ static void get_pll_config(unsigned long x, unsigned long y, * FPGA only supports 7 predefined pixel clocks, and clock select is * in bit 4:0 of new register 0x802a8. */ -static unsigned int display_ctrl_adjust(struct drm_device *dev, - struct drm_display_mode *mode, - unsigned int ctrl) +static u32 display_ctrl_adjust(struct drm_device *dev, + struct drm_display_mode *mode, + u32 ctrl) { - unsigned long x, y; + u64 x, y; u32 pll1; /* bit[31:0] of PLL */ u32 pll2; /* bit[63:32] of PLL */ struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); @@ -358,12 +357,12 @@ static unsigned int display_ctrl_adjust(struct drm_device *dev, static void hibmc_crtc_mode_set_nofb(struct drm_crtc *crtc) { - unsigned int val; + u32 val; struct drm_display_mode *mode = &crtc->state->mode; struct drm_device *dev = crtc->dev; 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; + u32 width = mode->hsync_end - mode->hsync_start; + u32 height = mode->vsync_end - mode->vsync_start; writel(format_pll_reg(), priv->mmio + HIBMC_CRT_PLL_CTRL); writel(HIBMC_FIELD(HIBMC_CRT_HORZ_TOTAL_TOTAL, mode->htotal - 1) | @@ -393,7 +392,7 @@ static void hibmc_crtc_mode_set_nofb(struct drm_crtc *crtc) static void hibmc_crtc_atomic_begin(struct drm_crtc *crtc, struct drm_crtc_state *old_state) { - unsigned int reg; + u32 reg; struct drm_device *dev = crtc->dev; struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); @@ -446,15 +445,15 @@ static void hibmc_crtc_load_lut(struct drm_crtc *crtc) struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); void __iomem *mmio = priv->mmio; u16 *r, *g, *b; - unsigned int reg; - int i; + u32 reg; + u32 i; r = crtc->gamma_store; g = r + crtc->gamma_size; b = g + crtc->gamma_size; for (i = 0; i < crtc->gamma_size; i++) { - unsigned int offset = i << 2; + u32 offset = i << 2; u8 red = *r++ >> 8; u8 green = *g++ >> 8; u8 blue = *b++ >> 8; -- 2.7.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH drm/hisilicon 1/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_de @ 2020-09-30 7:13 ` Tian Tao 0 siblings, 0 replies; 10+ messages in thread From: Tian Tao @ 2020-09-30 7:13 UTC (permalink / raw) To: airlied, daniel, tzimmermann, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel Cc: linuxarm Consistently Use the same style of variable type in hibmc_drm_de.c. Signed-off-by: Tian Tao <tiantao6@hisilicon.com> --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 59 +++++++++++++------------- 1 file changed, 29 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c index a3a9e0a..c54f93d 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c @@ -23,15 +23,15 @@ #include "hibmc_drm_regs.h" struct hibmc_display_panel_pll { - unsigned long M; - unsigned long N; - unsigned long OD; - unsigned long POD; + u64 M; + u64 N; + u64 OD; + u64 POD; }; struct hibmc_dislay_pll_config { - unsigned long hdisplay; - unsigned long vdisplay; + u64 hdisplay; + u64 vdisplay; u32 pll1_config_value; u32 pll2_config_value; }; @@ -102,7 +102,7 @@ static void hibmc_plane_atomic_update(struct drm_plane *plane, struct drm_plane_state *state = plane->state; u32 reg; s64 gpu_addr = 0; - unsigned int line_l; + u32 line_l; struct hibmc_drm_private *priv = to_hibmc_drm_private(plane->dev); struct drm_gem_vram_object *gbo; @@ -155,10 +155,10 @@ static const struct drm_plane_helper_funcs hibmc_plane_helper_funcs = { .atomic_update = hibmc_plane_atomic_update, }; -static void hibmc_crtc_dpms(struct drm_crtc *crtc, int dpms) +static void hibmc_crtc_dpms(struct drm_crtc *crtc, u32 dpms) { struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); - unsigned int reg; + u32 reg; reg = readl(priv->mmio + HIBMC_CRT_DISP_CTL); reg &= ~HIBMC_CRT_DISP_CTL_DPMS_MASK; @@ -172,7 +172,7 @@ static void hibmc_crtc_dpms(struct drm_crtc *crtc, int dpms) static void hibmc_crtc_atomic_enable(struct drm_crtc *crtc, struct drm_crtc_state *old_state) { - unsigned int reg; + u32 reg; struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); @@ -191,7 +191,7 @@ static void hibmc_crtc_atomic_enable(struct drm_crtc *crtc, static void hibmc_crtc_atomic_disable(struct drm_crtc *crtc, struct drm_crtc_state *old_state) { - unsigned int reg; + u32 reg; struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); hibmc_crtc_dpms(crtc, HIBMC_CRT_DPMS_OFF); @@ -212,7 +212,7 @@ static enum drm_mode_status hibmc_crtc_mode_valid(struct drm_crtc *crtc, const struct drm_display_mode *mode) { - int i = 0; + u32 i = 0; int vrefresh = drm_mode_vrefresh(mode); if (vrefresh < 59 || vrefresh > 61) @@ -227,9 +227,9 @@ hibmc_crtc_mode_valid(struct drm_crtc *crtc, return MODE_BAD; } -static unsigned int format_pll_reg(void) +static u32 format_pll_reg(void) { - unsigned int pllreg = 0; + u32 pllreg = 0; struct hibmc_display_panel_pll pll = {0}; /* @@ -249,7 +249,7 @@ static unsigned int format_pll_reg(void) return pllreg; } -static void set_vclock_hisilicon(struct drm_device *dev, unsigned long pll) +static void set_vclock_hisilicon(struct drm_device *dev, u64 pll) { u32 val; struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); @@ -279,11 +279,10 @@ static void set_vclock_hisilicon(struct drm_device *dev, unsigned long pll) writel(val, priv->mmio + CRT_PLL1_HS); } -static void get_pll_config(unsigned long x, unsigned long y, - u32 *pll1, u32 *pll2) +static void get_pll_config(u64 x, u64 y, u32 *pll1, u32 *pll2) { - int i; - int count = ARRAY_SIZE(hibmc_pll_table); + u32 i; + u32 count = ARRAY_SIZE(hibmc_pll_table); for (i = 0; i < count; i++) { if (hibmc_pll_table[i].hdisplay == x && @@ -306,11 +305,11 @@ static void get_pll_config(unsigned long x, unsigned long y, * FPGA only supports 7 predefined pixel clocks, and clock select is * in bit 4:0 of new register 0x802a8. */ -static unsigned int display_ctrl_adjust(struct drm_device *dev, - struct drm_display_mode *mode, - unsigned int ctrl) +static u32 display_ctrl_adjust(struct drm_device *dev, + struct drm_display_mode *mode, + u32 ctrl) { - unsigned long x, y; + u64 x, y; u32 pll1; /* bit[31:0] of PLL */ u32 pll2; /* bit[63:32] of PLL */ struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); @@ -358,12 +357,12 @@ static unsigned int display_ctrl_adjust(struct drm_device *dev, static void hibmc_crtc_mode_set_nofb(struct drm_crtc *crtc) { - unsigned int val; + u32 val; struct drm_display_mode *mode = &crtc->state->mode; struct drm_device *dev = crtc->dev; 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; + u32 width = mode->hsync_end - mode->hsync_start; + u32 height = mode->vsync_end - mode->vsync_start; writel(format_pll_reg(), priv->mmio + HIBMC_CRT_PLL_CTRL); writel(HIBMC_FIELD(HIBMC_CRT_HORZ_TOTAL_TOTAL, mode->htotal - 1) | @@ -393,7 +392,7 @@ static void hibmc_crtc_mode_set_nofb(struct drm_crtc *crtc) static void hibmc_crtc_atomic_begin(struct drm_crtc *crtc, struct drm_crtc_state *old_state) { - unsigned int reg; + u32 reg; struct drm_device *dev = crtc->dev; struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); @@ -446,15 +445,15 @@ static void hibmc_crtc_load_lut(struct drm_crtc *crtc) struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); void __iomem *mmio = priv->mmio; u16 *r, *g, *b; - unsigned int reg; - int i; + u32 reg; + u32 i; r = crtc->gamma_store; g = r + crtc->gamma_size; b = g + crtc->gamma_size; for (i = 0; i < crtc->gamma_size; i++) { - unsigned int offset = i << 2; + u32 offset = i << 2; u8 red = *r++ >> 8; u8 green = *g++ >> 8; u8 blue = *b++ >> 8; -- 2.7.4 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH drm/hisilicon 1/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_de 2020-09-30 7:13 ` Tian Tao @ 2020-10-14 11:58 ` Thomas Zimmermann -1 siblings, 0 replies; 10+ messages in thread From: Thomas Zimmermann @ 2020-10-14 11:58 UTC (permalink / raw) To: Tian Tao Cc: airlied, daniel, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel, linuxarm Hi, reviews take a while as I'm very busy ATM. On Wed, 30 Sep 2020 15:13:07 +0800 Tian Tao <tiantao6@hisilicon.com> wrote: > Consistently Use the same style of variable type in hibmc_drm_de.c. > > Signed-off-by: Tian Tao <tiantao6@hisilicon.com> > --- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 59 > +++++++++++++------------- 1 file changed, 29 insertions(+), 30 deletions(-) > > diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c index a3a9e0a..c54f93d > 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > @@ -23,15 +23,15 @@ > #include "hibmc_drm_regs.h" > > struct hibmc_display_panel_pll { > - unsigned long M; > - unsigned long N; > - unsigned long OD; > - unsigned long POD; > + u64 M; > + u64 N; > + u64 OD; > + u64 POD; > }; > > struct hibmc_dislay_pll_config { > - unsigned long hdisplay; > - unsigned long vdisplay; > + u64 hdisplay; > + u64 vdisplay; > u32 pll1_config_value; > u32 pll2_config_value; > }; > @@ -102,7 +102,7 @@ static void hibmc_plane_atomic_update(struct drm_plane > *plane, struct drm_plane_state *state = plane->state; > u32 reg; > s64 gpu_addr = 0; > - unsigned int line_l; > + u32 line_l; > struct hibmc_drm_private *priv = to_hibmc_drm_private(plane->dev); > struct drm_gem_vram_object *gbo; > > @@ -155,10 +155,10 @@ static const struct drm_plane_helper_funcs > hibmc_plane_helper_funcs = { .atomic_update = hibmc_plane_atomic_update, > }; > > -static void hibmc_crtc_dpms(struct drm_crtc *crtc, int dpms) > +static void hibmc_crtc_dpms(struct drm_crtc *crtc, u32 dpms) > { > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > - unsigned int reg; > + u32 reg; > > reg = readl(priv->mmio + HIBMC_CRT_DISP_CTL); > reg &= ~HIBMC_CRT_DISP_CTL_DPMS_MASK; > @@ -172,7 +172,7 @@ static void hibmc_crtc_dpms(struct drm_crtc *crtc, int > dpms) static void hibmc_crtc_atomic_enable(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > - unsigned int reg; > + u32 reg; > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > > hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); > @@ -191,7 +191,7 @@ static void hibmc_crtc_atomic_enable(struct drm_crtc > *crtc, static void hibmc_crtc_atomic_disable(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > - unsigned int reg; > + u32 reg; > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > > hibmc_crtc_dpms(crtc, HIBMC_CRT_DPMS_OFF); > @@ -212,7 +212,7 @@ static enum drm_mode_status > hibmc_crtc_mode_valid(struct drm_crtc *crtc, > const struct drm_display_mode *mode) > { > - int i = 0; > + u32 i = 0; This is a counter against ARRAY_SIZE. i should be of type 'size_t'. > int vrefresh = drm_mode_vrefresh(mode); > > if (vrefresh < 59 || vrefresh > 61) > @@ -227,9 +227,9 @@ hibmc_crtc_mode_valid(struct drm_crtc *crtc, > return MODE_BAD; > } > > -static unsigned int format_pll_reg(void) > +static u32 format_pll_reg(void) > { > - unsigned int pllreg = 0; > + u32 pllreg = 0; > struct hibmc_display_panel_pll pll = {0}; > > /* > @@ -249,7 +249,7 @@ static unsigned int format_pll_reg(void) > return pllreg; > } > > -static void set_vclock_hisilicon(struct drm_device *dev, unsigned long pll) > +static void set_vclock_hisilicon(struct drm_device *dev, u64 pll) > { > u32 val; > struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); > @@ -279,11 +279,10 @@ static void set_vclock_hisilicon(struct drm_device > *dev, unsigned long pll) writel(val, priv->mmio + CRT_PLL1_HS); > } > > -static void get_pll_config(unsigned long x, unsigned long y, > - u32 *pll1, u32 *pll2) > +static void get_pll_config(u64 x, u64 y, u32 *pll1, u32 *pll2) > { > - int i; > - int count = ARRAY_SIZE(hibmc_pll_table); > + u32 i; > + u32 count = ARRAY_SIZE(hibmc_pll_table); These variables should also be size_t. > > for (i = 0; i < count; i++) { > if (hibmc_pll_table[i].hdisplay == x && > @@ -306,11 +305,11 @@ static void get_pll_config(unsigned long x, unsigned > long y, > * FPGA only supports 7 predefined pixel clocks, and clock select is > * in bit 4:0 of new register 0x802a8. > */ > -static unsigned int display_ctrl_adjust(struct drm_device *dev, > - struct drm_display_mode *mode, > - unsigned int ctrl) > +static u32 display_ctrl_adjust(struct drm_device *dev, > + struct drm_display_mode *mode, > + u32 ctrl) > { > - unsigned long x, y; > + u64 x, y; > u32 pll1; /* bit[31:0] of PLL */ > u32 pll2; /* bit[63:32] of PLL */ > struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); > @@ -358,12 +357,12 @@ static unsigned int display_ctrl_adjust(struct > drm_device *dev, > static void hibmc_crtc_mode_set_nofb(struct drm_crtc *crtc) > { > - unsigned int val; > + u32 val; > struct drm_display_mode *mode = &crtc->state->mode; > struct drm_device *dev = crtc->dev; > 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; > + u32 width = mode->hsync_end - mode->hsync_start; > + u32 height = mode->vsync_end - mode->vsync_start; > > writel(format_pll_reg(), priv->mmio + HIBMC_CRT_PLL_CTRL); > writel(HIBMC_FIELD(HIBMC_CRT_HORZ_TOTAL_TOTAL, mode->htotal - 1) | > @@ -393,7 +392,7 @@ static void hibmc_crtc_mode_set_nofb(struct drm_crtc > *crtc) static void hibmc_crtc_atomic_begin(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > - unsigned int reg; > + u32 reg; > struct drm_device *dev = crtc->dev; > struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); > > @@ -446,15 +445,15 @@ static void hibmc_crtc_load_lut(struct drm_crtc *crtc) > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > void __iomem *mmio = priv->mmio; > u16 *r, *g, *b; > - unsigned int reg; > - int i; > + u32 reg; > + u32 i; This one's correct because crtc->gamma_size is uint32_t. With my comments addressed: Acked-by: Thomas Zimmermann <tzimmermann@suse.de> Best regards Thomas > > r = crtc->gamma_store; > g = r + crtc->gamma_size; > b = g + crtc->gamma_size; > > for (i = 0; i < crtc->gamma_size; i++) { > - unsigned int offset = i << 2; > + u32 offset = i << 2; > u8 red = *r++ >> 8; > u8 green = *g++ >> 8; > u8 blue = *b++ >> 8; -- 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH drm/hisilicon 1/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_de @ 2020-10-14 11:58 ` Thomas Zimmermann 0 siblings, 0 replies; 10+ messages in thread From: Thomas Zimmermann @ 2020-10-14 11:58 UTC (permalink / raw) To: Tian Tao Cc: airlied, linux-kernel, dri-devel, linuxarm, xinliang.liu, kraxel, alexander.deucher, tglx Hi, reviews take a while as I'm very busy ATM. On Wed, 30 Sep 2020 15:13:07 +0800 Tian Tao <tiantao6@hisilicon.com> wrote: > Consistently Use the same style of variable type in hibmc_drm_de.c. > > Signed-off-by: Tian Tao <tiantao6@hisilicon.com> > --- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 59 > +++++++++++++------------- 1 file changed, 29 insertions(+), 30 deletions(-) > > diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c index a3a9e0a..c54f93d > 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c > @@ -23,15 +23,15 @@ > #include "hibmc_drm_regs.h" > > struct hibmc_display_panel_pll { > - unsigned long M; > - unsigned long N; > - unsigned long OD; > - unsigned long POD; > + u64 M; > + u64 N; > + u64 OD; > + u64 POD; > }; > > struct hibmc_dislay_pll_config { > - unsigned long hdisplay; > - unsigned long vdisplay; > + u64 hdisplay; > + u64 vdisplay; > u32 pll1_config_value; > u32 pll2_config_value; > }; > @@ -102,7 +102,7 @@ static void hibmc_plane_atomic_update(struct drm_plane > *plane, struct drm_plane_state *state = plane->state; > u32 reg; > s64 gpu_addr = 0; > - unsigned int line_l; > + u32 line_l; > struct hibmc_drm_private *priv = to_hibmc_drm_private(plane->dev); > struct drm_gem_vram_object *gbo; > > @@ -155,10 +155,10 @@ static const struct drm_plane_helper_funcs > hibmc_plane_helper_funcs = { .atomic_update = hibmc_plane_atomic_update, > }; > > -static void hibmc_crtc_dpms(struct drm_crtc *crtc, int dpms) > +static void hibmc_crtc_dpms(struct drm_crtc *crtc, u32 dpms) > { > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > - unsigned int reg; > + u32 reg; > > reg = readl(priv->mmio + HIBMC_CRT_DISP_CTL); > reg &= ~HIBMC_CRT_DISP_CTL_DPMS_MASK; > @@ -172,7 +172,7 @@ static void hibmc_crtc_dpms(struct drm_crtc *crtc, int > dpms) static void hibmc_crtc_atomic_enable(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > - unsigned int reg; > + u32 reg; > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > > hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); > @@ -191,7 +191,7 @@ static void hibmc_crtc_atomic_enable(struct drm_crtc > *crtc, static void hibmc_crtc_atomic_disable(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > - unsigned int reg; > + u32 reg; > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > > hibmc_crtc_dpms(crtc, HIBMC_CRT_DPMS_OFF); > @@ -212,7 +212,7 @@ static enum drm_mode_status > hibmc_crtc_mode_valid(struct drm_crtc *crtc, > const struct drm_display_mode *mode) > { > - int i = 0; > + u32 i = 0; This is a counter against ARRAY_SIZE. i should be of type 'size_t'. > int vrefresh = drm_mode_vrefresh(mode); > > if (vrefresh < 59 || vrefresh > 61) > @@ -227,9 +227,9 @@ hibmc_crtc_mode_valid(struct drm_crtc *crtc, > return MODE_BAD; > } > > -static unsigned int format_pll_reg(void) > +static u32 format_pll_reg(void) > { > - unsigned int pllreg = 0; > + u32 pllreg = 0; > struct hibmc_display_panel_pll pll = {0}; > > /* > @@ -249,7 +249,7 @@ static unsigned int format_pll_reg(void) > return pllreg; > } > > -static void set_vclock_hisilicon(struct drm_device *dev, unsigned long pll) > +static void set_vclock_hisilicon(struct drm_device *dev, u64 pll) > { > u32 val; > struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); > @@ -279,11 +279,10 @@ static void set_vclock_hisilicon(struct drm_device > *dev, unsigned long pll) writel(val, priv->mmio + CRT_PLL1_HS); > } > > -static void get_pll_config(unsigned long x, unsigned long y, > - u32 *pll1, u32 *pll2) > +static void get_pll_config(u64 x, u64 y, u32 *pll1, u32 *pll2) > { > - int i; > - int count = ARRAY_SIZE(hibmc_pll_table); > + u32 i; > + u32 count = ARRAY_SIZE(hibmc_pll_table); These variables should also be size_t. > > for (i = 0; i < count; i++) { > if (hibmc_pll_table[i].hdisplay == x && > @@ -306,11 +305,11 @@ static void get_pll_config(unsigned long x, unsigned > long y, > * FPGA only supports 7 predefined pixel clocks, and clock select is > * in bit 4:0 of new register 0x802a8. > */ > -static unsigned int display_ctrl_adjust(struct drm_device *dev, > - struct drm_display_mode *mode, > - unsigned int ctrl) > +static u32 display_ctrl_adjust(struct drm_device *dev, > + struct drm_display_mode *mode, > + u32 ctrl) > { > - unsigned long x, y; > + u64 x, y; > u32 pll1; /* bit[31:0] of PLL */ > u32 pll2; /* bit[63:32] of PLL */ > struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); > @@ -358,12 +357,12 @@ static unsigned int display_ctrl_adjust(struct > drm_device *dev, > static void hibmc_crtc_mode_set_nofb(struct drm_crtc *crtc) > { > - unsigned int val; > + u32 val; > struct drm_display_mode *mode = &crtc->state->mode; > struct drm_device *dev = crtc->dev; > 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; > + u32 width = mode->hsync_end - mode->hsync_start; > + u32 height = mode->vsync_end - mode->vsync_start; > > writel(format_pll_reg(), priv->mmio + HIBMC_CRT_PLL_CTRL); > writel(HIBMC_FIELD(HIBMC_CRT_HORZ_TOTAL_TOTAL, mode->htotal - 1) | > @@ -393,7 +392,7 @@ static void hibmc_crtc_mode_set_nofb(struct drm_crtc > *crtc) static void hibmc_crtc_atomic_begin(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > - unsigned int reg; > + u32 reg; > struct drm_device *dev = crtc->dev; > struct hibmc_drm_private *priv = to_hibmc_drm_private(dev); > > @@ -446,15 +445,15 @@ static void hibmc_crtc_load_lut(struct drm_crtc *crtc) > struct hibmc_drm_private *priv = to_hibmc_drm_private(crtc->dev); > void __iomem *mmio = priv->mmio; > u16 *r, *g, *b; > - unsigned int reg; > - int i; > + u32 reg; > + u32 i; This one's correct because crtc->gamma_size is uint32_t. With my comments addressed: Acked-by: Thomas Zimmermann <tzimmermann@suse.de> Best regards Thomas > > r = crtc->gamma_store; > g = r + crtc->gamma_size; > b = g + crtc->gamma_size; > > for (i = 0; i < crtc->gamma_size; i++) { > - unsigned int offset = i << 2; > + u32 offset = i << 2; > u8 red = *r++ >> 8; > u8 green = *g++ >> 8; > u8 blue = *b++ >> 8; -- 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 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH drm/hisilicon 2/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_drv 2020-09-30 7:13 ` Tian Tao @ 2020-09-30 7:13 ` Tian Tao -1 siblings, 0 replies; 10+ messages in thread From: Tian Tao @ 2020-09-30 7:13 UTC (permalink / raw) To: airlied, daniel, tzimmermann, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel Cc: linuxarm Consistently Use the same style of variable type in hibmc_drm_de.c and hibmc_drm_de.h. Signed-off-by: Tian Tao <tiantao6@hisilicon.com> --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 13 ++++++------- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 8 ++++---- 2 files changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c index 5632bce..0c1b40d 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c @@ -121,12 +121,11 @@ static void hibmc_kms_fini(struct hibmc_drm_private *priv) /* * It can operate in one of three modes: 0, 1 or Sleep. */ -void hibmc_set_power_mode(struct hibmc_drm_private *priv, - unsigned int power_mode) +void hibmc_set_power_mode(struct hibmc_drm_private *priv, u32 power_mode) { - unsigned int control_value = 0; + u32 control_value = 0; void __iomem *mmio = priv->mmio; - unsigned int input = 1; + u32 input = 1; if (power_mode > HIBMC_PW_MODE_CTL_MODE_SLEEP) return; @@ -144,8 +143,8 @@ void hibmc_set_power_mode(struct hibmc_drm_private *priv, void hibmc_set_current_gate(struct hibmc_drm_private *priv, unsigned int gate) { - unsigned int gate_reg; - unsigned int mode; + u32 gate_reg; + u32 mode; void __iomem *mmio = priv->mmio; /* Get current power mode. */ @@ -170,7 +169,7 @@ void hibmc_set_current_gate(struct hibmc_drm_private *priv, unsigned int gate) static void hibmc_hw_config(struct hibmc_drm_private *priv) { - unsigned int reg; + u32 reg; /* On hardware reset, power mode 0 is default. */ hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h index 6a63502..5c4030d 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h @@ -33,8 +33,8 @@ struct hibmc_drm_private { /* hw */ void __iomem *mmio; void __iomem *fb_map; - unsigned long fb_base; - unsigned long fb_size; + u64 fb_base; + u64 fb_size; /* drm */ struct drm_device *dev; @@ -56,9 +56,9 @@ static inline struct hibmc_drm_private *to_hibmc_drm_private(struct drm_device * } void hibmc_set_power_mode(struct hibmc_drm_private *priv, - unsigned int power_mode); + u32 power_mode); void hibmc_set_current_gate(struct hibmc_drm_private *priv, - unsigned int gate); + u32 gate); int hibmc_de_init(struct hibmc_drm_private *priv); int hibmc_vdac_init(struct hibmc_drm_private *priv); -- 2.7.4 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH drm/hisilicon 2/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_drv @ 2020-09-30 7:13 ` Tian Tao 0 siblings, 0 replies; 10+ messages in thread From: Tian Tao @ 2020-09-30 7:13 UTC (permalink / raw) To: airlied, daniel, tzimmermann, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel Cc: linuxarm Consistently Use the same style of variable type in hibmc_drm_de.c and hibmc_drm_de.h. Signed-off-by: Tian Tao <tiantao6@hisilicon.com> --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 13 ++++++------- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 8 ++++---- 2 files changed, 10 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c index 5632bce..0c1b40d 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c @@ -121,12 +121,11 @@ static void hibmc_kms_fini(struct hibmc_drm_private *priv) /* * It can operate in one of three modes: 0, 1 or Sleep. */ -void hibmc_set_power_mode(struct hibmc_drm_private *priv, - unsigned int power_mode) +void hibmc_set_power_mode(struct hibmc_drm_private *priv, u32 power_mode) { - unsigned int control_value = 0; + u32 control_value = 0; void __iomem *mmio = priv->mmio; - unsigned int input = 1; + u32 input = 1; if (power_mode > HIBMC_PW_MODE_CTL_MODE_SLEEP) return; @@ -144,8 +143,8 @@ void hibmc_set_power_mode(struct hibmc_drm_private *priv, void hibmc_set_current_gate(struct hibmc_drm_private *priv, unsigned int gate) { - unsigned int gate_reg; - unsigned int mode; + u32 gate_reg; + u32 mode; void __iomem *mmio = priv->mmio; /* Get current power mode. */ @@ -170,7 +169,7 @@ void hibmc_set_current_gate(struct hibmc_drm_private *priv, unsigned int gate) static void hibmc_hw_config(struct hibmc_drm_private *priv) { - unsigned int reg; + u32 reg; /* On hardware reset, power mode 0 is default. */ hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h index 6a63502..5c4030d 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h @@ -33,8 +33,8 @@ struct hibmc_drm_private { /* hw */ void __iomem *mmio; void __iomem *fb_map; - unsigned long fb_base; - unsigned long fb_size; + u64 fb_base; + u64 fb_size; /* drm */ struct drm_device *dev; @@ -56,9 +56,9 @@ static inline struct hibmc_drm_private *to_hibmc_drm_private(struct drm_device * } void hibmc_set_power_mode(struct hibmc_drm_private *priv, - unsigned int power_mode); + u32 power_mode); void hibmc_set_current_gate(struct hibmc_drm_private *priv, - unsigned int gate); + u32 gate); int hibmc_de_init(struct hibmc_drm_private *priv); int hibmc_vdac_init(struct hibmc_drm_private *priv); -- 2.7.4 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH drm/hisilicon 2/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_drv 2020-09-30 7:13 ` Tian Tao @ 2020-10-14 12:02 ` Thomas Zimmermann -1 siblings, 0 replies; 10+ messages in thread From: Thomas Zimmermann @ 2020-10-14 12:02 UTC (permalink / raw) To: Tian Tao Cc: airlied, daniel, kraxel, alexander.deucher, tglx, dri-devel, xinliang.liu, linux-kernel, linuxarm Hi On Wed, 30 Sep 2020 15:13:08 +0800 Tian Tao <tiantao6@hisilicon.com> wrote: > Consistently Use the same style of variable type in hibmc_drm_de.c and > hibmc_drm_de.h. > > Signed-off-by: Tian Tao <tiantao6@hisilicon.com> > --- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 13 ++++++------- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 8 ++++---- > 2 files changed, 10 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c > b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c index 5632bce..0c1b40d > 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c > +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c > @@ -121,12 +121,11 @@ static void hibmc_kms_fini(struct hibmc_drm_private > *priv) /* > * It can operate in one of three modes: 0, 1 or Sleep. > */ > -void hibmc_set_power_mode(struct hibmc_drm_private *priv, > - unsigned int power_mode) > +void hibmc_set_power_mode(struct hibmc_drm_private *priv, u32 power_mode) > { > - unsigned int control_value = 0; > + u32 control_value = 0; > void __iomem *mmio = priv->mmio; > - unsigned int input = 1; > + u32 input = 1; > > if (power_mode > HIBMC_PW_MODE_CTL_MODE_SLEEP) > return; > @@ -144,8 +143,8 @@ void hibmc_set_power_mode(struct hibmc_drm_private > *priv, > void hibmc_set_current_gate(struct hibmc_drm_private *priv, unsigned int > gate) { > - unsigned int gate_reg; > - unsigned int mode; > + u32 gate_reg; > + u32 mode; > void __iomem *mmio = priv->mmio; > > /* Get current power mode. */ > @@ -170,7 +169,7 @@ void hibmc_set_current_gate(struct hibmc_drm_private > *priv, unsigned int gate) > static void hibmc_hw_config(struct hibmc_drm_private *priv) > { > - unsigned int reg; > + u32 reg; > > /* On hardware reset, power mode 0 is default. */ > hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); > diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h > b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h index 6a63502..5c4030d > 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h > +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h > @@ -33,8 +33,8 @@ struct hibmc_drm_private { > /* hw */ > void __iomem *mmio; > void __iomem *fb_map; > - unsigned long fb_base; > - unsigned long fb_size; > + u64 fb_base; > + u64 fb_size; resource_size_t would be the correct type here. With my comments addressed: Acked-by: Thomas Zimmermann <tzimmermann@suse.de> Best regards Thomas > > /* drm */ > struct drm_device *dev; > @@ -56,9 +56,9 @@ static inline struct hibmc_drm_private > *to_hibmc_drm_private(struct drm_device * } > > void hibmc_set_power_mode(struct hibmc_drm_private *priv, > - unsigned int power_mode); > + u32 power_mode); > void hibmc_set_current_gate(struct hibmc_drm_private *priv, > - unsigned int gate); > + u32 gate); > > int hibmc_de_init(struct hibmc_drm_private *priv); > int hibmc_vdac_init(struct hibmc_drm_private *priv); -- 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 ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH drm/hisilicon 2/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_drv @ 2020-10-14 12:02 ` Thomas Zimmermann 0 siblings, 0 replies; 10+ messages in thread From: Thomas Zimmermann @ 2020-10-14 12:02 UTC (permalink / raw) To: Tian Tao Cc: airlied, linux-kernel, dri-devel, linuxarm, xinliang.liu, kraxel, alexander.deucher, tglx Hi On Wed, 30 Sep 2020 15:13:08 +0800 Tian Tao <tiantao6@hisilicon.com> wrote: > Consistently Use the same style of variable type in hibmc_drm_de.c and > hibmc_drm_de.h. > > Signed-off-by: Tian Tao <tiantao6@hisilicon.com> > --- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 13 ++++++------- > drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 8 ++++---- > 2 files changed, 10 insertions(+), 11 deletions(-) > > diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c > b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c index 5632bce..0c1b40d > 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c > +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c > @@ -121,12 +121,11 @@ static void hibmc_kms_fini(struct hibmc_drm_private > *priv) /* > * It can operate in one of three modes: 0, 1 or Sleep. > */ > -void hibmc_set_power_mode(struct hibmc_drm_private *priv, > - unsigned int power_mode) > +void hibmc_set_power_mode(struct hibmc_drm_private *priv, u32 power_mode) > { > - unsigned int control_value = 0; > + u32 control_value = 0; > void __iomem *mmio = priv->mmio; > - unsigned int input = 1; > + u32 input = 1; > > if (power_mode > HIBMC_PW_MODE_CTL_MODE_SLEEP) > return; > @@ -144,8 +143,8 @@ void hibmc_set_power_mode(struct hibmc_drm_private > *priv, > void hibmc_set_current_gate(struct hibmc_drm_private *priv, unsigned int > gate) { > - unsigned int gate_reg; > - unsigned int mode; > + u32 gate_reg; > + u32 mode; > void __iomem *mmio = priv->mmio; > > /* Get current power mode. */ > @@ -170,7 +169,7 @@ void hibmc_set_current_gate(struct hibmc_drm_private > *priv, unsigned int gate) > static void hibmc_hw_config(struct hibmc_drm_private *priv) > { > - unsigned int reg; > + u32 reg; > > /* On hardware reset, power mode 0 is default. */ > hibmc_set_power_mode(priv, HIBMC_PW_MODE_CTL_MODE_MODE0); > diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h > b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h index 6a63502..5c4030d > 100644 --- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h > +++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h > @@ -33,8 +33,8 @@ struct hibmc_drm_private { > /* hw */ > void __iomem *mmio; > void __iomem *fb_map; > - unsigned long fb_base; > - unsigned long fb_size; > + u64 fb_base; > + u64 fb_size; resource_size_t would be the correct type here. With my comments addressed: Acked-by: Thomas Zimmermann <tzimmermann@suse.de> Best regards Thomas > > /* drm */ > struct drm_device *dev; > @@ -56,9 +56,9 @@ static inline struct hibmc_drm_private > *to_hibmc_drm_private(struct drm_device * } > > void hibmc_set_power_mode(struct hibmc_drm_private *priv, > - unsigned int power_mode); > + u32 power_mode); > void hibmc_set_current_gate(struct hibmc_drm_private *priv, > - unsigned int gate); > + u32 gate); > > int hibmc_de_init(struct hibmc_drm_private *priv); > int hibmc_vdac_init(struct hibmc_drm_private *priv); -- 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 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-10-14 12:02 UTC | newest] Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-09-30 7:13 [PATCH drm/hisilicon 0/2] Use the same style of variable type Tian Tao 2020-09-30 7:13 ` Tian Tao 2020-09-30 7:13 ` [PATCH drm/hisilicon 1/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_de Tian Tao 2020-09-30 7:13 ` Tian Tao 2020-10-14 11:58 ` Thomas Zimmermann 2020-10-14 11:58 ` Thomas Zimmermann 2020-09-30 7:13 ` [PATCH drm/hisilicon 2/2] drm/hisilicon: Use the same style of variable type in hibmc_drm_drv Tian Tao 2020-09-30 7:13 ` Tian Tao 2020-10-14 12:02 ` Thomas Zimmermann 2020-10-14 12:02 ` Thomas Zimmermann
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.