Re: [Qemu-devel] [PATCH 0/3] dp8393x update

2015-01-02 Thread Peter Maydell
On 2 January 2015 at 11:33, Laurent Vivier wrote: > Le 02/01/2015 11:19, Peter Maydell a écrit : >> On 2 January 2015 at 09:25, Laurent Vivier wrote: >>> Using AddressSpace is really a very good idea, in fact, but I don't like >>> the way you pass it to the device (a qdev_set_prop()). >>> >>> I t

Re: [Qemu-devel] [PATCH 0/3] dp8393x update

2015-01-02 Thread Laurent Vivier
Le 02/01/2015 11:19, Peter Maydell a écrit : > On 2 January 2015 at 09:25, Laurent Vivier wrote: >> Using AddressSpace is really a very good idea, in fact, but I don't like >> the way you pass it to the device (a qdev_set_prop()). >> >> I think we should do as it is done in PCI. This must be manag

Re: [Qemu-devel] [PATCH 0/3] dp8393x update

2015-01-02 Thread Peter Maydell
On 2 January 2015 at 09:25, Laurent Vivier wrote: > Using AddressSpace is really a very good idea, in fact, but I don't like > the way you pass it to the device (a qdev_set_prop()). > > I think we should do as it is done in PCI. This must be managed at > sysbus level, not at the device level. Act

Re: [Qemu-devel] [PATCH 0/3] dp8393x update

2015-01-02 Thread Laurent Vivier
Le 02/01/2015 02:34, Laurent Vivier a écrit : > Hi Hervé, > > Le 01/01/2015 22:01, Hervé Poussineau a écrit : >> Hi Laurent, >> >> Le 29/12/2014 01:39, Laurent Vivier a écrit : >>> This is a series of patches I wrote to use dp8393x (SONIC) with >>> Quadra 800 emulation. I think it is interesting t

Re: [Qemu-devel] [PATCH 0/3] dp8393x update

2015-01-01 Thread Laurent Vivier
Hi Hervé, Le 01/01/2015 22:01, Hervé Poussineau a écrit : > Hi Laurent, > > Le 29/12/2014 01:39, Laurent Vivier a écrit : >> This is a series of patches I wrote to use dp8393x (SONIC) with >> Quadra 800 emulation. I think it is interesting to share them with the >> mainline. >> >> Qdev'ifying all

Re: [Qemu-devel] [PATCH 0/3] dp8393x update

2015-01-01 Thread Hervé Poussineau
Hi Laurent, Le 29/12/2014 01:39, Laurent Vivier a écrit : This is a series of patches I wrote to use dp8393x (SONIC) with Quadra 800 emulation. I think it is interesting to share them with the mainline. Qdev'ifying allows to remove the annoying warning: "requested NIC (anonymous, model dp83932)

[Qemu-devel] [PATCH 0/3] dp8393x update

2014-12-28 Thread Laurent Vivier
This is a series of patches I wrote to use dp8393x (SONIC) with Quadra 800 emulation. I think it is interesting to share them with the mainline. Qdev'ifying allows to remove the annoying warning: "requested NIC (anonymous, model dp83932) was not created (not supported by this machine?)" [PATCH 1