<_florent_>
davidc__: re CommUDP, that's probably a mistake, i'll test with one socket and will fix if working
freemint has quit [Ping timeout: 245 seconds]
freemint has joined #litex
freemint has quit [Ping timeout: 240 seconds]
freemint has joined #litex
freemint has quit [Ping timeout: 265 seconds]
freemint has joined #litex
ambro718 has joined #litex
<davidc__>
_florent_: I have it tested with one socket, and it works for me. Though - I don't know what you're using it with (because it was binding to a weird port, anything that talked to it had to hardcode the return port)
<davidc__>
_florent_: saw your patch - the socket.bind should be superfluous because sendto() is called first. The current patch won't support two clients on one host, or the client and server on the same host, since they'll try and use the same UDP port
<davidc__>
IIRC, just dropping socket.bind() should be fine
CarlFK has quit [Read error: No route to host]
freemint has quit [Ping timeout: 276 seconds]
nrossi has quit [Quit: Connection closed for inactivity]
freemint has joined #litex
freemint has quit [Ping timeout: 245 seconds]
CarlFK has joined #litex
freemint has joined #litex
freemint has quit [Ping timeout: 240 seconds]
<_florent_>
davidc__: thanks, i'll test without socket.bind()
<davidc__>
_florent_: it might require a tweak to the server code; some of the server code I saw was explicitly specifying the port on the client-side (which is strange)
<davidc__>
rather than just returning to whatever the src port on the client was
<_florent_>
davidc__: feel free to create issue/send pull request on this if you think things could be improved