Re: [Qemu-devel] [PATCH v3 0/7] Kconfig dependencies for MIPS machines (but Malta)

[Aleksandar Markovic]

On May 28, 2019 8:49 AM, "Philippe Mathieu-Daudé" <address@hidden> wrote:
>
> > On 5/27/19 8:31 PM, Aleksandar Markovic wrote:
> > > What would be the status of this series? I am willing to integrate all
> > > this, including all followups, but have hard time sorting out what
> > > happend after this submission, what is the most complete version etc.
> > > Can you enlighten me please?
> >
> > Thanks for your interest in this series :)
> >
> > I was chatting with Thomas about this series earlier today!
> > I addressed your comment about the ITU/CPS devices, then wanted to join
> > all the Malta patches in the same series, but there are too many for a
> > single series so I'll keep them separate.
>
> After some sleep I could remember why I postponed this series:
>
> 1/ making the ITU selectable gives:
>
>   /usr/bin/ld: target/mips/op_helper.o: in function `helper_mtc0_saar':
>   ./target/mips/op_helper.c:1614: undefined reference to `itc_reconfigure'
>   /usr/bin/ld: target/mips/op_helper.o: in function `helper_mthc0_saar':
>   ./target/mips/op_helper.c:1631: undefined reference to `itc_reconfigure'
>   collect2: error: ld returned 1 exit status
>
> Anyway it is now selectable but has to be always selected.
>
> 2/ I thought only the Malta board was deeply mixed with the X86 ACPI
> code, but I later noticed the Fulong is too.
>
>   /usr/bin/ld: ../hw/isa/vt82c686.o: in function `vt82c686b_pm_realize':
>   ./hw/isa/vt82c686.c:381: undefined reference to `acpi_pm_tmr_init'
>   /usr/bin/ld: ./hw/isa/vt82c686.c:382: undefined reference to
> `acpi_pm1_evt_init'
>   /usr/bin/ld: ../hw/isa/vt82c686.o: in function `pm_update_sci':
>   ./hw/isa/vt82c686.c:195: undefined reference to `acpi_pm1_evt_get_sts'
>   collect2: error: ld returned 1 exit status
>
> Both VT82C686B/PIIX4 southbridges require considerable cleanup.
> I think I now have something workable, but I have to sell my patches
> correctly to the different subsystem maintainers ;)
>

I appreciate all this very much!

I don't want to rush you in any way, but just want to tell you that the
next MIPS pull request is planned for mid or end of the next week - so if
you judge some parts can be independently wrapped up by that time, by no
means send such set of patches. But again, it is up to you, no obligations
and no questions asked.

Au revoir,
Aleksandar

> Regards,
>
> Phil.