Hardy Griech

Results 98 comments of Hardy Griech

> @rgrr There still seems to be no changes since the last review, do you have any challenges with git? No... sorry... I'm currently too busy to work on the...

: > The outline of functions is the same, but their implementation differ in several places. I suppose this PR content helped in giving insight in how the problem could...

So this example uses the most recent NCM driver? (you can identify it with my name in the copyright (Hardy Griech)) What is on the host side?

One more thing: @HiFiPhile introduced one significant change during reformat. CFG_TUD_NCM_IN_NTB_MAX_SIZE/CFG_TUD_NCM_OUT_NTB_MAX_SIZE was changed from "(2 * TCP_MSS + 100)" to "3200". Could you please test with the old values?

> I have tested CFG_TUD_NCM_IN_NTB_MAX_SIZE/CFG_TUD_NCM_OUT_NTB_MAX_SIZE 3200, but not work. Please set those values to "(2 * TCP_MSS + 100)" and try again. > Something may be important. CFG_TUD_NCM_IN_MAX_DATAGRAMS_PER_NTB/CFG_TUD_NCM_OUT_MAX_DATAGRAMS_PER_NTB default 8/6,...

> In tud_network_xmit_cb, just process one datagram and will call tud_network_recv_cb to alloc data of this one datagram. I didn't notice that in my previous tests. So in my test,...

> I think the recv data flow is not very suitable for rtos. Do you have any good ideas? Perhaps this glue code helps?: https://github.com/rgrr/yapicoprobe/blob/master/src/net/net_glue.c

> There are servial ways. > ... Sorry, I cannot follow... to whom are you responding? The glue code above also uses FreeRTOS. Why not use it? Tested and works...

> The glue code is useful to me. I want more details but I don't know how to get lwip and tinyusb code in your project. Check other parts of...

Hmmm... @electretmike : which NCM are you using? The old one or the recent one? Recent one has my name in the copyright (Hardy Griech). If you are at the...