linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Thomas Huehn <thomas@net.t-labs.tu-berlin.de>
To: Christian Lamparter <chunkeey@googlemail.com>
Cc: linville@tuxdriver.com, linux-wireless@vger.kernel.org,
	johannes.berg@intel.com, nbd@openwrt.org
Subject: Re: [PATCH 2/2] mac80211: Add transmit power control support (TPC) for ACK packets
Date: Sat, 28 Jul 2012 16:36:31 +0200	[thread overview]
Message-ID: <5013F8EF.10404@net.t-labs.tu-berlin.de> (raw)
In-Reply-To: <201207272336.09898.chunkeey@googlemail.com>

Hi Christian,

Christian Lamparter schrieb:

> On Friday 27 July 2012 18:00:09 Thomas Huehn wrote:
>> This patch adds support to annotate and change the power-level of ACK
>> packets. To map hardware specific ack power capabilities to mac80211
>> one new hw flag is defined:
>>
>> @IEEE80211_HW_SUPPORTS_TPC_ACK_GLOBAL:
>> 	Power level of ack packets is adjustable and processed in a
>> 	global manner, per hw rather than per ack packet or link.
>>
>> The struct hw_config is extended by ack_power a global power level in
>> dBm to be used for all acknowledgement_packets. A new config change
>> event IEEE80211_CONF_CHANGE_ACK_POWER is defined to reacte with driver
>> specific function calls to ack_power changes from the mac80211.
> 
> I know this is kind of an odd. But as carl9170 has TPC settings
> for BlockACK, CF-End and QoS-Null frames, it wouldn't be possible
> to extend "ACK" to "response".
> 
> NB: The HW has also TPC control registers for ERP(RTS/CTS)
> but that's sort of a different issue.


I just double checked with the registers for Atheros ath5k and ath9k
chips and it is register at address offset: 0x80E8 and holds the
transmit powers for self- generated response frames: ACK_PWR (ACK
self-generated response frames), CTS_PWR (CTS self-generated response
frames) and CHRIP_PWR (Chirp self-generated response frames). So at
least for those chips, I can not see an separate TPC setting .. lets say
for BlockACK. Maybe the Atheros Guys can help out here ?
If my proposed FLAG is to generous we can introduce more hardware
specific ones. My TPC algorithm does consider DATA and ACK packets at
the moment as they are most relevant for interference management but
could be extended to other types.


Greetings Thomas

  reply	other threads:[~2012-07-28 14:36 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-27 16:00 [PATCH 0/2] mac80211: Add transmit power control (TPC) support Thomas Huehn
2012-07-27 16:00 ` [PATCH 1/2] mac80211: Add support of transmit power control (TPC) per data packet Thomas Huehn
2012-07-30  7:34   ` Johannes Berg
2012-07-30 11:29     ` Thomas Huehn
2012-07-30 13:38       ` Johannes Berg
2012-07-27 16:00 ` [PATCH 2/2] mac80211: Add transmit power control support (TPC) for ACK packets Thomas Huehn
2012-07-27 21:36   ` Christian Lamparter
2012-07-28 14:36     ` Thomas Huehn [this message]
2012-07-30  7:36   ` Johannes Berg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5013F8EF.10404@net.t-labs.tu-berlin.de \
    --to=thomas@net.t-labs.tu-berlin.de \
    --cc=chunkeey@googlemail.com \
    --cc=johannes.berg@intel.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=nbd@openwrt.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).