All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tan, Ley Foon <ley.foon.tan@intel.com>
To: u-boot@lists.denx.de
Subject: [PATCH] arm: socfpga: arria10: Add save_boot_params()
Date: Fri, 6 Mar 2020 00:33:11 +0000	[thread overview]
Message-ID: <MN2PR11MB4509564971FE25C64EA22EA1CCE30@MN2PR11MB4509.namprd11.prod.outlook.com> (raw)
In-Reply-To: <8a3f5586-4c92-5972-1a49-916abd63608d@denx.de>



> -----Original Message-----
> From: Marek Vasut <marex@denx.de>
> Sent: Thursday, March 5, 2020 8:23 PM
> To: Ley Foon Tan <lftan.linux@gmail.com>
> Cc: Tan, Ley Foon <ley.foon.tan@intel.com>; u-boot at lists.denx.de; Simon
> Goldschmidt <simon.k.r.goldschmidt@gmail.com>; See, Chin Liang
> <chin.liang.see@intel.com>; Chee, Tien Fong <tien.fong.chee@intel.com>
> Subject: Re: [PATCH] arm: socfpga: arria10: Add save_boot_params()
> 
> On 3/5/20 9:56 AM, Ley Foon Tan wrote:
> Hi,
> 
> [...]
> 
> >>>>>> Aren't those already defined in include/configs/socfpga_common.h ?
> >>>>> socfpga_common.h have this:
> >>>>> #define CONFIG_SYS_INIT_RAM_SIZE        (0x40000 -
> >>>> CONFIG_SYS_SPL_MALLOC_SIZE)
> >>>>>
> >>>>> But, we can't use it here.
> >>>>
> >>>> Why ?
> >>> CONFIG_SYS_INIT_RAM_SIZE is minus with
> CONFIG_SYS_SPL_MALLOC_SIZE, not 256KB (0x40000).
> >>
> >> Then define the init ram size macro somewhere in
> >> arch/arm/mach-socfpga/include and use it in both the configs/ and
> >> your code ? :)
> > Look at existing header files under arch/arm/mach-socfpga/include/,
> > can add to base_addr_a10.h.
> 
> Isn't that base address common to A10 and CV/AV ?
No, CV/AV uses base_addr_ac5.h.

Regards
Ley Foon

  reply	other threads:[~2020-03-06  0:33 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-26 19:01 [PATCH] arm: socfpga: arria10: Add save_boot_params() Ley Foon Tan
2020-03-01 15:12 ` Marek Vasut
2020-03-02  7:20   ` Tan, Ley Foon
2020-03-02 10:15     ` Marek Vasut
2020-03-03  9:14       ` Ley Foon Tan
2020-03-03 12:12         ` Marek Vasut
2020-03-04  0:36           ` Tan, Ley Foon
2020-03-04 12:18             ` Marek Vasut
2020-03-05  8:56               ` Ley Foon Tan
2020-03-05 12:22                 ` Marek Vasut
2020-03-06  0:33                   ` Tan, Ley Foon [this message]
2020-03-06  0:43                     ` Marek Vasut
2020-03-06  3:15                       ` Tan, Ley Foon

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=MN2PR11MB4509564971FE25C64EA22EA1CCE30@MN2PR11MB4509.namprd11.prod.outlook.com \
    --to=ley.foon.tan@intel.com \
    --cc=u-boot@lists.denx.de \
    /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.