All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabio Estevam <festevam@gmail.com>
To: Paul Liu <paul.liu@linaro.org>,
	Schrempf Frieder <frieder.schrempf@kontron.de>
Cc: Fabio Estevam <festevam@denx.de>, Stefano Babic <sbabic@denx.de>,
	 Kirill Kapranov <kirill.kapranov@compulab.co.il>,
	Uri Mashiach <uri.mashiach@compulab.co.il>,
	 Valentin Raevsky <valentin@compulab.co.il>,
	U-Boot-Denx <u-boot@lists.denx.de>, Harald Seiler <hws@denx.de>
Subject: Re: [PATCH 1/2] imx8mm-cl-iot-gate: Do not build fip.bin by default
Date: Wed, 18 Aug 2021 18:39:05 -0300	[thread overview]
Message-ID: <CAOMZO5CbfGuZjOT-rVLB96FK058Wn4tS9SPdbHONQNW-rWWHRQ@mail.gmail.com> (raw)
In-Reply-To: <CALG-KHxGjqYjFGtH6tAuVqvC_fDLVafwRpsKqoNR746AR7UKFw@mail.gmail.com>

Hi Paul,

On Wed, Aug 18, 2021 at 6:32 PM Paul Liu <paul.liu@linaro.org> wrote:
>
> Hi Fabio,
>
> I got several boards. With all different PN. But all of them are 2GB memory. And the recent master doesn't boot on one of my board. I haven't tried all of the combinations.

With the U-Boot from Compulab, it reports 4GB. With mainline U-Boot it
reports 2GB, so yes, there is an issue indeed.

However, I don't see a hang.

> After bisect, I found commit e27bddff breaks the boot. It just hang there.

Adding Frieder as the author of the patch.

> But if I change PHYS_SDRAM_SIZE to 1GB then it can boot. However this is not the solution because it limits the memory usage. So I might need to take some time look into what happened and spend more time to test on other PN. Before start testing your patches.
> And because you can boot the master branch fine so I'm not sure what happened on my side.

Yes, please fix the memory issue first. Thanks!

  reply	other threads:[~2021-08-18 21:39 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-13  0:59 [PATCH 1/2] imx8mm-cl-iot-gate: Do not build fip.bin by default Fabio Estevam
2021-08-13  0:59 ` [PATCH 2/2] imx8mm-cl-iot-gate: Add documentation Fabio Estevam
2021-08-13 10:07   ` Paul Liu
2021-08-13  9:59 ` [PATCH 1/2] imx8mm-cl-iot-gate: Do not build fip.bin by default Paul Liu
2021-08-13 12:15   ` Fabio Estevam
2021-08-15 19:26     ` Paul Liu
2021-08-16 21:09       ` Fabio Estevam
2021-08-16 21:29         ` Tom Rini
2021-08-16 22:41         ` Fabio Estevam
2021-08-17  0:50           ` Paul Liu
2021-08-17 18:32             ` Fabio Estevam
2021-08-18  0:08               ` Paul Liu
2021-08-18 12:24                 ` Fabio Estevam
2021-08-18 21:32                   ` Paul Liu
2021-08-18 21:39                     ` Fabio Estevam [this message]
2021-08-19  0:27                       ` Fabio Estevam
2021-08-19  7:14                         ` Frieder Schrempf
2021-08-19 20:51                           ` Paul Liu
2021-08-20  2:06                             ` Paul Liu

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=CAOMZO5CbfGuZjOT-rVLB96FK058Wn4tS9SPdbHONQNW-rWWHRQ@mail.gmail.com \
    --to=festevam@gmail.com \
    --cc=festevam@denx.de \
    --cc=frieder.schrempf@kontron.de \
    --cc=hws@denx.de \
    --cc=kirill.kapranov@compulab.co.il \
    --cc=paul.liu@linaro.org \
    --cc=sbabic@denx.de \
    --cc=u-boot@lists.denx.de \
    --cc=uri.mashiach@compulab.co.il \
    --cc=valentin@compulab.co.il \
    /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.