Re: [PATCH] virtio_mmio: add ACPI probing

2015-08-03 Thread G Gregory
On 28 July 2015 at 11:27, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015 at 11:12:33AM +0100, Peter Maydell wrote: On 28 July 2015 at 11:08, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015 at 10:44:02AM +0100, Graeme Gregory wrote: Added the match table and

Re: [PATCH] virtio_mmio: add ACPI probing

2015-08-03 Thread Shannon Zhao
On 2015/7/29 5:10, Peter Maydell wrote: On 28 July 2015 at 21:28, G Gregory graeme.greg...@linaro.org wrote: On 28 July 2015 at 21:12, Peter Maydell peter.mayd...@linaro.org wrote: Mmm. I'm not terribly happy about stuff being in QEMU before the ACPI spec for it has been finalised. We

Re: [PATCH] virtio_mmio: add ACPI probing

2015-08-03 Thread G Gregory
On 28 July 2015 at 21:12, Peter Maydell peter.mayd...@linaro.org wrote: On 28 July 2015 at 11:27, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015 at 11:12:33AM +0100, Peter Maydell wrote: On 28 July 2015 at 11:08, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-29 Thread Michael S. Tsirkin
On Tue, Jul 28, 2015 at 10:44:02AM +0100, Graeme Gregory wrote: Added the match table and pointers for ACPI probing to the driver. This uses the same identifier for virt devices as being used for qemu ARM64 ACPI support.

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-29 Thread Peter Maydell
On 28 July 2015 at 11:33, G Gregory graeme.greg...@linaro.org wrote: We assigned LNRO in ASWG to avoid collisions with our prototypes/real platforms so it makes sense to me to switch to QEMU. So just to check, if we switch virtio-mmio from an LNRO0005 ID to a QEMU ID we aren't going to

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-29 Thread Michael S. Tsirkin
On Wed, Jul 29, 2015 at 06:52:29PM +0100, Peter Maydell wrote: On 28 July 2015 at 11:33, G Gregory graeme.greg...@linaro.org wrote: We assigned LNRO in ASWG to avoid collisions with our prototypes/real platforms so it makes sense to me to switch to QEMU. So just to check, if we switch

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-29 Thread Michael S. Tsirkin
On Wed, Jul 29, 2015 at 10:01:35AM +0800, Shannon Zhao wrote: On 2015/7/29 5:10, Peter Maydell wrote: On 28 July 2015 at 21:28, G Gregory graeme.greg...@linaro.org wrote: On 28 July 2015 at 21:12, Peter Maydell peter.mayd...@linaro.org wrote: Mmm. I'm not terribly happy about stuff

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-28 Thread Peter Maydell
On 28 July 2015 at 11:08, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015 at 10:44:02AM +0100, Graeme Gregory wrote: Added the match table and pointers for ACPI probing to the driver. This uses the same identifier for virt devices as being used for qemu ARM64 ACPI support.

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-28 Thread Michael S. Tsirkin
On Tue, Jul 28, 2015 at 10:44:02AM +0100, Graeme Gregory wrote: Added the match table and pointers for ACPI probing to the driver. This uses the same identifier for virt devices as being used for qemu ARM64 ACPI support.

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-28 Thread Michael S. Tsirkin
On Tue, Jul 28, 2015 at 11:12:33AM +0100, Peter Maydell wrote: On 28 July 2015 at 11:08, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015 at 10:44:02AM +0100, Graeme Gregory wrote: Added the match table and pointers for ACPI probing to the driver. This uses the same

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-28 Thread Peter Maydell
On 28 July 2015 at 11:27, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015 at 11:12:33AM +0100, Peter Maydell wrote: On 28 July 2015 at 11:08, Michael S. Tsirkin m...@redhat.com wrote: On Tue, Jul 28, 2015 at 10:44:02AM +0100, Graeme Gregory wrote: Added the match table and

Re: [PATCH] virtio_mmio: add ACPI probing

2015-07-28 Thread Peter Maydell
On 28 July 2015 at 21:28, G Gregory graeme.greg...@linaro.org wrote: On 28 July 2015 at 21:12, Peter Maydell peter.mayd...@linaro.org wrote: Mmm. I'm not terribly happy about stuff being in QEMU before the ACPI spec for it has been finalised. We should not be picking stuff randomly on the