lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <201203121606.26926.sr@denx.de> Date: Mon, 12 Mar 2012 16:06:26 +0100 From: Stefan Roese <sr@...x.de> To: Rob Herring <robherring2@...il.com> Cc: netdev@...r.kernel.org, Giuseppe Cavallaro <peppe.cavallaro@...com>, Viresh Kumar <viresh.kumar@...com>, devicetree-discuss@...abs.org, linux-arm-kernel@...ts.infradead.org Subject: Re: [PATCH] stmmac: Add device-tree support Hi Rob, On Monday 12 March 2012 15:34:59 Rob Herring wrote: > On 03/12/2012 09:05 AM, Stefan Roese wrote: > > This patch adds support to configure the STMMAC ethernet driver via > > device-tree instead of platform_data. > > > > Currently, only the properties needed on SPEAr600 are provided. All > > other properties should be added once needed on other platforms. > > > > Signed-off-by: Stefan Roese <sr@...x.de> > > Cc: Giuseppe Cavallaro <peppe.cavallaro@...com> > > Cc: Viresh Kumar <viresh.kumar@...com> > > --- > > > > Documentation/devicetree/bindings/net/stmmac.txt | 38 +++++++++++ > > .../net/ethernet/stmicro/stmmac/stmmac_platform.c | 66 > > +++++++++++++++++++- 2 files changed, 103 insertions(+), 1 deletion(-) > > create mode 100644 Documentation/devicetree/bindings/net/stmmac.txt > > > > diff --git a/Documentation/devicetree/bindings/net/stmmac.txt > > b/Documentation/devicetree/bindings/net/stmmac.txt new file mode 100644 > > index 0000000..386a47f > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/net/stmmac.txt > > @@ -0,0 +1,38 @@ > > +* STMicroelectronics 10/100/1000 Ethernet driver (GMAC) > > + > > +Required properties: > > +- compatible: Should be "stm,gmac" > > This is too generic. This should be 1 string per version of h/w. Viresh, Giuseppe, can you please suggest a proper string for the SPEAr600 STMMAC core, including version? > 'stm' should be 'st' according to vendor-prefixes.txt. Okay. > > +- reg: Address and length of the register set for the device > > +- interrupt-parent: Should be the phandle for the interrupt controller > > + that services interrupts for this device > > +- interrupts: Should contain the STMMAC interrupts > > +- interrupt-names: Should contain the interrupt names "macirq" > > + "eth_wake_irq" if this interrupt is supported in the "interrupts" > > + property > > You should be able to tell this from the compatible string and number of > interrupts. Yes. Currently the driver uses platform_get_irq_byname() to register the irq's. That's why I added these properties. Is there something wrong with using it this way? > > +- phy-mode: String, operation mode of the PHY interface. > > + Supported values are: "mii", "rmii", "gmii", "rgmii". > > +- phy-addr: MDIO address of the PHY > > This is normally probed or the mdio bus is a sub-node of the MAC node. > See arch/powerpc/boot/dts/mpc8377_mds.dts for an example. Okay, I'll rework this. > > + > > +Optional properties: > > +- stm,prog-burst-len: Specify the burst length > > +- stm,has-gmac: Indicates that the controller supports 1000Mbps > > +- stm,has-pmt: Indicates that the controller supports power management > > I think these should all be encoded by the compatible string. You mean that by defining a specific compatible string (e.g. "st,gmac-spear600"), these settings are implicitly set? And should therefore be omitted from the dts? Thanks, Stefan -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists