Hi Tom, On 9 December 2014 at 08:04, Simon Glass <[email protected]> wrote: > On 8 December 2014 at 00:23, Martin Dorwig <[email protected]> wrote: >> this is an atempt to make the export of functions typesafe. >> I replaced the jumptable void ** by a struct (jt_funcs) with function >> pointers. >> The EXPORT_FUNC macro now has 3 fixed parameters and one >> variadic parameter >> The first is the name of the exported function, >> the rest of the parameters are used to format a functionpointer >> in the jumptable, >> >> the EXPORT_FUNC macros are expanded three times, >> 1. to declare the members of the struct >> 2. to initialize the structmember pointers >> 3. to call the functions in stubs.c >> >> Signed-off-by: Martin Dorwig <[email protected]> > > Acked-by: Simon Glass <[email protected]> > > Tested on Pit (ARM-based) and chromebook_link (x86). > > Tested-by: Simon Glass <[email protected]> > > Everything seems to build OK but for two 'exceeding area limit' > problems which I assume we can ignore. >
I see this is assigned to you in patchwork. Unfortunately with driver model I2C this patch is not quite right any more. Since this is my fault I will take on tweaking it and sending v5. >> --- >> >> Changes in v4: >> - add forward decl. for struct spi_slave to export.h >> >> Changes in v3: >> - install_hdlr/free_hdlr must be dummy, since they do not exist >> - blackfin cpu.c must include exports.h >> - rebased to master >> - take CONFIG_DM_SPI into account >> >> Changes in v2: >> - redesign the way functions are exported to standalone applications Regards, Simon _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot
