On Thursday 29 September 2016, Felix Fietkau wrote:
> On 2016-09-08 12:54, Kalle Valo wrote:
> > Felix Fietkau <n...@nbd.name> writes:
> > 
> >> Signed-off-by: Felix Fietkau <n...@nbd.name>
> >> ---
> >>  .../bindings/net/wireless/mediatek,mt76.txt        | 26 
> >> ++++++++++++++++++++++
> >>  1 file changed, 26 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 0000000..d51c35f
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/net/wireless/mediatek,mt76.txt
> >> @@ -0,0 +1,26 @@
> >> +* 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
> >> +- mediatek,2ghz: Override the 2.4 GHz band capability from EEPROM
> >> +- mediatek,5ghz: Override the 5 GHz band capability from EEPROM
> >> +- mediatek,mtd-eeprom: Specify a MTD partition + offset containing EEPROM 
> >> data
> >> +
> >> +&pcie {
> >> +  status = "okay";
> >> +
> >> +  pcie0 {
> >> +          mt76@0,0 {
> >> +                  reg = <0x0000 0 0 0 0>;

Maybe have an examplep of a real register address other than zero?

> >> +                  device_type = "pci";
> >> +                  mediatek,mtd-eeprom = <&factory 0x8000>;
> >> +                  mediatek,2ghz = <0>;

It's not clear what the possible values for the 2ghz property are,
can you be more verbose in the description? How is <0> different
from no property?

        Arnd

Reply via email to