diff options
author | Dave Airlie <airlied@redhat.com> | 2012-01-05 04:55:22 -0500 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-01-06 05:01:35 -0500 |
commit | cc1f71942944890c7e05fc55dc4427c94b63d4f1 (patch) | |
tree | 2e9e1b347adb6e4457a02da6e5c2fcbdda851a28 /drivers/gpu/drm | |
parent | 3df96909b75835d487a9178761622b0cbd7310d4 (diff) |
drm: introduce drm_can_sleep and use in intel/radeon drivers. (v2)
So we have a few places where the drm drivers would like to sleep to
be nice to the system, mainly in the modesetting paths, but we also
have two cases were atomic modesetting must take place, panic writing
and kernel debugger. So provide a central inline to determine if a
sleep or delay should be used and use this in the intel and radeon drivers.
v2: drop intel_drv.h MSLEEP macro, nobody uses it.
Based on patch from Michel Dänzer <michel.daenzer@amd.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43941
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm')
-rw-r--r-- | drivers/gpu/drm/i915/intel_drv.h | 9 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/atom.c | 2 |
2 files changed, 3 insertions, 8 deletions
diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h index 5ac8a164a1ee..1348705faf6b 100644 --- a/drivers/gpu/drm/i915/intel_drv.h +++ b/drivers/gpu/drm/i915/intel_drv.h | |||
@@ -40,7 +40,7 @@ | |||
40 | ret__ = -ETIMEDOUT; \ | 40 | ret__ = -ETIMEDOUT; \ |
41 | break; \ | 41 | break; \ |
42 | } \ | 42 | } \ |
43 | if (W && !(in_atomic() || in_dbg_master())) msleep(W); \ | 43 | if (W && drm_can_sleep()) msleep(W); \ |
44 | } \ | 44 | } \ |
45 | ret__; \ | 45 | ret__; \ |
46 | }) | 46 | }) |
@@ -48,13 +48,6 @@ | |||
48 | #define wait_for(COND, MS) _wait_for(COND, MS, 1) | 48 | #define wait_for(COND, MS) _wait_for(COND, MS, 1) |
49 | #define wait_for_atomic(COND, MS) _wait_for(COND, MS, 0) | 49 | #define wait_for_atomic(COND, MS) _wait_for(COND, MS, 0) |
50 | 50 | ||
51 | #define MSLEEP(x) do { \ | ||
52 | if (in_dbg_master()) \ | ||
53 | mdelay(x); \ | ||
54 | else \ | ||
55 | msleep(x); \ | ||
56 | } while (0) | ||
57 | |||
58 | #define KHz(x) (1000*x) | 51 | #define KHz(x) (1000*x) |
59 | #define MHz(x) KHz(1000*x) | 52 | #define MHz(x) KHz(1000*x) |
60 | 53 | ||
diff --git a/drivers/gpu/drm/radeon/atom.c b/drivers/gpu/drm/radeon/atom.c index 14cc88aaf3a7..d1bd239cd9e9 100644 --- a/drivers/gpu/drm/radeon/atom.c +++ b/drivers/gpu/drm/radeon/atom.c | |||
@@ -665,6 +665,8 @@ static void atom_op_delay(atom_exec_context *ctx, int *ptr, int arg) | |||
665 | SDEBUG(" count: %d\n", count); | 665 | SDEBUG(" count: %d\n", count); |
666 | if (arg == ATOM_UNIT_MICROSEC) | 666 | if (arg == ATOM_UNIT_MICROSEC) |
667 | udelay(count); | 667 | udelay(count); |
668 | else if (!drm_can_sleep()) | ||
669 | mdelay(count); | ||
668 | else | 670 | else |
669 | msleep(count); | 671 | msleep(count); |
670 | } | 672 | } |