lkml.org 
[lkml]   [2005]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] configurable debug info from radeonfb old driver
Michael Iatrou <m.iatrou@freemail.gr> wrote:
>
> Hi,
>
> Currently, radeonfb old driver always prints debugging informations. This
> patch makes debug info reporting configurable.
>
>
> diff -urN linux-2.6.13-rc6/drivers/video/Kconfig linux-2.6.13-rc6.new/drivers/video/Kconfig
> --- linux-2.6.13-rc6/drivers/video/Kconfig 2005-08-14 00:48:34.000000000 +0300
> +++ linux-2.6.13-rc6.new/drivers/video/Kconfig 2005-08-14 00:54:10.000000000 +0300
> @@ -936,6 +936,15 @@
> There is a product page at
> <http://www.ati.com/na/pages/products/pc/radeon32/index.html>.
>
> +config FB_RADEON_OLD_DEBUG
> + bool "Enable debug output from Old Radeon driver"
> + depends on FB_RADEON_OLD
> + default n
> + help
> + Say Y here if you want the Radeon driver to output all sorts
> + of debugging informations to provide to the maintainer when
> + something goes wrong.
> +
> config FB_RADEON
> tristate "ATI Radeon display support"
> depends on FB && PCI
> diff -urN linux-2.6.13-rc6/drivers/video/radeonfb.c linux-2.6.13-rc6.new/drivers/video/radeonfb.c
> --- linux-2.6.13-rc6/drivers/video/radeonfb.c 2005-06-19 14:49:29.000000000 +0300
> +++ linux-2.6.13-rc6.new/drivers/video/radeonfb.c 2005-08-14 00:55:16.000000000 +0300
> @@ -80,7 +80,11 @@
> #include <video/radeon.h>
> #include <linux/radeonfb.h>
>
> -#define DEBUG 1
> +#ifdef CONFIG_FB_RADEON_OLD_DEBUG
> +#define DEBUG 1
> +#else
> +#define DEBUG 0
> +#endif
>
> #if DEBUG
> #define RTRACE printk

That's probably a bit fancier than we really need. How about we just set
DEBUG to zero?

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-08-14 10:32    [W:0.037 / U:0.044 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site