On 3/14/2011 5:59 PM, Tony Lindgren wrote:
* Cousson, Benoit<b-cous...@ti.com> [110314 01:39]:
Hi Tony,
On 3/11/2011 9:34 PM, Tony Lindgren wrote:
Remove duplicate mux modes to make the binary smaller:
text data bss dec hex filename
9378 24472 0 33850 843a mux44xx.o
9378 19104 0 28482 6f42 mux44xx.o
Cc: Benoit Cousson<b-cous...@ti.com>
Signed-off-by: Tony Lindgren<t...@atomide.com>
---
arch/arm/mach-omap2/mux44xx.c | 282 +----------------------------------------
1 files changed, 6 insertions(+), 276 deletions(-)
diff --git a/arch/arm/mach-omap2/mux44xx.c b/arch/arm/mach-omap2/mux44xx.c
index c322e7b..9a66445 100644
--- a/arch/arm/mach-omap2/mux44xx.c
+++ b/arch/arm/mach-omap2/mux44xx.c
@@ -755,25 +755,9 @@ static struct omap_ball __initdata omap4_core_cbl_ball[] =
{
#endif
/*
- * Superset of all mux modes for omap4 ES2.0
+ * Signals different on ES2.0 compared to superset
I didn't do it originally due to the huge amount of differences
between the two versions and the impact at runtime it will imply to
fix the modified entries at boot time. It might still be interesting
to measure it.
A quick test shows the difference is 0.007324218 - 0.007202148 seconds.. :)
Whaoo, 122us, that chip is really fast :-)
Since ES1 is no longer used on any board except the early one, that
you probably still have :-), it might be better to consider ES2 as
the superset and then store ES1 diff as the subset. It will avoid
any performance impact for the latest devices.
Well the performance impact is minimal in this case. But from cutting
down the size point of view that makes sense as then we could later on
optionally compile out OMAP_PACKAGE_CBL.
Sounds like that's a separate incremental patch on top of this one
though.
OK, I'll try to make that for 2.6.40.
Benoit
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
