[alsa-devel] davici-mcasp: "tx-num-evt" confusion with number of serializers

Michal Bachraty michal.bachraty at streamunlimited.com
Thu Feb 28 12:02:00 CET 2013


On Thursday, February 28, 2013 10:32:15 Mike Looijmans wrote:

> That's a big improvement over "rx_num_evt". The first time I saw this
> struct, I thought I had to set rx_num_evt to the number of receiving AXR
> pins in use.
> 
> Maybe call it "rx_min_fifo_depth" because just "depth" might be
> interpreted as the capaciity or maximum size of the fifo, while in fact
> the parameter is a minimum amount of data to be put into the FIFO
> buffer. Of maybe "rx_fifo_thd" to indicate it's a threshhold. And use
> the same name for the "tx" part as well (tx_fifo_thd or tx_fifo_min_depth).

 "rx_min_fifo_depth" or  "tx_min_fifo_depth"  seem to be fine.
If I'll have time, I'll prepare patch. That's cosmetic upgrade, but should 
help others.

Michal.


More information about the Alsa-devel mailing list