[PATCH v2 2/3] hvc/xen: fix error path in xen_hvc_init() to always register frontend driver

[David Woodhouse]

From: David Woodhouse <d...@amazon.co.uk>

The xen_hvc_init() function should always register the frontend driver,
even when there's no primary console — as there may be secondary consoles.
(Qemu can always add secondary consoles, but only the toolstack can add
the primary because it's special.)

Signed-off-by: David Woodhouse <d...@amazon.co.uk>
Reviewed-by: Juergen Gross <jgr...@suse.com>
Cc: sta...@vger.kernel.org
---
 drivers/tty/hvc/hvc_xen.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/hvc/hvc_xen.c b/drivers/tty/hvc/hvc_xen.c
index f24e285b6441..4a768b504263 100644
--- a/drivers/tty/hvc/hvc_xen.c
+++ b/drivers/tty/hvc/hvc_xen.c
@@ -588,7 +588,7 @@ static int __init xen_hvc_init(void)
                ops = &dom0_hvc_ops;
                r = xen_initial_domain_console_init();
                if (r < 0)
-                       return r;
+                       goto register_fe;
                info = vtermno_to_xencons(HVC_COOKIE);
        } else {
                ops = &domU_hvc_ops;
@@ -597,7 +597,7 @@ static int __init xen_hvc_init(void)
                else
                        r = xen_pv_console_init();
                if (r < 0)
-                       return r;
+                       goto register_fe;
 
                info = vtermno_to_xencons(HVC_COOKIE);
                info->irq = bind_evtchn_to_irq_lateeoi(info->evtchn);
@@ -622,6 +622,7 @@ static int __init xen_hvc_init(void)
        }
 
        r = 0;
+ register_fe:
 #ifdef CONFIG_HVC_XEN_FRONTEND
        r = xenbus_register_frontend(&xencons_driver);
 #endif
-- 
2.40.1