On 04/09/2015 12:58 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]. [1]: http://git.kernel.org/cgit/linux/kernel/git/bluetooth/bluetooth-next.git/tree/net/mac802154/driver-ops.h#n61 > Regards > > Marcel > -- Varka Bhadram ------------------------------------------------------------------------------------------------------------------------------- [ C-DAC is on Social-Media too. Kindly follow us at: Facebook: https://www.facebook.com/CDACINDIA & Twitter: @cdacindia ] This e-mail is for the sole use of the intended recipient(s) and may contain confidential and privileged information. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies and the original message. Any unauthorized review, use, disclosure, dissemination, forwarding, printing or copying of this email is strictly prohibited and appropriate legal action will be taken. ------------------------------------------------------------------------------------------------------------------------------- -- 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