On 04/09/2015 01:16 PM, Marcel Holtmann wrote:
> Hi Varka,
>
>>>> Available 802.15.4 devices support multiple power levels,
>>>> which includes positive and negative values.This patch fix
>>>> set_txpower() operation arugument type to signed from int
>>>>
>>>> Signed-off-by: Varka Bhadram <varkab@xxxxxxx>
>>>> ---
>>>> include/net/mac802154.h | 2 +-
>>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>> diff --git a/include/net/mac802154.h b/include/net/mac802154.h
>>>> index fb4e8a3d..1213419 100644
>>>> --- a/include/net/mac802154.h
>>>> +++ b/include/net/mac802154.h
>>>> @@ -213,7 +213,7 @@ struct ieee802154_ops {
>>>> int (*set_hw_addr_filt)(struct ieee802154_hw *hw,
>>>> struct ieee802154_hw_addr_filt *filt,
>>>> unsigned long changed);
>>>> - int (*set_txpower)(struct ieee802154_hw *hw, int db);
>>>> + int (*set_txpower)(struct ieee802154_hw *hw, s8 dbm);
>>> the commit message is pretty much non-sense. int is as signed as s8.
>> The netlink attribute for the power level is s8. At mac level the values
>> stored in s8 only [1].
> sure, but the commit message is still non-sense.
Ok. I will change the commit and also i need to add the change to at86rf230 driver.
--
Thanks
Varka Bhadram
--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
