Return-path: Received: from mail.kernel.org ([198.145.29.99]:59008 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757812AbdJRCCu (ORCPT ); Tue, 17 Oct 2017 22:02:50 -0400 MIME-Version: 1.0 In-Reply-To: <1589476.LyXlx7DqTg@debian64> References: <20171006110249.31013-1-nbd@nbd.name> <20171013190749.lzzw4zjzt7fkp3hs@rob-hp-laptop> <3828919b-d634-9f03-22ad-1515219c31d4@nbd.name> <1589476.LyXlx7DqTg@debian64> From: Rob Herring Date: Tue, 17 Oct 2017 21:02:28 -0500 Message-ID: (sfid-20171018_040256_034213_A51EE238) Subject: Re: [PATCH v6 1/3] dt-bindings: net: add mt76 wireless device binding To: Christian Lamparter Cc: Felix Fietkau , linux-wireless , Kalle Valo , "devicetree@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-wireless-owner@vger.kernel.org List-ID: On Sat, Oct 14, 2017 at 10:43 AM, Christian Lamparter wrote: > On Saturday, October 14, 2017 9:20:46 AM CEST Felix Fietkau wrote: >> On 2017-10-13 21:07, Rob Herring wrote: >> > On Fri, Oct 06, 2017 at 01:02:47PM +0200, Felix Fietkau wrote: >> >> Add documentation describing how device tree can be used to configure >> >> wireless chips supported by the mt76 driver. >> >> >> >> Signed-off-by: Felix Fietkau >> >> --- >> >> .../bindings/net/wireless/mediatek,mt76.txt | 24 ++++++++++++++++++++++ >> >> 1 file changed, 24 insertions(+) >> >> create mode 100644 Documentation/devicetree/bindings/net/wireless/mediatek,mt76.txt >> >> >> >> diff --git a/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.txt b/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.txt >> >> new file mode 100644 >> >> index 000000000000..19522ab97d62 >> >> --- /dev/null >> >> +++ b/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.txt >> >> @@ -0,0 +1,24 @@ >> >> +* MediaTek mt76xx devices >> >> + >> >> +This node provides properties for configuring the MediaTek mt76xx wireless >> >> +device. The node is expected to be specified as a child node of the PCI >> >> +controller to which the wireless chip is connected. >> >> + >> >> +Optional properties: >> >> + >> >> +- mac-address: See ethernet.txt in the parent directory >> >> +- local-mac-address: See ethernet.txt in the parent directory >> >> +- ieee80211-freq-limit: See ieee80211.txt >> >> +- mediatek,mtd-eeprom: Specify a MTD partition + offset containing EEPROM data >> > >> > MTD is a Linuxism. And is an EEPROM the only supported device? I'd >> > suggest naming if after what the data contains. >> PCI cards with this kind of wireless chip usually come with some form of >> EEPROM or use the on-chip OTP ROM. >> This property is for the case where the chip is built into an embedded >> device and the data that would otherwise be on an EEPROM is stored on a >> MTD partition instead. >> The EEPROM data itself contains multiple things: calibration data, >> hardware capabilities, MAC address, etc. >> I couldn't think of a better name for it, do you have any suggestions? Naming is hard. > This sort of reminds me of the failed ath9k nvmem patches: > https://patchwork.kernel.org/patch/9622127/ > > Which uses the nvmem system. > > https://github.com/torvalds/linux/blob/master/Documentation/nvmem/nvmem.txt > > Rob, would this be acceptable? Yeah, alignment with other drivers is a good thing. Rob