From: Dirk Behme <dirk.behme@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] mtd: Update MTD infrastructure to support 64bit device size
Date: Mon, 22 Jun 2009 21:49:10 +0200 [thread overview]
Message-ID: <4A3FE036.1060009@googlemail.com> (raw)
In-Reply-To: <200906222126.32562.sr@denx.de>
Stefan Roese wrote:
> On Monday 22 June 2009 20:50:30 Scott Wood wrote:
>> Dirk Behme wrote:
>>> Because every non-U-Boot expert who just downloads recent git and
>>> compiles it will get the warning and will wonder if something is wrong
>>> or not. So yes, the check doesn't hurt, but the warning might confuse
>>> the users, which are non-experts.
>> And the answer to that user is that something *is* wrong, and should be
>> reported to the board maintainer so they can enable 64-bit printf.
>
> Ack.
Ok. So you tell me we shall enable 64-bit printf for all OMAP3 boards
that show this warning?
What I still miss: If this warning indicates that something *is*
wrong, why not run MAKEALL and enable it in config for all boards that
show this warning? What's the advantage of having all maintainers
sending patches for their boards?
Best regards
Dirk
>>>> We could get rid of it of course if we would enable this 64bit printf
>>>> format on all platforms and removed this option completely. This would
>>>> cost a bit of image size but would make things easier/simpler. Just my
>>>> 0.02$...
>> Agreed. Or once we get kconfig we can have it be selected by any code
>> that requires it.
>
> Good idea. Thanks Scott.
>
> Best regards,
> Stefan
>
> =====================================================================
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
> =====================================================================
>
next prev parent reply other threads:[~2009-06-22 19:49 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-11 14:03 [U-Boot] [PATCH] mtd: Update MTD infrastructure to support 64bit device size Stefan Roese
2009-05-11 16:27 ` Mike Frysinger
2009-05-11 16:38 ` Stefan Roese
2009-05-11 16:57 ` Mike Frysinger
2009-05-11 20:06 ` Scott Wood
2009-05-11 17:59 ` Wolfgang Denk
2009-05-12 4:39 ` Stefan Roese
2009-05-12 5:11 ` Mike Frysinger
2009-05-12 8:14 ` Wolfgang Denk
2009-05-12 8:26 ` Stefan Roese
2009-06-03 21:49 ` Wolfgang Denk
2009-06-20 5:33 ` Dirk Behme
2009-06-20 8:52 ` Jean-Christophe PLAGNIOL-VILLARD
2009-06-20 17:45 ` Dirk Behme
2009-06-22 12:01 ` Stefan Roese
2009-06-22 18:10 ` Dirk Behme
2009-06-22 18:21 ` Stefan Roese
2009-06-28 5:47 ` Dirk Behme
2009-06-29 14:35 ` Stefan Roese
2009-06-22 12:55 ` Stefan Roese
2009-06-22 18:12 ` Dirk Behme
2009-06-22 18:25 ` Stefan Roese
2009-06-22 18:38 ` Dirk Behme
2009-06-22 18:50 ` Scott Wood
2009-06-22 19:26 ` Stefan Roese
2009-06-22 19:49 ` Dirk Behme [this message]
2009-06-22 19:52 ` Scott Wood
2009-06-22 20:04 ` Dirk Behme
2009-06-22 20:08 ` Scott Wood
2009-06-22 20:15 ` Dirk Behme
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=4A3FE036.1060009@googlemail.com \
--to=dirk.behme@googlemail.com \
--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