Re: [PATCH] spice: simplify chardev setup

2020-07-23 Thread Christophe de Dinechin
On 2020-07-22 at 13:18 CEST, Gerd Hoffmann wrote... > On Wed, Jul 22, 2020 at 12:19:43PM +0200, Christophe de Dinechin wrote: >> >> On 2020-07-22 at 11:20 CEST, Christophe de Dinechin wrote... >> > On 2020-07-22 at 10:49 CEST, Gerd Hoffmann wrote... >> >> Initialize spice before chardevs. That

Re: [PATCH] spice: simplify chardev setup

2020-07-22 Thread Gerd Hoffmann
On Wed, Jul 22, 2020 at 11:20:31AM +0200, Christophe de Dinechin wrote: > > On 2020-07-22 at 10:49 CEST, Gerd Hoffmann wrote... > > Initialize spice before chardevs. That allows to register the spice > > chardevs directly in the init function and removes the need to maintain > > a linked list of

Re: [PATCH] spice: simplify chardev setup

2020-07-22 Thread Gerd Hoffmann
On Wed, Jul 22, 2020 at 12:19:43PM +0200, Christophe de Dinechin wrote: > > On 2020-07-22 at 11:20 CEST, Christophe de Dinechin wrote... > > On 2020-07-22 at 10:49 CEST, Gerd Hoffmann wrote... > >> Initialize spice before chardevs. That allows to register the spice > >> chardevs directly in the

Re: [PATCH] spice: simplify chardev setup

2020-07-22 Thread Christophe de Dinechin
On 2020-07-22 at 11:20 CEST, Christophe de Dinechin wrote... > On 2020-07-22 at 10:49 CEST, Gerd Hoffmann wrote... >> Initialize spice before chardevs. That allows to register the spice >> chardevs directly in the init function and removes the need to maintain >> a linked list of chardevs just

Re: [PATCH] spice: simplify chardev setup

2020-07-22 Thread Christophe de Dinechin
On 2020-07-22 at 10:49 CEST, Gerd Hoffmann wrote... > Initialize spice before chardevs. That allows to register the spice > chardevs directly in the init function and removes the need to maintain > a linked list of chardevs just for registration. > > Signed-off-by: Gerd Hoffmann Looks good to

[PATCH] spice: simplify chardev setup

2020-07-22 Thread Gerd Hoffmann
Initialize spice before chardevs. That allows to register the spice chardevs directly in the init function and removes the need to maintain a linked list of chardevs just for registration. Signed-off-by: Gerd Hoffmann --- include/chardev/spice.h | 1 - include/ui/qemu-spice.h | 1 -