[PATCH 04/52] gdbv850sim/console/console-io.c: Fix warnings
Joel Sherrill
joel.sherrill at oarcorp.com
Wed Oct 8 21:35:28 UTC 2014
All of the BSPs using console-polled.c will be addressed by a single
patch in the next revision. I added <bsp/console-polled.h>, included
it from the appropriate driver, and added static to the printk()
support method in all 14 BSPs. That should clobber a few more
warnings.
--joel
On 10/8/2014 3:26 PM, Gedare Bloom wrote:
> On Wed, Oct 8, 2014 at 4:20 PM, Joel Sherrill <joel.sherrill at oarcorp.com> wrote:
>> On 10/8/2014 3:10 PM, Gedare Bloom wrote:
>>> On Wed, Oct 8, 2014 at 3:15 PM, Joel Sherrill <joel.sherrill at oarcorp.com> wrote:
>>>> ---
>>>> c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c | 9 ++++++++-
>>>> 1 file changed, 8 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c b/c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c
>>>> index c178197..a66c800 100644
>>>> --- a/c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c
>>>> +++ b/c/src/lib/libbsp/v850/gdbv850sim/console/console-io.c
>>>> @@ -12,6 +12,14 @@
>>>> #include <bsp/syscall.h>
>>>>
>>>> /*
>>>> + * Prototypes
>>>> + */
>>>> +void console_initialize_hardware(void);
>>>> +void console_outbyte_polled(int port, char ch);
>>>> +int console_inbyte_nonblocking(int port);
>>>> +void console_output_char(char c);
>>>> +
>>> These prototype additions should go to a header file, or the functions
>>> should be made static.
>> The first three are the assumed interface for the single port
>> polled console driver framework. They can't be static.
>>
>> c/src/lib/libbsp/shared/console-polled.c
>>
>> Any suggestions for a new header file to add them to?
>> I suppose we could add console-polled.h for private use?
> Sure.
>
>> FWIW Adding this will impact 14 BSPs and likely fix 3*14 warnings.
>> But only one other patch (0013-gdbamsim).
>>
>> The fourth should be able to be static. I should have caught that.
>>>> +/*
>>>> * console_initialize_hardware
>>>> *
>>>> * This routine initializes the console hardware.
>>>> @@ -38,7 +46,6 @@ void console_outbyte_polled(
>>>> *
>>>> * This routine polls for a character.
>>>> */
>>>> -
>>>> int console_inbyte_nonblocking(
>>>> int port
>>>> )
>>>> --
>>>> 1.9.3
>>>>
>>>> _______________________________________________
>>>> devel mailing list
>>>> devel at rtems.org
>>>> http://lists.rtems.org/mailman/listinfo/devel
>> --
>> 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
>>
>>
>> _______________________________________________
>> devel mailing list
>> devel at rtems.org
>> http://lists.rtems.org/mailman/listinfo/devel
--
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