Hi Felipe, On ven., mars 25 2016, Felipe Balbi <balbi@xxxxxxxxxx> wrote: > Hi, > > Peter Griffin <peter.griffin@xxxxxxxxxx> writes: >> Otherwise generic-xhci and xhci-platform which have no data get wrongly >> detected as XHCI_PLAT_TYPE_MARVELL_ARMADA by xhci_plat_type_is(). >> >> This fixes a regression in v4.5 for STiH407 family SoC's which use the >> synopsis dwc3 IP, whereby the disable_clk error path gets taken due to >> wrongly being detected as XHCI_PLAT_TYPE_MARVELL_ARMADA and the hcd never >> gets added. >> >> I suspect this will also fix other dwc3 DT platforms such as Exynos, >> although I've only tested on STih410 SoC. >> >> Fixes: 4efb2f694114 ("usb: host: xhci-plat: add struct xhci_plat_priv") >> Cc: stable@xxxxxxxxxxxxxxx >> Cc: gregory.clement@xxxxxxxxxxxxxxxxxx >> Cc: yoshihiro.shimoda.uh@xxxxxxxxxxx >> Signed-off-by: Peter Griffin <peter.griffin@xxxxxxxxxx> >> --- >> drivers/usb/host/xhci-plat.h | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/usb/host/xhci-plat.h b/drivers/usb/host/xhci-plat.h >> index 5a2e2e3..529c3c4 100644 >> --- a/drivers/usb/host/xhci-plat.h >> +++ b/drivers/usb/host/xhci-plat.h >> @@ -14,7 +14,7 @@ >> #include "xhci.h" /* for hcd_to_xhci() */ >> >> enum xhci_plat_type { >> - XHCI_PLAT_TYPE_MARVELL_ARMADA, >> + XHCI_PLAT_TYPE_MARVELL_ARMADA = 1, >> XHCI_PLAT_TYPE_RENESAS_RCAR_GEN2, >> XHCI_PLAT_TYPE_RENESAS_RCAR_GEN3, > > aren't these platforms using device tree ? Why aren't these just > different compatible strings ? According to 4efb2f69411456d35051e9047c15157c9a5ba217 "usb: host: xhci-plat: add struct xhci_plat_priv" : This patch adds struct xhci_plat_priv to simplify the code to match platform specific variables. For now, this patch adds a member "type" in the structure Gregory > > -- > balbi -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html