This is the mail archive of the ecos-discuss@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Multicast support in AT91 EMAC Ethernet driver


Hello,

we are busy adding multicast support to the AT91 EMAC Ethernet driver.

As code size is often important for eCos users, I was thinking to add an #ifdef around the multicast code.
The setting is in /io/eth/current/cdl/eth_drivers.cdl: CYGINT_IO_ETH_MULTICAST.
But that is always used this way: 'implements CYGINT_IO_ETH_MULTICAST' in the driver cdl (e.g. /devs/eth/cl/cs8900a/current/cdl/cl_cs8900a_eth_drivers.cdl).


The problem is that you cannot add/remove that option yourself: from ecos.ecc: '# This value cannot be modified here.'

So once somebody as extended a driver to support multicast, that code is present in your binary (control() function), even if you don't need it.
That's OK for me. But I don't it's ok for everybody??
Then we could change the cdl option.


Kind regards,
Jürgen


-- Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]