All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] coreboot: only init usb if necessary
@ 2018-11-12 12:44 Christian Gmeiner
  2018-11-12 13:44 ` Bin Meng
  0 siblings, 1 reply; 5+ messages in thread
From: Christian Gmeiner @ 2018-11-12 12:44 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>
---
 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 a6fd3a849a..915341fe12 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 (IS_ENABLED(CONFIG_USB_KEYBOARD))
+		usb_init();
 
 	board_final_cleanup();
 
-- 
2.19.1

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

* [U-Boot] [PATCH] coreboot: only init usb if necessary
  2018-11-12 12:44 [U-Boot] [PATCH] coreboot: only init usb if necessary Christian Gmeiner
@ 2018-11-12 13:44 ` Bin Meng
  2018-11-12 14:39   ` Christian Gmeiner
  0 siblings, 1 reply; 5+ messages in thread
From: Bin Meng @ 2018-11-12 13:44 UTC (permalink / raw)
  To: u-boot

Hi Christian/Thomas,

On Mon, Nov 12, 2018 at 8:44 PM Christian Gmeiner
<christian.gmeiner@gmail.com> wrote:
>
> 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>
> ---
>  arch/x86/cpu/coreboot/coreboot.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>

Looks good to me. Can we also fix the efi-payload one?

Regards,
Bin

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

* [U-Boot] [PATCH] coreboot: only init usb if necessary
  2018-11-12 13:44 ` Bin Meng
@ 2018-11-12 14:39   ` Christian Gmeiner
  2018-11-12 14:59     ` Bin Meng
  0 siblings, 1 reply; 5+ messages in thread
From: Christian Gmeiner @ 2018-11-12 14:39 UTC (permalink / raw)
  To: u-boot

Hi Bin

Am Mo., 12. Nov. 2018 um 14:44 Uhr schrieb Bin Meng <bmeng.cn@gmail.com>:
>
> Hi Christian/Thomas,
>
> On Mon, Nov 12, 2018 at 8:44 PM Christian Gmeiner
> <christian.gmeiner@gmail.com> wrote:
> >
> > 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>
> > ---
> >  arch/x86/cpu/coreboot/coreboot.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
>
> Looks good to me. Can we also fix the efi-payload one?
>

Sure - do you prefer two patches (one for coreboot and one for x86:
efi: payload) or
shall we put both changes into one patch?

-- 
greets
--
Christian Gmeiner, MSc

https://christian-gmeiner.info

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

* [U-Boot] [PATCH] coreboot: only init usb if necessary
  2018-11-12 14:39   ` Christian Gmeiner
@ 2018-11-12 14:59     ` Bin Meng
  2018-11-30  4:03       ` Bin Meng
  0 siblings, 1 reply; 5+ messages in thread
From: Bin Meng @ 2018-11-12 14:59 UTC (permalink / raw)
  To: u-boot

Hi Christian,

On Mon, Nov 12, 2018 at 10:39 PM Christian Gmeiner
<christian.gmeiner@gmail.com> wrote:
>
> Hi Bin
>
> Am Mo., 12. Nov. 2018 um 14:44 Uhr schrieb Bin Meng <bmeng.cn@gmail.com>:
> >
> > Hi Christian/Thomas,
> >
> > On Mon, Nov 12, 2018 at 8:44 PM Christian Gmeiner
> > <christian.gmeiner@gmail.com> wrote:
> > >
> > > 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>
> > > ---
> > >  arch/x86/cpu/coreboot/coreboot.c | 3 ++-
> > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > >
> >
> > Looks good to me. Can we also fix the efi-payload one?
> >
>
> Sure - do you prefer two patches (one for coreboot and one for x86:
> efi: payload) or
> shall we put both changes into one patch?

I don't have preference. You can use either way :)

Regards,
Bin

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

* [U-Boot] [PATCH] coreboot: only init usb if necessary
  2018-11-12 14:59     ` Bin Meng
@ 2018-11-30  4:03       ` Bin Meng
  0 siblings, 0 replies; 5+ messages in thread
From: Bin Meng @ 2018-11-30  4:03 UTC (permalink / raw)
  To: u-boot

Hi Christian,

On Mon, Nov 12, 2018 at 10:59 PM Bin Meng <bmeng.cn@gmail.com> wrote:
>
> Hi Christian,
>
> On Mon, Nov 12, 2018 at 10:39 PM Christian Gmeiner
> <christian.gmeiner@gmail.com> wrote:
> >
> > Hi Bin
> >
> > Am Mo., 12. Nov. 2018 um 14:44 Uhr schrieb Bin Meng <bmeng.cn@gmail.com>:
> > >
> > > Hi Christian/Thomas,
> > >
> > > On Mon, Nov 12, 2018 at 8:44 PM Christian Gmeiner
> > > <christian.gmeiner@gmail.com> wrote:
> > > >
> > > > 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>
> > > > ---
> > > >  arch/x86/cpu/coreboot/coreboot.c | 3 ++-
> > > >  1 file changed, 2 insertions(+), 1 deletion(-)
> > > >
> > >
> > > Looks good to me. Can we also fix the efi-payload one?
> > >
> >
> > Sure - do you prefer two patches (one for coreboot and one for x86:
> > efi: payload) or
> > shall we put both changes into one patch?
>
> I don't have preference. You can use either way :)

Since I don't see new version patch posted to include the EFI payload,
I just did this and sent a series @
http://patchwork.ozlabs.org/project/uboot/list/?series=78952

Regards,
Bin

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

end of thread, other threads:[~2018-11-30  4:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-12 12:44 [U-Boot] [PATCH] coreboot: only init usb if necessary Christian Gmeiner
2018-11-12 13:44 ` Bin Meng
2018-11-12 14:39   ` Christian Gmeiner
2018-11-12 14:59     ` Bin Meng
2018-11-30  4:03       ` 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.