svn commit: r234238 - in stable/9/sys: conf modules/mps

Dimitry Andric dim at FreeBSD.org
Fri Apr 13 21:47:15 UTC 2012


Author: dim
Date: Fri Apr 13 21:47:14 2012
New Revision: 234238
URL: http://svn.freebsd.org/changeset/base/234238

Log:
  MFC r233354:
  
  Work around the following clang warning in mps(4):
  
    sys/dev/mps/mps_sas.c:861:1: error: function 'mpssas_discovery_timeout' is not needed and will not be emitted [-Werror,-Wunneeded-internal-declaration]
    mpssas_discovery_timeout(void *data)
    ^
  
  Because the driver is obtained from upstream, we don't want to modify
  it; just silence the warning instead, it is harmless.

Modified:
  stable/9/sys/conf/files
  stable/9/sys/conf/kern.mk
  stable/9/sys/modules/mps/Makefile
Directory Properties:
  stable/9/sys/   (props changed)
  stable/9/sys/conf/   (props changed)

Modified: stable/9/sys/conf/files
==============================================================================
--- stable/9/sys/conf/files	Fri Apr 13 21:35:24 2012	(r234237)
+++ stable/9/sys/conf/files	Fri Apr 13 21:47:14 2012	(r234238)
@@ -1464,7 +1464,8 @@ dev/mps/mps.c			optional mps
 dev/mps/mps_config.c		optional mps
 dev/mps/mps_mapping.c		optional mps
 dev/mps/mps_pci.c		optional mps pci
-dev/mps/mps_sas.c		optional mps
+dev/mps/mps_sas.c		optional mps \
+	compile-with "${NORMAL_C} ${NO_WUNNEEDED_INTERNAL_DECL}"
 dev/mps/mps_sas_lsi.c		optional mps
 dev/mps/mps_table.c		optional mps
 dev/mps/mps_user.c		optional mps

Modified: stable/9/sys/conf/kern.mk
==============================================================================
--- stable/9/sys/conf/kern.mk	Fri Apr 13 21:35:24 2012	(r234237)
+++ stable/9/sys/conf/kern.mk	Fri Apr 13 21:47:14 2012	(r234238)
@@ -23,6 +23,7 @@ NO_WSHIFT_COUNT_OVERFLOW=	-Wno-shift-cou
 NO_WUNUSED_VALUE=		-Wno-unused-value
 NO_WSELF_ASSIGN=		-Wno-self-assign
 NO_WFORMAT_SECURITY=		-Wno-format-security
+NO_WUNNEEDED_INTERNAL_DECL=	-Wno-unneeded-internal-declaration
 # Several other warnings which might be useful in some cases, but not severe
 # enough to error out the whole kernel build.  Display them anyway, so there is
 # some incentive to fix them eventually.

Modified: stable/9/sys/modules/mps/Makefile
==============================================================================
--- stable/9/sys/modules/mps/Makefile	Fri Apr 13 21:35:24 2012	(r234237)
+++ stable/9/sys/modules/mps/Makefile	Fri Apr 13 21:47:14 2012	(r234238)
@@ -12,3 +12,6 @@ SRCS+=	device_if.h bus_if.h pci_if.h
 DEBUG_FLAGS += -g
 
 .include <bsd.kmod.mk>
+
+CWARNFLAGS.mps_sas.c=	${NO_WUNNEEDED_INTERNAL_DECL}
+CWARNFLAGS+=		${CWARNFLAGS.${.IMPSRC:T}}


More information about the svn-src-stable mailing list