public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] common/xyzmodem.c, ymodem, slow behavior receiving bytes
Date: Tue, 13 Nov 2012 08:09:19 +0100	[thread overview]
Message-ID: <20121113070919.796A02002E1@gemini.denx.de> (raw)
In-Reply-To: <20121113001651.GA21177@angel3>

Dear Angelo Dureghello,

please don't top-post / full quote.

In message <20121113001651.GA21177@angel3> you wrote:
> 
> seems that mcf5307, working at 90Mhz, is not fast enough when "-Os -g" 
> compile options are set.
> 
> I changed for test config.mk from
> 
> DBGFLAGS= -g # -DDEBUG
> OPTFLAGS= -Os #-fomit-frame-pointer
> 
> into
> 
> DBGFLAGS= #-g # -DDEBUG
> OPTFLAGS= -O2 #-Os #-fomit-frame-pointer
> 
> common compiles now with -O2 and ymodem works fine again at 115200.

Removing -g makes no sense in this cotext.  It has no impact on the
generated code.

I am really surprised about your claim that the -O2 compiled code is
actually running faster than the -Os compiled one on a low-end system
as yours (90 MHz CPU clock, 8 kB cache size).

Which exact tool chain are you using to build the code?

> Also, i don't understand why "-g" is set by default. 

It is set because it is useful to some (those in the need of debugging
their code) and does not hurt others.

> Is there a way to override/customize this options for this cpu ?

I am not convinced that it makes sense to change settings on a per-cpu
base.  A 90 MHz CPU should be more than sufficient to receive data at
115kbps.  I can only compare against 50 MHz PowerQuicc I systems
(which is about the lowest end machines I have at hands now), and
there no such problem exists.

It would be good to understand exactly where the problem is coming
from.  I don't think that the -Os setting is the core of the problem;
I tend to suspect rather your tool chain or your serial driver or such.

> Or i have to definitely step down to 57600 ?

There should be no need for that.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Lispers are among  the  best  grads  of  the  Sweep-It-Under-Someone-
Else's-Carpet  School of Simulated Simplicity. [Was that sufficiently
incendiary? :-)]  - Larry Wall in <1992Jan10.201804.11926@netlabs.com

  parent reply	other threads:[~2012-11-13  7:09 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-10 13:40 [U-Boot] common/xyzmodem.c, ymodem, slow behavior receiving bytes Angelo Dureghello
2012-11-13  0:16 ` Angelo Dureghello
2012-11-13  1:41   ` Marek Vasut
2012-11-13  7:09   ` Wolfgang Denk [this message]
2012-11-14  9:47     ` Angelo Dureghello
2012-11-14 10:28       ` Henrik Nordström
2012-11-14 11:05       ` Wolfgang Denk
2012-11-19 21:54         ` Angelo Dureghello
2012-11-19 22:58           ` Wolfgang Denk
2012-11-22 14:38             ` Angelo Dureghello
2012-11-23 22:57             ` Angelo Dureghello
2012-11-23 23:04               ` Wolfgang Denk

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=20121113070919.796A02002E1@gemini.denx.de \
    --to=wd@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