On 10/24/2015 12:40 AM, Florian Fainelli wrote: > On 23/10/15 14:37, Hauke Mehrtens wrote: >> From: Rafał Miłecki <zajec5@xxxxxxxxx> >> >> Signed-off-by: Rafał Miłecki <zajec5@xxxxxxxxx> >> Signed-off-by: Hauke Mehrtens <hauke@xxxxxxxxxx> >> --- > > [snip] > >> + >> + switch (chipinfo->id) { >> + case BCMA_CHIP_ID_BCM4707: > > All the code above cries for being a proper USB PHY driver, is there a > particular reason why it needs to be living in bcma-hcd.c other than > being convenient? No, there isn't. I will try to create a usb phy driver and connect it with bcma-hcd.c. Is there already a phy driver which does this stuff? I think this PHY is shared with the 3rd PCIe controller on some SoCs. > >> + if (chipinfo->rev == 4) { >> + /* For NS-B0, USB3 PLL Block */ >> + bcma_wait_reg(bus, ccb->mii + 0x000, 0x0100, 0x0000, 1000); >> + iowrite32(0x587e8000, ccb->mii + 0x004); >> + >> + /* Clear ana_pllSeqStart */ >> + bcma_wait_reg(bus, ccb->mii + 0x000, 0x0100, 0x0000, 1000); >> + iowrite32(0x58061000, ccb->mii + 0x004); > > > --- > Florian > -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html