From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 4 Aug 2020 09:44:57 -0400 Subject: [PATCH 1/2] IPQ40xx: Add DRAM detection & FDT fixup In-Reply-To: References: <20200803115220.342329-1-robert.marko@sartura.hr> <20200803143953.GR6965@bill-the-cat> Message-ID: <20200804134457.GY6965@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Tue, Aug 04, 2020 at 01:20:56PM +0200, Robert Marko wrote: > On Mon, Aug 3, 2020 at 4:39 PM Tom Rini wrote: > > > > On Mon, Aug 03, 2020 at 01:52:19PM +0200, Robert Marko wrote: > > > > > Fixup the Linux FDT with the detection of onboard DRAM as > > > provided by SBL (Secondary boot loader) by reading > > > the shared-memory region. > > > > > > Imported from Snapdragon target. > > > > > > Signed-off-by: Robert Marko > > > Cc: Luka Perkov > > > --- > > > arch/arm/mach-ipq40xx/Makefile | 1 + > > > arch/arm/mach-ipq40xx/dram.c | 99 +++++++++++++++++++++++ > > > arch/arm/mach-ipq40xx/include/mach/dram.h | 12 +++ > > > 3 files changed, 112 insertions(+) > > > create mode 100644 arch/arm/mach-ipq40xx/dram.c > > > create mode 100644 arch/arm/mach-ipq40xx/include/mach/dram.h > > > > Is this as-is? If so, we need to figure out something or another to > > make sharing the code, rather than copying, possible. How are these > > both grouped in the kernel? Thanks! > > Yes, the Snapdragon target uses the identical code. > The kernel does not have code for this but rather relies on the > bootloader to do it. Right, but I mean what does the kernel do to avoid copying identical code between these two platforms? I guess the answer is that for arch/arm64 everything has been pushed out of arch/arm64/. So in this case, the dram code should get moved to our drivers/ddr/ or drivers/ram/ depending on which place "dram.c" really belongs best. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: