From: Philippe CORNU <philippe.cornu@st.com> To: Peter Rosin <peda@axentia.se>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org> Cc: Yannick FERTRE <yannick.fertre@st.com>, Benjamin Gaignard <benjamin.gaignard@linaro.org>, Vincent ABRIOU <vincent.abriou@st.com>, "David Airlie" <airlied@linux.ie>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, Daniel Vetter <daniel.vetter@intel.com>, Jani Nikula <jani.nikula@linux.intel.com>, Sean Paul <seanpaul@chromium.org>, Lionel Landwerlin <lionel.g.landwerlin@intel.com>, Boris Brezillon <boris.brezillon@free-electrons.com> Subject: Re: [PATCH v5 13/14] drm: stm: remove dead code and pointless local lut storage Date: Mon, 17 Jul 2017 09:47:06 +0000 [thread overview] Message-ID: <2b3b4bbc-df98-ac01-1740-294f73acf8fc@st.com> (raw) In-Reply-To: <20170713162538.22788-14-peda@axentia.se> On 07/13/2017 06:25 PM, Peter Rosin wrote: > The redundant fb helper .load_lut is no longer used, and can not > work right without also providing the fb helpers .gamma_set and > .gamma_get thus rendering the code in this driver suspect. > > Just remove the dead code. > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> > Signed-off-by: Peter Rosin <peda@axentia.se> Acked-by: Philippe Cornu <philippe.cornu@st.com> Note: we will update stm32 clut support after your patch. Many thanks. > --- > drivers/gpu/drm/stm/ltdc.c | 12 ------------ > drivers/gpu/drm/stm/ltdc.h | 1 - > 2 files changed, 13 deletions(-) > > diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c > index 533176015cbb..3e95b4d1f4cc 100644 > --- a/drivers/gpu/drm/stm/ltdc.c > +++ b/drivers/gpu/drm/stm/ltdc.c > @@ -375,17 +375,6 @@ static irqreturn_t ltdc_irq(int irq, void *arg) > * DRM_CRTC > */ > > -static void ltdc_crtc_load_lut(struct drm_crtc *crtc) > -{ > - struct ltdc_device *ldev = crtc_to_ltdc(crtc); > - unsigned int i, lay; > - > - for (lay = 0; lay < ldev->caps.nb_layers; lay++) > - for (i = 0; i < 256; i++) > - reg_write(ldev->regs, LTDC_L1CLUTWR + lay * LAY_OFS, > - ldev->clut[i]); > -} > - > static void ltdc_crtc_atomic_enable(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > @@ -525,7 +514,6 @@ static void ltdc_crtc_atomic_flush(struct drm_crtc *crtc, > } > > static struct drm_crtc_helper_funcs ltdc_crtc_helper_funcs = { > - .load_lut = ltdc_crtc_load_lut, > .mode_set_nofb = ltdc_crtc_mode_set_nofb, > .atomic_flush = ltdc_crtc_atomic_flush, > .atomic_enable = ltdc_crtc_atomic_enable, > diff --git a/drivers/gpu/drm/stm/ltdc.h b/drivers/gpu/drm/stm/ltdc.h > index d7a9c736ac1e..620ca5555abf 100644 > --- a/drivers/gpu/drm/stm/ltdc.h > +++ b/drivers/gpu/drm/stm/ltdc.h > @@ -27,7 +27,6 @@ struct ltdc_device { > struct drm_panel *panel; > struct mutex err_lock; /* protecting error_status */ > struct ltdc_caps caps; > - u32 clut[256]; /* color look up table */ > u32 error_status; > u32 irq_status; > }; >
WARNING: multiple messages have this Message-ID (diff)
From: Philippe CORNU <philippe.cornu@st.com> To: Peter Rosin <peda@axentia.se>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org> Cc: Boris Brezillon <boris.brezillon@free-electrons.com>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, Yannick FERTRE <yannick.fertre@st.com>, Daniel Vetter <daniel.vetter@intel.com>, Vincent ABRIOU <vincent.abriou@st.com> Subject: Re: [PATCH v5 13/14] drm: stm: remove dead code and pointless local lut storage Date: Mon, 17 Jul 2017 09:47:06 +0000 [thread overview] Message-ID: <2b3b4bbc-df98-ac01-1740-294f73acf8fc@st.com> (raw) In-Reply-To: <20170713162538.22788-14-peda@axentia.se> On 07/13/2017 06:25 PM, Peter Rosin wrote: > The redundant fb helper .load_lut is no longer used, and can not > work right without also providing the fb helpers .gamma_set and > .gamma_get thus rendering the code in this driver suspect. > > Just remove the dead code. > > Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> > Signed-off-by: Peter Rosin <peda@axentia.se> Acked-by: Philippe Cornu <philippe.cornu@st.com> Note: we will update stm32 clut support after your patch. Many thanks. > --- > drivers/gpu/drm/stm/ltdc.c | 12 ------------ > drivers/gpu/drm/stm/ltdc.h | 1 - > 2 files changed, 13 deletions(-) > > diff --git a/drivers/gpu/drm/stm/ltdc.c b/drivers/gpu/drm/stm/ltdc.c > index 533176015cbb..3e95b4d1f4cc 100644 > --- a/drivers/gpu/drm/stm/ltdc.c > +++ b/drivers/gpu/drm/stm/ltdc.c > @@ -375,17 +375,6 @@ static irqreturn_t ltdc_irq(int irq, void *arg) > * DRM_CRTC > */ > > -static void ltdc_crtc_load_lut(struct drm_crtc *crtc) > -{ > - struct ltdc_device *ldev = crtc_to_ltdc(crtc); > - unsigned int i, lay; > - > - for (lay = 0; lay < ldev->caps.nb_layers; lay++) > - for (i = 0; i < 256; i++) > - reg_write(ldev->regs, LTDC_L1CLUTWR + lay * LAY_OFS, > - ldev->clut[i]); > -} > - > static void ltdc_crtc_atomic_enable(struct drm_crtc *crtc, > struct drm_crtc_state *old_state) > { > @@ -525,7 +514,6 @@ static void ltdc_crtc_atomic_flush(struct drm_crtc *crtc, > } > > static struct drm_crtc_helper_funcs ltdc_crtc_helper_funcs = { > - .load_lut = ltdc_crtc_load_lut, > .mode_set_nofb = ltdc_crtc_mode_set_nofb, > .atomic_flush = ltdc_crtc_atomic_flush, > .atomic_enable = ltdc_crtc_atomic_enable, > diff --git a/drivers/gpu/drm/stm/ltdc.h b/drivers/gpu/drm/stm/ltdc.h > index d7a9c736ac1e..620ca5555abf 100644 > --- a/drivers/gpu/drm/stm/ltdc.h > +++ b/drivers/gpu/drm/stm/ltdc.h > @@ -27,7 +27,6 @@ struct ltdc_device { > struct drm_panel *panel; > struct mutex err_lock; /* protecting error_status */ > struct ltdc_caps caps; > - u32 clut[256]; /* color look up table */ > u32 error_status; > u32 irq_status; > }; > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2017-07-17 9:47 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-07-13 16:25 [PATCH v5 00/14] improve the fb_setcmap helper Peter Rosin 2017-07-13 16:25 ` Peter Rosin 2017-07-13 16:25 ` [PATCH v5 01/14] drm: rename, adjust and export drm_atomic_replace_property_blob Peter Rosin 2017-07-13 16:25 ` [PATCH v5 02/14] drm/atomic-helper: update lut props directly in ..._legacy_gamma_set Peter Rosin 2017-07-13 16:25 ` [PATCH v5 03/14] drm/fb-helper: separate the fb_setcmap helper into atomic and legacy paths Peter Rosin 2017-07-14 13:54 ` Daniel Vetter 2017-08-03 22:49 ` Peter Rosin 2017-08-04 9:38 ` Daniel Vetter 2017-08-04 9:38 ` Daniel Vetter 2017-08-04 10:30 ` [PATCH] staging: vboxvideo: remove dead gamma lut code Peter Rosin 2017-08-04 10:45 ` [RESEND PATCH] " Peter Rosin 2017-08-05 11:11 ` Hans de Goede 2017-08-05 11:11 ` Hans de Goede 2017-08-07 9:21 ` Daniel Vetter 2017-08-08 11:54 ` Peter Rosin 2017-08-09 15:14 ` Daniel Vetter 2017-08-09 15:14 ` Daniel Vetter 2017-07-13 16:25 ` [PATCH v5 04/14] drm: amd: remove dead code and pointless local lut storage Peter Rosin 2017-07-14 14:06 ` Alex Deucher 2017-07-14 14:06 ` Alex Deucher 2017-07-13 16:25 ` [PATCH v5 05/14] drm: armada: remove dead empty functions Peter Rosin 2017-07-13 16:25 ` [PATCH v5 06/14] drm: ast: remove dead code and pointless local lut storage Peter Rosin 2017-07-13 16:25 ` [PATCH v5 07/14] drm: cirrus: " Peter Rosin 2017-07-13 16:25 ` Peter Rosin 2017-07-13 16:25 ` [PATCH v5 08/14] drm: gma500: " Peter Rosin 2017-07-13 16:25 ` [PATCH v5 09/14] drm: i915: " Peter Rosin 2017-07-13 16:25 ` [PATCH v5 10/14] drm: mgag200: " Peter Rosin 2017-07-13 16:25 ` [PATCH v5 11/14] drm: nouveau: " Peter Rosin 2017-07-13 16:25 ` [PATCH v5 12/14] drm: radeon: " Peter Rosin 2017-07-14 14:06 ` Alex Deucher 2017-07-14 14:06 ` Alex Deucher 2017-07-13 16:25 ` [PATCH v5 13/14] drm: stm: " Peter Rosin 2017-07-17 9:47 ` Philippe CORNU [this message] 2017-07-17 9:47 ` Philippe CORNU 2017-07-13 16:25 ` [PATCH v5 14/14] drm: remove unused and redundant callbacks Peter Rosin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=2b3b4bbc-df98-ac01-1740-294f73acf8fc@st.com \ --to=philippe.cornu@st.com \ --cc=airlied@linux.ie \ --cc=benjamin.gaignard@linaro.org \ --cc=boris.brezillon@free-electrons.com \ --cc=daniel.vetter@intel.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=jani.nikula@linux.intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=lionel.g.landwerlin@intel.com \ --cc=peda@axentia.se \ --cc=seanpaul@chromium.org \ --cc=vincent.abriou@st.com \ --cc=yannick.fertre@st.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.