public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Detlev Zundel <dzu@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] [patch] Fix NC650 nand support
Date: Fri, 20 Apr 2007 17:14:09 +0200	[thread overview]
Message-ID: <m2irbr3wou.fsf@sowhat.denx.de> (raw)
In-Reply-To: <20070420135320.E5A9935363E@atlas.denx.de> (Wolfgang Denk's message of "Fri\, 20 Apr 2007 15\:53\:20 +0200")

Hi Wolfgang,

> In message <200704201338.51115.sr@denx.de> you wrote:
>> 
>> You might want to clean up the compiler warnings too (ELDK 4.1)? ;-)
>
> Done. [4.2 actually :-) ]

Hm.  I am eager to see the changes. If you *really* cleaned *all*
warnings then I am sure you also fixed the warning in extable.c and
while doing this noticed that the whole fixup only works because we
have a maximum of one fixup entry (./drivers/tsi108_pci.c, I cannot
find any user of addr_probe anywhere).  Otherwise the sorting of the
table is missing and thus the searching with a bisecting algorithm
will not work as is.

I am *really looking* forward to the patch fixing that :)

Cheers
  Detlev

-- 
PUBLIC NOTICE AS REQUIRED BY LAW:    Any Use of  This Product,  in Any Manner
Whatsoever, Will Increase the Amount of Disorder in the Universe. Although No
Liability Is  Implied Herein,  the Consumer Is Warned  That This Process Will
Ultimately Lead to the Heat Death of the Universe.
--
DENX Software Engineering GmbH, HRB 165235 Munich, CEO: Wolfgang Denk
Office: Kirchenstr. 5, D-82194 Groebenzell, Germany +49 8142 66989 40

  reply	other threads:[~2007-04-20 15:14 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-20 10:07 [U-Boot-Users] [patch] Fix NC650 nand support Detlev Zundel
2007-04-20 11:38 ` Stefan Roese
2007-04-20 13:53   ` Wolfgang Denk
2007-04-20 15:14     ` Detlev Zundel [this message]
2007-04-20 15:49       ` Wolfgang Denk
2007-04-22 23:52         ` Zang Roy-r61911
2007-04-23 13:00           ` Detlev Zundel
2007-04-24  6:32             ` Zang Roy-r61911

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=m2irbr3wou.fsf@sowhat.denx.de \
    --to=dzu@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