Peter Maydell <[email protected]> writes: > On 24 October 2012 09:42, Markus Armbruster <[email protected]> wrote: >> Peter Maydell <[email protected]> writes: >> >>> A lot of the ARM board models follow the pattern of having a >>> single common init function which is called with various >>> parameters from the QEMUMachine init function for several board >>> model variants. The change to QEMUMachineInitArgs in commit 5f072e >>> took the fairly mechanical approach of splitting the QEMUMachineInitArgs >>> fields out in each variant's init function to pass to an unchanged >>> common init function. This is a bit repetitive, so clean them up >>> to just pass the QEMUMachineInitArgs directly to the common >>> function instead. >> >> A quick grep finds similar cleanup opportunities in >> >> hw/exynos4_boards.c exynos4_boards_init_common() >> hw/pc_piix.c pc_init1() >> hw/sun4m.c sun4m_hw_init() >> hw/sun4u.c sun4uv_init() >> hw/xtensa_lx60.c lx_init() >> hw/xtensa_sim.c sim_init() > > I deliberately stuck to the ARM boards on the basis that the > others were a little out of my jurisdiction. I missed the > exynos4 though.
Fair enough. Cc'ing some relevant maintainers.
