On Mon, 2018-04-09 at 15:40 +1000, Nicholas Piggin wrote:
> This delay was in the very first OPAL console commit 6.5 years ago.
> The firmware console has hardened sufficiently to remove it.
>
Reviewed-by: Benjamin Herrenschmidt <b...@kernel.crashing.org>
> Signed-off-by: Nicholas Piggin <npig...@gmail.com>
> ---
> drivers/tty/hvc/hvc_opal.c | 8 +-------
> 1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/drivers/tty/hvc/hvc_opal.c b/drivers/tty/hvc/hvc_opal.c
> index e151cfacf2a7..436b98258e60 100644
> --- a/drivers/tty/hvc/hvc_opal.c
> +++ b/drivers/tty/hvc/hvc_opal.c
> @@ -307,14 +307,8 @@ static int udbg_opal_getc(void)
> int ch;
> for (;;) {
> ch = udbg_opal_getc_poll();
> - if (ch == -1) {
> - /* This shouldn't be needed...but... */
> - volatile unsigned long delay;
> - for (delay=0; delay < 2000000; delay++)
> - ;
> - } else {
> + if (ch != -1)
> return ch;
> - }
> }
> }
>
