[PATCH:xf86-video-mach64 1/2] Remove unused atidecoder.c & atidecoder.h
Defined a enum & string array, neither of which were used anywhere. Reported by Solaris linker -z guidance option: ld: guidance: removal of unused file recommended: atidecoder.o Signed-off-by: Alan Coopersmith --- src/Makefile.am |3 +-- src/atidecoder.c | 50 -- src/atidecoder.h | 51 --- 3 files changed, 1 insertion(+), 103 deletions(-) delete mode 100644 src/atidecoder.c delete mode 100644 src/atidecoder.h diff --git a/src/Makefile.am b/src/Makefile.am index 4c1498f..f1a255f 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -50,7 +50,7 @@ mach64_drv_ladir = @moduledir@/drivers mach64_drv_la_SOURCES = \ atibus.c atichip.c atiprobe.c atividmem.c \ atiadjust.c atiaudio.c aticlock.c aticonfig.c aticonsole.c \ - atidac.c atidecoder.c atidsp.c atii2c.c \ + atidac.c atidsp.c atii2c.c \ atilock.c atimach64.c atimach64accel.c atimach64cursor.c \ atimach64i2c.c atimach64io.c atimach64xv.c atimode.c atipreinit.c \ atiprint.c atirgb514.c atiscreen.c atituner.c atiutil.c ativalid.c \ @@ -70,7 +70,6 @@ EXTRA_DIST = \ aticrtc.h \ aticursor.h \ atidac.h \ - atidecoder.h \ atidga.h \ atidri.h \ atidripriv.h \ diff --git a/src/atidecoder.c b/src/atidecoder.c deleted file mode 100644 index 6419fbc..000 --- a/src/atidecoder.c +++ /dev/null @@ -1,50 +0,0 @@ -/* - * Copyright 2003 through 2004 by Marc Aurele La France (TSI @ UQV), t...@xfree86.org - * - * Permission to use, copy, modify, distribute, and sell this software and its - * documentation for any purpose is hereby granted without fee, provided that - * the above copyright notice appear in all copies and that both that copyright - * notice and this permission notice appear in supporting documentation, and - * that the name of Marc Aurele La France not be used in advertising or - * publicity pertaining to distribution of the software without specific, - * written prior permission. Marc Aurele La France makes no representations - * about the suitability of this software for any purpose. It is provided - * "as-is" without express or implied warranty. - * - * MARC AURELE LA FRANCE DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS SOFTWARE, - * INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO - * EVENT SHALL MARC AURELE LA FRANCE BE LIABLE FOR ANY SPECIAL, INDIRECT OR - * CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, - * DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER - * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR - * PERFORMANCE OF THIS SOFTWARE. - */ - -#ifdef HAVE_CONFIG_H -#include "config.h" -#endif - -#include "atidecoder.h" - -/* - * Video decoder definitions. - */ -const char *ATIDecoderNames[] = -{ -"No decoder", -"BrookTree BT819", -"Brooktree BT829", -"Brooktree BT829A", -"Philips SA7111", -"Philips SA7112", -"ATI Rage Theater", -"Unknown type (7)", -"Unknown type (8)", -"Unknown type (9)", -"Unknown type (10)", -"Unknown type (11)", -"Unknown type (12)", -"Unknown type (13)", -"Unknown type (14)", -"Unknown type (15)" -}; diff --git a/src/atidecoder.h b/src/atidecoder.h deleted file mode 100644 index 6ade55b..000 --- a/src/atidecoder.h +++ /dev/null @@ -1,51 +0,0 @@ -/* - * Copyright 2003 by Marc Aurele La France (TSI @ UQV), t...@xfree86.org - * - * Permission to use, copy, modify, distribute, and sell this software and its - * documentation for any purpose is hereby granted without fee, provided that - * the above copyright notice appear in all copies and that both that copyright - * notice and this permission notice appear in supporting documentation, and - * that the name of Marc Aurele La France not be used in advertising or - * publicity pertaining to distribution of the software without specific, - * written prior permission. Marc Aurele La France makes no representations - * about the suitability of this software for any purpose. It is provided - * "as-is" without express or implied warranty. - * - * MARC AURELE LA FRANCE DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS SOFTWARE, - * INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO - * EVENT SHALL MARC AURELE LA FRANCE BE LIABLE FOR ANY SPECIAL, INDIRECT OR - * CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, - * DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER - * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR - * PERFORMANCE OF THIS SOFTWARE. - */ - -#ifndef ___ATIDECODER_H___ -#define ___ATIDECODER_H___ 1 - -/* - * Video decoder definitions. - */ -typedef enum -{ -ATI_DECODER_NONE, -ATI_DECODER_BT819, -ATI_DECODER_BT829, -ATI_DECODER_BT829A, -ATI_DECODER_SA7111, -ATI_DECODER_SA7112, -ATI_DECODER_THEATE
Re: [PATCH:xf86-video-mach64 1/2] Remove unused atidecoder.c & atidecoder.h
On Thu, Dec 20, 2012 at 1:43 AM, Alan Coopersmith wrote: > Defined a enum & string array, neither of which were used anywhere. > > Reported by Solaris linker -z guidance option: > ld: guidance: removal of unused file recommended: atidecoder.o > > Signed-off-by: Alan Coopersmith Reviewed-by: Alex Deucher > --- > src/Makefile.am |3 +-- > src/atidecoder.c | 50 -- > src/atidecoder.h | 51 --- > 3 files changed, 1 insertion(+), 103 deletions(-) > delete mode 100644 src/atidecoder.c > delete mode 100644 src/atidecoder.h > > diff --git a/src/Makefile.am b/src/Makefile.am > index 4c1498f..f1a255f 100644 > --- a/src/Makefile.am > +++ b/src/Makefile.am > @@ -50,7 +50,7 @@ mach64_drv_ladir = @moduledir@/drivers > mach64_drv_la_SOURCES = \ > atibus.c atichip.c atiprobe.c atividmem.c \ > atiadjust.c atiaudio.c aticlock.c aticonfig.c aticonsole.c \ > - atidac.c atidecoder.c atidsp.c atii2c.c \ > + atidac.c atidsp.c atii2c.c \ > atilock.c atimach64.c atimach64accel.c atimach64cursor.c \ > atimach64i2c.c atimach64io.c atimach64xv.c atimode.c atipreinit.c \ > atiprint.c atirgb514.c atiscreen.c atituner.c atiutil.c ativalid.c \ > @@ -70,7 +70,6 @@ EXTRA_DIST = \ > aticrtc.h \ > aticursor.h \ > atidac.h \ > - atidecoder.h \ > atidga.h \ > atidri.h \ > atidripriv.h \ > diff --git a/src/atidecoder.c b/src/atidecoder.c > deleted file mode 100644 > index 6419fbc..000 > --- a/src/atidecoder.c > +++ /dev/null > @@ -1,50 +0,0 @@ > -/* > - * Copyright 2003 through 2004 by Marc Aurele La France (TSI @ UQV), > t...@xfree86.org > - * > - * Permission to use, copy, modify, distribute, and sell this software and > its > - * documentation for any purpose is hereby granted without fee, provided that > - * the above copyright notice appear in all copies and that both that > copyright > - * notice and this permission notice appear in supporting documentation, and > - * that the name of Marc Aurele La France not be used in advertising or > - * publicity pertaining to distribution of the software without specific, > - * written prior permission. Marc Aurele La France makes no representations > - * about the suitability of this software for any purpose. It is provided > - * "as-is" without express or implied warranty. > - * > - * MARC AURELE LA FRANCE DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS > SOFTWARE, > - * INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO > - * EVENT SHALL MARC AURELE LA FRANCE BE LIABLE FOR ANY SPECIAL, INDIRECT OR > - * CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF > USE, > - * DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER > - * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR > - * PERFORMANCE OF THIS SOFTWARE. > - */ > - > -#ifdef HAVE_CONFIG_H > -#include "config.h" > -#endif > - > -#include "atidecoder.h" > - > -/* > - * Video decoder definitions. > - */ > -const char *ATIDecoderNames[] = > -{ > -"No decoder", > -"BrookTree BT819", > -"Brooktree BT829", > -"Brooktree BT829A", > -"Philips SA7111", > -"Philips SA7112", > -"ATI Rage Theater", > -"Unknown type (7)", > -"Unknown type (8)", > -"Unknown type (9)", > -"Unknown type (10)", > -"Unknown type (11)", > -"Unknown type (12)", > -"Unknown type (13)", > -"Unknown type (14)", > -"Unknown type (15)" > -}; > diff --git a/src/atidecoder.h b/src/atidecoder.h > deleted file mode 100644 > index 6ade55b..000 > --- a/src/atidecoder.h > +++ /dev/null > @@ -1,51 +0,0 @@ > -/* > - * Copyright 2003 by Marc Aurele La France (TSI @ UQV), t...@xfree86.org > - * > - * Permission to use, copy, modify, distribute, and sell this software and > its > - * documentation for any purpose is hereby granted without fee, provided that > - * the above copyright notice appear in all copies and that both that > copyright > - * notice and this permission notice appear in supporting documentation, and > - * that the name of Marc Aurele La France not be used in advertising or > - * publicity pertaining to distribution of the software without specific, > - * written prior permission. Marc Aurele La France makes no representations > - * about the suitability of this software for any purpose. It is provided > - * "as-is" without express or implied warranty. > - * > - * MARC AURELE LA FRANCE DISCLAIMS ALL WARRANTIES WITH REGARD TO THIS > SOFTWARE, > - * INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO > - * EVENT SHALL MARC AURELE LA FRANCE BE LIABLE FOR ANY SPECIAL, INDIRECT OR > - * CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF > USE, > - * DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER > - * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION W