FIX: arm64: dts: hi3660: enable gpio_hubv2 by default for usb
authorGuodong Xu <guodong.xu@linaro.org>
Tue, 22 Aug 2017 23:51:33 +0000 (07:51 +0800)
committerDouglas RAILLARD <douglas.raillard@arm.com>
Tue, 14 Aug 2018 15:32:08 +0000 (16:32 +0100)
This patch fixes the problem of when using UEFI, USB host doesn't work.

This patch removes gpio_hubv1. gpio_hubv1 is only useful to revision 1
(v1) of hikey960. But revision 1 is experimental, and is not available in
the market any more.

This patch enables gpio_hubv2 by default by deleting status = "disabled".
gpio_hubv2 is used by USB host/client switching. Corresponding driver code
is in drivers/misc/microchip/hub/hub_usb5734.c

Previously, HiSilicon bootloader reads board's id. According boardid
(v1 or v2), it modifies DTB node. Eg. when it reads a v2 board, it changes
node gpio_hubv2 status to: status = "ok".

But when we use UEFI, UEFI doesn't do the boardid detection, and it doesn't
change DTB. So, without this patch, both gpio_hubv2 and gpio_hubv1 are
"disabled", and switching from USB OTG to host always fails.

Signed-off-by: Guodong Xu <guodong.xu@linaro.org>
arch/arm64/boot/dts/hisilicon/hi3660.dtsi

index 3c3af3ff741e0e312b0206349f89b977b1058192..61638e0b845547e618aa9042b50daacd072f6ec9 100644 (file)
                        status = "ok";
                };
 
-               hubv1: gpio_hubv1 {
-                       compatible = "hisilicon,gpio_hubv1";
-                       typc_vbus_int_gpio,typec-gpios = <&gpio25 2 0>;
-                       typc_vbus_enable_val = <0>;
-                       hub_vdd12_en_gpio = <&gpio9 1 0>;
-                       hub_vdd33_en_gpio = <&gpio9 2 0>;
-                       status = "disabled";
-               };
-
                hubv2: gpio_hubv2 {
                        compatible = "hisilicon,gpio_hubv2";
                        typc_vbus_int_gpio,typec-gpios = <&gpio25 2 0>;
                        hub_reset_en_gpio = <&gpio4 4 0>;
                        pinctrl-names = "default";
                        pinctrl-0 = <&usbhub5734_pmx_func>;
-                       status = "disabled";
                };
 
                i2c3: i2c@fdf0c000 {