[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v5 05/17] spapr/drc: Drop spapr_drc_attach() fdt
From: |
David Gibson |
Subject: |
Re: [qemu-s390x] [PATCH v5 05/17] spapr/drc: Drop spapr_drc_attach() fdt argument |
Date: |
Wed, 20 Feb 2019 20:57:00 +1100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Wed, Feb 20, 2019 at 10:01:23AM +0100, Greg Kurz wrote:
> On Wed, 20 Feb 2019 14:22:19 +1100
> David Gibson <address@hidden> wrote:
>
> > On Tue, Feb 19, 2019 at 06:17:58PM +0100, Greg Kurz wrote:
> > > All DRC subtypes have been converted to generate the FDT fragment at
> > > configure connector time instead of attach time. The fdt and fdt_offset
> > > arguments of spapr_drc_attach() aren't needed anymore. Drop them and
> > > make the implementation of the dt_populate() method mandatory.
> > >
> > > Signed-off-by: Greg Kurz <address@hidden>
> >
> > I've applied the first 5 patches to ppc-for-4.0, but as a followup...
> >
> > [...]
> > > @@ -1113,8 +1104,6 @@ static void rtas_ibm_configure_connector(PowerPCCPU
> > > *cpu,
> > >
> > > drck = SPAPR_DR_CONNECTOR_GET_CLASS(drc);
> > >
> > > - g_assert(drc->fdt || drck->dt_populate);
> > > -
> > > if (!drc->fdt) {
> >
> > ..you can now remove this conditional, since it will always be true.
> >
>
> Hmm... I'm afraid this is not true since configure-connector is supposed
> to be called several times according to PAPR. And this is exactly what
> the code does: first return the node name to the guest, then all properties
> and subnodes one at a time...
Oh, duh, of course. Forget I said that.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- [qemu-s390x] [PATCH v5 00/17] spapr: Add support for PHB hotplug, Greg Kurz, 2019/02/19
- [qemu-s390x] [PATCH v5 06/17] xics: Write source state to KVM at claim time, Greg Kurz, 2019/02/19
- [qemu-s390x] [PATCH v5 04/17] spapr/pci: Generate FDT fragment at configure connector time, Greg Kurz, 2019/02/19
- [qemu-s390x] [PATCH v5 07/17] spapr: Expose the name of the interrupt controller node, Greg Kurz, 2019/02/19
- [qemu-s390x] [PATCH v5 08/17] spapr_irq: Expose the phandle of the interrupt controller, Greg Kurz, 2019/02/19
- [qemu-s390x] [PATCH v5 11/17] spapr: populate PHB DRC entries for root DT node, Greg Kurz, 2019/02/19