public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Gabor Juhos <juhosg@openwrt.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [mips] malta configuration works in qemu-1.5 does not in qemu-1.6
Date: Mon, 11 Nov 2013 11:25:44 +0100	[thread overview]
Message-ID: <5280B0A8.30105@openwrt.org> (raw)
In-Reply-To: <CACUy__X2x5_KAfWEEn7WGubs+XS=yv8pY_43CQXk3R2am_=_bw@mail.gmail.com>

2013.11.11. 10:52 keltez?ssel, Daniel Schwierzeck ?rta:
> 2013/11/10 Alon Bar-Lev <alon.barlev@gmail.com>:
>> Hello,
>>
>> master is: 15c5cdf from Fri Nov 8 15:25:29 2013 -0500
>> u-boot-2013.10
>>
>> Working in qemu-1.5.3.
>> Does not work in qemu-1.6.1.
>>
>> The command prompt is not displayed, 100% cpu consumption.
>>
>> ---
>> $ qemu-system-mips -M malta -nographic -drive if=pflash,file=/tmp/flash -m
>> 256
>>
>>
>> U-Boot 2013.10-00249-g15c5cdf (Nov 10 2013 - 18:00:48)
>>
>> Board: MIPS Malta CoreLV (Qemu)
>> DRAM:  256 MiB
>> ---
>>
>> Any idea what the difference? any fix pending somewhere?
>>
> 
> it is obviously am upstream change in qemu, which requires a fix in u-boot.
> 
> Paul or Gabor, do you have an idea?

It is broken since the following change in qemu:

commit a427338b222b43197c2776cbc996936df0302f51
Author: Paul Burton <paul.burton@imgtec.com>
Date:   Fri Jun 14 08:30:44 2013 +0100

    mips_malta: correct reading MIPS revision at 0x1fc00010

    Rather than modifying the BIOS code at its original location, copy it
    for the 0x1fc00000 region & modify the copy. This means the original
    ROM code is correctly readable at 0x1e000010 whilst the MIPS revision
    is readable at 0x1fc00010.

    Additionally the code previously operated on target memory which would
    later be overwritten by the BIOS image upon CPU reset if the -bios
    argument was used to specify the BIOS image. This led to the written
    MIPS revision being lost. Copying using rom_copy when -bios is used
    fixes this issue.

    Signed-off-by: Paul Burton <paul.burton@imgtec.com>
    Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
    Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>

Due to that change, we are getting a CPU exception while probing the
flash at the 0x1fc00000 address.

Changing the MALTA_FLASH_BASE constant to 0x1e000000 in
'arch/mips/include/asm/malta.h' fixes the problem. I will send a patch for that.

-Gabor

  reply	other threads:[~2013-11-11 10:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-10 16:09 [U-Boot] [mips] malta configuration works in qemu-1.5 does not in qemu-1.6 Alon Bar-Lev
2013-11-11  9:52 ` Daniel Schwierzeck
2013-11-11 10:25   ` Gabor Juhos [this message]
2013-11-11 10:35     ` Alon Bar-Lev
2013-11-11 11:14     ` Paul Burton
2013-11-11 14:05       ` Gabor Juhos

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=5280B0A8.30105@openwrt.org \
    --to=juhosg@openwrt.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox