diff options
author | Ronald G. Minnich <rminnich@gmail.com> | 2013-01-30 14:29:34 -0800 |
---|---|---|
committer | David Hendricks <dhendrix@chromium.org> | 2013-01-31 23:18:16 +0100 |
commit | 79e36d90608a929c33b655c6fb6376f33f332e6f (patch) | |
tree | 185114440365b321d96e7643564a42debc1edb9d /src/mainboard/amd/parmer/agesawrapper.h | |
parent | bc3abbbaf05123f87cee143845868bf6b95fdd3e (diff) |
Improve how our printk calls do_div by using constants.
The do_div code has a nice optimization in it when it is called with
constants. The current highly generalized use of it defeats those
optimizations and causes trouble on ARM, resulting in a complex and
buggy code path.
Since we only need to print in bases 8, 10, and 16, do a minor
restructuring of the code so that we call do_div with constants.
If you need base 2, print in base 16 and do it in your head. :-)
This fixes an ongoing problem with ARM, will not harm X86, and will
help PPC should we ever want to support it again.
Plus, I don't have to ever try to understand the div64 assembly and where
it's going wrong :-)
Change-Id: I6a480011916eb0834e05c5bb10909d83330fe797
Signed-off-by: Ronald G. Minnich <rminnich@gmail.com>
Reviewed-on: http://review.coreboot.org/2235
Tested-by: build bot (Jenkins)
Reviewed-by: David Hendricks <dhendrix@chromium.org>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/mainboard/amd/parmer/agesawrapper.h')
0 files changed, 0 insertions, 0 deletions