* [PATCH 24/81] drm/fb helper: don't call drm_crtc_helper_set_config [not found] <1342016944-23395-1-git-send-email-daniel.vetter@ffwll.ch> @ 2012-07-11 14:28 ` Daniel Vetter 2012-07-18 15:49 ` Daniel Vetter 2012-07-11 14:28 ` [PATCH 25/81] drm: remove the list_head from drm_mode_set Daniel Vetter 1 sibling, 1 reply; 6+ messages in thread From: Daniel Vetter @ 2012-07-11 14:28 UTC (permalink / raw) To: Intel Graphics Development; +Cc: Daniel Vetter, dri-devel Go through the interface vtable instead, because not everyone might be using the crtc helper code. Cc: dri-devel@lists.freedesktop.org Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch> --- drivers/gpu/drm/drm_fb_helper.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c index 5683b7f..bf97c0a 100644 --- a/drivers/gpu/drm/drm_fb_helper.c +++ b/drivers/gpu/drm/drm_fb_helper.c @@ -228,7 +228,7 @@ bool drm_fb_helper_restore_fbdev_mode(struct drm_fb_helper *fb_helper) int i, ret; for (i = 0; i < fb_helper->crtc_count; i++) { struct drm_mode_set *mode_set = &fb_helper->crtc_info[i].mode_set; - ret = drm_crtc_helper_set_config(mode_set); + ret = mode_set->crtc->funcs->set_config(mode_set); if (ret) error = true; } -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 24/81] drm/fb helper: don't call drm_crtc_helper_set_config 2012-07-11 14:28 ` [PATCH 24/81] drm/fb helper: don't call drm_crtc_helper_set_config Daniel Vetter @ 2012-07-18 15:49 ` Daniel Vetter 2012-07-18 15:50 ` Daniel Vetter 0 siblings, 1 reply; 6+ messages in thread From: Daniel Vetter @ 2012-07-18 15:49 UTC (permalink / raw) To: Intel Graphics Development; +Cc: Daniel Vetter, dri-devel Hi Dave, Can you please pick this one up for -next? There are a few other issues where the fb helper is a bit too intimate with the crtc helper (i.e. it depends upon callbacks and behaviour only provided when using the crtc helpers). But this one here is the only one I can't work around in i915.ko and hence would block merging the modeset rewrite for 3.7. I've put a todo down to untangle the fb helper a bit better from the crtc helper, but that's for a later patch series. Yours, Daneil PS: Can you also pick up the other drm patch in this series? That unused list_head confused me quite a bit until I've noticed that it's ... unused. On Wed, Jul 11, 2012 at 04:28:07PM +0200, Daniel Vetter wrote: > Go through the interface vtable instead, because not everyone might be > using the crtc helper code. > > Cc: dri-devel@lists.freedesktop.org > Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch> > --- > drivers/gpu/drm/drm_fb_helper.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c > index 5683b7f..bf97c0a 100644 > --- a/drivers/gpu/drm/drm_fb_helper.c > +++ b/drivers/gpu/drm/drm_fb_helper.c > @@ -228,7 +228,7 @@ bool drm_fb_helper_restore_fbdev_mode(struct drm_fb_helper *fb_helper) > int i, ret; > for (i = 0; i < fb_helper->crtc_count; i++) { > struct drm_mode_set *mode_set = &fb_helper->crtc_info[i].mode_set; > - ret = drm_crtc_helper_set_config(mode_set); > + ret = mode_set->crtc->funcs->set_config(mode_set); > if (ret) > error = true; > } > -- > 1.7.7.6 > -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 24/81] drm/fb helper: don't call drm_crtc_helper_set_config 2012-07-18 15:49 ` Daniel Vetter @ 2012-07-18 15:50 ` Daniel Vetter 2012-07-20 1:17 ` Dave Airlie 0 siblings, 1 reply; 6+ messages in thread From: Daniel Vetter @ 2012-07-18 15:50 UTC (permalink / raw) To: Intel Graphics Development, Dave Airlie; +Cc: Daniel Vetter, dri-devel Actually including Dave on the recipient list might work bettter ;-) On Wed, Jul 18, 2012 at 05:49:22PM +0200, Daniel Vetter wrote: > Hi Dave, > > Can you please pick this one up for -next? There are a few other issues > where the fb helper is a bit too intimate with the crtc helper (i.e. it > depends upon callbacks and behaviour only provided when using the crtc > helpers). But this one here is the only one I can't work around in i915.ko > and hence would block merging the modeset rewrite for 3.7. > > I've put a todo down to untangle the fb helper a bit better from the crtc > helper, but that's for a later patch series. > > Yours, Daneil > > PS: Can you also pick up the other drm patch in this series? That unused > list_head confused me quite a bit until I've noticed that it's ... unused. > > On Wed, Jul 11, 2012 at 04:28:07PM +0200, Daniel Vetter wrote: > > Go through the interface vtable instead, because not everyone might be > > using the crtc helper code. > > > > Cc: dri-devel@lists.freedesktop.org > > Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch> > > --- > > drivers/gpu/drm/drm_fb_helper.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c > > index 5683b7f..bf97c0a 100644 > > --- a/drivers/gpu/drm/drm_fb_helper.c > > +++ b/drivers/gpu/drm/drm_fb_helper.c > > @@ -228,7 +228,7 @@ bool drm_fb_helper_restore_fbdev_mode(struct drm_fb_helper *fb_helper) > > int i, ret; > > for (i = 0; i < fb_helper->crtc_count; i++) { > > struct drm_mode_set *mode_set = &fb_helper->crtc_info[i].mode_set; > > - ret = drm_crtc_helper_set_config(mode_set); > > + ret = mode_set->crtc->funcs->set_config(mode_set); > > if (ret) > > error = true; > > } > > -- > > 1.7.7.6 > > > > -- > Daniel Vetter > Mail: daniel@ffwll.ch > Mobile: +41 (0)79 365 57 48 -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 24/81] drm/fb helper: don't call drm_crtc_helper_set_config 2012-07-18 15:50 ` Daniel Vetter @ 2012-07-20 1:17 ` Dave Airlie 0 siblings, 0 replies; 6+ messages in thread From: Dave Airlie @ 2012-07-20 1:17 UTC (permalink / raw) To: Daniel Vetter; +Cc: Daniel Vetter, Intel Graphics Development, dri-devel On Thu, Jul 19, 2012 at 1:50 AM, Daniel Vetter <daniel@ffwll.ch> wrote: > Actually including Dave on the recipient list might work bettter ;-) > > On Wed, Jul 18, 2012 at 05:49:22PM +0200, Daniel Vetter wrote: >> Hi Dave, >> >> Can you please pick this one up for -next? There are a few other issues >> where the fb helper is a bit too intimate with the crtc helper (i.e. it >> depends upon callbacks and behaviour only provided when using the crtc >> helpers). But this one here is the only one I can't work around in i915.ko >> and hence would block merging the modeset rewrite for 3.7. >> >> I've put a todo down to untangle the fb helper a bit better from the crtc >> helper, but that's for a later patch series. >> >> Yours, Daneil >> >> PS: Can you also pick up the other drm patch in this series? That unused >> list_head confused me quite a bit until I've noticed that it's ... unused. >> Merged. Thanks. Dave. ^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 25/81] drm: remove the list_head from drm_mode_set [not found] <1342016944-23395-1-git-send-email-daniel.vetter@ffwll.ch> 2012-07-11 14:28 ` [PATCH 24/81] drm/fb helper: don't call drm_crtc_helper_set_config Daniel Vetter @ 2012-07-11 14:28 ` Daniel Vetter 2012-07-20 1:23 ` Dave Airlie 1 sibling, 1 reply; 6+ messages in thread From: Daniel Vetter @ 2012-07-11 14:28 UTC (permalink / raw) To: Intel Graphics Development; +Cc: Daniel Vetter, dri-devel It's unused. At it confused me quite a bit until I've discovered that. Cc: dri-devel@lists.freedesktop.org Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch> --- include/drm/drm_crtc.h | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h index bac55c2..a1a0386 100644 --- a/include/drm/drm_crtc.h +++ b/include/drm/drm_crtc.h @@ -676,8 +676,6 @@ struct drm_plane { * This is used to set modes. */ struct drm_mode_set { - struct list_head head; - struct drm_framebuffer *fb; struct drm_crtc *crtc; struct drm_display_mode *mode; -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 25/81] drm: remove the list_head from drm_mode_set 2012-07-11 14:28 ` [PATCH 25/81] drm: remove the list_head from drm_mode_set Daniel Vetter @ 2012-07-20 1:23 ` Dave Airlie 0 siblings, 0 replies; 6+ messages in thread From: Dave Airlie @ 2012-07-20 1:23 UTC (permalink / raw) To: Daniel Vetter; +Cc: Intel Graphics Development, dri-devel On Thu, Jul 12, 2012 at 12:28 AM, Daniel Vetter <daniel.vetter@ffwll.ch> wrote: > It's unused. At it confused me quite a bit until I've discovered that. Applied thanks, Dave. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-07-20 1:23 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1342016944-23395-1-git-send-email-daniel.vetter@ffwll.ch>
2012-07-11 14:28 ` [PATCH 24/81] drm/fb helper: don't call drm_crtc_helper_set_config Daniel Vetter
2012-07-18 15:49 ` Daniel Vetter
2012-07-18 15:50 ` Daniel Vetter
2012-07-20 1:17 ` Dave Airlie
2012-07-11 14:28 ` [PATCH 25/81] drm: remove the list_head from drm_mode_set Daniel Vetter
2012-07-20 1:23 ` Dave Airlie
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox