public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [mips][cfi] Unable to write into flash under qemu
Date: Mon, 18 Jul 2016 10:38:51 +0200	[thread overview]
Message-ID: <578C959B.3020404@denx.de> (raw)
In-Reply-To: <CAOazyz1j8X-Z_hf5jw70ZYB3QXzdCSnhqjkaU5j3+rxmOO+vgQ@mail.gmail.com>

Hi Alon, Hi Daniel,

On 10.07.2016 22:50, Alon Bar-Lev wrote:
> On 5 July 2016 at 19:10, Daniel Schwierzeck
> <daniel.schwierzeck@gmail.com> wrote:
>>
>> Hi Alon,
>>
>> +cc Paul, Stefan
>>
>> Am 05.07.2016 um 14:04 schrieb Alon Bar-Lev:
>>> Hi,
>>>
>>> Found the debug setting.
>>>
>>> When I try to erase flash under qemu I get the following error, the
>>> erase actually succeeds in the backed file, I see sector as 0xff.
>>>
>>> Can anyone check if problem is within u-boot or qemu?
>>>
>>> Thanks!
>>
>> I've found the problem. Paul sent a fix [1] for pflash in June 2013
>> to make 32 Bit access working with YAMON bootloader.
>>
>> The fix mirrors the flash status value in the upper 16 Bits of the return
>> value. Thus U-Boot gets a value of 0x800080 in flash_full_status_check()
>> after a block erase command. U-Boot compares the status value in
>> flash_isequal() with the expected value of 0x80 but that fails.
>>
>> Because of this I think pflash support for Malta is broken since Qemu 1.6.0.
>>
>> A possible fix could be this:
>
> Hi,
> Thanks again.
> How do we merge this fix?

Daniel, could you please send a proper patch (non-inlined) for this so
that I can push it to upstream? Or did I miss this patch?

Thanks,
Stefan

  reply	other threads:[~2016-07-18  8:38 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-05  0:08 [U-Boot] [mips][cfi] Unable to write into flash under qemu Alon Bar-Lev
2016-07-05 12:04 ` Alon Bar-Lev
2016-07-05 16:10   ` Daniel Schwierzeck
2016-07-05 16:50     ` Alon Bar-Lev
2016-07-05 17:05       ` Daniel Schwierzeck
2016-07-05 17:17         ` Alon Bar-Lev
2016-07-05 17:34           ` Daniel Schwierzeck
2016-07-05 17:57             ` Alon Bar-Lev
2016-07-10 20:50     ` Alon Bar-Lev
2016-07-18  8:38       ` Stefan Roese [this message]
2016-07-18 12:13         ` Daniel Schwierzeck

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=578C959B.3020404@denx.de \
    --to=sr@denx.de \
    --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