devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
To: Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>
Cc: Lior Amsalem <alior-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>,
	Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>,
	Ike Pan <ike.pan-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>,
	Nadav Haklai <nadavh-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>,
	David Marlin <dmarlin-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	Yehuda Yitschak <yehuday-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>,
	Jani Monoses
	<jani.monoses-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>,
	Simon Guinot
	<simon.guinot-jKBdWWKqtFpg9hUCZPvPmw@public.gmane.org>,
	Tawfik Bayouk <tawfik-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>,
	Dan Frazier
	<dann.frazier-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>,
	Eran Ben-Avi <benavi-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>,
	Leif Lindholm <leif.lindholm-5wv7dgnIgG8@public.gmane.org>,
	Sebastian Hesselbarth
	<sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Jon Masters <jcm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
	Florian Fainelli
	<florian-p3rKhJxN3npAfugRpC6u6w@public.gmane.org>,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	Chris Van Hoof <vanhoof-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org>,
	Maen Suleiman <maen-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>,
	Shadi Ammouri <shadi-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org>
Subject: Re: [PATCH] arm: mvebu: support for the new Armada XP evaluation board(DB-MV784MP-GP)
Date: Thu, 31 Jan 2013 10:50:25 +0100	[thread overview]
Message-ID: <20130131095025.GC20242@lunn.ch> (raw)
In-Reply-To: <20130130181035.GP7717-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>

On Wed, Jan 30, 2013 at 01:10:35PM -0500, Jason Cooper wrote:
> 
> Gregory,
> 
> Adding devicetree-discuss.
> 
> Guys, how do you prefer to handle plug-in RAM modules?  describe the
> soldered-in amount, or?
> 
> On Wed, Jan 30, 2013 at 06:35:17PM +0100, Gregory CLEMENT wrote:
> > On 01/30/2013 06:33 PM, Jason Cooper wrote:
> > > On Wed, Jan 30, 2013 at 06:26:14PM +0100, Gregory CLEMENT wrote:
> > >> This is the new Armada XP evaluation board from Marvell. It comes with
> > >> a RS232 port over USB, a SATA link, an internal SSD, 4 Ethernet
> > >> Gigabit links.
> > >>
> > >> Support for USB (Host and device), SDIO, PCIe will be added as drivers
> > >> when they become available for Armada XP in mainline.
> > >>
> > >> Signed-off-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
> > >> ---
> > >>  arch/arm/boot/dts/Makefile         |    1 +
> > >>  arch/arm/boot/dts/armada-xp-gp.dts |   94 ++++++++++++++++++++++++++++++++++++
> > >>  2 files changed, 95 insertions(+)
> > >>  create mode 100644 arch/arm/boot/dts/armada-xp-gp.dts
> > >>
> ...
> > >> +	memory {
> > >> +		device_type = "memory";
> > >> +		reg = <0x00000000 0xC0000000>; /* 3 GB */
> > > 
> > > Is this the soldered on the board amount, or the amount with an
> > > expandable slot filled?
> > 
> > It is not soldered, so you can change the amount of memory
> 
> Well, there's something we don't encounter too much in the embedded
> world.  ;-)  I'm inclined to think the entry here should be the soldered
> on amount, and the bootloader would (hypothetically) change the DT to
> the probed amount.  But I haven't seen a precedent set yet...

Hi Jason

We have the same issue with OpenBox AX3. All the devices we have in
our hands have 1GB soldered down, plus 2GB on expansion. The DT
currently has the full 3GB.

However, there has been discussion of reducing this down to 1GB since
both Thomas and Gregory have unstable systems with the full 3GB, but
mine box is stable with the full 3GB. So i don't know how strong a
precedent that sets....

	  Andrew

  parent reply	other threads:[~2013-01-31  9:50 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1359566774-27669-1-git-send-email-gregory.clement@free-electrons.com>
     [not found] ` <1359566774-27669-2-git-send-email-gregory.clement@free-electrons.com>
     [not found]   ` <20130130173306.GO7717@titan.lakedaemon.net>
     [not found]     ` <510959D5.1010305@free-electrons.com>
2013-01-30 18:10       ` [PATCH] arm: mvebu: support for the new Armada XP evaluation board(DB-MV784MP-GP) Jason Cooper
     [not found]         ` <20130130181035.GP7717-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2013-01-31  9:50           ` Andrew Lunn [this message]
     [not found]             ` <20130131095025.GC20242-g2DYL2Zd6BY@public.gmane.org>
2013-01-31 10:07               ` Thomas Petazzoni
2013-01-31 12:02                 ` Gregory CLEMENT
     [not found]                   ` <510A5D38.9040608-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-01-31 12:44                     ` Jason Cooper
2013-01-31 12:07           ` Gregory CLEMENT
     [not found]             ` <510A5E87.60509-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-01-31 12:45               ` Jason Cooper
     [not found]                 ` <20130131124542.GB7717-u4khhh1J0LxI1Ri9qeTfzeTW4wlIGRCZ@public.gmane.org>
2013-01-31 12:50                   ` Gregory CLEMENT
     [not found]                     ` <510A689B.6040407-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2013-01-31 12:52                       ` Jason Cooper
2013-01-31 16:22                       ` Andrew Lunn
     [not found]                         ` <20130131162254.GD20242-g2DYL2Zd6BY@public.gmane.org>
2013-01-31 16:38                           ` Andrew Lunn
     [not found]                             ` <20130131163814.GE20242-g2DYL2Zd6BY@public.gmane.org>
2013-01-31 21:08                               ` Arnd Bergmann

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=20130131095025.GC20242@lunn.ch \
    --to=andrew-g2dyl2zd6by@public.gmane.org \
    --cc=alior-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org \
    --cc=benavi-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org \
    --cc=dann.frazier-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=dmarlin-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=florian-p3rKhJxN3npAfugRpC6u6w@public.gmane.org \
    --cc=ike.pan-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org \
    --cc=jani.monoses-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org \
    --cc=jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org \
    --cc=jcm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=leif.lindholm-5wv7dgnIgG8@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=maen-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org \
    --cc=nadavh-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org \
    --cc=sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=shadi-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org \
    --cc=simon.guinot-jKBdWWKqtFpg9hUCZPvPmw@public.gmane.org \
    --cc=tawfik-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.org \
    --cc=vanhoof-Z7WLFzj8eWMS+FvcfC7Uqw@public.gmane.org \
    --cc=yehuday-eYqpPyKDWXRBDgjK7y7TUQ@public.gmane.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).