[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Allen Martin]

This allows systems to pause autoboot with USB keyboard.  Tested on
tegra2 seaboard.

Signed-off-by: Allen Martin <amartin@nvidia.com>
---
 README               |    5 +++++
 arch/arm/lib/board.c |    9 +++++++++
 2 files changed, 14 insertions(+), 0 deletions(-)

diff --git a/README b/README
index 882483b..40e8481 100644
--- a/README
+++ b/README
@@ -1132,6 +1132,11 @@ The following options need to be configured:
 		CONFIG_USB_EHCI_TXFIFO_THRESH enables setting of the
 		txfilltuning field in the EHCI controller on reset.
 
+		CONFIG_USB_INIT causes USB to be initialized
+		automatically before the main command loop.  This is
+		useful if you have a USB keyboard and you want to use
+		it to stop autoboot.
+
 - USB Device:
 		Define the below if you wish to use the USB console.
 		Once firmware is rebuilt from a serial console issue the
diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
index 3d78274..6787048 100644
--- a/arch/arm/lib/board.c
+++ b/arch/arm/lib/board.c
@@ -64,6 +64,10 @@
 #include "../drivers/net/lan91c96.h"
 #endif
 
+#ifdef CONFIG_USB_INIT
+#include <usb.h>
+#endif
+
 DECLARE_GLOBAL_DATA_PTR;
 
 ulong monitor_flash_len;
@@ -548,6 +552,11 @@ void board_init_r(gd_t *id, ulong dest_addr)
 	api_init();
 #endif
 
+#ifdef CONFIG_USB_INIT
+	/* this needs to be before console init for USB kbd to work as stdin */
+	usb_init();
+#endif
+
 	console_init_r();	/* fully init console as a device */
 
 #if defined(CONFIG_ARCH_MISC_INIT)
-- 
1.7.5.4

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Marek Vasut]

> This allows systems to pause autoboot with USB keyboard.  Tested on
> tegra2 seaboard.
> 
> Signed-off-by: Allen Martin <amartin@nvidia.com>

Can't you just add "usb reset" to preboot env?

M
> ---
>  README               |    5 +++++
>  arch/arm/lib/board.c |    9 +++++++++
>  2 files changed, 14 insertions(+), 0 deletions(-)
> 
> diff --git a/README b/README
> index 882483b..40e8481 100644
> --- a/README
> +++ b/README
> @@ -1132,6 +1132,11 @@ The following options need to be configured:
>  		CONFIG_USB_EHCI_TXFIFO_THRESH enables setting of the
>  		txfilltuning field in the EHCI controller on reset.
> 
> +		CONFIG_USB_INIT causes USB to be initialized
> +		automatically before the main command loop.  This is
> +		useful if you have a USB keyboard and you want to use
> +		it to stop autoboot.
> +
>  - USB Device:
>  		Define the below if you wish to use the USB console.
>  		Once firmware is rebuilt from a serial console issue the
> diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
> index 3d78274..6787048 100644
> --- a/arch/arm/lib/board.c
> +++ b/arch/arm/lib/board.c
> @@ -64,6 +64,10 @@
>  #include "../drivers/net/lan91c96.h"
>  #endif
> 
> +#ifdef CONFIG_USB_INIT
> +#include <usb.h>
> +#endif
> +
>  DECLARE_GLOBAL_DATA_PTR;
> 
>  ulong monitor_flash_len;
> @@ -548,6 +552,11 @@ void board_init_r(gd_t *id, ulong dest_addr)
>  	api_init();
>  #endif
> 
> +#ifdef CONFIG_USB_INIT
> +	/* this needs to be before console init for USB kbd to work as stdin */
> +	usb_init();
> +#endif
> +
>  	console_init_r();	/* fully init console as a device */
> 
>  #if defined(CONFIG_ARCH_MISC_INIT)

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Wolfgang Denk]

Dear Allen Martin,

In message <1324515956-1642-1-git-send-email-amartin@nvidia.com> you wrote:
> This allows systems to pause autoboot with USB keyboard.  Tested on
> tegra2 seaboard.
> 
> Signed-off-by: Allen Martin <amartin@nvidia.com>
> ---
>  README               |    5 +++++
>  arch/arm/lib/board.c |    9 +++++++++
>  2 files changed, 14 insertions(+), 0 deletions(-)

This modification is in no way ARM specific, so if we accept it, it
must be applied to all supported architectures.

Where exacly is this feature nneded?  You don;t enable it in any of
the board configurations...

Also, should we then not auto-enable this if CONFIG_USB_KEYBOARD is
set?

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I can type faster than I can move a  mouse,  so  I  find  menu-driven
drawing packages time consuming and frustrating.      - W. R. Stevens

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Igor Grinberg]

Hi Wolfgang,

On 12/22/11 10:06, Wolfgang Denk wrote:
> Dear Allen Martin,
> 
> In message <1324515956-1642-1-git-send-email-amartin@nvidia.com> you wrote:
>> This allows systems to pause autoboot with USB keyboard.  Tested on
>> tegra2 seaboard.
>>
>> Signed-off-by: Allen Martin <amartin@nvidia.com>
>> ---
>>  README               |    5 +++++
>>  arch/arm/lib/board.c |    9 +++++++++
>>  2 files changed, 14 insertions(+), 0 deletions(-)
> 
> This modification is in no way ARM specific, so if we accept it, it
> must be applied to all supported architectures.
> 
> Where exacly is this feature nneded?  You don;t enable it in any of
> the board configurations...
> 
> Also, should we then not auto-enable this if CONFIG_USB_KEYBOARD is
> set?

Right, IMHO, there is no need for a new config option -
just make the USB init depend on CONFIG_USB_KEYBOARD.


-- 
Regards,
Igor.

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Allen Martin]

> > This allows systems to pause autoboot with USB keyboard.  Tested on
> > tegra2 seaboard.
> >
> > Signed-off-by: Allen Martin <amartin@nvidia.com>
> 
> Can't you just add "usb reset" to preboot env?

The point is to be able to use USB keyboard to stop autoboot on systems where USB keyboard is the primary (or only) input device.  This needs to happen before BOOTCOMMAND gets parsed.

-Allen
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Moffett, Kyle D]

--
Curious about my work on the Debian powerpcspe port?
I'm keeping a blog here: http://pureperl.blogspot.com/
>>> This allows systems to pause autoboot with USB keyboard.  Tested on
>>> tegra2 seaboard.
>>> 
>>> Signed-off-by: Allen Martin <amartin@nvidia.com>
>> 
>> Can't you just add "usb reset" to preboot env?
> 
> The point is to be able to use USB keyboard to stop autoboot on systems
> where USB keyboard is the primary (or only) input device.  This needs
> to happen before BOOTCOMMAND gets parsed.

I think the "preboot" environment variable already *is* run before
starting the autoboot timer?

Cheers,
Kyle Moffett

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Marek Vasut]

> --
> Curious about my work on the Debian powerpcspe port?
> I'm keeping a blog here: http://pureperl.blogspot.com/
> 
> >>> This allows systems to pause autoboot with USB keyboard.  Tested on
> >>> tegra2 seaboard.
> >>> 
> >>> Signed-off-by: Allen Martin <amartin@nvidia.com>
> >> 
> >> Can't you just add "usb reset" to preboot env?
> > 
> > The point is to be able to use USB keyboard to stop autoboot on systems
> > where USB keyboard is the primary (or only) input device.  This needs
> > to happen before BOOTCOMMAND gets parsed.
> 
> I think the "preboot" environment variable already *is* run before
> starting the autoboot timer?

Yep.

M

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Allen Martin]

> > >> Can't you just add "usb reset" to preboot env?
> > >
> > > The point is to be able to use USB keyboard to stop autoboot on
> > > systems where USB keyboard is the primary (or only) input device.
> > > This needs to happen before BOOTCOMMAND gets parsed.
> >
> > I think the "preboot" environment variable already *is* run before
> > starting the autoboot timer?
> 
> Yep.
> 

I also had to stick it before console init because console makes decisions about the input device based on what's been registered when it initializes (at least with iomux turned on, I didn't try it without).

Maybe a better fix is to have console reevaluate things when the usb keyboard driver registers itself?  I'm open to suggestions.

-Allen
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Allen Martin]

> > This allows systems to pause autoboot with USB keyboard.  Tested on
> > tegra2 seaboard.
> >
> > Signed-off-by: Allen Martin <amartin@nvidia.com>
> > ---
> >  README               |    5 +++++
> >  arch/arm/lib/board.c |    9 +++++++++
> >  2 files changed, 14 insertions(+), 0 deletions(-)
> 
> This modification is in no way ARM specific, so if we accept it, it must be
> applied to all supported architectures.
> 
Sounds reasonable, I can make patches for other architectures, but I can 
really only test ARM and x86

> Where exacly is this feature nneded?  You don;t enable it in any of 
> the board configurations...

It's intended for systems that use USB keyboard as the primary input device.  
I have a tegra2 device like that, I haven't sent out the patch to enable it yet 
because it depends on Simon Glass's tegra USB patches, which are still in review.

> Also, should we then not auto-enable this if CONFIG_USB_KEYBOARD is set?

Sure, that sounds reasonable.

-Allen
nvpublic

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Allen Martin]

> > > > The point is to be able to use USB keyboard to stop autoboot on
> > > > systems where USB keyboard is the primary (or only) input device.
> > > > This needs to happen before BOOTCOMMAND gets parsed.
> > >
> > > I think the "preboot" environment variable already *is* run before
> > > starting the autoboot timer?
> >
> > Yep.
> >
> 
> I also had to stick it before console init because console makes
> decisions about the input device based on what's been registered when it
> initializes (at least with iomux turned on, I didn't try it without).
> 
> Maybe a better fix is to have console reevaluate things when the usb
> keyboard driver registers itself?  I'm open to suggestions.

I don't fully grok how console/stdin gets assigned, but is this possibly
the problem? (from drv_usb_kbd_init()):

                /* Check if this is the standard input device. */
                if (strcmp(stdinname, DEVNAME))
                        return 1;

                /* Reassign the console */
                if (overwrite_console())
                        return 1;

                error = console_assign(stdin, DEVNAME);

This seems incompatible with iomux, shouldn't these decisions be made in the
console driver?  In my case, stdin is set to "serial,tegra-kbc,usbkbd" so this
strcmp won't match.

-Allen
nvpublic

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Wolfgang Denk]

Dear Allen Martin,

In message <3C7A7ACA8617D24290826EC008B5CD083E5DCDAF40@HQMAIL03.nvidia.com> you wrote:
> 
> I don't fully grok how console/stdin gets assigned, but is this possibly
> the problem? (from drv_usb_kbd_init()):
> 
>                 /* Check if this is the standard input device. */
>                 if (strcmp(stdinname, DEVNAME))
>                         return 1;
> 
>                 /* Reassign the console */
>                 if (overwrite_console())
>                         return 1;
> 
>                 error = console_assign(stdin, DEVNAME);
> 
> This seems incompatible with iomux, shouldn't these decisions be made in the
> console driver?  In my case, stdin is set to "serial,tegra-kbc,usbkbd" so this
> strcmp won't match.

Indeed, this code predates the iomux implementation, and nobody
noticed yet that it needs fixing.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
They're usually so busy thinking about what  happens  next  that  the
only  time they ever find out what is happening now is when they come
to look back on it.                 - Terry Pratchett, _Wyrd Sisters_

[U-Boot] [PATCH] USB: add CONFIG_USB_INIT to autoinitialize USB before main_loop

[Allen Martin]

> > This seems incompatible with iomux, shouldn't these decisions be made
> in the
> > console driver?  In my case, stdin is set to "serial,tegra-kbc,usbkbd"
> so this
> > strcmp won't match.
> 
> Indeed, this code predates the iomux implementation, and nobody
> noticed yet that it needs fixing.

Ok thanks, I'll work on fixing this, as it seems a lot less painful
than modifying the init sequence for a dozen different architectures
I know nothing about.

-Allen

nvpublic