phy: usb: Fix missing elements in BCM4908 USB init array
authorSam Edwards <cfsworks@gmail.com>
Fri, 4 Oct 2024 03:41:30 +0000 (20:41 -0700)
committerVinod Koul <vkoul@kernel.org>
Mon, 7 Oct 2024 06:13:46 +0000 (11:43 +0530)
The Broadcom USB PHY driver contains a lookup table
(`reg_bits_map_tables`) to resolve register bitmaps unique to certain
versions of the USB PHY as found in various Broadcom chip families. A
recent commit (see 'fixes' tag) introduced two new elements to each chip
family in this table -- except for one: BCM4908. This resulted in the
xHCI controller not being initialized correctly, causing a panic on
boot.

The next patch will update this table to use designated initializers in
order to prevent this from happening again. For now, just add back the
missing array elements to resolve the regression.

Fixes: 4536fe9640b6 ("phy: usb: suppress OC condition for 7439b2")
Signed-off-by: Sam Edwards <CFSworks@gmail.com>
Reviewed-by: Justin Chen <justin.chen@broadcom.com>
Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
Link: https://lore.kernel.org/r/20241004034131.1363813-2-CFSworks@gmail.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/broadcom/phy-brcm-usb-init.c

index 39536b6..5ebb3a6 100644 (file)
@@ -220,6 +220,8 @@ usb_reg_bits_map_table[BRCM_FAMILY_COUNT][USB_CTRL_SELECTOR_COUNT] = {
                0, /* USB_CTRL_SETUP_SCB2_EN_MASK */
                0, /* USB_CTRL_SETUP_SS_EHCI64BIT_EN_MASK */
                0, /* USB_CTRL_SETUP_STRAP_IPP_SEL_MASK */
+               0, /* USB_CTRL_SETUP_OC3_DISABLE_PORT0_MASK */
+               0, /* USB_CTRL_SETUP_OC3_DISABLE_PORT1_MASK */
                0, /* USB_CTRL_SETUP_OC3_DISABLE_MASK */
                0, /* USB_CTRL_PLL_CTL_PLL_IDDQ_PWRDN_MASK */
                0, /* USB_CTRL_USB_PM_BDC_SOFT_RESETB_MASK */