All of lore.kernel.org
 help / color / mirror / Atom feed
From: michael@walle.cc (Michael Walle)
To: linux-arm-kernel@lists.infradead.org
Subject: orion/kirkwood and device tree
Date: Wed, 2 Nov 2011 23:03:04 +0100	[thread overview]
Message-ID: <201111022303.04957.michael@walle.cc> (raw)
In-Reply-To: <20111102165057.GU32165@titan.lakedaemon.net>

Am Mittwoch 02 November 2011, 17:50:57 schrieb Jason:
> Michael,
> 
> On Mon, Oct 31, 2011 at 11:50:28PM +0100, Michael Walle wrote:
> > i've already ported some marvell devices to DT. spi-orion, orion-wdt,
> > rtc-mv and mv_cesa. Atm i'm struggling with how to pass
> > kirkwood_mbus_dram_info to the device drivers (the old method is to pass
> > it through platform_data)
> 
> Do you have a public git tree I could pull from, by chance?  I don't
> care about the state, I'd like to learn by example and start pitching
> in.

yeah i pushed it to github:
https://github.com/mwalle/linux/tree/kirkwood-devtree

Am Dienstag 01 November 2011, 07:25:23 schrieben Andrew:
> I have some changes in this area in my tree
Andrew, is your repository public?

> We could maybe put this table into DT?
I don't know if these informations belong to the DT or to generic kirkwood 
arch support (eg. mach-kirkwood/board-dt.c).

In both cases there needs to be some functions to retrieve these properties 
(instead of passing a pointer to some structure around). But i don't know if 
this fits the linux device drivers concept, eg. to be independent from any 
architecture.

So if any kernel hacker is reading this, i'm happy for every hint :) The 
problem is that most kirkwood/orion SoC devices needs a memory map to set up 
its internal memory windows for DMA access. Actually it only uses one window, 
which addresses the main memory. The arch-{kirkwood,orion} code stores the 
needed informations within a static *_mbus_dram_info variable, which is then 
passed as a pointer within platform_data to the device driver.

Is there a best practice for doing such things with device tree aware drivers?

-- 
Michael

  reply	other threads:[~2011-11-02 22:03 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20111031105740.GC29402@lunn.ch>
     [not found] ` <20111031152100.GR32165@titan.lakedaemon.net>
     [not found]   ` <20111031164042.GD29402@lunn.ch>
2011-10-31 22:50     ` orion/kirkwood and device tree Michael Walle
2011-10-31 22:54       ` Russell King - ARM Linux
2011-11-01  6:25       ` Andrew Lunn
2011-11-02 16:50       ` Jason
2011-11-02 22:03         ` Michael Walle [this message]
2011-11-03 18:15           ` memory map in fdt was: " Jason
2011-11-03 18:15             ` Jason
2011-11-03 18:47           ` Jason
2011-11-03 21:50             ` Michael Walle
2011-11-04  9:21               ` Simon Guinot
2011-11-16 23:34                 ` Michael Walle
2011-11-04 14:01               ` Nicolas Pitre
2011-11-06 23:12                 ` Michael Walle
2011-11-06 16:05       ` Andrew Lunn
2011-11-06 22:40         ` Michael Walle
2011-11-07  6:27           ` Andrew Lunn
2011-11-07  2:35         ` Nicolas Pitre
2011-11-07  6:28           ` Andrew Lunn
2011-11-06 16:07       ` [PATCH 1/1] [orion] Consolidate the address map setup on Orion based platforms Andrew Lunn

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=201111022303.04957.michael@walle.cc \
    --to=michael@walle.cc \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.