linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: Andrew Lunn <andrew@lunn.ch>, Jason Cooper <jason@lakedaemon.net>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
	"devicetree-discuss@lists.ozlabs.org"
	<devicetree-discuss@lists.ozlabs.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Jason Gunthorpe <jgunthorpe@obsidianresearch.com>,
	Lennert Buytenhek <buytenh@wantstofly.org>,
	"linuxppc-dev@lists.ozlabs.org list"
	<linuxppc-dev@lists.ozlabs.org>,
	David Miller <davem@davemloft.net>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 2/2] net: mv643xx_eth: proper initialization for Kirkwood SoCs
Date: Fri, 24 May 2013 19:25:11 +0200	[thread overview]
Message-ID: <519FA277.3010802@gmail.com> (raw)
In-Reply-To: <20130524171300.GD18614@n2100.arm.linux.org.uk>

On 05/24/2013 07:13 PM, Russell King - ARM Linux wrote:
> Do you really want that on ARM?  Given the fiasco with the location of
> the registers, are you sure you want to place more trust in that
> direction?  Does it give you a warm fuzzy feeling to know that you
> might have to work out some way to patch vendor supplied bytecode?

Don't get me wrong. I want mv643xx_eth DT or even platform_data to
evolve to a fully self configured driver not depending on proper u-boot
setup at all.

But I don't want to go that road now and I wonder if it might be safer
for us (and PPC guys) if we start mv643xx_eth over from scratch one day.

For this patch set, I want a basic DT binding that works. Patching the
driver to play with kirkwood loosing the MAC and other important
registers is not my main concern here. If clearing that one bit doesn't
help for all kirkwood boards, I'd rather leave the non-gating
workaround.

mv643xx_eth not knowing DT for ARM is stalling last important bits for
Orion SoCs. I want this to go in first as with David another maintainer
is involved.

Sebastian

  reply	other threads:[~2013-05-24 17:25 UTC|newest]

Thread overview: 93+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1367854420-8006-1-git-send-email-sebastian.hesselbarth@gmail.com>
2013-05-21 16:41 ` [PATCH v4 00/12] net: mv643xx_eth DT support and fixes Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 01/12] net: mv643xx_eth: use phy_disconnect instead of phy_detach Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 02/12] net: mv643xx_eth: use managed devm_ioremap for port registers Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 03/12] net: mv643xx_eth: add phy_node to platform_data struct Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 04/12] net: mv643xx_eth: use of_phy_connect if phy_node present Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 05/12] net: mv643xx_eth: add DT parsing support Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 06/12] ARM: dove: add gigabit ethernet and mvmdio device tree nodes Sebastian Hesselbarth
2013-05-21 17:48     ` Andrew Lunn
2013-05-22  9:43       ` Sebastian Hesselbarth
2013-05-22 10:04         ` tiejun.chen
2013-05-22 10:13           ` Sebastian Hesselbarth
2013-05-22 13:10             ` Jason Cooper
2013-05-22 16:59               ` Jason Gunthorpe
2013-05-22 17:01                 ` Jason Cooper
2013-05-22 17:32                 ` Sebastian Hesselbarth
2013-05-22 17:35                   ` Jason Cooper
2013-05-22 17:42                     ` Sebastian Hesselbarth
2013-05-22 17:48                       ` Jason Cooper
2013-05-22 18:44                         ` Sebastian Hesselbarth
2013-05-22 18:49                           ` Jason Cooper
2013-05-22 18:55                             ` Sebastian Hesselbarth
2013-05-22 18:58                               ` Jason Cooper
2013-05-22 19:52                         ` Sebastian Hesselbarth
2013-05-22 18:24                   ` Jason Gunthorpe
2013-05-22 18:51                     ` Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 07/12] ARM: kirkwood: " Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 08/12] ARM: orion5x: " Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 09/12] ARM: dove: remove legacy mv643xx_eth setup Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 10/12] ARM: kirkwood: remove legacy clk alias for mv643xx_eth Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 11/12] ARM: kirkwood: remove redundant DT board files Sebastian Hesselbarth
2013-05-22 20:36     ` Simon Baatz
2013-05-22 20:55       ` Sebastian Hesselbarth
2013-05-22 21:02         ` Jason Cooper
2013-05-22 21:17           ` Sebastian Hesselbarth
2013-05-21 16:41   ` [PATCH v4 12/12] ARM: orion5x: remove legacy mv643xx_eth board setup Sebastian Hesselbarth
2013-05-22 16:16   ` [PATCH v4 00/12] net: mv643xx_eth DT support and fixes Andrew Lunn
2013-05-22 20:04   ` [PATCH 1/2] ARM: kirkwood: proper retain MAC address workaround on DT ethernet Sebastian Hesselbarth
2013-05-22 20:04     ` [PATCH 2/2] net: mv643xx_eth: proper initialization for Kirkwood SoCs Sebastian Hesselbarth
2013-05-22 20:16       ` Jason Gunthorpe
2013-05-22 21:02         ` Sebastian Hesselbarth
2013-05-23 16:01         ` Jason Cooper
2013-05-23 17:11           ` Jason Gunthorpe
2013-05-23 17:23             ` Jason Cooper
2013-05-23 17:53               ` Jason Gunthorpe
2013-05-23 18:40                 ` Jason Cooper
2013-05-23 19:01                   ` Jason Gunthorpe
2013-05-24 16:46                     ` Jason Cooper
2013-05-24 16:53                       ` Andrew Lunn
2013-05-24 17:03                         ` Jason Cooper
2013-05-24 17:33                       ` Jason Gunthorpe
2013-05-28 18:02                         ` Jason Cooper
2013-05-23 22:40                   ` Sebastian Hesselbarth
2013-05-24 11:03                     ` Linus Walleij
2013-05-24 17:01                       ` Jason Cooper
2013-05-24 17:13                         ` Russell King - ARM Linux
2013-05-24 17:25                           ` Sebastian Hesselbarth [this message]
2013-05-24 16:53                     ` Jason Cooper
2013-05-26  4:04     ` [PATCH 1/2] ARM: kirkwood: proper retain MAC address workaround on DT ethernet David Miller
2013-05-26 20:06       ` Sebastian Hesselbarth
2013-05-27  9:23         ` David Miller
2013-05-27  9:39           ` Benjamin Herrenschmidt
2013-05-27 10:24             ` Sebastian Hesselbarth
2013-05-27 11:50               ` Benjamin Herrenschmidt
2013-05-27 12:47                 ` Arnd Bergmann
2013-05-27 21:50                   ` Benjamin Herrenschmidt
2013-05-27 22:12                     ` Sebastian Hesselbarth
2013-05-27 22:17                     ` David Miller
2013-05-27 20:18                 ` David Miller
2013-05-27 21:48                   ` Benjamin Herrenschmidt
2013-05-27  9:38         ` Benjamin Herrenschmidt
2013-05-29 19:32   ` [PATCH v5 00/13] net: mv643xx_eth DT support and fixes Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 01/13] net: mv643xx_eth: use phy_disconnect instead of phy_detach Sebastian Hesselbarth
2013-05-29 20:00       ` Jason Cooper
2013-05-29 19:32     ` [PATCH v5 02/13] net: mv643xx_eth: use managed devm_ioremap for port registers Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 03/13] net: mv643xx_eth: add phy_node to platform_data struct Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 04/13] net: mv643xx_eth: use of_phy_connect if phy_node present Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 05/13] net: mv643xx_eth: proper initialization for Kirkwood SoCs Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 06/13] net: mv643xx_eth: add DT parsing support Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 07/13] ARM: dove: add gigabit ethernet and mvmdio device tree nodes Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 08/13] ARM: kirkwood: " Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 09/13] ARM: orion5x: " Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 10/13] ARM: dove: remove legacy mv643xx_eth setup Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 11/13] ARM: kirkwood: remove legacy clk alias for mv643xx_eth Sebastian Hesselbarth
2013-05-29 19:32     ` [PATCH v5 12/13] ARM: kirkwood: remove redundant DT board files Sebastian Hesselbarth
2013-05-30  9:06       ` Arnaud Ebalard
2013-05-30  9:08         ` Sebastian Hesselbarth
2013-05-30 19:37         ` Jason Cooper
2013-05-30 22:28           ` Arnaud Ebalard
2013-05-31 11:54             ` Jason Cooper
2013-05-29 19:32     ` [PATCH v5 13/13] ARM: orion5x: remove legacy mv643xx_eth board setup Sebastian Hesselbarth
2013-05-31  0:55     ` [PATCH v5 00/13] net: mv643xx_eth DT support and fixes David Miller
2013-05-31  6:28       ` Sebastian Hesselbarth
2013-05-31  9:32         ` David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=519FA277.3010802@gmail.com \
    --to=sebastian.hesselbarth@gmail.com \
    --cc=andrew@lunn.ch \
    --cc=buytenh@wantstofly.org \
    --cc=davem@davemloft.net \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=jason@lakedaemon.net \
    --cc=jgunthorpe@obsidianresearch.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).