RTOS/rtems
1
0
Fork 0
forked from Imagelibrary/rtems
Code Pull Requests Activity

sonic.c: Fix warning on gcc 6

Browse Source 
GCC 6.0 previews give the following warning on sonic.c:

../../../../../rtems/c/src/libchip/network/sonic.c:837:11: warning:
statement is indented as if it were guarded by... [-Wmisleading-indentation]

This was discussed here:

https://lists.rtems.org/pipermail/devel/2016-March/014004.html

and we concluded it would be better to rewrite the entire snippet.
This commit is contained in:
Martin Galvan
2016-05-06 17:08:47 -03:00
parent f65f803a26
commit b4d7d5d52e
1 changed files with 11 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
c/src/libchip/network/sonic.c
View File

@@ -826,19 +826,17 @@ SONIC_STATIC void sonic_rda_wait(
* driver has to move the RRP back *two* entries to
* reuse the receive buffer holding the giant packet.
*/
for (i = 0 ; i < 2 ; i++) {
if ((*sc->read_register)( rp, SONIC_REG_RRP ) ==
(*sc->read_register)( rp, SONIC_REG_RSA ))
(*sc->write_register)(
rp,
SONIC_REG_RRP,
(*sc->read_register)( rp, SONIC_REG_REA )
);
(*sc->write_register)(
rp,
SONIC_REG_RRP,
(*sc->read_register)(rp, SONIC_REG_RRP) - sizeof(ReceiveResource_t)
);
for (i = 0; i < 2; ++i) {
uint32_t rrp = (*sc->read_register)( rp, SONIC_REG_RRP );
const uint32_t rsa = (*sc->read_register)( rp, SONIC_REG_RSA );
if (rrp == rsa) {
const uint32_t rea = (*sc->read_register)( rp, SONIC_REG_REA );
(*sc->write_register)( rp, SONIC_REG_RRP, rea );
}
rrp = (*sc->read_register)( rp, SONIC_REG_RRP );
(*sc->write_register)( rp, SONIC_REG_RRP, rrp - sizeof(ReceiveResource_t) );
}
/*