[Intel-gfx] [PATCH] drm/i915: Make driver less chatty

[yakui]

On Tue, 2009-06-30 at 14:38 +0800, Keith Packard wrote:
> On Tue, 2009-06-30 at 09:33 +0800, yakui_zhao wrote:
> 
> > >         DRM_DEBUG(fmt, args...)
> > >         DRM_DEBUG_DRIVER(prefix, fmt, args...)
> > >         DRM_DEBUG_KMS(prefix, fmt, args...)
> > >         DRM_DEBUG_MODE(prefix, fmt, args...)
> > > but that should probably be done as a separate patch.
> > Agree with what Jesse said.
> >     Will you please use the macro definition mentioned by Jesse so that
> > we can get the different drm debug info.
> >     For example:
> >          The DRM_DEBUG_DRIVER can be used in the file of
> > i915_dma/i915_drv.c
> >          The DRM_DEBUG_KMS can be used in the file related with KMS.
> 
> I'd like to see documentation added to the DRM header file that explains
> when each of these should be used -- in particular, what's the
> difference between DRM_DEBUG_KMS and DRM_DEBUG_MODE?
The DRM_DEBUG_MODE can be used in the code related with generic drm
mode. For example: the file of drm_mode/drm_crtc_helper 
The DRM_DEBUG_KMS macro definition can be used in the file that is
related with the output device in KMS mode. For example:
intel_lvds/intel_sdvo/intel_tv/intel_hdmi.

Thanks.
  Yakui
>