[coreboot] Flashrom jedec probe patch + AT29C010A logs

Maciej Pijanka maciej.pijanka at gmail.com
Wed Jun 3 01:09:43 CEST 2009

On 03/06/2009, Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net> wrote:
> On 03.06.2009 00:58, Maciej Pijanka wrote:
>> On 03/06/2009, Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
>> wrote:
>>> If timing is not set (0), maybe fail as well? That would be accomplished
>>> by killing the above if branch and changing the message below to say
>>> "negative/ininitialized value".
>> sure, but that will fail/skip silently most chips so we also should
>> change printf_debug to normal one otherwise nobody without -V notice
> Yes! For the "not set" and the "negative value" case, this should
> definitely be printf, not printf_debug.
>> that they was skipped not probe failed
>> (maybe we need more than two levels of verbosity?)
>> or some nice way to notify user when -V is not used something like
>> '%d chips probe were skipped, use -V to figure why'
> I think your suggestion of using prinf instead of printf_debug is best.

updated patch in attachment

Maciej Pijanka, PLD-Linux Developer, Reg Linux user #133161
POE/Perl user
-------------- next part --------------
A non-text attachment was scrubbed...
Name: probe_timing.patch
Type: application/octet-stream
Size: 26645 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20090603/e90f93ce/attachment.obj>

More information about the coreboot mailing list