123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136 |
- From e8624859dde2ad07633dac7ec86629a516411ea1 Mon Sep 17 00:00:00 2001
- From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
- Date: Wed, 21 Sep 2016 18:01:43 +0200
- Subject: [PATCH] USB: bcma: drop Northstar PHY 2.0 initialization code
- MIME-Version: 1.0
- Content-Type: text/plain; charset=UTF-8
- Content-Transfer-Encoding: 8bit
- This driver should initialize controller only, PHY initialization should
- be handled by separated PHY driver. We already have phy-bcm-ns-usb2 in
- place so let it makes its duty.
- Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
- Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
- ---
- drivers/usb/host/bcma-hcd.c | 80 ++++++++++++++-------------------------------
- 1 file changed, 25 insertions(+), 55 deletions(-)
- --- a/drivers/usb/host/bcma-hcd.c
- +++ b/drivers/usb/host/bcma-hcd.c
- @@ -239,44 +239,10 @@ static int bcma_hcd_usb20_old_arm_init(s
- return 0;
- }
-
- -static void bcma_hcd_init_chip_arm_phy(struct bcma_device *dev)
- -{
- - struct bcma_device *arm_core;
- - void __iomem *dmu;
- -
- - arm_core = bcma_find_core(dev->bus, BCMA_CORE_ARMCA9);
- - if (!arm_core) {
- - dev_err(&dev->dev, "can not find ARM Cortex A9 ihost core\n");
- - return;
- - }
- -
- - dmu = ioremap_nocache(arm_core->addr_s[0], 0x1000);
- - if (!dmu) {
- - dev_err(&dev->dev, "can not map ARM Cortex A9 ihost core\n");
- - return;
- - }
- -
- - /* Unlock DMU PLL settings */
- - iowrite32(0x0000ea68, dmu + 0x180);
- -
- - /* Write USB 2.0 PLL control setting */
- - iowrite32(0x00dd10c3, dmu + 0x164);
- -
- - /* Lock DMU PLL settings */
- - iowrite32(0x00000000, dmu + 0x180);
- -
- - iounmap(dmu);
- -}
- -
- -static void bcma_hcd_init_chip_arm_hc(struct bcma_device *dev)
- +static void bcma_hcd_usb20_ns_init_hc(struct bcma_device *dev)
- {
- u32 val;
-
- - /*
- - * Delay after PHY initialized to ensure HC is ready to be configured
- - */
- - usleep_range(1000, 2000);
- -
- /* Set packet buffer OUT threshold */
- val = bcma_read32(dev, 0x94);
- val &= 0xffff;
- @@ -287,20 +253,33 @@ static void bcma_hcd_init_chip_arm_hc(st
- val = bcma_read32(dev, 0x9c);
- val |= 1;
- bcma_write32(dev, 0x9c, val);
- +
- + /*
- + * Broadcom initializes PHY and then waits to ensure HC is ready to be
- + * configured. In our case the order is reversed. We just initialized
- + * controller and we let HCD initialize PHY, so let's wait (sleep) now.
- + */
- + usleep_range(1000, 2000);
- }
-
- -static void bcma_hcd_init_chip_arm(struct bcma_device *dev)
- +/**
- + * bcma_hcd_usb20_ns_init - Initialize Northstar USB 2.0 controller
- + */
- +static int bcma_hcd_usb20_ns_init(struct bcma_hcd_device *bcma_hcd)
- {
- - bcma_core_enable(dev, 0);
- + struct bcma_device *core = bcma_hcd->core;
- + struct bcma_chipinfo *ci = &core->bus->chipinfo;
- + struct device *dev = &core->dev;
- +
- + bcma_core_enable(core, 0);
-
- - if (dev->bus->chipinfo.id == BCMA_CHIP_ID_BCM4707 ||
- - dev->bus->chipinfo.id == BCMA_CHIP_ID_BCM53018) {
- - if (dev->bus->chipinfo.pkg == BCMA_PKG_ID_BCM4707 ||
- - dev->bus->chipinfo.pkg == BCMA_PKG_ID_BCM4708)
- - bcma_hcd_init_chip_arm_phy(dev);
- + if (ci->id == BCMA_CHIP_ID_BCM4707 ||
- + ci->id == BCMA_CHIP_ID_BCM53018)
- + bcma_hcd_usb20_ns_init_hc(core);
-
- - bcma_hcd_init_chip_arm_hc(dev);
- - }
- + of_platform_default_populate(dev->of_node, NULL, dev);
- +
- + return 0;
- }
-
- static void bcma_hci_platform_power_gpio(struct bcma_device *dev, bool val)
- @@ -373,16 +352,7 @@ static int bcma_hcd_usb20_init(struct bc
- if (dma_set_mask_and_coherent(dev->dma_dev, DMA_BIT_MASK(32)))
- return -EOPNOTSUPP;
-
- - switch (dev->id.id) {
- - case BCMA_CORE_NS_USB20:
- - bcma_hcd_init_chip_arm(dev);
- - break;
- - case BCMA_CORE_USB20_HOST:
- - bcma_hcd_init_chip_mips(dev);
- - break;
- - default:
- - return -ENODEV;
- - }
- + bcma_hcd_init_chip_mips(dev);
-
- /* In AI chips EHCI is addrspace 0, OHCI is 1 */
- ohci_addr = dev->addr_s[0];
- @@ -451,7 +421,7 @@ static int bcma_hcd_probe(struct bcma_de
- err = -ENOTSUPP;
- break;
- case BCMA_CORE_NS_USB20:
- - err = bcma_hcd_usb20_init(usb_dev);
- + err = bcma_hcd_usb20_ns_init(usb_dev);
- break;
- case BCMA_CORE_NS_USB30:
- err = bcma_hcd_usb30_init(usb_dev);
|