* [U-Boot] [PATCH 1/2] coreboot: only init usb if necessary
@ 2018-11-30 4:07 Bin Meng
2018-11-30 4:07 ` [U-Boot] [PATCH 2/2] efi: payload: " Bin Meng
2018-11-30 7:43 ` [U-Boot] [PATCH 1/2] coreboot: " Christian Gmeiner
0 siblings, 2 replies; 6+ messages in thread
From: Bin Meng @ 2018-11-30 4:07 UTC (permalink / raw)
To: u-boot
From: Thomas RIENOESSL <thomas.rienoessl@bachmann.info>
Up until now the call to initialize the USB subsystem was hardcoded
for U-Boot running as a coreboot payload. This was used to enable
the use of a USB keyboard in the U-Boot shell. However not all boards
might need this functionality. As initializing the USB subsystem can
take a considerable amount of time (several seconds on some boards),
we now initialize the USB subsystem only if U-Boot is configured to
use USB keyboards.
Signed-off-by: Thomas RIENOESSL <thomas.rienoessl@bachmann.info>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---
arch/x86/cpu/coreboot/coreboot.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/x86/cpu/coreboot/coreboot.c b/arch/x86/cpu/coreboot/coreboot.c
index aaf0d07..4c6ed0b 100644
--- a/arch/x86/cpu/coreboot/coreboot.c
+++ b/arch/x86/cpu/coreboot/coreboot.c
@@ -77,7 +77,8 @@ int last_stage_init(void)
timestamp_add_to_bootstage();
/* start usb so that usb keyboard can be used as input device */
- usb_init();
+ if (CONFIG_IS_ENABLED(USB_KEYBOARD))
+ usb_init();
board_final_cleanup();
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 2/2] efi: payload: only init usb if necessary
2018-11-30 4:07 [U-Boot] [PATCH 1/2] coreboot: only init usb if necessary Bin Meng
@ 2018-11-30 4:07 ` Bin Meng
2018-11-30 7:43 ` Christian Gmeiner
2018-11-30 7:43 ` [U-Boot] [PATCH 1/2] coreboot: " Christian Gmeiner
1 sibling, 1 reply; 6+ messages in thread
From: Bin Meng @ 2018-11-30 4:07 UTC (permalink / raw)
To: u-boot
Up until now the call to initialize the USB subsystem was hardcoded
for U-Boot running as an EFI payload. This was used to enable the
use of a USB keyboard in the U-Boot shell. However not all boards
might need this functionality. As initializing the USB subsystem can
take a considerable amount of time (several seconds on some boards),
we now initialize the USB subsystem only if U-Boot is configured to
use USB keyboards.
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
---
arch/x86/cpu/efi/payload.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/x86/cpu/efi/payload.c b/arch/x86/cpu/efi/payload.c
index c323c7b..225aef7 100644
--- a/arch/x86/cpu/efi/payload.c
+++ b/arch/x86/cpu/efi/payload.c
@@ -166,7 +166,8 @@ int reserve_arch(void)
int last_stage_init(void)
{
/* start usb so that usb keyboard can be used as input device */
- usb_init();
+ if (CONFIG_IS_ENABLED(USB_KEYBOARD))
+ usb_init();
return 0;
}
--
2.7.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 1/2] coreboot: only init usb if necessary
2018-11-30 4:07 [U-Boot] [PATCH 1/2] coreboot: only init usb if necessary Bin Meng
2018-11-30 4:07 ` [U-Boot] [PATCH 2/2] efi: payload: " Bin Meng
@ 2018-11-30 7:43 ` Christian Gmeiner
2018-12-10 2:17 ` Bin Meng
1 sibling, 1 reply; 6+ messages in thread
From: Christian Gmeiner @ 2018-11-30 7:43 UTC (permalink / raw)
To: u-boot
Am Fr., 30. Nov. 2018 um 05:02 Uhr schrieb Bin Meng <bmeng.cn@gmail.com>:
>
> From: Thomas RIENOESSL <thomas.rienoessl@bachmann.info>
>
> Up until now the call to initialize the USB subsystem was hardcoded
> for U-Boot running as a coreboot payload. This was used to enable
> the use of a USB keyboard in the U-Boot shell. However not all boards
> might need this functionality. As initializing the USB subsystem can
> take a considerable amount of time (several seconds on some boards),
> we now initialize the USB subsystem only if U-Boot is configured to
> use USB keyboards.
>
> Signed-off-by: Thomas RIENOESSL <thomas.rienoessl@bachmann.info>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
I totally forgot to send an updated version of this patch - thanks for
doing this Bin!
Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
--
greets
--
Christian Gmeiner, MSc
https://christian-gmeiner.info
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 2/2] efi: payload: only init usb if necessary
2018-11-30 4:07 ` [U-Boot] [PATCH 2/2] efi: payload: " Bin Meng
@ 2018-11-30 7:43 ` Christian Gmeiner
2018-12-10 2:17 ` Bin Meng
0 siblings, 1 reply; 6+ messages in thread
From: Christian Gmeiner @ 2018-11-30 7:43 UTC (permalink / raw)
To: u-boot
Am Fr., 30. Nov. 2018 um 05:02 Uhr schrieb Bin Meng <bmeng.cn@gmail.com>:
>
> Up until now the call to initialize the USB subsystem was hardcoded
> for U-Boot running as an EFI payload. This was used to enable the
> use of a USB keyboard in the U-Boot shell. However not all boards
> might need this functionality. As initializing the USB subsystem can
> take a considerable amount of time (several seconds on some boards),
> we now initialize the USB subsystem only if U-Boot is configured to
> use USB keyboards.
>
> Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
> ---
>
> arch/x86/cpu/efi/payload.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/x86/cpu/efi/payload.c b/arch/x86/cpu/efi/payload.c
> index c323c7b..225aef7 100644
> --- a/arch/x86/cpu/efi/payload.c
> +++ b/arch/x86/cpu/efi/payload.c
> @@ -166,7 +166,8 @@ int reserve_arch(void)
> int last_stage_init(void)
> {
> /* start usb so that usb keyboard can be used as input device */
> - usb_init();
> + if (CONFIG_IS_ENABLED(USB_KEYBOARD))
> + usb_init();
>
> return 0;
> }
> --
> 2.7.4
>
--
greets
--
Christian Gmeiner, MSc
https://christian-gmeiner.info
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 1/2] coreboot: only init usb if necessary
2018-11-30 7:43 ` [U-Boot] [PATCH 1/2] coreboot: " Christian Gmeiner
@ 2018-12-10 2:17 ` Bin Meng
0 siblings, 0 replies; 6+ messages in thread
From: Bin Meng @ 2018-12-10 2:17 UTC (permalink / raw)
To: u-boot
On Fri, Nov 30, 2018 at 3:43 PM Christian Gmeiner
<christian.gmeiner@gmail.com> wrote:
>
> Am Fr., 30. Nov. 2018 um 05:02 Uhr schrieb Bin Meng <bmeng.cn@gmail.com>:
> >
> > From: Thomas RIENOESSL <thomas.rienoessl@bachmann.info>
> >
> > Up until now the call to initialize the USB subsystem was hardcoded
> > for U-Boot running as a coreboot payload. This was used to enable
> > the use of a USB keyboard in the U-Boot shell. However not all boards
> > might need this functionality. As initializing the USB subsystem can
> > take a considerable amount of time (several seconds on some boards),
> > we now initialize the USB subsystem only if U-Boot is configured to
> > use USB keyboards.
> >
> > Signed-off-by: Thomas RIENOESSL <thomas.rienoessl@bachmann.info>
> > Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
> > Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>
> I totally forgot to send an updated version of this patch - thanks for
> doing this Bin!
>
> Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
>
applied to u-boot-x86, thanks!
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 2/2] efi: payload: only init usb if necessary
2018-11-30 7:43 ` Christian Gmeiner
@ 2018-12-10 2:17 ` Bin Meng
0 siblings, 0 replies; 6+ messages in thread
From: Bin Meng @ 2018-12-10 2:17 UTC (permalink / raw)
To: u-boot
On Fri, Nov 30, 2018 at 3:43 PM Christian Gmeiner
<christian.gmeiner@gmail.com> wrote:
>
> Am Fr., 30. Nov. 2018 um 05:02 Uhr schrieb Bin Meng <bmeng.cn@gmail.com>:
> >
> > Up until now the call to initialize the USB subsystem was hardcoded
> > for U-Boot running as an EFI payload. This was used to enable the
> > use of a USB keyboard in the U-Boot shell. However not all boards
> > might need this functionality. As initializing the USB subsystem can
> > take a considerable amount of time (several seconds on some boards),
> > we now initialize the USB subsystem only if U-Boot is configured to
> > use USB keyboards.
> >
> > Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
>
> Reviewed-by: Christian Gmeiner <christian.gmeiner@gmail.com>
applied to u-boot-x86, thanks!
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2018-12-10 2:17 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-30 4:07 [U-Boot] [PATCH 1/2] coreboot: only init usb if necessary Bin Meng
2018-11-30 4:07 ` [U-Boot] [PATCH 2/2] efi: payload: " Bin Meng
2018-11-30 7:43 ` Christian Gmeiner
2018-12-10 2:17 ` Bin Meng
2018-11-30 7:43 ` [U-Boot] [PATCH 1/2] coreboot: " Christian Gmeiner
2018-12-10 2:17 ` Bin Meng
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.