diff --git a/master/domain.c b/master/domain.c index 799795fcc7284c3892ea0cb7d70daaa7768a20ab..5b9f766446a503cd7ab8c567c13513eae78febbc 100644 --- a/master/domain.c +++ b/master/domain.c @@ -205,7 +205,7 @@ ec_slave_t *EtherCAT_rt_register_slave_field( ec_master_t *master; const ec_sync_t *sync; const ec_field_t *field; - unsigned int field_idx, found, i, j; + unsigned int field_idx, i, j; uint32_t field_offset; if (!field_count) { @@ -234,8 +234,7 @@ ec_slave_t *EtherCAT_rt_register_slave_field( } field_idx = 0; - found = 0; - for (i = 0; type->sync_managers[i] && !found; i++) { + for (i = 0; type->sync_managers[i]; i++) { sync = type->sync_managers[i]; field_offset = 0; for (j = 0; sync->fields[j]; j++) { @@ -252,9 +251,10 @@ ec_slave_t *EtherCAT_rt_register_slave_field( } } - printk(KERN_ERR "EtherCAT: Slave %i (\"%s %s\") has less than %i fields of" - " type %i, starting at %i!\n", slave->ring_position, - vendor_name, product_name, field_count, field_type, field_index); + printk(KERN_ERR "EtherCAT: Slave %i (\"%s %s\") has less than %i field(s)" + " of type %i, starting at %i (only %i)!\n", slave->ring_position, + vendor_name, product_name, field_count, field_type, field_index, + field_idx); return NULL; } diff --git a/master/types.c b/master/types.c index 4de1fa2c6b206fce6c2c99641d0aeaced8f90f63..14aeeded5d42e166b5193afe6922f7605ae970d2 100644 --- a/master/types.c +++ b/master/types.c @@ -88,12 +88,12 @@ const ec_sync_t el31X2_sm3 = { const ec_slave_type_t Beckhoff_EL3102 = { "Beckhoff", "EL3102", "2x Analog Input diff.", - {&mailbox_sm0, &mailbox_sm1, &el31X2_sm2, &el31X2_sm2, NULL} + {&mailbox_sm0, &mailbox_sm1, &el31X2_sm2, &el31X2_sm3, NULL} }; const ec_slave_type_t Beckhoff_EL3162 = { "Beckhoff", "EL3102", "2x Analog Input", - {&mailbox_sm0, &mailbox_sm1, &el31X2_sm2, &el31X2_sm2, NULL} + {&mailbox_sm0, &mailbox_sm1, &el31X2_sm2, &el31X2_sm3, NULL} }; /*****************************************************************************/