Re: [PATCH] Add support for USB Keyboard attached to UHCI

"jidong xiao" <[email protected]>
Newsgroups gmane.linux.kernel.debugging
Message-ID <[email protected]>
On Nov 21, 2007 8:01 PM, Konstantin Baydarov <[email protected]> wrote:
>
> On Wed, 7 Nov 2007 17:11:37 +0300
> Konstantin Baydarov <[email protected]> wrote:
>
> > On Mon, 5 Nov 2007 10:54:42 -0800 (PST)
> > Aaron Young <[email protected]> wrote:
> >
> > >
> > >   Would it be possible for you to wait for my new kdb/usb
> > > infastructure code and re-implement your UHCI code against it at
> > > that time? I could attempt to take an in depth look at it at that
> > > time perhaps...
> > >
> > >   Thanks,
> > >
> > >   -Aaron
> >
> >   Yes I'll intergrate UCHI code to the new KDB.
> > ---------------------------
> > Use http://oss.sgi.com/ecartis to modify your settings or to
> > unsubscribe.
>
>  UCHI support integrated to kdb-v4.4-2.6.24-rc2. Tested on i386 UP PC.
>

> @@ -135,6 +261,9 @@ get_usb_char(void)
>         int     ret;
>         unsigned char keycode, spec;
>         extern u_short plain_map[], shift_map[], ctrl_map[];
> +       int     poll_ret, i2, j, max;
> +
> +       ret = -1;
>
>         /*
>          * Loop through all the USB keyboard(s) and return
> @@ -148,20 +277,27 @@ get_usb_char(void)
>                         continue;
>
>                 /* Transfer char */
> -                ret = (*kdb_usb_kbds[i].poll_func)(kdb_usb_kbds[i].urb);
> -                if (ret == -1) /* error or no characters, try the next kbd */
> +                poll_ret = (*kdb_usb_kbds[i].poll_func)(kdb_usb_kbds[i].urb);
> +                if (poll_ret == -1) /* error or no characters, try the next kbd */
>                         continue;
>

It seems once get_usb_char() is called,the poll_func i.e.
uhci_kdb_poll_char might be invoked, but I am wondering who calls
get_usb_char()?I didn't find any place where get_usb_call() is
invoked.

Regards
Jason
---------------------------
Use http://oss.sgi.com/ecartis to modify your settings or to unsubscribe.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.