[rtems commit] greth: Make local functions static, remove overflow and correct printf call.
Joel Sherrill
joel at rtems.org
Fri Mar 21 15:14:54 UTC 2014
Module: rtems
Branch: master
Commit: e4b5085344b8bcf7c53bb7b575535db92036d019
Changeset: http://git.rtems.org/rtems/commit/?id=e4b5085344b8bcf7c53bb7b575535db92036d019
Author: Daniel Cederman <cederman at gaisler.com>
Date: Thu Mar 13 14:35:43 2014 +0100
greth: Make local functions static, remove overflow and correct printf call.
---
.../lib/libbsp/sparc/leon3/leon_greth/leon_greth.c | 2 +-
c/src/libchip/network/greth.c | 14 ++++++++------
2 files changed, 9 insertions(+), 7 deletions(-)
diff --git a/c/src/lib/libbsp/sparc/leon3/leon_greth/leon_greth.c b/c/src/lib/libbsp/sparc/leon3/leon_greth/leon_greth.c
index ea36118..77e4f49 100644
--- a/c/src/lib/libbsp/sparc/leon3/leon_greth/leon_greth.c
+++ b/c/src/lib/libbsp/sparc/leon3/leon_greth/leon_greth.c
@@ -47,7 +47,7 @@ int rtems_leon_greth_driver_attach(
*(volatile int *) base_addr = 0;
*(volatile int *) base_addr = GRETH_CTRL_RST;
*(volatile int *) base_addr = 0;
- leon_greth_configuration.base_address = base_addr;
+ leon_greth_configuration.base_address = (void*)base_addr;
leon_greth_configuration.vector = eth_irq; /* on LEON vector is IRQ no. */
leon_greth_configuration.txd_count = TDA_COUNT;
leon_greth_configuration.rxd_count = RDA_COUNT;
diff --git a/c/src/libchip/network/greth.c b/c/src/libchip/network/greth.c
index 66581c7..e957dc3 100644
--- a/c/src/libchip/network/greth.c
+++ b/c/src/libchip/network/greth.c
@@ -88,7 +88,7 @@ extern void ipalign(struct mbuf *m);
#endif
const struct timespec greth_tan = {
GRETH_AUTONEGO_TIMEOUT_MS/1000,
- GRETH_AUTONEGO_TIMEOUT_MS*1000000
+ (GRETH_AUTONEGO_TIMEOUT_MS % 1000) *1000000
};
/* For optimizing the autonegotiation time */
@@ -178,7 +178,7 @@ static char *almalloc(int sz)
/* GRETH interrupt handler */
-void greth_interrupt_handler (void *arg)
+static void greth_interrupt_handler (void *arg)
{
uint32_t status;
uint32_t ctrl;
@@ -258,7 +258,7 @@ static void print_init_info(struct greth_softc *sc)
}
#ifdef GRETH_AUTONEGO_PRINT_TIME
if ( sc->auto_neg ) {
- printf("Autonegotiation Time: %dms\n", sc->auto_neg_time.tv_sec*1000 +
+ printf("Autonegotiation Time: %ldms\n", sc->auto_neg_time.tv_sec*1000 +
sc->auto_neg_time.tv_nsec/1000000);
}
#endif
@@ -533,7 +533,7 @@ void ipalign(struct mbuf *m)
}
#endif
-void
+static void
greth_Daemon (void *arg)
{
struct ether_header *eh;
@@ -544,7 +544,9 @@ greth_Daemon (void *arg)
rtems_event_set events;
rtems_interrupt_level level;
int first;
+#ifdef CPU_U32_FIX
unsigned int tmp;
+#endif
for (;;)
{
@@ -627,7 +629,7 @@ again:
tmp = GRETH_MEM_LOAD(4+(uintptr_t)eh);
tmp = GRETH_MEM_LOAD(8+(uintptr_t)eh);
tmp = GRETH_MEM_LOAD(12+(uintptr_t)eh);
-
+ (void)tmp;
ipalign(m); /* Align packet on 32-bit boundary */
}
#endif
@@ -738,7 +740,7 @@ sendpacket (struct ifnet *ifp, struct mbuf *m)
}
-int
+static int
sendpacket_gbit (struct ifnet *ifp, struct mbuf *m)
{
struct greth_softc *dp = ifp->if_softc;
More information about the vc
mailing list