On 2019-04-02 02:16, Bob Copeland wrote:
Hi Balaji,
I started playing with these patches a bit -- they could be useful for
my use case, but I had a question:
+ * @power: tx power (in dBm) to be used for sending data traffic. If
tx power
+ * is not provided, the default per-interface tx power setting will
be
+ * overriding. Driver should be picking up the lowest tx power,
either tx
+ * power per-interface or per-station.
Should this really dBm? It's a bit asymmetric for WIPHY_TX_POWER_LEVEL
to
use mBm and this to use dBm, and I might want to adjust in half-dB
steps if
supported by hardware. Also allocating an s16 is a bit much for dBm.
if user will send mBm value then driver has to convert to dBm because
firmware will expect the value in dBm.
Please refer johannes comments on following patch " [EXT] Re: [PATCH]
iw: Add support for controlling tx power for per station"
for the reason why we opted dBm.
Yeah, s16 is required if units in mBm. i will modify to s8 in next
version patch.
Regards,
Balaji.
