[rtems commit] m68k/shared/misc/memProbe.c: Add prototype to eliminate warning

Joel Sherrill joel.sherrill at OARcorp.com
Mon Jun 2 12:45:46 UTC 2014


On 6/2/2014 1:53 AM, Sebastian Huber wrote:
> On 2014-06-02 03:24, Joel Sherrill wrote:
>> On Jun 1, 2014 8:00 PM, Gedare Bloom <gedare at rtems.org> wrote:
>>  >
>>  > On Sun, Jun 1, 2014 at 7:10 PM, Joel Sherrill <joel at rtems.org> wrote:
>>  > > Module:    rtems
>>  > > Branch:    master
>>  > > Commit:    6ddc4dab2085e9db28b2470f05aaa54652d68a84
>>  > > Changeset:
>> http://git.rtems.org/rtems/commit/?id=6ddc4dab2085e9db28b2470f05aaa54652d68a84
>>  > >
>>  > > Author:    Joel Sherrill <joel.sherrill at oarcorp.com>
>>  > > Date:      Sun Jun  1 18:18:00 2014 -0500
>>  > >
>>  > > m68k/shared/misc/memProbe.c: Add prototype to eliminate warning
>>  > >
>>  > > ---
>>  > >
>>  > >  c/src/lib/libcpu/m68k/shared/misc/memProbe.c |    2 ++
>>  > >  1 files changed, 2 insertions(+), 0 deletions(-)
>>  > >
>>  > > diff --git a/c/src/lib/libcpu/m68k/shared/misc/memProbe.c
>> b/c/src/lib/libcpu/m68k/shared/misc/memProbe.c
>>  > > index 5c52911..d352f76 100644
>>  > > --- a/c/src/lib/libcpu/m68k/shared/misc/memProbe.c
>>  > > +++ b/c/src/lib/libcpu/m68k/shared/misc/memProbe.c
>>  > > @@ -4,6 +4,7 @@
>>  > >
>>  > >  #include <bsp.h>
>>  > >  #include <string.h>
>>  > > +#include <rtems/m68k/sim.h>
>>  > >
>>  > >  #if (M68K_COLDFIRE_ARCH == 1)
>>  > >  # define EXCEPTION_FRAME_PC_OFFSET "4"
>>  > > @@ -16,6 +17,7 @@ int memProbeByte(void *from, void *to);
>>  > >  int memProbeShort(void *from, void *to);
>>  > >  int memProbeLong(void *from, void *to);
>>  > >  int memProbeCatcher(void);
>>  > > +rtems_status_code bspExtMemProbe(void *addr, int write, int size, void
>> *pval);
>>  > >
>>  > We should generally prefer to declare functions in a header file, or
>>  > make them static if not exported.
>>
>> Got a suggestion? Did I miss an existing prototype?
>>
> I also think that adding prototypes to source files to silence this warning is 
> not the right thing and makes it worse.
>
> These functions seem to be unused.  So why not delete this file?
>
Are they used by EPICS?

-- 
Joel Sherrill, Ph.D.             Director of Research & Development
joel.sherrill at OARcorp.com        On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
Support Available                (256) 722-9985




More information about the devel mailing list