Skip to content

Fixing compiler warnings. #166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
allenhuffman wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fixing compiler warnings. #166

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
73d006d
Fixing compiler warnings.
allenhuffman Oct 4, 2025
bd54ec8
Fix compiler warning for static functions
allenhuffman Oct 4, 2025
5e1df9a
Fixing compiler warnings from missing prototypes
allenhuffman Oct 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 8 additions & 2 deletions Ethernet/socket.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,11 @@
//#define SOCK_ANY_PORT_NUM 0xC000;
#define SOCK_ANY_PORT_NUM 0xC000

// Static prototypes
static int8_t connect_IO_6(uint8_t sn, uint8_t * addr, uint16_t port, uint8_t addrlen);
static int32_t sendto_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port, uint8_t addrlen);
static int32_t recvfrom_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port, uint8_t *addrlen);

static uint16_t sock_any_port = SOCK_ANY_PORT_NUM;
static uint16_t sock_io_mode = 0;
static uint16_t sock_is_sending = 0;
Expand Down Expand Up @@ -196,8 +201,8 @@ inline uint8_t inline_CheckAddrlen_W6x00(void) {

int8_t socket(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag) {

uint8_t taddr[16];
uint16_t local_port = 0;
//uint8_t taddr[16];
//uint16_t local_port = 0;
CHECK_SOCKNUM();
switch (protocol & 0x0F) {
#ifdef IPV6_AVAILABLE
Expand Down Expand Up @@ -1033,6 +1038,7 @@ static int32_t recvfrom_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t *
while (getSn_CR(sn));

#else
(void)addrlen; // Unused
if (sock_remained_size[sn] == 0) {
wiz_recv_data(sn, head, 8);
setSn_CR(sn, Sn_CR_RECV);
Expand Down
6 changes: 3 additions & 3 deletions Ethernet/socket.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ int8_t listen(uint8_t sn);
In block io mode, it does not return until connection is completed. \n
In Non-block io mode(@ref SF_IO_NONBLOCK), it returns @ref SOCK_BUSY immediately.
*/
static int8_t connect_IO_6(uint8_t sn, uint8_t * addr, uint16_t port, uint8_t addrlen);
//static int8_t connect_IO_6(uint8_t sn, uint8_t * addr, uint16_t port, uint8_t addrlen);
//int8_t connect(uint8_t sn, uint8_t * addr, uint16_t port, uint8_t addrlen);

/**
Expand Down Expand Up @@ -401,7 +401,7 @@ int32_t recv(uint8_t sn, uint8_t * buf, uint16_t len);
In non-block io mode(@ref SF_IO_NONBLOCK), It return @ref SOCK_BUSY immediately when SOCKET transimttable buffer size is not enough.
*/
//int32_t sendto(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port, uint8_t addrlen);
static int32_t sendto_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port, uint8_t addrlen);
//static int32_t sendto_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port, uint8_t addrlen);

/**
@ingroup WIZnet_socket_APIs
Expand Down Expand Up @@ -436,7 +436,7 @@ static int32_t sendto_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * ad
In non-block io mode(@ref SF_IO_NONBLOCK), it return @ref SOCK_BUSY immediately when SOCKET RX buffer is empty. \n
*/
//int32_t recvfrom(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port, uint8_t *addrlen);
static int32_t recvfrom_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port, uint8_t *addrlen);
//static int32_t recvfrom_IO_6(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port, uint8_t *addrlen);


/////////////////////////////
Expand Down
32 changes: 26 additions & 6 deletions Ethernet/wizchip_conf.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,24 @@
//M20150401 : Remove ; in the default callback function such as wizchip_cris_enter(), wizchip_cs_select() and etc.
/////////////

// Prototypes to avoid compiler warnings:
void wizchip_cris_enter(void);
void wizchip_cris_exit(void);
void wizchip_cs_select(void);
void wizchip_cs_deselect(void);
iodata_t wizchip_bus_readdata(uint32_t AddrSel);
void wizchip_bus_writedata(uint32_t AddrSel, iodata_t wb);
void wizchip_bus_read_buf(uint32_t AddrSel, iodata_t* buf, int16_t len, uint8_t addrinc);
void wizchip_bus_write_buf(uint32_t AddrSel, iodata_t* buf, int16_t len, uint8_t addrinc);
iodata_t wizchip_bus_readbyte(void);
uint8_t wizchip_spi_readbyte(void);
void wizchip_spi_writebyte(uint8_t wb);
void wizchip_spi_readburst(uint8_t* pBuf, uint16_t len);
void wizchip_spi_writeburst(uint8_t* pBuf, uint16_t len);
void wizchip_qspi_read(uint8_t opcode, uint16_t addr, uint8_t* pBuf, uint16_t len);
void wizchip_qspi_write(uint8_t opcode, uint16_t addr, uint8_t* pBuf, uint16_t len);
void reg_wizchip_busbuf_cbfunc(void(*busbuf_rb)(uint32_t AddrSel, iodata_t* pBuf, int16_t len, uint8_t addrinc), void (*busbuf_wb)(uint32_t AddrSel, iodata_t* pBuf, int16_t len, uint8_t addrinc));

/**
@brief Default function to enable interrupt.
@note This function help not to access wrong address. If you do not describe this function or register any functions,
Expand Down Expand Up @@ -127,7 +145,7 @@ void wizchip_bus_writedata(uint32_t AddrSel, iodata_t wb) {
@sa wizchip_bus_write_buf()
*/
void wizchip_bus_read_buf(uint32_t AddrSel, iodata_t* buf, int16_t len, uint8_t addrinc) {
uint16_t i;
int16_t i;
if (addrinc) {
addrinc = sizeof(iodata_t);
}
Expand All @@ -151,7 +169,7 @@ void wizchip_bus_read_buf(uint32_t AddrSel, iodata_t* buf, int16_t len, uint8_t
@sa wizchip_bus_read_buf()
*/
void wizchip_bus_write_buf(uint32_t AddrSel, iodata_t* buf, int16_t len, uint8_t addrinc) {
uint16_t i;
int16_t i;
if (addrinc) {
addrinc = sizeof(iodata_t);
}
Expand Down Expand Up @@ -179,7 +197,7 @@ uint8_t wizchip_spi_readbyte(void) {
null function is called.
*/
//void wizchip_spi_writebyte(uint8_t wb) {};
void wizchip_spi_writebyte(uint8_t wb) {}
void wizchip_spi_writebyte(uint8_t wb) { (void)wb; }

/**
@brief Default function to burst read in SPI interface.
Expand Down Expand Up @@ -221,14 +239,14 @@ void wizchip_spi_writeburst(uint8_t* pBuf, uint16_t len) {}
@note This function help not to access wrong address. If you do not describe this function or register any functions,
null function is called.
*/
void wizchip_qspi_read(uint8_t opcode, uint16_t addr, uint8_t* pBuf, uint16_t len) {}
void wizchip_qspi_read(uint8_t opcode, uint16_t addr, uint8_t* pBuf, uint16_t len) { (void)opcode; (void)addr; (void)pBuf; (void)len; }

/**
@brief Default function to write in QSPI interface.
@note This function help not to access wrong address. If you do not describe this function or register any functions,
null function is called.
*/
void wizchip_qspi_write(uint8_t opcode, uint16_t addr, uint8_t* pBuf, uint16_t len) {}
void wizchip_qspi_write(uint8_t opcode, uint16_t addr, uint8_t* pBuf, uint16_t len) { (void)opcode; (void)addr; (void)pBuf; (void)len; }

#endif
/**
Expand Down Expand Up @@ -270,7 +288,9 @@ _WIZCHIP WIZCHIP = {
//wizchip_bus_readbyte,
//wizchip_bus_writebyte
wizchip_bus_readdata,
wizchip_bus_writedata
wizchip_bus_writedata,
NULL,
NULL
},

}
Expand Down