On Thu, Feb 27, 2014 at 09:39:00PM +0100, Uwe Kleine-König wrote: > Hello, > > this is v2 of the series introducing devfs_create_partitions. With the > new syntax creating partitions is easier to parse for humans, it has > support to create partitions without holes inbetween without the need to > explicitly calculate the offsets and the code generated for the callers > is smaller. > > I compared the size of arch/arm/boards/a9m2410/a9m2410.o with and > without the series with the kernel's bloat-o-meter, with the following > result: > > add/remove: 1/0 grow/shrink: 0/1 up/down: 84/-104 (-20) > function old new delta > a9m2410_nand0_partitions - 84 +84 > a9m2410_devices_init 296 192 -104 > > (Well, the size advantage is a *bit* relativized by the growth of fs/devfs-core.o: > > add/remove: 3/0 grow/shrink: 0/1 up/down: 508/-124 (384) > function old new delta > __devfs_add_partition.part - 376 +376 > devfs_create_partitions - 128 +128 > new - 4 +4 > devfs_add_partition 296 172 -124 > > Don't know what "new" is above.) > > Compared to (implicit) v1 of this series I dropped the compound > literals and used named static arrays instead as wished by Sebastian > Hesselbarth and Sascha Hauer. Applied this series. I skipped the pca100 patch as this one has the partition layout in the devicetree now. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
