diff options
author | Damien George <damien.p.george@gmail.com> | 2014-09-30 22:51:47 +0100 |
---|---|---|
committer | Damien George <damien.p.george@gmail.com> | 2014-09-30 22:51:47 +0100 |
commit | 9d2bf9c40573bd43856ad42c9f5eaad3b8370a24 (patch) | |
tree | c43c8eac0cdc7677382479b0af4288a3c7490691 /drivers | |
parent | 3a1c4c5bc6212310f5c0469300263290758217d7 (diff) | |
download | micropython-9d2bf9c40573bd43856ad42c9f5eaad3b8370a24.tar.gz micropython-9d2bf9c40573bd43856ad42c9f5eaad3b8370a24.zip |
drivers, wiznet5k: Wrap exported functions in a macro for renaming.
3rd party drivers should not export generic names like "close".
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/wiznet5k/ethernet/socket.c | 40 | ||||
-rw-r--r-- | drivers/wiznet5k/ethernet/socket.h | 33 | ||||
-rw-r--r-- | drivers/wiznet5k/internet/dns/dns.c | 10 |
3 files changed, 43 insertions, 40 deletions
diff --git a/drivers/wiznet5k/ethernet/socket.c b/drivers/wiznet5k/ethernet/socket.c index 20877c50e0..3dcfe33dca 100644 --- a/drivers/wiznet5k/ethernet/socket.c +++ b/drivers/wiznet5k/ethernet/socket.c @@ -87,7 +87,7 @@ static uint8_t sock_pack_info[_WIZCHIP_SOCK_NUM_] = {0,}; -int8_t socket(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag) +int8_t WIZCHIP_EXPORT(socket)(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag) { CHECK_SOCKNUM(); switch(protocol) @@ -132,7 +132,7 @@ int8_t socket(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag) break; } } - close(sn); + WIZCHIP_EXPORT(close)(sn); setSn_MR(sn, (protocol | (flag & 0xF0))); if(!port) { @@ -150,7 +150,7 @@ int8_t socket(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag) return (int8_t)sn; } -int8_t close(uint8_t sn) +int8_t WIZCHIP_EXPORT(close)(uint8_t sn) { CHECK_SOCKNUM(); @@ -166,7 +166,7 @@ int8_t close(uint8_t sn) return SOCK_OK; } -int8_t listen(uint8_t sn) +int8_t WIZCHIP_EXPORT(listen)(uint8_t sn) { CHECK_SOCKNUM(); CHECK_SOCKMODE(Sn_MR_TCP); @@ -177,7 +177,7 @@ int8_t listen(uint8_t sn) { if(getSn_CR(sn) == SOCK_CLOSED) { - close(sn); + WIZCHIP_EXPORT(close)(sn); return SOCKERR_SOCKCLOSED; } } @@ -185,7 +185,7 @@ int8_t listen(uint8_t sn) } -int8_t connect(uint8_t sn, uint8_t * addr, uint16_t port) +int8_t WIZCHIP_EXPORT(connect)(uint8_t sn, uint8_t * addr, uint16_t port) { CHECK_SOCKNUM(); CHECK_SOCKMODE(Sn_MR_TCP); @@ -236,7 +236,7 @@ int8_t connect(uint8_t sn, uint8_t * addr, uint16_t port) return SOCK_OK; } -int8_t disconnect(uint8_t sn) +int8_t WIZCHIP_EXPORT(disconnect)(uint8_t sn) { CHECK_SOCKNUM(); CHECK_SOCKMODE(Sn_MR_TCP); @@ -249,14 +249,14 @@ int8_t disconnect(uint8_t sn) { if(getSn_IR(sn) & Sn_IR_TIMEOUT) { - close(sn); + WIZCHIP_EXPORT(close)(sn); return SOCKERR_TIMEOUT; } } return SOCK_OK; } -int32_t send(uint8_t sn, uint8_t * buf, uint16_t len) +int32_t WIZCHIP_EXPORT(send)(uint8_t sn, uint8_t * buf, uint16_t len) { uint8_t tmp=0; uint16_t freesize=0; @@ -284,7 +284,7 @@ int32_t send(uint8_t sn, uint8_t * buf, uint16_t len) } else if(tmp & Sn_IR_TIMEOUT) { - close(sn); + WIZCHIP_EXPORT(close)(sn); return SOCKERR_TIMEOUT; } else return SOCK_BUSY; @@ -297,7 +297,7 @@ int32_t send(uint8_t sn, uint8_t * buf, uint16_t len) tmp = getSn_SR(sn); if ((tmp != SOCK_ESTABLISHED) && (tmp != SOCK_CLOSE_WAIT)) { - close(sn); + WIZCHIP_EXPORT(close)(sn); return SOCKERR_SOCKSTATUS; } if( (sock_io_mode & (1<<sn)) && (len > freesize) ) return SOCK_BUSY; @@ -315,7 +315,7 @@ int32_t send(uint8_t sn, uint8_t * buf, uint16_t len) } -int32_t recv(uint8_t sn, uint8_t * buf, uint16_t len) +int32_t WIZCHIP_EXPORT(recv)(uint8_t sn, uint8_t * buf, uint16_t len) { uint8_t tmp = 0; uint16_t recvsize = 0; @@ -336,13 +336,13 @@ int32_t recv(uint8_t sn, uint8_t * buf, uint16_t len) if(recvsize != 0) break; else if(getSn_TX_FSR(sn) == getSn_TxMAX(sn)) { - close(sn); + WIZCHIP_EXPORT(close)(sn); return SOCKERR_SOCKSTATUS; } } else { - close(sn); + WIZCHIP_EXPORT(close)(sn); return SOCKERR_SOCKSTATUS; } } @@ -357,7 +357,7 @@ int32_t recv(uint8_t sn, uint8_t * buf, uint16_t len) return len; } -int32_t sendto(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port) +int32_t WIZCHIP_EXPORT(sendto)(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port) { uint8_t tmp = 0; uint16_t freesize = 0; @@ -436,7 +436,7 @@ int32_t sendto(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t -int32_t recvfrom(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port) +int32_t WIZCHIP_EXPORT(recvfrom)(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port) { uint8_t mr; uint8_t head[8]; @@ -506,7 +506,7 @@ int32_t recvfrom(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16 sock_remained_size[sn] = (sock_remained_size[sn] <<8) + head[1]; if(sock_remained_size[sn] > 1514) { - close(sn); + WIZCHIP_EXPORT(close)(sn); return SOCKFATAL_PACKLEN; } sock_pack_info[sn] = PACK_FIRST; @@ -555,7 +555,7 @@ int32_t recvfrom(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16 } -int8_t ctlsocket(uint8_t sn, ctlsock_type cstype, void* arg) +int8_t WIZCHIP_EXPORT(ctlsocket)(uint8_t sn, ctlsock_type cstype, void* arg) { uint8_t tmp = 0; CHECK_SOCKNUM(); @@ -598,7 +598,7 @@ int8_t ctlsocket(uint8_t sn, ctlsock_type cstype, void* arg) return SOCK_OK; } -int8_t setsockopt(uint8_t sn, sockopt_type sotype, void* arg) +int8_t WIZCHIP_EXPORT(setsockopt)(uint8_t sn, sockopt_type sotype, void* arg) { // M20131220 : Remove warning //uint8_t tmp; @@ -651,7 +651,7 @@ int8_t setsockopt(uint8_t sn, sockopt_type sotype, void* arg) return SOCK_OK; } -int8_t getsockopt(uint8_t sn, sockopt_type sotype, void* arg) +int8_t WIZCHIP_EXPORT(getsockopt)(uint8_t sn, sockopt_type sotype, void* arg) { CHECK_SOCKNUM(); switch(sotype) diff --git a/drivers/wiznet5k/ethernet/socket.h b/drivers/wiznet5k/ethernet/socket.h index 40df8a872d..932224ecd0 100644 --- a/drivers/wiznet5k/ethernet/socket.h +++ b/drivers/wiznet5k/ethernet/socket.h @@ -78,8 +78,11 @@ * Following figure shows network flow diagram by WIZnet SOCKET API. * @image html WIZnet_SOCKET.jpg "<WIZnet SOCKET API>" */ -#ifndef _SOCKET_H_ -#define _SOCKET_H_ +#ifndef _WIZCHIP_SOCKET_H_ +#define _WIZCHIP_SOCKET_H_ + +// use this macro for exported names to avoid name clashes +#define WIZCHIP_EXPORT(name) wizchip_ ## name #include "wizchip_conf.h" @@ -147,7 +150,7 @@ * @ref SOCKERR_SOCKMODE - Not support socket mode as TCP, UDP, and so on. \n * @ref SOCKERR_SOCKFLAG - Invaild socket flag. */ -int8_t socket(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag); +int8_t WIZCHIP_EXPORT(socket)(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag); /** * @ingroup WIZnet_socket_APIs @@ -159,7 +162,7 @@ int8_t socket(uint8_t sn, uint8_t protocol, uint16_t port, uint8_t flag); * @return @b Success : @ref SOCK_OK \n * @b Fail : @ref SOCKERR_SOCKNUM - Invalid socket number */ -int8_t close(uint8_t sn); +int8_t WIZCHIP_EXPORT(close)(uint8_t sn); /** * @ingroup WIZnet_socket_APIs @@ -172,7 +175,7 @@ int8_t close(uint8_t sn); * @b Fail :\n @ref SOCKERR_SOCKINIT - Socket is not initialized \n * @ref SOCKERR_SOCKCLOSED - Socket closed unexpectedly. */ -int8_t listen(uint8_t sn); +int8_t WIZCHIP_EXPORT(listen)(uint8_t sn); /** * @ingroup WIZnet_socket_APIs @@ -195,7 +198,7 @@ int8_t listen(uint8_t sn); * @ref SOCKERR_TIMEOUT - Timeout occurred during request connection\n * @ref SOCK_BUSY - In non-block io mode, it returned immediately\n */ -int8_t connect(uint8_t sn, uint8_t * addr, uint16_t port); +int8_t WIZCHIP_EXPORT(connect)(uint8_t sn, uint8_t * addr, uint16_t port); /** * @ingroup WIZnet_socket_APIs @@ -212,7 +215,7 @@ int8_t connect(uint8_t sn, uint8_t * addr, uint16_t port); * @ref SOCKERR_TIMEOUT - Timeout occurred \n * @ref SOCK_BUSY - Socket is busy. */ -int8_t disconnect(uint8_t sn); +int8_t WIZCHIP_EXPORT(disconnect)(uint8_t sn); /** * @ingroup WIZnet_socket_APIs @@ -232,7 +235,7 @@ int8_t disconnect(uint8_t sn); * @ref SOCKERR_DATALEN - zero data length \n * @ref SOCK_BUSY - Socket is busy. */ -int32_t send(uint8_t sn, uint8_t * buf, uint16_t len); +int32_t WIZCHIP_EXPORT(send)(uint8_t sn, uint8_t * buf, uint16_t len); /** * @ingroup WIZnet_socket_APIs @@ -254,7 +257,7 @@ int32_t send(uint8_t sn, uint8_t * buf, uint16_t len); * @ref SOCKERR_DATALEN - zero data length \n * @ref SOCK_BUSY - Socket is busy. */ -int32_t recv(uint8_t sn, uint8_t * buf, uint16_t len); +int32_t WIZCHIP_EXPORT(recv)(uint8_t sn, uint8_t * buf, uint16_t len); /** * @ingroup WIZnet_socket_APIs @@ -282,7 +285,7 @@ int32_t recv(uint8_t sn, uint8_t * buf, uint16_t len); * @ref SOCKERR_TIMEOUT - Timeout occurred \n * @ref SOCK_BUSY - Socket is busy. */ -int32_t sendto(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port); +int32_t WIZCHIP_EXPORT(sendto)(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t port); /** * @ingroup WIZnet_socket_APIs @@ -312,7 +315,7 @@ int32_t sendto(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t * @ref SOCKERR_SOCKNUM - Invalid socket number \n * @ref SOCKBUSY - Socket is busy. */ -int32_t recvfrom(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port); +int32_t WIZCHIP_EXPORT(recvfrom)(uint8_t sn, uint8_t * buf, uint16_t len, uint8_t * addr, uint16_t *port); ///////////////////////////// @@ -399,7 +402,7 @@ typedef enum * @return @b Success @ref SOCK_OK \n * @b fail @ref SOCKERR_ARG - Invalid argument\n */ -int8_t ctlsocket(uint8_t sn, ctlsock_type cstype, void* arg); +int8_t WIZCHIP_EXPORT(ctlsocket)(uint8_t sn, ctlsock_type cstype, void* arg); /** * @ingroup WIZnet_socket_APIs @@ -427,7 +430,7 @@ int8_t ctlsocket(uint8_t sn, ctlsock_type cstype, void* arg); * - @ref SOCKERR_SOCKOPT - Invalid socket option or its value \n * - @ref SOCKERR_TIMEOUT - Timeout occurred when sending keep-alive packet \n */ -int8_t setsockopt(uint8_t sn, sockopt_type sotype, void* arg); +int8_t WIZCHIP_EXPORT(setsockopt)(uint8_t sn, sockopt_type sotype, void* arg); /** * @ingroup WIZnet_socket_APIs @@ -461,6 +464,6 @@ int8_t setsockopt(uint8_t sn, sockopt_type sotype, void* arg); * When SO_PACKINFO value is PACK_FIRST and the return value of recvfrom() is zero, * This means the zero byte UDP data(UDP Header only) received. */ -int8_t getsockopt(uint8_t sn, sockopt_type sotype, void* arg); +int8_t WIZCHIP_EXPORT(getsockopt)(uint8_t sn, sockopt_type sotype, void* arg); -#endif // _SOCKET_H_ +#endif // _WIZCHIP_SOCKET_H_ diff --git a/drivers/wiznet5k/internet/dns/dns.c b/drivers/wiznet5k/internet/dns/dns.c index 48f486e0ba..c0ad570c00 100644 --- a/drivers/wiznet5k/internet/dns/dns.c +++ b/drivers/wiznet5k/internet/dns/dns.c @@ -521,21 +521,21 @@ int8_t DNS_run(uint8_t * dns_ip, uint8_t * name, uint8_t * ip_from_dns) hal_sys_tick = HAL_GetTick(); // Socket open - socket(DNS_SOCKET, Sn_MR_UDP, 0, 0); + WIZCHIP_EXPORT(socket)(DNS_SOCKET, Sn_MR_UDP, 0, 0); #ifdef _DNS_DEBUG_ printf("> DNS Query to DNS Server : %d.%d.%d.%d\r\n", dns_ip[0], dns_ip[1], dns_ip[2], dns_ip[3]); #endif len = dns_makequery(0, (char *)name, pDNSMSG, MAX_DNS_BUF_SIZE); - sendto(DNS_SOCKET, pDNSMSG, len, dns_ip, IPPORT_DOMAIN); + WIZCHIP_EXPORT(sendto)(DNS_SOCKET, pDNSMSG, len, dns_ip, IPPORT_DOMAIN); while (1) { if ((len = getSn_RX_RSR(DNS_SOCKET)) > 0) { if (len > MAX_DNS_BUF_SIZE) len = MAX_DNS_BUF_SIZE; - len = recvfrom(DNS_SOCKET, pDNSMSG, len, ip, &port); + len = WIZCHIP_EXPORT(recvfrom)(DNS_SOCKET, pDNSMSG, len, ip, &port); #ifdef _DNS_DEBUG_ printf("> Receive DNS message from %d.%d.%d.%d(%d). len = %d\r\n", ip[0], ip[1], ip[2], ip[3],port,len); #endif @@ -556,10 +556,10 @@ int8_t DNS_run(uint8_t * dns_ip, uint8_t * name, uint8_t * ip_from_dns) #ifdef _DNS_DEBUG_ printf("> DNS Timeout\r\n"); #endif - sendto(DNS_SOCKET, pDNSMSG, len, dns_ip, IPPORT_DOMAIN); + WIZCHIP_EXPORT(sendto)(DNS_SOCKET, pDNSMSG, len, dns_ip, IPPORT_DOMAIN); } } - close(DNS_SOCKET); + WIZCHIP_EXPORT(close)(DNS_SOCKET); // Return value // 0 > : failed / 1 - success return ret; |