[U-Boot] [RFC] mmc: Remove ops from struct mmc and put in mmc_ops
Tom Rini
trini at ti.com
Thu Feb 27 14:50:07 CET 2014
On Thu, Feb 27, 2014 at 04:04:55AM +0000, Rommel G Custodio wrote:
> Dear Pantelis Antoniou
>
> Pantelis Antoniou <panto <at> antoniou-consulting.com> writes:
>
> >
> > Remove the in-structure ops and put them in mmc_ops with
> > a constant pointer to it.
> >
> > This makes the mmc structure smaller as well as conserving
> > code space (in theory).
>
> First build is latest master. Second build was with your patch. Compare size
> output.
>
>
> [local (powerpc-linux-gnuspe)]$ powerpc-linux-gnuspe-size obj-
> P5040DS_SPIFLASH_master/u-boot
> text data bss dec hex filename
> 463352 39236 314064 816652 c760c obj-P5040DS_SPIFLASH_master/u-boot
>
>
> [local (powerpc-linux-gnuspe)]$ powerpc-linux-gnuspe-size obj-
> P5040DS_SPIFLASH/u-boot
> text data bss dec hex filename
> 463364 39256 314064 816684 c762c obj-P5040DS_SPIFLASH/u-boot
The size increase on ARM is smaller, but still there too.
So we aren't making the code smaller nor the binaries. Marek, was this
your request to clean up or ? Will this help make device model stuff
easier down the line?
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140227/21d4073b/attachment.pgp>
More information about the U-Boot
mailing list