[Nouveau] [PATCH] drm/nouveau: use drm debug levels

Francisco Jerez currojerez at riseup.net
Sun Dec 13 11:56:13 PST 2009


Maarten Maathuis <madman2003 at gmail.com> writes:

> - Use driver level (0x2) for NV_DEBUG instead of all levels
> - Create a NV_DEBUG_KMS for KMS level (04) and use them in modesetting code
> - Remove a few odd NV_TRACE calls and replace with NV_DEBUG_KMS
>
> Signed-off-by: Maarten Maathuis <madman2003 at gmail.com>
> ---
>  drivers/gpu/drm/nouveau/nouveau_bios.c      |   12 +++++-----
>  drivers/gpu/drm/nouveau/nouveau_connector.c |    8 +++---
>  drivers/gpu/drm/nouveau/nouveau_dp.c        |   28 +++++++++++++-------------
>  drivers/gpu/drm/nouveau/nouveau_drv.h       |   14 +++++++++++-
>  drivers/gpu/drm/nouveau/nouveau_fbcon.c     |    2 +-
>  drivers/gpu/drm/nouveau/nv04_crtc.c         |   10 ++++----
>  drivers/gpu/drm/nouveau/nv04_dac.c          |    8 ++----
>  drivers/gpu/drm/nouveau/nv04_dfp.c          |    8 ++++--
>  drivers/gpu/drm/nouveau/nv04_display.c      |    4 +-
>  drivers/gpu/drm/nouveau/nv17_tv.c           |    4 +-
>  drivers/gpu/drm/nouveau/nv50_crtc.c         |   22 ++++++++++----------
>  drivers/gpu/drm/nouveau/nv50_cursor.c       |    8 +++---
>  drivers/gpu/drm/nouveau/nv50_dac.c          |   20 +++++++++---------
>  drivers/gpu/drm/nouveau/nv50_display.c      |   20 +++++++++---------
>  drivers/gpu/drm/nouveau/nv50_sor.c          |   16 +++++++-------
>  15 files changed, 97 insertions(+), 87 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nouveau_bios.c b/drivers/gpu/drm/nouveau/nouveau_bios.c
> index 5eec5ed..62fadbb 100644
>
> [...]
>
> diff --git a/drivers/gpu/drm/nouveau/nv04_dac.c b/drivers/gpu/drm/nouveau/nv04_dac.c
> index a5fa517..e6b690e 100644
> --- a/drivers/gpu/drm/nouveau/nv04_dac.c
> +++ b/drivers/gpu/drm/nouveau/nv04_dac.c
> @@ -205,7 +205,7 @@ out:
>  	NVWriteVgaSeq(dev, 0, NV_VIO_SR_CLOCK_INDEX, saved_seq1);
>  
>  	if (blue == 0x18) {
> -		NV_TRACE(dev, "Load detected on head A\n");
> +		NV_DEBUG_KMS(dev, "Load detected on head A\n");
>  		return connector_status_connected;
>  	}

This is a bit inconsistent with the other "Load detected" messages
below (in nv17_dac_detect), which are NV_INFO.

>  
> @@ -350,13 +350,11 @@ static void nv04_dac_mode_set(struct drm_encoder *encoder,
>  			      struct drm_display_mode *mode,
>  			      struct drm_display_mode *adjusted_mode)
>  {
> -	struct nouveau_encoder *nv_encoder = nouveau_encoder(encoder);
>  	struct drm_device *dev = encoder->dev;
>  	struct drm_nouveau_private *dev_priv = dev->dev_private;
>  	int head = nouveau_crtc(encoder->crtc)->index;
>  
> -	NV_TRACE(dev, "%s called for encoder %d\n", __func__,
> -		      nv_encoder->dcb->index);
> +	NV_DEBUG_KMS(dev, "Output mode on CRTC %d:\n", head);
>  

I don't understand what that debugging message means, I guess you didn't
mean to do that?

>
> [...]
>
> diff --git a/drivers/gpu/drm/nouveau/nv17_tv.c b/drivers/gpu/drm/nouveau/nv17_tv.c
> index 46cfd9c..7c4d7c0 100644
> --- a/drivers/gpu/drm/nouveau/nv17_tv.c
> +++ b/drivers/gpu/drm/nouveau/nv17_tv.c
> @@ -219,7 +219,7 @@ static void  nv17_tv_dpms(struct drm_encoder *encoder, int mode)
>  		return;
>  	nouveau_encoder(encoder)->last_dpms = mode;
>  
> -	NV_TRACE(dev, "Setting dpms mode %d on TV encoder (output %d)\n",
> +	NV_DEBUG_KMS(dev, "Setting dpms mode %d on TV encoder (output %d)\n",
>  		 mode, nouveau_encoder(encoder)->dcb->index);
>  
>  	regs->ptv_200 &= ~1;

This is inconsistent too with the DPMS messages in nv04_dfp/dac/tv.c,
which are NV_INFO (I'm OK either way but IMHO it would be nice to keep
all the DPMS messages at the same debug level).

>
> [...]
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/nouveau/attachments/20091213/657d7bef/attachment.pgp 


More information about the Nouveau mailing list