All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Barker <pbarker@toganlabs.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] U-boot 2017.07 not working on Raspberry Pi
Date: Fri, 4 Aug 2017 13:58:50 +0100	[thread overview]
Message-ID: <CAPyrkgxo=U7qJb06ic9Y2=7DCd78==2p3bt9O-+eB8+kg3Eguw@mail.gmail.com> (raw)
In-Reply-To: <CAPnjgZ1qrdT5nrLkkHWjN99V_ypXNrQOytzgvjh+Xcdy__NV6Q@mail.gmail.com>

On Thu, Aug 3, 2017 at 2:45 PM, Simon Glass <sjg@chromium.org> wrote:
> Hi,
>
> On 3 August 2017 at 03:37, Mark Kettenis <mark.kettenis@xs4all.nl> wrote:
>>> Date: Thu, 3 Aug 2017 19:02:06 +1000
>>> From: Jonathan Gray <jsg@jsg.id.au>
>>>
>>>
>>> We had to revert 25877d4e4c45451c5398aec3de50e0d5befe0e9f to add
>>> CONFIG_OF_EMBED back to get OpenBSD to continue working on the rpi3.
>>
>> That revert is in particular necessary for passthrough of the device
>> tree from the Raspberry Pi firmwre to work.  I haven't investigated
>> this further yet, but it seems some of the relocation code is
>> conditionalized on CONFIG_OF_EMBED, and I suspect that when
>> CONFIG_OF_EMBED is absent, u-boot overwrites the device tree when it
>> relocates itself.
>
> I'm not sure why it needs CONFIG_OF_EMBED. Certainly U-Boot does
> relocate the DT when this is not set. But it should not be set for
> production images, only for debugging.
>

I've tried reverting just "dm: arm: rpi: Drop CONFIG_OF_EMBED"
(25877d4e) but I'm still left without a working keyboard in u-boot -
hitting keys during the countdown does not interrupt the boot process.
However the kernel can be booted with just that one commit reverted.

I've also looking into the CONFIG_OF_BOARD option. This is currently
disabled for raspberrypi devices which means that
board_fdt_blob_setup() in board/raspberrypi/rpi/rpi.c is not called.
Enabling CONFIG_OF_BOARD leads to a crash at boot with no output from
u-boot, the monitor connected to the raspberrypi just shows the colour
palette.

I can look into this further but it probably needs to wait until I can
order some RS232 adaptors and get a serial port setup for early debug
output.

Thanks,

-- 
Paul Barker
Co-Founder & Principal Engineer
Togán Labs Ltd

  parent reply	other threads:[~2017-08-04 12:58 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-02 12:13 [U-Boot] U-boot 2017.07 not working on Raspberry Pi Paul Barker
2017-08-03  9:02 ` Jonathan Gray
2017-08-03  9:37   ` Mark Kettenis
2017-08-03 13:45     ` Simon Glass
2017-08-03 15:42       ` Stephen Warren
2017-08-03 17:42         ` Tom Rini
2017-08-21  2:59           ` Simon Glass
2017-08-25  1:46             ` Simon Glass
2017-08-25  9:46               ` Paul Barker
2017-08-25 12:25                 ` Jonathan Gray
2017-08-25 16:01                 ` Stephen Warren
2017-08-21  5:06           ` Heiko Schocher
2017-08-04 12:58       ` Paul Barker [this message]
2017-08-04 13:18         ` Mark Kettenis

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='CAPyrkgxo=U7qJb06ic9Y2=7DCd78==2p3bt9O-+eB8+kg3Eguw@mail.gmail.com' \
    --to=pbarker@toganlabs.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.