* Re: [PATCH] drm/i915: Remove obsolete code from intelfb_alloc()
[not found] <1450536645_2420@s16502780>
@ 2015-12-19 15:02 ` Chris Wilson
2015-12-21 16:03 ` Daniel Vetter
0 siblings, 1 reply; 3+ messages in thread
From: Chris Wilson @ 2015-12-19 15:02 UTC (permalink / raw)
To: Lukas Wunner; +Cc: Daniel Vetter, intel-gfx
On Sat, Dec 19, 2015 at 03:40:39PM +0100, Lukas Wunner wrote:
> Clean up after 0c82312f3f15 ("drm/i915: Pin the ifbdev for the
> info->system_base GGTT mmapping"):
>
> At each of the remaining "goto out" in intelfb_alloc(), fb can only be
> either an ERR_PTR or NULL, so the call to drm_framebuffer_unreference()
> is now obsolete.
>
> Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Lukas Wunner <lukas@wunner.de>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] drm/i915: Remove obsolete code from intelfb_alloc()
2015-12-19 15:02 ` [PATCH] drm/i915: Remove obsolete code from intelfb_alloc() Chris Wilson
@ 2015-12-21 16:03 ` Daniel Vetter
0 siblings, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2015-12-21 16:03 UTC (permalink / raw)
To: Chris Wilson, Lukas Wunner, intel-gfx, Daniel Vetter
On Sat, Dec 19, 2015 at 03:02:49PM +0000, Chris Wilson wrote:
> On Sat, Dec 19, 2015 at 03:40:39PM +0100, Lukas Wunner wrote:
> > Clean up after 0c82312f3f15 ("drm/i915: Pin the ifbdev for the
> > info->system_base GGTT mmapping"):
> >
> > At each of the remaining "goto out" in intelfb_alloc(), fb can only be
> > either an ERR_PTR or NULL, so the call to drm_framebuffer_unreference()
> > is now obsolete.
> >
> > Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > Signed-off-by: Lukas Wunner <lukas@wunner.de>
> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Queued for -next, thanks for the patch.
-Daniel
> -Chris
>
> --
> Chris Wilson, Intel Open Source Technology Centre
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] drm/i915: Remove obsolete code from intelfb_alloc()
@ 2015-12-19 14:40 Lukas Wunner
0 siblings, 0 replies; 3+ messages in thread
From: Lukas Wunner @ 2015-12-19 14:40 UTC (permalink / raw)
To: intel-gfx; +Cc: Daniel Vetter
Clean up after 0c82312f3f15 ("drm/i915: Pin the ifbdev for the
info->system_base GGTT mmapping"):
At each of the remaining "goto out" in intelfb_alloc(), fb can only be
either an ERR_PTR or NULL, so the call to drm_framebuffer_unreference()
is now obsolete.
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Lukas Wunner <lukas@wunner.de>
---
drivers/gpu/drm/i915/intel_fbdev.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_fbdev.c b/drivers/gpu/drm/i915/intel_fbdev.c
index bea75ca..09840f4 100644
--- a/drivers/gpu/drm/i915/intel_fbdev.c
+++ b/drivers/gpu/drm/i915/intel_fbdev.c
@@ -119,7 +119,7 @@ static int intelfb_alloc(struct drm_fb_helper *helper,
{
struct intel_fbdev *ifbdev =
container_of(helper, struct intel_fbdev, helper);
- struct drm_framebuffer *fb = NULL;
+ struct drm_framebuffer *fb;
struct drm_device *dev = helper->dev;
struct drm_i915_private *dev_priv = to_i915(dev);
struct drm_mode_fb_cmd2 mode_cmd = {};
@@ -171,8 +171,6 @@ static int intelfb_alloc(struct drm_fb_helper *helper,
out:
mutex_unlock(&dev->struct_mutex);
- if (!IS_ERR_OR_NULL(fb))
- drm_framebuffer_unreference(fb);
return ret;
}
--
1.8.5.2 (Apple Git-48)
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-12-21 16:03 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1450536645_2420@s16502780>
2015-12-19 15:02 ` [PATCH] drm/i915: Remove obsolete code from intelfb_alloc() Chris Wilson
2015-12-21 16:03 ` Daniel Vetter
2015-12-19 14:40 Lukas Wunner
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.