All of lore.kernel.org
 help / color / mirror / Atom feed
* kbdif
@ 2008-02-13 17:41 Samuel Thibault
  2008-02-13 18:14 ` kbdif Markus Armbruster
  0 siblings, 1 reply; 5+ messages in thread
From: Samuel Thibault @ 2008-02-13 17:41 UTC (permalink / raw)
  To: xen-devel

Mmm, I can't find any event related to buttons in kbdif.h, was it just
forgotten?

Samuel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: kbdif
  2008-02-13 17:41 kbdif Samuel Thibault
@ 2008-02-13 18:14 ` Markus Armbruster
  2008-02-13 18:23   ` kbdif Samuel Thibault
  0 siblings, 1 reply; 5+ messages in thread
From: Markus Armbruster @ 2008-02-13 18:14 UTC (permalink / raw)
  To: Samuel Thibault; +Cc: xen-devel

Samuel Thibault <samuel.thibault@eu.citrix.com> writes:

> Mmm, I can't find any event related to buttons in kbdif.h, was it just
> forgotten?
>
> Samuel

XENKBD_TYPE_KEY, BTN_LEFT etc.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: kbdif
  2008-02-13 18:14 ` kbdif Markus Armbruster
@ 2008-02-13 18:23   ` Samuel Thibault
  2008-02-14  8:42     ` kbdif Markus Armbruster
  0 siblings, 1 reply; 5+ messages in thread
From: Samuel Thibault @ 2008-02-13 18:23 UTC (permalink / raw)
  To: Markus Armbruster; +Cc: xen-devel

Markus Armbruster, le Wed 13 Feb 2008 19:14:05 +0100, a écrit :
> Samuel Thibault <samuel.thibault@eu.citrix.com> writes:
> 
> > Mmm, I can't find any event related to buttons in kbdif.h, was it just
> > forgotten?
> >
> > Samuel
> 
> XENKBD_TYPE_KEY, BTN_LEFT etc.

Ah ok.
Is there a reason for requiring fb and kbd to be initialised "in
parallel"?  I mean, hw/xenfb.c requires the kbd to be created, then
the fb to be created, then the kbd to be initialized, then the fb to be
initialized, then the kbd to be connected, then the fb to be connected.

I would have thought that creating/initializing/connecting kbd first and
then eventually fb would be allowed.

Samuel

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: kbdif
  2008-02-13 18:23   ` kbdif Samuel Thibault
@ 2008-02-14  8:42     ` Markus Armbruster
  2008-02-14 10:06       ` kbdif Samuel Thibault
  0 siblings, 1 reply; 5+ messages in thread
From: Markus Armbruster @ 2008-02-14  8:42 UTC (permalink / raw)
  To: Samuel Thibault; +Cc: xen-devel

Samuel Thibault <samuel.thibault@eu.citrix.com> writes:

[...]
> Is there a reason for requiring fb and kbd to be initialised "in
> parallel"?  I mean, hw/xenfb.c requires the kbd to be created, then
> the fb to be created, then the kbd to be initialized, then the fb to be
> initialized, then the kbd to be connected, then the fb to be connected.
>
> I would have thought that creating/initializing/connecting kbd first and
> then eventually fb would be allowed.
>
> Samuel

I guess it could be done that way too.

The state machine in hw/xenfb.c has the code in execution order, to
make it easier to read in one forward pass.  And it groups the kbd and
fb steps together, to highlight their similarities.

Any problems with this execution order?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: kbdif
  2008-02-14  8:42     ` kbdif Markus Armbruster
@ 2008-02-14 10:06       ` Samuel Thibault
  0 siblings, 0 replies; 5+ messages in thread
From: Samuel Thibault @ 2008-02-14 10:06 UTC (permalink / raw)
  To: Markus Armbruster; +Cc: xen-devel

Markus Armbruster, le Thu 14 Feb 2008 09:42:04 +0100, a écrit :
> Samuel Thibault <samuel.thibault@eu.citrix.com> writes:
> > Is there a reason for requiring fb and kbd to be initialised "in
> > parallel"?  I mean, hw/xenfb.c requires the kbd to be created, then
> > the fb to be created, then the kbd to be initialized, then the fb to be
> > initialized, then the kbd to be connected, then the fb to be connected.
> >
> > I would have thought that creating/initializing/connecting kbd first and
> > then eventually fb would be allowed.
> 
> Any problems with this execution order?

Yes: in the code I'm currently writing I have

	kbd_dev = init_kbdfront(nodename);
	fb_dev = init_fbfront(nodename, width, height, etc.);

And this can't work currently, I instead have to run init_kbdfront and
init_fbfront in threads so as to get them running in parallel.

Samuel

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2008-02-14 10:06 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-02-13 17:41 kbdif Samuel Thibault
2008-02-13 18:14 ` kbdif Markus Armbruster
2008-02-13 18:23   ` kbdif Samuel Thibault
2008-02-14  8:42     ` kbdif Markus Armbruster
2008-02-14 10:06       ` kbdif Samuel Thibault

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.