On 12/05/2015 08:22, mreza...@redhat.com wrote:
> From: Miroslav Rezanina <mreza...@redhat.com>
>
> mips build fail with link error in case PARALLEL_CONFIG is disabled as
> hw/mips/mips_jazz.c calls parallel_mm_init. Due to dependecies to content
> of parallel.c we can't simply move it to hw/isa/isa-devices.c.
>
> This patch adds stubs/parallel.c file that contains stub version of
> parallel_mm_init. This ensure successful build with PARALLEL_CONFIG disabled.
>
> Signed-off-by: Miroslav Rezanina <mreza...@redhat.com>
> ---
> stubs/Makefile.objs | 1 +
> stubs/parallel.c | 8 ++++++++
> 2 files changed, 9 insertions(+)
> create mode 100644 stubs/parallel.c
>
> diff --git a/stubs/Makefile.objs b/stubs/Makefile.objs
> index 8beff4c..ad4e110 100644
> --- a/stubs/Makefile.objs
> +++ b/stubs/Makefile.objs
> @@ -24,6 +24,7 @@ stub-obj-y += mon-printf.o
> stub-obj-y += mon-set-error.o
> stub-obj-y += monitor-init.o
> stub-obj-y += notify-event.o
> +stub-obj-y += parallel.o
> stub-obj-$(CONFIG_SPICE) += qemu-chr-open-spice.o
> stub-obj-y += qtest.o
> stub-obj-y += reset.o
> diff --git a/stubs/parallel.c b/stubs/parallel.c
> new file mode 100644
> index 0000000..8293d52
> --- /dev/null
> +++ b/stubs/parallel.c
> @@ -0,0 +1,8 @@
> +#include "hw/i386/pc.h"
> +
> +bool parallel_mm_init(MemoryRegion *address_space,
> + hwaddr base, int it_shift, qemu_irq irq,
> + CharDriverState *chr)
> +{
> + return false;
> +}
>
I think removing CONFIG_PARALLEL from a board that hardcodes its
presence makes little sense, so I would just drop this patch.
Paolo
