linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: ben-linux@fluff.org (Ben Dooks)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] s5pv210: Change the base ram address to 0x3000'0000
Date: Fri, 28 May 2010 07:02:23 +0100	[thread overview]
Message-ID: <20100528060223.GU4720@trinity.fluff.org> (raw)
In-Reply-To: <AANLkTimBlk4zAA_dwvENfmhr-bUb3VXRjJ5wsxI76Oih@mail.gmail.com>

On Fri, May 28, 2010 at 02:25:41PM +0900, Kyungmin Park wrote:
> On Fri, May 28, 2010 at 1:20 PM, Ben Dooks <ben-linux@fluff.org> wrote:
> > On Fri, May 28, 2010 at 01:02:24PM +0900, Kyungmin Park wrote:
> >> On Fri, May 28, 2010 at 12:53 PM, Ben Dooks <ben-linux@fluff.org> wrote:
> >> > On Fri, May 28, 2010 at 11:35:59AM +0900, Kyungmin Park wrote:
> >> >> s5pc110 (aka s5pv210) has 2 DRAM port and used it both usually.
> >> >> Assume DMC0 starts with 0x2000'0000 with 128MiB.
> >> >> DMC1 starts with 0x4000'0000 with 128MiB.
> >> >> Note that DMC1 has to start address 0x4000'0000 at least.
> >> >>
> >> >> Then there's too much memory hole 0x1800'0000 (128MiB + 256MiB)
> >> >>
> >> >> To reduce memory waste, the DMC0 start with 0x3000'0000.
> >
> > but you just said DMC0 starts at 0x2... ? Is this reconfigurable via
> > software, if so can it be moved once initialisation si done? how fixed
> > is this?
> 
> DMC0 can set the its start address at bootloader, it can range from
> 0x2000'0000 to 0x3fff'ffff.
> 
> My goal is that reduce the 256MiB memory area and can use the full
> memory without HIGHMEM and 2G/2G split configuration.

If I remember correctly, te device has 4x256MiB banks, of which the
population is external on S5PV210 or packaging with the S5PC1xx series.

On the split situation, is it really that bad to have a 2G/2G split,
are you currently running into a situation where a single process
really needs that much address space (especially given the fact that
there can only ever be 1GiB of real memory in the system)?

On the higmem issue, I don't think we've ever had to bother with HIGMEM
even on a SMDKV210 with 1GiB of memory fitted when we first did the support.

> >
> >> Hi,
> >>
> >> >
> >> > We should be using SPARSEMEM now, is it not reducing any wastage
> >> > in virtual memory space?
> >>
> >> Please consider the older kernel also. and most systems are designed
> >> for 0x3000'0000.
> >
> > Ok, will a kernel with the base-address set to 0x3... work when loaded
> > on a system with RAM starting at 0x2... ? I'm not sure the answer to this
> > question.
> 
> Then can't detect last 2Gib memory, only detect first 2Gib memory at
> kernel 2.6.29.

Not sure how to deal with this one.
 
> >
> > I'd rather try and get one kernel working on all systems with the S5PV210
> > if possible instead of havign to pick which ones have certain base addresses
> >
> >
> >> >
> >> > How will the user change their board to deal with this change to hte
> >> > kernel?
> >>
> >> Now we got the these memory
> >> DMC0 has 128MiB and DMC1 has 512MiB.
> >> Then it uses the memory from 0x2000'0000 to 0x5fff'ffff. it used 1GiB
> >> memory address space.
> >> Of course we can enable thie HIGHMEM but older kernel doesn't support it fully.
> >> Also I hope to use the 3G/1G memory scheme.
> >
> > Given you can only get 1G of real memory into a S5PV210, is it really
> > necessary to go to 3G/1G? What problems are there with 2/2 split?
> 
> Right it's best but can't. It's hardware problem. DMC0 has different
> memory with 166MHz and DMC1 has 200MHz
> It can't change it.

Yes, but that's an allocation issue not a layout issue. Whatever is
allocating pages needs to know to prefer the DMC1 memory.
 
> >
> >> How to you think?
> >
> > I thought that when the s5pv210 was submitted we enabled the SPARSMEM
> > support so that whatever DRAM configuration we could handle it within
> > a single kernel.
> 
> I also hope to use single kernel for s5p series. and that's reason I
> hope to change it.
> With some reason. mobile division has different memory types and DMC0
> should be used fixed memory.
> and only DMC1 can modify form 2Gib to 4Gib.
> 
> Thank you,
> Kyungmin Park

-- 
-- 
Ben

Q:      What's a light-year?
A:      One-third less calories than a regular year.

  reply	other threads:[~2010-05-28  6:02 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-28  2:35 [PATCH] s5pv210: Change the base ram address to 0x3000'0000 Kyungmin Park
2010-05-28  3:53 ` Ben Dooks
2010-05-28  4:02   ` Kyungmin Park
2010-05-28  4:20     ` Ben Dooks
2010-05-28  5:25       ` Kyungmin Park
2010-05-28  6:02         ` Ben Dooks [this message]
2010-05-28  6:19           ` Kyungmin Park
2010-05-28  7:09             ` Ben Dooks
2010-05-28 14:17               ` Kyungmin Park
2010-05-31  0:18 ` Kukjin Kim
2010-05-31  0:39 ` Ben Dooks
2010-05-31  0:47   ` Kyungmin Park
2010-05-31  0:56     ` Kyungmin Park

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=20100528060223.GU4720@trinity.fluff.org \
    --to=ben-linux@fluff.org \
    --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).