On Thu, 12 Dec 2019 10:13:27 -0800
Florian Fainelli <f.faine...@gmail.com> wrote:

Hi Florian,

thanks for chiming in!

> On 12/12/19 3:59 AM, Matthias Brugger wrote:
> > 
> > 
> > On 12/12/2019 11:41, Andre Przywara wrote:  
> >> On Wed, 11 Dec 2019 10:23:16 +0100
> >> Matthias Brugger <matthias....@gmail.com> wrote:
> >>
> >> Hi,
> >>  
> >>> On 09/12/2019 14:33, Sascha Dewald wrote:  
> >>>> Hello Amit,
> >>>>
> >>>> no problem.
> >>>>
> >>>> Is there some git feature branch, to have a look ?    
> >>>
> >>> Yes I'd love to have a look as well :)  
> >>
> >> We will send an RFC later this week, Amit is doing some cleanup now. At 
> >> the moment transfers work up to 256 packets (375KB when using TFTP), at 
> >> this point it breaks, because recycling DMA descriptors is not working for 
> >> some reason. I haven't found any documentation for the MAC, so I am 
> >> reverse engineering the Linux driver and doing printf and tcpdump 
> >> debugging right now. Looking forward to people joining us in this effort 
> >> ;-)
> >>  
> > 
> > Thanks for the update. I CC'ed Florian who I think is the SW expert on that
> > chip. Maybe he can help you.  
> 
> Transfers work for up to 256 packets in which direction? Are you
> properly consuming descriptors that were produced by the RDMA (on
> receive), 

Yes, not sure we do it properly, but we do.

The problem I see is that the TDMA_CONS_INDEX does not increase after it 
reaches 256, although RDMA_PROD_INDEX does. Is there any cleanup in the actual 
descriptors needed?
Did I get this right that those indices are just linear "progress counters", 
and not actual descriptor indices? I see the Linux code masks them with 0xffff, 
not with the number of descriptors.

Will try to dig a bit deeper on this.

> conversely, polling the TDMA consumer index on transmit?

Yes, I switched over to that (from polling for the DMA_OWN bit).

> Can you share your code somewhere so I could see if there are obvious 
> problems?

Of course, we just need to clean it up a bit, and it's getting late for Amit 
today. We will CC: you on the post tomorrow.

Thanks for you help!

Cheers,
Andre.

Reply via email to