> On Wed, May 15, 2019 at 11:43:55AM +0200, Lorenzo Bianconi wrote: > > > On Mon, May 13, 2019 at 11:48:37AM +0200, Stanislaw Gruszka wrote: > > > > On Mon, May 13, 2019 at 10:41:28AM +0200, Lorenzo Bianconi wrote: > > > > > > Lorenzo Bianconi <lorenzo@xxxxxxxxxx> writes: > > > > > > > > > > > > > Introduce a knob in mt7603 debugfs in order to enable/disable > > > > > > > edcca processing > > > > > > > > > > > > > > Signed-off-by: Lorenzo Bianconi <lorenzo@xxxxxxxxxx> > > > > > > > > > > > > It's good to explain what edcca does and how the file is used supposed > > > > > > to be used. In other words, have a small introduction for the user. > > > > > > > > > > Hi Kalle, > > > > > > > > > > edcca is used for adjusting energy detect based on CCA thresholds. > > > > > The code was already there so I just reported the acronym. > > > > > > > > What for it is needed ? > > > > > > Care to comment why EDCCA is needed at all ? > > > > > > Taking that debugfs file that enable it is read-only, it looks like > > > feature that nobody needs nor tests. > > > > already fixed in v2 > > https://patchwork.kernel.org/patch/10940645/ > > I'm aware of this patch and other one for mt76x02. But so far in the > sources EDCCA is disabled for mt76x02 without possibility to enable it > (and this permission file issue was pointed by Kalle during review, not > by someone who want to test EDCCA). So again, what for EDCCA is needed ? As I have already written in a previous email, ED/CCA is used to control tx power according to the CCA MIB counters (e.g do not transmit if the channel busy time is higher than 90% for given amount of time in a row). I guess it is required by ETSI regulatory. Regarding file permission for mt76x02 debugfs edcca node is a typo. Lorenzo > > Stanislaw
Attachment:
signature.asc
Description: PGP signature
