All of lore.kernel.org
 help / color / mirror / Atom feed
From: Evgeny Voevodin <e.voevodin@samsung.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	qemu-devel@nongnu.org, kyungmin.park@samsung.com,
	paul@codesourcery.com, m.kozlov@samsung.com,
	jehyung.lee@samsung.com, d.solodkiy@samsung.com
Subject: Re: [Qemu-devel] [PATCH v15 00/10] ARM: Samsung Exynos4210-based boards support.
Date: Wed, 15 Feb 2012 08:16:44 +0400	[thread overview]
Message-ID: <4F3B31AC.5070806@samsung.com> (raw)
In-Reply-To: <4F3B2530.7070300@samsung.com>

On 15.02.2012 07:23, Evgeny Voevodin wrote:
> On 14.02.2012 23:51, Peter Maydell wrote:
>> On 10 February 2012 05:36, Evgeny Voevodin<e.voevodin@samsung.com> wrote:
>>> This set of patches adds support for Samsung Exynos4210-based boards
>>> NURI and SMDKC210.
>>> Tested on Linux kernel v3.x series.
>>> Boot smdkc210 board with root filesystem on NFS disk and serial port
>>> #0 redirected to terminal
>>> # qemu-system-arm -kernel ./zImage -append "console=ttySAC0,115200n8
>>> ip=dhcp root=/dev/nfs nfsroot=10.0.2.2:/srv/nfs/ rw " -serial stdio
>>> -M smdkc210
>> Have you tried booting this with the 'earlyprintk' kernel option?
>> I tried this with a 3.2.0 kernel and the exynos4 defconfig and it
>> crashes (without 'earlyprintk' is fine). What seems to be happening
>> is that the printk() in s3c_init_cpu() ends up taking an abort
>> trying to read the UART status register in the waituart macro.
>> This load has worked fine up until this point (with the same
>> address being used for the load) but I have a feeling this is the
>> first printk after we've done some fiddling with the memory mapping
>> of devices. Presumably without 'earlyprintk' we don't try to
>> do this output and will dump the whole printk buffer later via
>> a different codepath.
>>
>> Possibly this is a kernel bug but I'll let you track it down :-)
>>
>> -- PMM
>>
>
> Some times ago we have tried it but not recently. Will check it and
> report. Thanks.
>

My suggestion is that this is a kernel issue. Try latest 3.3-rc3. It 
works with "earlyprintk", though I can't see any valuable difference in 
output.

Also, make attention to what config says:

"Say Y here if you want the debug print routines to direct
their output to UART 0. The port must have been initialised
by the boot-loader before use."

We do not have appropriate bootloader, though 3.3-rc3 works without it.

-- 
Kind regards,
Evgeny Voevodin,
Leading Software Engineer,
ASWG, Moscow R&D center, Samsung Electronics
e-mail: e.voevodin@samsung.com

  reply	other threads:[~2012-02-15  4:16 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-10  5:36 [Qemu-devel] [PATCH v15 00/10] ARM: Samsung Exynos4210-based boards support Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 01/10] ARM: exynos4210: IRQ subsystem support Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 02/10] ARM: Samsung exynos4210-based boards emulation Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 03/10] ARM: exynos4210: UART support Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 04/10] ARM: exynos4210: PWM support Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 05/10] ARM: exynos4210: basic Power Management Unit implementation Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 06/10] ARM: exynos4210: MCT support Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 07/10] hw/lan9118: Add basic 16-bit mode support Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 08/10] hw/exynos4210.c: Add LAN support for SMDKC210 Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 09/10] Exynos4210: added display controller implementation Evgeny Voevodin
2012-02-10  5:36 ` [Qemu-devel] [PATCH v15 10/10] MAINTAINERS: Add maintainers for Exynos SOC Evgeny Voevodin
2012-02-14 11:49   ` Andreas Färber
2012-02-14 10:27 ` [Qemu-devel] [PATCH v15 00/10] ARM: Samsung Exynos4210-based boards support Evgeny Voevodin
2012-02-14 10:32   ` Peter Maydell
2012-02-14 10:36     ` Evgeny Voevodin
2012-02-14 19:51 ` Peter Maydell
2012-02-15  3:23   ` Evgeny Voevodin
2012-02-15  4:16     ` Evgeny Voevodin [this message]
2012-02-15  8:56       ` Peter Maydell
2012-02-15  9:38         ` Evgeny Voevodin
2012-02-15 10:44 ` Peter Maydell

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=4F3B31AC.5070806@samsung.com \
    --to=e.voevodin@samsung.com \
    --cc=d.solodkiy@samsung.com \
    --cc=jehyung.lee@samsung.com \
    --cc=kyungmin.park@samsung.com \
    --cc=m.kozlov@samsung.com \
    --cc=paul@codesourcery.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /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.