From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mout.kundenserver.de ([212.227.17.24]:60094 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750870AbaK1KXx (ORCPT ); Fri, 28 Nov 2014 05:23:53 -0500 From: Arnd Bergmann To: Kalle Valo Cc: ath10k@lists.infradead.org, devicetree@vger.kernel.org, linux-wireless@vger.kernel.org, toshik@chromium.org Subject: Re: [PATCH v4 1/2] dt: bindings: add ath10k wireless device Date: Fri, 28 Nov 2014 11:23:08 +0100 Message-ID: <5247714.HacITJUipJ@wuerfel> (sfid-20141128_112358_615379_5E9C98F0) In-Reply-To: <87fvd3sl9c.fsf@kamboji.qca.qualcomm.com> References: <20141127120727.17546.61202.stgit@potku.adurom.net> <3202372.2AaH3c4qv1@wuerfel> <87fvd3sl9c.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Sender: linux-wireless-owner@vger.kernel.org List-ID: On Friday 28 November 2014 11:54:39 Kalle Valo wrote: > > Right now ath10k does not support that, but in the future we might want > to add mac address as well. We need to do some testing with the firmware > to make sure that we can safely change the "main" address from ath10k. This is from Documentation/devicetree/bindings/net/ethernet.txt: - local-mac-address: array of 6 bytes, specifies the MAC address that was assigned to the network device; - mac-address: array of 6 bytes, specifies the MAC address that was last used by the boot program; should be used in cases where the MAC address assigned to the device by the boot program is different from the "local-mac-address" property; If the address is fixed, you might still be able to use the local-mac-address property to communicate the main address. Normally you only have to pass the mac address in DT if the device itself doesn't know the address. Do you know where the main address is stored? If it's always known to the card, we don't need to pass it, and if the card doesn't know it, I suspect it would be safe to change it. > And because of Virtual AP (mBSSID) feature we actually would need to > provide multiple addresses, not just one. Maybe with addr_mask like > struct wiphy has? > > * @perm_addr: permanent MAC address of this device > * @addr_mask: If the device supports multiple MAC addresses by masking, > * set this to a mask with variable bits set to 1, e.g. if the last > * four bits are variable then set it to 00-00-00-00-00-0f. The actual > * variable bits shall be determined by the interfaces added, with > * interfaces not matching the mask being rejected to be brought up. > > /* permanent MAC address(es) */ > u8 perm_addr[ETH_ALEN]; > u8 addr_mask[ETH_ALEN]; We don't have a common binding for this yet, I think drivers that do this at the moment just know how many addresses they are allowed to take. If the mask is generally considered useful, we could probably add that to the binding though. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH v4 1/2] dt: bindings: add ath10k wireless device Date: Fri, 28 Nov 2014 11:23:08 +0100 Message-ID: <5247714.HacITJUipJ@wuerfel> References: <20141127120727.17546.61202.stgit@potku.adurom.net> <3202372.2AaH3c4qv1@wuerfel> <87fvd3sl9c.fsf@kamboji.qca.qualcomm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: In-Reply-To: <87fvd3sl9c.fsf-HodKDYzPHsUD5k0oWYwrnHL1okKdlPRT@public.gmane.org> Sender: linux-wireless-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Kalle Valo Cc: ath10k-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-wireless-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, toshik-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org List-Id: devicetree@vger.kernel.org On Friday 28 November 2014 11:54:39 Kalle Valo wrote: > > Right now ath10k does not support that, but in the future we might want > to add mac address as well. We need to do some testing with the firmware > to make sure that we can safely change the "main" address from ath10k. This is from Documentation/devicetree/bindings/net/ethernet.txt: - local-mac-address: array of 6 bytes, specifies the MAC address that was assigned to the network device; - mac-address: array of 6 bytes, specifies the MAC address that was last used by the boot program; should be used in cases where the MAC address assigned to the device by the boot program is different from the "local-mac-address" property; If the address is fixed, you might still be able to use the local-mac-address property to communicate the main address. Normally you only have to pass the mac address in DT if the device itself doesn't know the address. Do you know where the main address is stored? If it's always known to the card, we don't need to pass it, and if the card doesn't know it, I suspect it would be safe to change it. > And because of Virtual AP (mBSSID) feature we actually would need to > provide multiple addresses, not just one. Maybe with addr_mask like > struct wiphy has? > > * @perm_addr: permanent MAC address of this device > * @addr_mask: If the device supports multiple MAC addresses by masking, > * set this to a mask with variable bits set to 1, e.g. if the last > * four bits are variable then set it to 00-00-00-00-00-0f. The actual > * variable bits shall be determined by the interfaces added, with > * interfaces not matching the mask being rejected to be brought up. > > /* permanent MAC address(es) */ > u8 perm_addr[ETH_ALEN]; > u8 addr_mask[ETH_ALEN]; We don't have a common binding for this yet, I think drivers that do this at the moment just know how many addresses they are allowed to take. If the mask is generally considered useful, we could probably add that to the binding though. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mout.kundenserver.de ([212.227.17.24]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1XuIiM-0005gZ-Dv for ath10k@lists.infradead.org; Fri, 28 Nov 2014 10:23:51 +0000 From: Arnd Bergmann Subject: Re: [PATCH v4 1/2] dt: bindings: add ath10k wireless device Date: Fri, 28 Nov 2014 11:23:08 +0100 Message-ID: <5247714.HacITJUipJ@wuerfel> In-Reply-To: <87fvd3sl9c.fsf@kamboji.qca.qualcomm.com> References: <20141127120727.17546.61202.stgit@potku.adurom.net> <3202372.2AaH3c4qv1@wuerfel> <87fvd3sl9c.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ath10k" Errors-To: ath10k-bounces+kvalo=adurom.com@lists.infradead.org To: Kalle Valo Cc: devicetree@vger.kernel.org, linux-wireless@vger.kernel.org, toshik@chromium.org, ath10k@lists.infradead.org On Friday 28 November 2014 11:54:39 Kalle Valo wrote: > > Right now ath10k does not support that, but in the future we might want > to add mac address as well. We need to do some testing with the firmware > to make sure that we can safely change the "main" address from ath10k. This is from Documentation/devicetree/bindings/net/ethernet.txt: - local-mac-address: array of 6 bytes, specifies the MAC address that was assigned to the network device; - mac-address: array of 6 bytes, specifies the MAC address that was last used by the boot program; should be used in cases where the MAC address assigned to the device by the boot program is different from the "local-mac-address" property; If the address is fixed, you might still be able to use the local-mac-address property to communicate the main address. Normally you only have to pass the mac address in DT if the device itself doesn't know the address. Do you know where the main address is stored? If it's always known to the card, we don't need to pass it, and if the card doesn't know it, I suspect it would be safe to change it. > And because of Virtual AP (mBSSID) feature we actually would need to > provide multiple addresses, not just one. Maybe with addr_mask like > struct wiphy has? > > * @perm_addr: permanent MAC address of this device > * @addr_mask: If the device supports multiple MAC addresses by masking, > * set this to a mask with variable bits set to 1, e.g. if the last > * four bits are variable then set it to 00-00-00-00-00-0f. The actual > * variable bits shall be determined by the interfaces added, with > * interfaces not matching the mask being rejected to be brought up. > > /* permanent MAC address(es) */ > u8 perm_addr[ETH_ALEN]; > u8 addr_mask[ETH_ALEN]; We don't have a common binding for this yet, I think drivers that do this at the moment just know how many addresses they are allowed to take. If the mask is generally considered useful, we could probably add that to the binding though. Arnd _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k