[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] serial: Poll USB devices if usbX serial port is missing
From: |
Andrei Borzenkov |
Subject: |
Re: [PATCH 2/2] serial: Poll USB devices if usbX serial port is missing |
Date: |
Sat, 12 Nov 2016 12:18:42 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
10.11.2016 16:45, Joonas Lahtinen пишет:
> If usbX serial port is missing, try to poll USB devices. This makes them
> appear at least on Intel NUC devices DN2820FYKH and DCCP847DYE.
>
> This fixes cases where you try to configure usbX serial at the beginning
> of a script, like:
>
> serial usb0 --speed=115200 --word=8 --parity=none --stop=1
> terminal_output --append serial_usb0
> terminal_input --append serial_usb0
>
> Without this patch the above would fail with:
>
> serial port `usb0' isn't found
>
> Strangely, adding 'usb' command before the serial initialization
> made it work, and even moving the terminal_output and input command
> to be before. This is due to the terminal_output command detecting
> "serial_usb" and running USB polling if the output is missing. 'usb'
> command from usbtest module always polls USB devices.
>
> As it is the logical thing to configure the serial port before adding
> it as an output or input, this patch makes things works as expected.
> Most online resources mentioning GRUB and USB serial adapters also do
> mention the above order, so there must be quite many struggling with
> this problem on platforms where USB needs to be polled in order for
> the detection to happen.
>
> Signed-off-by: Joonas Lahtinen <address@hidden>
> ---
> grub-core/term/serial.c | 21 ++++++++++++++++++++-
> 1 file changed, 20 insertions(+), 1 deletion(-)
>
> diff --git a/grub-core/term/serial.c b/grub-core/term/serial.c
> index db80b3b..1f856a2 100644
> --- a/grub-core/term/serial.c
> +++ b/grub-core/term/serial.c
> @@ -191,6 +191,7 @@ grub_cmd_serial (grub_extcmd_context_t ctxt, int argc,
> char **args)
> struct grub_serial_port *port;
> struct grub_serial_config config;
> grub_err_t err;
> + int again;
>
> if (state[OPTION_UNIT].set)
> {
> @@ -212,7 +213,25 @@ grub_cmd_serial (grub_extcmd_context_t ctxt, int argc,
> char **args)
> if (!name)
> name = "com0";
>
> - port = grub_serial_find (name);
> + again = 0;
> + while(1)
> + {
> + port = grub_serial_find (name);
> +
> + if (port || again)
> + break;
> +
> + if (grub_memcmp (name, "usb", sizeof ("usb") - 1) == 0)
> + {
> + grub_usb_poll_devices(1);
This creates hard dependency of serial module to usb. Please use
grub_term_poll_usb as terminal module does.
> + again = 1;
> + }
> + else
> + {
> + break;
> + }
> + }
> +
> if (!port)
> return grub_error (GRUB_ERR_BAD_ARGUMENT,
> N_("serial port `%s' isn't found"),
>
Re: [PATCH 1/2] .gitignore: Add grub-core/build-grub-module-verifier, Andrei Borzenkov, 2016/11/12