RE: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board

From	James Hartley <James.Hartley@imgtec.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board
Date	2016-10-13 16:50 +0200
Message-ID	<srPuy-4ML-23@gated-at.bofh.it> (permalink)
References	<spg7T-6F-1@gated-at.bofh.it> <spg7T-6F-13@gated-at.bofh.it> <sqJKy-2la-7@gated-at.bofh.it> <srICJ-rU-3@gated-at.bofh.it> <srJS9-18s-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Rahul,

> -----Original Message-----
> From: linux-mips-bounce@linux-mips.org [mailto:linux-mips-bounce@linux-
> mips.org] On Behalf Of James Hartley
> Sent: 13 October 2016 09:50
> To: Rahul Bedarkar; Rob Herring
> Cc: Ralf Baechle; Mark Rutland; linux-mips@linux-mips.org;
> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: RE: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board
> 
> Hi Rahul,
> 
> > -----Original Message-----
> > From: Rahul Bedarkar
> > Sent: 13 October 2016 08:02
> > To: Rob Herring
> > Cc: Ralf Baechle; Mark Rutland; James Hartley;
> > linux-mips@linux-mips.org; devicetree@vger.kernel.org;
> > linux-kernel@vger.kernel.org
> > Subject: Re: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk
> > board
> >
> > Hi,
> >
> > On Monday 10 October 2016 07:51 PM, Rob Herring wrote:
> > >> +
> > >> +	memory {
> > >
> > > Is 0 the actual base, or that gets filled in by bootloader? If the
> > > formet, add unit address.
> > >
> >
> > Bootloader (uboot) can override or fixup memory node. But with version
> > of bootloader I tested with, base address is hardcoded to 0 and only
> > size may get changed. But since booloader can override or fixup memory
> > node, I assume we don't add unit address in this case.
> >
> > >> +		device_type = "memory";
> > >> +		reg =  <0x00000000 0x08000000>;
> >
> > I now realized that size is incorrectly specified in memory node. It
> > should be 256MB and not 128MB. I will fix this in v2.
> 
> This is board dependent, and since you have already explained that the boot
> loader is expected to configure this, I would recommend leaving it at 128MB
> (which I believe is the size of the smallest DDR part in use on existing boards).
> This would mean that if for some reason the boot loader did not adjust the
> settings, it would still boot on all currently known boards.

So it seems the 128MB boards are not CI40 based, so since this dts is CI40 specific, it's fine to have 256MB.

James.

> 
> James.
> 
> >
> > Thanks,
> > Rahul

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] MIPS: DTS: add base device tree for Pistachio SoC Rahul Bedarkar <rahul.bedarkar@imgtec.com> - 2016-10-06 14:40 +0200
  [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board Rahul Bedarkar <rahul.bedarkar@imgtec.com> - 2016-10-06 14:40 +0200
    Re: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board Rob Herring <robh@kernel.org> - 2016-10-10 16:30 +0200
      Re: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board Rahul Bedarkar <Rahul.Bedarkar@imgtec.com> - 2016-10-13 09:30 +0200
        RE: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board James Hartley <James.Hartley@imgtec.com> - 2016-10-13 10:50 +0200
          RE: [PATCH 2/2] MIPS: DTS: img: add device tree for Marduk board James Hartley <James.Hartley@imgtec.com> - 2016-10-13 16:50 +0200

csiph-web