dany/u-boot-megous
1
0
Fork 0
mirror of https://xff.cz/git/u-boot/ synced 2025-09-01 16:52:14 +02:00
Code Issues Packages Projects Releases Wiki Activity

common/console.c: discard volatile

Browse Source 
Avoid errors of like

common/console.c: In function ‘console_record_reset’:
common/console.c:615:16: error: passing argument 1 of ‘membuff_purge’
discards ‘volatile’ qualifier from pointer target type
[-Werror=discarded-qualifiers]
  615 |  membuff_purge(&gd->console_out);
      |                ^~~~~~~~~~~~~~~~

by casting to non-volatile.

The volatile property stems from declarations like

arch/arm/include/asm/global_data.h:114:

But there is no need to treat gd->console_out and gd->console_in as
volatile in the context of common/console.c.

Fixes: b612312816 ("console: Add a function to read a line of the output / eof")
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Heinrich Schuchardt
2020-02-12 18:23:49 +01:00
committed by Tom Rini
parent 58b209cf60
commit a3a9e04657
1 changed files with 13 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
common/console.c
View File

@@ -401,7 +401,7 @@ int getc(void)
if (gd->console_in.start) { if (gd->console_in.start) {
int ch; int ch;
ch = membuff_getbyte(&gd->console_in); ch = membuff_getbyte((struct membuff *)&gd->console_in);
if (ch != -1) if (ch != -1)
return 1; return 1;
} }
@@ -426,7 +426,7 @@ int tstc(void)
return 0; return 0;
#ifdef CONFIG_CONSOLE_RECORD #ifdef CONFIG_CONSOLE_RECORD
if (gd->console_in.start) { if (gd->console_in.start) {
if (membuff_peekbyte(&gd->console_in) != -1) if (membuff_peekbyte((struct membuff *)&gd->console_in) != -1)
return 1; return 1;
} }
#endif #endif
@@ -518,7 +518,7 @@ void putc(const char c)
return; return;
#ifdef CONFIG_CONSOLE_RECORD #ifdef CONFIG_CONSOLE_RECORD
if ((gd->flags & GD_FLG_RECORD) && gd->console_out.start) if ((gd->flags & GD_FLG_RECORD) && gd->console_out.start)
membuff_putbyte(&gd->console_out, c); membuff_putbyte((struct membuff *)&gd->console_out, c);
#endif #endif
#ifdef CONFIG_SILENT_CONSOLE #ifdef CONFIG_SILENT_CONSOLE
if (gd->flags & GD_FLG_SILENT) { if (gd->flags & GD_FLG_SILENT) {
@@ -569,7 +569,7 @@ void puts(const char *s)
return; return;
#ifdef CONFIG_CONSOLE_RECORD #ifdef CONFIG_CONSOLE_RECORD
if ((gd->flags & GD_FLG_RECORD) && gd->console_out.start) if ((gd->flags & GD_FLG_RECORD) && gd->console_out.start)
membuff_put(&gd->console_out, s, strlen(s)); membuff_put((struct membuff *)&gd->console_out, s, strlen(s));
#endif #endif
#ifdef CONFIG_SILENT_CONSOLE #ifdef CONFIG_SILENT_CONSOLE
if (gd->flags & GD_FLG_SILENT) { if (gd->flags & GD_FLG_SILENT) {
@@ -602,18 +602,20 @@ int console_record_init(void)
{ {
int ret; int ret;
ret = membuff_new(&gd->console_out, CONFIG_CONSOLE_RECORD_OUT_SIZE); ret = membuff_new((struct membuff *)&gd->console_out,
CONFIG_CONSOLE_RECORD_OUT_SIZE);
if (ret) if (ret)
return ret; return ret;
ret = membuff_new(&gd->console_in, CONFIG_CONSOLE_RECORD_IN_SIZE); ret = membuff_new((struct membuff *)&gd->console_in,
CONFIG_CONSOLE_RECORD_IN_SIZE);
return ret; return ret;
} }
void console_record_reset(void) void console_record_reset(void)
{ {
membuff_purge(&gd->console_out); membuff_purge((struct membuff *)&gd->console_out);
membuff_purge(&gd->console_in); membuff_purge((struct membuff *)&gd->console_in);
} }
void console_record_reset_enable(void) void console_record_reset_enable(void)
@@ -624,12 +626,13 @@ void console_record_reset_enable(void)
int console_record_readline(char *str, int maxlen) int console_record_readline(char *str, int maxlen)
{ {
return membuff_readline(&gd->console_out, str, maxlen, ' '); return membuff_readline((struct membuff *)&gd->console_out, str,
maxlen, ' ');
} }
int console_record_avail(void) int console_record_avail(void)
{ {
return membuff_avail(&gd->console_out); return membuff_avail((struct membuff *)&gd->console_out);
} }
#endif #endif