Re: sys/dev/amr build error with Clang

2012-08-31 Thread John Baldwin
On Friday, August 31, 2012 5:13:01 am deeptec...@gmail.com wrote: > Dimitry Andric wrote: > > The one call to get the callout to amr_periodic() started seems to have > > been commented out in r239912: > > > > http://svnweb.freebsd.org/base/head/sys/dev/amr/amr.c?r1=239912&r2=239911&pathrev=2399

Re: sys/dev/amr build error with Clang

2012-08-31 Thread Scott Long
On Aug 31, 2012, at 2:43 AM, Dimitry Andric wrote: > On 2012-08-31 09:34, deeptec...@gmail.com wrote: >> /usr/src/sys/modules/amr/../../dev/amr/amr.c:970:1: error: function >> 'amr_periodic' is not needed and will not be emitted >> [-Werror,-Wunneeded-internal-declaration] > > The o

Re: sys/dev/amr build error with Clang

2012-08-31 Thread deeptech71
Dimitry Andric wrote: The one call to get the callout to amr_periodic() started seems to have been commented out in r239912: http://svnweb.freebsd.org/base/head/sys/dev/amr/amr.c?r1=239912&r2=239911&pathrev=239912 If the function isn't necessary anymore, it could just be deleted, or #ifdef'

Re: sys/dev/amr build error with Clang

2012-08-31 Thread Dimitry Andric
On 2012-08-31 09:34, deeptec...@gmail.com wrote: /usr/src/sys/modules/amr/../../dev/amr/amr.c:970:1: error: function 'amr_periodic' is not needed and will not be emitted [-Werror,-Wunneeded-internal-declaration] The one call to get the callout to amr_periodic() started seems to

sys/dev/amr build error with Clang

2012-08-31 Thread deeptech71
Hey, Baldie: /usr/src/sys/modules/amr/../../dev/amr/amr.c:970:1: error: function 'amr_periodic' is not needed and will not be emitted [-Werror,-Wunneeded-internal-declaration] amr_periodic(void *data) ^ 1 error generated. *** [amr.o] Error code 1 Stop in /usr/src/sys/modules/amr. ___