From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] M29EW flash is detected as 0xFF
Date: Fri, 01 Mar 2013 08:18:54 +0100 [thread overview]
Message-ID: <5130565E.5050301@denx.de> (raw)
In-Reply-To: <CAD6G_RTEU_SxcZnBMN3vV+bfd2yrVvZaWyU3dbogNTsN0MEepg@mail.gmail.com>
On 01.03.2013 08:12, Jagan Teki wrote:
>>> I made a fix for hw cfi issue when device is operating in 8-bit mode,
>>> If the above changes are looks fine we may have this fix will solve
>>> the M29EW flashes operates under 8-bit mode.
>>>
>>> Could send your comments/views.
>>
>> Not sure if I understood you correctly. So I have to ask some questions:
>>
>
> Firstly, apologies from my side by sending too many status updates
> that makes you confusion.
No problem. I have been slow on reacting.
>> Did you use this patch?
>
> Yes, I have applied the same patch,
>
>>
>> http://patchwork.ozlabs.org/patch/140863/
>>
>> Did it apply without issues to mainline U-Boot? Or did you have to make
>> some changes so that it can be applied? And did it fix the issues you
>> have seen before?
>
> I have manually applied those changes as the index values are differ.
> I haven't added any newly code other than the patch changes.
If you had to make some changes to this patch, then please send the
changes / updated patch version as a new version (v3) to the list. So
that it applies clean now. And please change the commit text accordingly.
> Yes it fixed the error by detecting correct manufacture and device ID.
Only that? IIRC, then without this patch, writing to the 8-bit flash
didn't work at all. But I don't have such a board, so I can't really
test myself.
>> And did you have some make some further changes as well?
>
> And apart from these patch change, I have changed one more this for
> numonyx M29EW fix
> for incorrect write buffer size..any way that I could send once these
> changes are pushed/reviewed.
Please send this additional change/fix as a separate patch to the list
as well.
Thanks,
Stefan
next prev parent reply other threads:[~2013-03-01 7:18 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-17 11:51 [U-Boot] M29EW flash is detected as 0xFF Jagan Teki
2013-02-18 9:04 ` Jagan Teki
2013-02-18 9:57 ` Jagan Teki
2013-02-20 13:57 ` Stefan Roese
2013-02-20 17:25 ` Jagan Teki
2013-02-20 17:31 ` Jagan Teki
2013-02-20 18:14 ` Stefan Roese
2013-02-21 11:57 ` Jagan Teki
2013-02-22 15:32 ` Jagan Teki
2013-02-28 19:28 ` Jagan Teki
2013-03-01 6:59 ` Stefan Roese
2013-03-01 7:12 ` Jagan Teki
2013-03-01 7:18 ` Stefan Roese [this message]
2013-03-01 11:28 ` Jagan Teki
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=5130565E.5050301@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