Merge branch 'fix_warnings' into 'stable-1.6'

Fix some warnings

See merge request etherlab.org/ethercat!153
This commit is contained in:
Florian Pose 2025-02-06 08:24:22 +00:00
commit fa7c40b63e
3 changed files with 23 additions and 21 deletions

View File

@ -77,7 +77,7 @@ static unsigned int off_ana_in;
static unsigned int off_ana_out;
static unsigned int off_dig_out;
const static ec_pdo_entry_reg_t domain1_regs[] = {
static const ec_pdo_entry_reg_t domain1_regs[] = {
#if EL3152_ALT_PDOS
{AnaInSlavePos, Beckhoff_EL3152, 0x6401, 1, &off_ana_in},
#else

View File

@ -27,6 +27,8 @@
#include "../../include/ecrt.h" // EtherCAT realtime interface
#include "../../include/ectty.h" // EtherCAT TTY interface
#include "serial.h"
/****************************************************************************/
// Optional features
@ -114,7 +116,7 @@ LIST_HEAD(handlers);
* Revision number: 0x00100000
*/
ec_pdo_entry_info_t el6002_pdo_entries[] = {
static const ec_pdo_entry_info_t el6002_pdo_entries[] = {
{0x7001, 0x01, 16}, /* Ctrl */
{0x7000, 0x11, 8}, /* Data Out 0 */
{0x7000, 0x12, 8}, /* Data Out 1 */
@ -209,14 +211,14 @@ ec_pdo_entry_info_t el6002_pdo_entries[] = {
{0x6010, 0x26, 8}, /* Data In 21 */
};
ec_pdo_info_t el6002_pdos[] = {
static const ec_pdo_info_t el6002_pdos[] = {
{0x1604, 23, el6002_pdo_entries + 0}, /* COM RxPDO-Map Outputs Ch.1 */
{0x1605, 23, el6002_pdo_entries + 23}, /* COM RxPDO-Map Outputs Ch.2 */
{0x1a04, 23, el6002_pdo_entries + 46}, /* COM TxPDO-Map Inputs Ch.1 */
{0x1a05, 23, el6002_pdo_entries + 69}, /* COM TxPDO-Map Inputs Ch.2 */
};
ec_sync_info_t el6002_syncs[] = {
static const ec_sync_info_t el6002_syncs[] = {
{0, EC_DIR_OUTPUT, 0, NULL, EC_WD_DISABLE},
{1, EC_DIR_INPUT, 0, NULL, EC_WD_DISABLE},
{2, EC_DIR_OUTPUT, 2, el6002_pdos + 0, EC_WD_DISABLE},
@ -275,7 +277,7 @@ el600x_baud_rate_t el600x_baud_rate[] = {
/****************************************************************************/
int el60xx_cflag_changed(void *data, tcflag_t cflag)
static int el60xx_cflag_changed(void *data, tcflag_t cflag)
{
el60xx_port_t *port = (el60xx_port_t *) data;
unsigned int data_bits, stop_bits;
@ -375,7 +377,7 @@ static ec_tty_operations_t el60xx_tty_ops = {
/****************************************************************************/
int el60xx_port_init(el60xx_port_t *port, ec_slave_config_t *sc,
static int el60xx_port_init(el60xx_port_t *port, ec_slave_config_t *sc,
ec_domain_t *domain, unsigned int slot_offset, const char *name)
{
int ret = 0;
@ -490,7 +492,7 @@ out_return:
/****************************************************************************/
void el60xx_port_clear(el60xx_port_t *port)
static void el60xx_port_clear(el60xx_port_t *port)
{
ectty_free(port->tty);
if (port->tx_data) {
@ -500,7 +502,7 @@ void el60xx_port_clear(el60xx_port_t *port)
/****************************************************************************/
void el60xx_port_run(el60xx_port_t *port, u8 *pd)
static void el60xx_port_run(el60xx_port_t *port, u8 *pd)
{
u16 status = EC_READ_U16(pd + port->off_status);
u8 *rx_data = pd + port->off_rx;
@ -658,7 +660,7 @@ void el60xx_port_run(el60xx_port_t *port, u8 *pd)
/****************************************************************************/
int el6002_init(el6002_t *el6002, ec_master_t *master, u16 position,
static int el6002_init(el6002_t *el6002, ec_master_t *master, u16 position,
ec_domain_t *domain, u32 vendor, u32 product)
{
int ret = 0, i;
@ -702,7 +704,7 @@ out_return:
/****************************************************************************/
void el6002_clear(el6002_t *el6002)
static void el6002_clear(el6002_t *el6002)
{
int i;
@ -713,7 +715,7 @@ void el6002_clear(el6002_t *el6002)
/****************************************************************************/
void el6002_run(el6002_t *el6002, u8 *pd)
static void el6002_run(el6002_t *el6002, u8 *pd)
{
int i;
@ -735,7 +737,7 @@ void run_serial_devices(u8 *pd)
/****************************************************************************/
int create_el6002_handler(ec_master_t *master, ec_domain_t *domain,
static int create_el6002_handler(ec_master_t *master, ec_domain_t *domain,
u16 position, u32 vendor, u32 product)
{
el6002_t *el6002;

View File

@ -44,7 +44,7 @@
// EtherCAT
static ec_master_t *master = NULL;
static ec_master_state_t master_state = {};
struct semaphore master_sem;
static struct semaphore master_sem;
static ec_domain_t *domain1 = NULL;
static ec_domain_state_t domain1_state = {};
@ -65,7 +65,7 @@ static unsigned int counter = 0;
/****************************************************************************/
void check_domain1_state(void)
static void check_domain1_state(void)
{
ec_domain_state_t ds;
@ -83,7 +83,7 @@ void check_domain1_state(void)
/****************************************************************************/
void check_master_state(void)
static void check_master_state(void)
{
ec_master_state_t ms;
@ -104,9 +104,9 @@ void check_master_state(void)
/****************************************************************************/
#if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 15, 0)
void cyclic_task(struct timer_list *t)
static void cyclic_task(struct timer_list *t)
#else
void cyclic_task(unsigned long data)
static void cyclic_task(unsigned long data)
#endif
{
// receive process data
@ -142,7 +142,7 @@ void cyclic_task(unsigned long data)
/****************************************************************************/
void send_callback(void *cb_data)
static void send_callback(void *cb_data)
{
ec_master_t *m = (ec_master_t *) cb_data;
down(&master_sem);
@ -152,7 +152,7 @@ void send_callback(void *cb_data)
/****************************************************************************/
void receive_callback(void *cb_data)
static void receive_callback(void *cb_data)
{
ec_master_t *m = (ec_master_t *) cb_data;
down(&master_sem);
@ -162,7 +162,7 @@ void receive_callback(void *cb_data)
/****************************************************************************/
int __init init_mini_module(void)
static int __init init_mini_module(void)
{
int ret = -1;
ec_slave_config_t *sc;
@ -229,7 +229,7 @@ out_return:
/****************************************************************************/
void __exit cleanup_mini_module(void)
static void __exit cleanup_mini_module(void)
{
printk(KERN_INFO PFX "Stopping...\n");