Return-path: Received: from mail2.candelatech.com ([208.74.158.173]:52196 "EHLO mail2.candelatech.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753728AbdBARgG (ORCPT ); Wed, 1 Feb 2017 12:36:06 -0500 Subject: Re: [PATCH v2 1/2] cfg80211: Add support to set tx power for a station associated To: Ashok Raj Nagarajan References: <1485888101-23454-1-git-send-email-arnagara@qti.qualcomm.com> <12efc3d8-2274-a6e8-6d74-c06bf89762fb@candelatech.com> <14f6b3acb8b26d0eb806148c0ee47fb0@codeaurora.org> Cc: Ashok Raj Nagarajan , linux-wireless@vger.kernel.org, johannes@sipsolutions.net, ath10k@lists.infradead.org From: Ben Greear Message-ID: <2b68dc12-44bf-f510-339a-3d987a88e8b5@candelatech.com> (sfid-20170201_183737_000911_FB9833B3) Date: Wed, 1 Feb 2017 09:36:00 -0800 MIME-Version: 1.0 In-Reply-To: <14f6b3acb8b26d0eb806148c0ee47fb0@codeaurora.org> Content-Type: text/plain; charset=windows-1252; format=flowed Sender: linux-wireless-owner@vger.kernel.org List-ID: On 02/01/2017 09:27 AM, Ashok Raj Nagarajan wrote: >>> +static int nl80211_parse_sta_txpower_setting(struct genl_info *info, >>> + struct station_parameters *params) >>> +{ >>> + struct cfg80211_registered_device *rdev = info->user_ptr[0]; >>> + enum nl80211_tx_power_setting type; >>> + int idx; >>> + >>> + if (!rdev->ops->set_tx_power || >>> + !wiphy_ext_feature_isset(&rdev->wiphy, >>> + NL80211_EXT_FEATURE_STA_TX_PWR)) >>> + return -EOPNOTSUPP; >> >> Maybe always let a user set to default value even if the driver does not >> support setting specific values? >> > > IMHO, having some default value in place of non-valid values may not be the right way. If a user or user-space script wants to always be able to initialize things to default values, it would be nice if it did not have to pay specific attention to whether the NIC supports STA_TX_PWR feature or not. Since a NIC that does not support this feature will always have sta TX power set to default by definition, that is why I think you should let the call succeed in that case. Thanks, Ben -- Ben Greear Candela Technologies Inc http://www.candelatech.com