diff options
| author |   Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-04-07 23:25:20 +0200 |
|---|---|---|
| committer |   Jani Nikula <jani.nikula@intel.com> | 2014-04-10 10:54:47 +0300 |
| commit | 004f388559a89ac17fa4c81fb8559a6599050a4e (patch) | |
| tree | 54ba44586dbcb970145d6f521748abac2f209bff /drivers/gpu/drm/drm_mm.c | |
| parent | DRM: armada: fix corruption while loading cursors (diff) | |
| download | linux-dev-004f388559a89ac17fa4c81fb8559a6599050a4e.tar.xz linux-dev-004f388559a89ac17fa4c81fb8559a6599050a4e.zip | |
drm/mm: Don't WARN if drm_mm_reserve_node
Jesse's BIOS fb reconstruction code actually relies on the -ENOSPC
return value to detect overlapping framebuffers (which the bios uses
always when lighting up more than one screen). All this fanciness
happens in intel_alloc_plane_obj in intel_display.c.
Since no one else uses this we can safely remove the WARN without
repercussions.
Reported-by: Ben Widawsky <benjamin.widawsky@intel.com>
Tested-by: Ben Widawsky <ben@bwidawsk.net>
Cc: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: Dave Airlie <airlied@redhat.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Acked-by: Dave Airlie <airlied@redhat.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Diffstat (limited to 'drivers/gpu/drm/drm_mm.c')
| -rw-r--r-- | drivers/gpu/drm/drm_mm.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/gpu/drm/drm_mm.c b/drivers/gpu/drm/drm_mm.c index 71e2d3fcd6ee..04a209e2b66d 100644 --- a/drivers/gpu/drm/drm_mm.c +++ b/drivers/gpu/drm/drm_mm.c @@ -207,8 +207,6 @@ int drm_mm_reserve_node(struct drm_mm *mm, struct drm_mm_node *node) return 0; } - WARN(1, "no hole found for node 0x%lx + 0x%lx\n", - node->start, node->size); return -ENOSPC; } EXPORT_SYMBOL(drm_mm_reserve_node); |