lkml.org 
[lkml]   [2005]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    SubjectRe: [PATCH] configurable debug info from radeonfb old driver
    Date
    When the date was Sunday 14 August 2005 11:25, Andrew Morton wrote:

    > 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?

    That's an option too.

    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 12:58:10.000000000 +0300
    @@ -80,7 +80,7 @@
    #include <video/radeon.h>
    #include <linux/radeonfb.h>

    -#define DEBUG 1
    +#define DEBUG 0

    #if DEBUG
    #define RTRACE printk
    --
    Michael Iatrou
    -
    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 12:05    [W:0.026 / U:59.460 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site