From f2a0c5438dc83171de1007a68e4f98e35b5a8fbe Mon Sep 17 00:00:00 2001 From: Leif Delgass Date: Sat, 26 Apr 2003 22:28:56 +0000 Subject: Ensure driver has been initialized (dev_private != NULL) before installing irq handler in DRM(irq_install). Modify all drivers to ensure irq handler is removed before cleanup and cleanup is called at takedown. Remove unused buffer private struct fields in i810, i830. Check for lock on init/cleanup in all drivers except i810/i830. The current DDX for i810 and i830 doesn't hold the lock on kernel init (FIXME?). --- shared-core/r128_irq.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'shared-core/r128_irq.c') diff --git a/shared-core/r128_irq.c b/shared-core/r128_irq.c index bfc30405..f3f84d8c 100644 --- a/shared-core/r128_irq.c +++ b/shared-core/r128_irq.c @@ -93,8 +93,9 @@ void r128_driver_irq_postinstall( drm_device_t *dev ) { void r128_driver_irq_uninstall( drm_device_t *dev ) { drm_r128_private_t *dev_priv = (drm_r128_private_t *)dev->dev_private; - if ( dev_priv ) { - /* Disable *all* interrupts */ - R128_WRITE( R128_GEN_INT_CNTL, 0 ); - } + if (!dev_priv) + return; + + /* Disable *all* interrupts */ + R128_WRITE( R128_GEN_INT_CNTL, 0 ); } -- cgit v1.2.3