Hi Andreas, On Sun, May 5, 2013 at 9:46 PM, Andreas Färber <afaer...@suse.de> wrote: > Am 05.05.2013 05:11, schrieb Peter Crosthwaite: >> On Sun, May 5, 2013 at 12:09 AM, Jean-Christophe DUBOIS >> <j...@tribudubois.net> wrote: >>> +void imx_fec_create(int nic, const hwaddr base, qemu_irq irq) >> >> does this compile with Werror? I thought there was a requirement that >> all non-static functions must have a fn prototype defined beforehand >> which I cant see in this patch. > > It went into 2/4 accidentally. :) >
Thanks, While your here I have an on topic question, do we want creation helpers like this? I notice that we progressively moving towards the state where machine models talk to QOM directly without wrapper init fns such as this.. Are functions like this compulsory, optional or deprecated? Regards, Peter > Andreas > > -- > SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany > GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg >
