linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: gerlando.falauto@keymile.com (Gerlando Falauto)
To: linux-arm-kernel@lists.infradead.org
Subject: DT version of kirkwood_ge0x_init()
Date: Tue, 04 Jun 2013 22:53:40 +0200	[thread overview]
Message-ID: <51AE53D4.5070208@keymile.com> (raw)
In-Reply-To: <51ADC2C2.6010106@gmail.com>

Hi Sebastian,

thanks for your answer and your hard work about this whole cleanup.

Two more questions though -- the whole pulling sequence and the 
mainlining process is still very confusing to me so I really have no 
idea where to look or what to expect.

1) Has this been pulled somewhere? You provided a pointer to the LKML 
but I assume this would pop up by way of an -arm tree... is that right?
I'd like to start using that.

2) About my old fixes for the IRQ conflicts on Kirkwood, I saw that 
Thomas Gleixner's rework was pulled into the "tip" tree, but that's more 
genirq related. Did you (or anyone else) also follow up on that and make 
the changes for Kirkwood in order to enable the separate mask registers? 
I'd like to test that (that's actually why I started looking at recent 
kernels with FDT support to begin with).
Or shall I just follow Thomas' proof of concept on irq-sun4i and do the 
same for Kirkwood?

Sorry, I am really more than lost when it's about tracking 
how/where/when changes are pulled. Any "mainlining for dummies" pointer 
would be more than appreciated.

Thank you,
Gerlando

On 06/04/2013 12:34 PM, Sebastian Hesselbarth wrote:
> On 06/04/13 12:18, Gerlando Falauto wrote:
>> I noticed how most of the DT-aware board-setup files only have a single
>> <board>_init() function, calling kirkwood_ge00_init() with a struct
>> mv643xx_eth_platform_data as a single argument.
>>
>> I was wondering -- is there a reason why we cannot remove all this
>> board-specific code and move all this to the DT?
>
> Gerlando,
>
> DT for mv643xx_eth is on the way (https://lkml.org/lkml/2013/5/29/527).
> We wait for the driver to surface to relax branch dependencies and then
> move all DT Orion SoCs to it.
>
>   > I would really love to have all our boards under a single
>   > CONFIG_<FAMILY>_DT and a single compatible string, with all the
>   > differences within the DTs itself -- no more #ifdef CONFIG_<BOARD>,
>   > no more of_machine_is_compatible("boardXXX").
>
> All those will happen if there is DT support for mv643xx_eth which
> is the only driver left without DT and board dependencies. But there
> will be no CONFIG_LACIE_DT or whatever, but just CONFIG_KIRKWOOD_DT
> and board dependent stuff described in the corresponding dts.
>
> Sebastian
>

  parent reply	other threads:[~2013-06-04 20:53 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-06-04 10:18 DT version of kirkwood_ge0x_init() Gerlando Falauto
2013-06-04 10:34 ` Sebastian Hesselbarth
2013-06-04 10:43   ` Jason Cooper
2013-06-04 11:59     ` Simon Guinot
2013-06-04 12:05       ` Jason Cooper
2013-06-04 12:18         ` Simon Guinot
2013-06-04 13:10         ` Jason Cooper
2013-06-04 20:53   ` Gerlando Falauto [this message]
2013-06-05  9:04     ` Sebastian Hesselbarth
2013-06-05  9:37       ` Gerlando Falauto
2013-06-05  9:45         ` Gregory CLEMENT
2013-06-05  9:49         ` Sebastian Hesselbarth
2013-06-05  9:55           ` Gerlando Falauto
2013-06-05 13:23       ` Jason Cooper

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=51AE53D4.5070208@keymile.com \
    --to=gerlando.falauto@keymile.com \
    --cc=linux-arm-kernel@lists.infradead.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).