[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180501162718.GA13139@rob-hp-laptop>
Date: Tue, 1 May 2018 11:27:18 -0500
From: Rob Herring <robh@...nel.org>
To: Chen-Yu Tsai <wens@...e.org>
Cc: Maxime Ripard <maxime.ripard@...tlin.com>,
Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>,
Giuseppe Cavallaro <peppe.cavallaro@...com>,
Mark Rutland <mark.rutland@....com>,
Mark Brown <broonie@...nel.org>,
linux-arm-kernel@...ts.infradead.org, linux-clk@...r.kernel.org,
devicetree@...r.kernel.org, netdev@...r.kernel.org,
Corentin Labbe <clabbe.montjoie@...il.com>,
Icenowy Zheng <icenowy@...c.io>
Subject: Re: [PATCH net-next v2 05/15] dt-bindings: net: dwmac-sun8i:
simplify description of syscon property
On Wed, May 02, 2018 at 12:12:17AM +0800, Chen-Yu Tsai wrote:
> The syscon property is used to point to the device that holds the glue
> layer control register known as the "EMAC (or GMAC) clock register".
>
> We do not need to explicitly list what compatible strings are needed, as
> this information is readily available in the user manuals. Also the
> "syscon" device type is more of an implementation detail. There are many
> ways to access a register not in a device's address range, the syscon
> interface being the most generic and unrestricted one.
>
> Simplify the description so that it says what it is supposed to
> describe.
>
> Signed-off-by: Chen-Yu Tsai <wens@...e.org>
> ---
> Documentation/devicetree/bindings/net/dwmac-sun8i.txt | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
Reviewed-by: Rob Herring <robh@...nel.org>
Powered by blists - more mailing lists