[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160812183732.GA3838@rob-hp-laptop>
Date: Fri, 12 Aug 2016 13:37:32 -0500
From: Rob Herring <robh@...nel.org>
To: Heiko Stübner <heiko@...ech.de>
Cc: Roger Chen <roger.chen@...k-chips.com>,
linux-rockchip@...ts.infradead.org, zhengxing@...k-chips.com,
pawel.moll@....com, mark.rutland@....com,
ijc+devicetree@...lion.org.uk, galak@...eaurora.org,
peppe.cavallaro@...com, alexandre.torgue@...com,
davem@...emloft.net, devicetree@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
netdev@...r.kernel.org
Subject: Re: [PATCH 1/3] net: stmmac: dwmac-rk: add rk3366 & rk3399 specific
data
On Thu, Aug 11, 2016 at 10:12:44AM +0200, Heiko Stübner wrote:
> Hi Roger,
>
> Am Donnerstag, 11. August 2016, 15:24:46 schrieb Roger Chen:
> > Add constants and callback functions for the dwmac on rk3228/rk3229 socs.
> > As can be seen, the base structure is the same, only registers and the
> > bits in them moved slightly.
> >
> > Signed-off-by: Roger Chen <roger.chen@...k-chips.com>
> > ---
> > .../devicetree/bindings/net/rockchip-dwmac.txt | 4 +-
> > drivers/net/ethernet/stmicro/stmmac/dwmac-rk.c | 226
> > +++++++++++++++++++++ 2 files changed, 228 insertions(+), 2 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/net/rockchip-dwmac.txt
> > b/Documentation/devicetree/bindings/net/rockchip-dwmac.txt index
> > cccd945..8c066e6 100644
> > --- a/Documentation/devicetree/bindings/net/rockchip-dwmac.txt
> > +++ b/Documentation/devicetree/bindings/net/rockchip-dwmac.txt
> > @@ -3,8 +3,8 @@ Rockchip SoC RK3288 10/100/1000 Ethernet driver(GMAC)
> > The device node has following properties.
> >
> > Required properties:
> > - - compatible: Can be one of "rockchip,rk3228-gmac",
> > "rockchip,rk3288-gmac", -
> > "rockchip,rk3368-gmac"
> > + - compatible: Can be one of "rockchip,rk3288-gmac",
>
> you're dropping the rk3228 here.
And please do 1 compatible per line unless multiple strings apply to a
given platform.
Rob
Powered by blists - more mailing lists