[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices |
Date: |
Thu, 17 Dec 2015 11:11:28 +0000 |
On 17 December 2015 at 10:28, Paolo Bonzini <address@hidden> wrote:
>
>
> On 17/12/2015 09:26, Peter Maydell wrote:
>>> > In any case, I would prefer qdev_bus_rename to stay in xlnx-zynqmp.c.
>> I disagree that it should be in xlnx-zynqmp.c. Either
>> (a) qdev already provides some reasonable mechanism for
>> SoC container like this to allow their users to get at
>> buses provided by their child objects, in which case we
>> should use it
>> (b) qdev doesn't provide such a mechanism, in which case
>> we need to provide one (either qdev_bus_rename or something
>> else if you have a better idea)
>>
>> But we definitely shouldn't have the SoC container code
>> messing around with the internals of the qdev objects.
>
> It's a hack and I don't want it to become a sanctioned way to do it.
> It's already messing around pretty heavily with qdev internals, see the
> line right after QLIST_INSERT_HEAD:
>
> QLIST_INSERT_HEAD(&dev->child_bus, spi_bus, sibling);
Well, that doesn't look good either. I think my point still
stands -- we should be providing proper infrastructure at
the qdev level to allow SoC container devices to do the
things they need to do, not just letting the containers
mess with the qdev internals.
thanks
-- PMM
- [Qemu-devel] [PATCH v5 0/6] Connect the SPI devices to ZynqMP, Alistair Francis, 2015/12/16
- [Qemu-devel] [PATCH v5 1/6] m25p80.c: Add sst25wf080 SPI flash device, Alistair Francis, 2015/12/16
- [Qemu-devel] [PATCH v5 2/6] ssi: Move ssi.h into a separate directory, Alistair Francis, 2015/12/16
- [Qemu-devel] [PATCH v5 3/6] xilinx_spips: Separate the state struct into a header, Alistair Francis, 2015/12/16
- [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Alistair Francis, 2015/12/16
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Paolo Bonzini, 2015/12/16
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Alistair Francis, 2015/12/16
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Peter Maydell, 2015/12/17
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Paolo Bonzini, 2015/12/17
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Paolo Bonzini, 2015/12/17
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Alistair Francis, 2015/12/18
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Paolo Bonzini, 2015/12/18
- Re: [Qemu-devel] [PATCH v5 5/6] xlnx-zynqmp: Connect the SPI devices, Peter Crosthwaite, 2015/12/19
[Qemu-devel] [PATCH v5 4/6] qdev: Add a function to rename busses, Alistair Francis, 2015/12/16
[Qemu-devel] [PATCH v5 6/6] xlnx-ep108: Connect the SPI Flash, Alistair Francis, 2015/12/16