[PATCH] usb: dwc3: remove extcon dependency

2013-09-10 Thread Heikki Krogerus
It is required by the OMAP glue driver, but it already depends on it. The core driver should not depend on it. This will allow the use of the PCI glue driver again. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/Kconfig | 1 - 1 file changed, 1 deletion

[PATCH] usb: dwc3: pci: add support for BayTrail

2013-09-17 Thread Heikki Krogerus
Add PCI id for Intel BayTrail. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/dwc3-pci.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/usb/dwc3/dwc3-pci.c b/drivers/usb/dwc3/dwc3-pci.c index 9b13812..997ebe4 100644 --- a/drivers/usb/dwc3/dwc3

[PATCH] usb: remove intel_mid_otg.h

2013-10-04 Thread Heikki Krogerus
It's not used anymore. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- include/linux/usb/intel_mid_otg.h | 180 -- 1 file changed, 180 deletions(-) delete mode 100644 include/linux/usb/intel_mid_otg.h diff --git a/include/linux/usb

[PATCH 2/3] usb: phy: generic: clean up the probe function

2013-11-15 Thread Heikki Krogerus
Remove an extra return from the bottom. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/phy/phy-generic.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/usb/phy/phy-generic.c b/drivers/usb/phy/phy-generic.c index 68c5548..2b96311 100644 --- a/drivers

[PATCH 0/3] usb: phy: generic: fix for dwc3-pci

2013-11-15 Thread Heikki Krogerus
Hi, phy-generic broke dwc3-pci after the gpio support was added. The last patch is fixing that issue. The other two are just cleanups. Thanks, Heikki Krogerus (3): usb: phy: generic: fix a compiler warning usb: phy: generic: clean up the probe function usb: dwc3: fix the glue drivers

[PATCH 1/3] usb: phy: generic: fix a compiler warning

2013-11-15 Thread Heikki Krogerus
Just because it annoys me. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/phy/phy-generic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy-generic.c b/drivers/usb/phy/phy-generic.c index fce3a9e..68c5548 100644

Re: [PATCH 1/2] usb: dwc3: core: continue probing if usb phy library returns -ENODEV/-ENXIO

2014-01-28 Thread Heikki Krogerus
Hi, On Mon, Jan 27, 2014 at 10:05:20AM -0600, Felipe Balbi wrote: Why would you need to know if the PHY drivers are needed or not explicitly in your controller driver? because, one way or another, they all do need it. Except for quirky ones like AM437x where a USB3 IP was hardwired into

Re: [PATCH 1/2] usb: dwc3: core: continue probing if usb phy library returns -ENODEV/-ENXIO

2014-01-29 Thread Heikki Krogerus
Hi, On Tue, Jan 28, 2014 at 10:30:36AM -0600, Felipe Balbi wrote: On Tue, Jan 28, 2014 at 05:32:30PM +0200, Heikki Krogerus wrote: On Mon, Jan 27, 2014 at 10:05:20AM -0600, Felipe Balbi wrote: For the controller drivers the PHYs are just a resource like any other. The controller drivers

[PATCH] xhci: Add BayTrail to list of Intel switchable hosts

2013-05-21 Thread Heikki Krogerus
, that contain commit 69e848c2090aebba5698a1620604c7dccb448684 Intel xhci: Support EHCI/xHCI port switching. Signed-off-by: Chew, Chiau Ee chiau.ee.c...@intel.com Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: sta...@vger.kernel.org --- drivers/usb/host/ehci-pci.c |3

Re: [PATCH] xhci: Add BayTrail to list of Intel switchable hosts

2013-05-22 Thread Heikki Krogerus
On Tue, May 21, 2013 at 10:37:55AM -0400, Alan Stern wrote: On Tue, 21 May 2013, Heikki Krogerus wrote: diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c index 595d210..a5708d9 100644 --- a/drivers/usb/host/ehci-pci.c +++ b/drivers/usb/host/ehci-pci.c @@ -322,7

Re: [PATCH] xhci: Add BayTrail to list of Intel switchable hosts

2013-05-22 Thread Heikki Krogerus
On Tue, May 21, 2013 at 04:26:43PM +0400, Sergei Shtylyov wrote: Like the xHCI controller on Intel Panther Point and Lynx Point chipsets, the xHCI controller on Intel BayTrail has also ports that can be switched between the EHCI host controller. s/between/to/ OK. Thanks, -- heikki --

Re: [PATCH] xhci: Add BayTrail to list of Intel switchable hosts

2013-05-28 Thread Heikki Krogerus
Hi Sarah, On Fri, May 24, 2013 at 09:55:25AM -0700, Sarah Sharp wrote: At this point the port switchover quirk is getting unwieldy. I know of at least two more platforms that will need the switchover quirk, and it's silly to keep adding them to the list. Heikki, can you change the code to

Re: [PATCH v2 06/11] USB: mxs-phy: add basic otg support

2012-09-12 Thread Heikki Krogerus
Hi, On Tue, Aug 28, 2012 at 03:03:12PM +0800, Richard Zhao wrote: +static int mxs_phy_set_host(struct usb_otg *otg, struct usb_bus *host) +{ Shouldn't you at least save the host pointer? otg-host = host; + return 0; +} + +static int mxs_phy_set_peripheral(struct usb_otg *otg, +

[PATCH] phy: don't return error from the stubs

2014-04-02 Thread Heikki Krogerus
is not enabled. Reported-by: Chew, Chiau Ee chiau.ee.c...@intel.com Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- include/linux/phy/phy.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h

Re: [PATCH RFC 3/4] xhci: Tune PHY for the DWC3-Exynos host controller

2014-04-16 Thread Heikki Krogerus
Hi, On Tue, Apr 15, 2014 at 06:24:11PM +0530, Vivek Gautam wrote: I had seen your patches in the mailing list, but i don't see any updated version of these patches. Are you planning to work on this above mentioned patch-series any time soon ? I'm sorry, I forgot this completely. I have not

[PATCHv2 5/6] base: platform: name the device already during allocation

2014-06-05 Thread Heikki Krogerus
This allows resources such as GPIOs and clocks, which can be matched based on the device name when requested, to be assigned even when PLATFORM_DEVID_AUTO is used. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org --- drivers/base

[PATCHv2 6/6] usb: dwc3: host: convey the PHYs to xhci

2014-06-05 Thread Heikki Krogerus
On some platforms a PHY may need to be handled also in the host controller driver. Exynos5420 SoC requires some PHY tuning based on the USB speed. This patch delivers dwc3's PHYs to the xhci platform device when it's created. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc

[PATCHv2 4/6] phy: remove the old lookup method

2014-06-05 Thread Heikki Krogerus
The users of the old method are now converted to the new one. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/phy/phy-bcm-kona-usb2.c | 2 +- drivers/phy/phy-core.c | 45 +++-- drivers/phy/phy-exynos-dp-video.c | 2

[PATCHv2 3/6] arm: omap3: twl: use the new lookup method with usb phy

2014-06-05 Thread Heikki Krogerus
Provide complete association for the phy and it's user (musb) with the new phy lookup method. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Tony Lindgren t...@atomide.com --- arch/arm/mach-omap2/twl-common.c | 18 -- 1 file changed, 8 insertions(+), 10

[PATCHv2 0/6] phy: simplified phy lookup

2014-06-05 Thread Heikki Krogerus
in clkdev.c. Vivek needs to handle the phys of dwc3 also in xhci driver on Exynos5420 SoC, so I'm resending these now. Heikki Krogerus (6): phy: safer to_phy() macro phy: improved lookup method arm: omap3: twl: use the new lookup method with usb phy phy: remove the old lookup method base

[PATCHv2 1/6] phy: safer to_phy() macro

2014-06-05 Thread Heikki Krogerus
This makes to_phy() macro work with other variable names besides dev. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- include/linux/phy/phy.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h index 2760744

[PATCHv2 2/6] phy: improved lookup method

2014-06-05 Thread Heikki Krogerus
Removes the need for the phys to be aware of their users even when not using DT. The method is copied from clkdev.c. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- Documentation/phy.txt | 66 --- drivers/phy/phy-core.c | 135

[RFC PATH 3/3] phy: ulpi: add support for NXP ISP170X USB PHY

2013-11-28 Thread Heikki Krogerus
This driver is based on drivers/power/isp1704_power.c. It simply converts the original driver to ulpi driver. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/phy/ulpi/Kconfig| 10 + drivers/phy/ulpi/Makefile | 1 + drivers/phy/ulpi/isp1704_ulpi.c | 446

[RFC PATH 2/3] usb: dwc3: add ULPI interface support

2013-11-28 Thread Heikki Krogerus
Registers ULPI interface with the ULPI abstraction layer if the HSPHY type is ULPI, which will create phy instance for usb2. Depends on Kishon's patch set adding support for generic PHY framework. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/Kconfig | 7

[RFC PATH 0/3] USB PHYs and PCI

2013-11-28 Thread Heikki Krogerus
at this stage, before I make any drivers on top of this thing. The isp1704_ulpi.c I made just as an example for now. Thanks, Heikki Krogerus (3): phy: add USB ULPI abstraction layer usb: dwc3: add ULPI interface support phy: ulpi: add support for NXP ISP170X USB PHY drivers/phy/Kconfig

[RFC PATH 1/3] phy: add USB ULPI abstraction layer

2013-11-28 Thread Heikki Krogerus
layer allows runtime detection. This makes it possible to take advantage of vendor specific functions of the PHYs with product specific drivers without the need for platform or device specific quirks. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/phy/Kconfig

Re: [RFC PATH 1/3] phy: add USB ULPI abstraction layer

2013-12-02 Thread Heikki Krogerus
Hi, On Mon, Dec 02, 2013 at 04:20:51PM +0530, Kishon Vijay Abraham I wrote: Hi, On Thursday 28 November 2013 09:29 PM, Heikki Krogerus wrote: ULPI PHY is an USB2 PHY that is accessed from the USB controller. ULPI PHYs allow discovery based on vendor and product ids which allows binding

Re: [RFC PATH 2/3] usb: dwc3: add ULPI interface support

2013-12-02 Thread Heikki Krogerus
Hi, On Mon, Dec 02, 2013 at 04:24:31PM +0530, Kishon Vijay Abraham I wrote: Hi, On Thursday 28 November 2013 09:29 PM, Heikki Krogerus wrote: snip diff --git a/drivers/usb/dwc3/Makefile b/drivers/usb/dwc3/Makefile index dd17601..8bb82bc 100644 --- a/drivers/usb/dwc3/Makefile +++ b

Re: [RFC PATH 0/3] USB PHYs and PCI

2013-12-02 Thread Heikki Krogerus
Hi, On Mon, Dec 02, 2013 at 04:33:26PM +0530, Kishon Vijay Abraham I wrote: Hi, On Thursday 28 November 2013 09:29 PM, Heikki Krogerus wrote: Hi guys, PCI does not give any information about the PHY but we still need to be able to take advantage of any possible vendor specific

Re: [PATCH 3/3] usb: phy-generic: Add ULPI VBUS support

2013-12-03 Thread Heikki Krogerus
On Mon, Dec 02, 2013 at 03:05:19PM +0800, Chris Ruehl wrote: @@ -154,6 +164,27 @@ int usb_phy_gen_create_phy(struct device *dev, struct usb_phy_gen_xceiv *nop, { int err; + if (nop-ulpi_vbus 0) { + unsigned int flags = 0; + + if (nop-ulpi_vbus 0x1)

Re: [PATCH v2 1/7] usb: dwc3: get usb_phy only if the platform indicates the presence of PHY's

2013-12-03 Thread Heikki Krogerus
Hi, On Thu, Oct 17, 2013 at 09:54:26AM -0500, Felipe Balbi wrote: On Wed, Oct 16, 2013 at 04:27:26PM +0300, Roger Quadros wrote: On 10/16/2013 04:10 PM, Kishon Vijay Abraham I wrote: Do you know if there are users of dwc3 other than exynos5250 and omap5? If not, we should get rid of the

Re: [PATCH v2 2/7] usb: dwc3: adapt dwc3 core to use Generic PHY Framework

2013-12-03 Thread Heikki Krogerus
Hi Kishon, On Wed, Oct 16, 2013 at 01:24:12AM +0530, Kishon Vijay Abraham I wrote: + count = of_property_match_string(node, phy-names, usb2-phy); + if (count = 0 || (pdata pdata-usb2_generic_phy)) { + dwc-usb2_generic_phy = devm_phy_get(dev, usb2-phy); + if

Re: [PATCH 3/3] usb: phy-generic: Add ULPI VBUS support

2013-12-04 Thread Heikki Krogerus
Hi Chris, On Wed, Dec 04, 2013 at 03:16:21PM +0800, Chris Ruehl wrote: On Tuesday, December 03, 2013 04:15 PM, Heikki Krogerus wrote: On Mon, Dec 02, 2013 at 03:05:19PM +0800, Chris Ruehl wrote: @@ -154,6 +164,27 @@ int usb_phy_gen_create_phy(struct device *dev, struct usb_phy_gen_xceiv *nop

Re: [PATCH v3 04/10] usb: dwc3: use quirks to know if a particualr platform doesn't have PHY

2013-12-04 Thread Heikki Krogerus
Hi guys, Kishon, sorry I did not see this v3 set. On Mon, Nov 25, 2013 at 03:31:24PM +0530, Kishon Vijay Abraham I wrote: There can be systems which does not have an external phy, so get phy only if no quirks are added that indicates the PHY is not present. Introduced two quirk flags to

Re: [PATCH v3 04/10] usb: dwc3: use quirks to know if a particualr platform doesn't have PHY

2013-12-04 Thread Heikki Krogerus
Hi, On Thu, Dec 05, 2013 at 12:04:46PM +0530, Kishon Vijay Abraham I wrote: On Wednesday 04 December 2013 08:10 PM, Heikki Krogerus wrote: On Mon, Nov 25, 2013 at 03:31:24PM +0530, Kishon Vijay Abraham I wrote: There can be systems which does not have an external phy, so get phy only

Re: [PATCH v2 1/7] usb: dwc3: get usb_phy only if the platform indicates the presence of PHY's

2013-12-09 Thread Heikki Krogerus
Hi, On Fri, Dec 06, 2013 at 02:15:30PM -0600, Felipe Balbi wrote: On Tue, Dec 03, 2013 at 12:39:50PM +0200, Heikki Krogerus wrote: On Thu, Oct 17, 2013 at 09:54:26AM -0500, Felipe Balbi wrote: On Wed, Oct 16, 2013 at 04:27:26PM +0300, Roger Quadros wrote: On 10/16/2013 04:10 PM, Kishon

Re: [PATCH v3 04/10] usb: dwc3: use quirks to know if a particualr platform doesn't have PHY

2013-12-09 Thread Heikki Krogerus
Hi, On Mon, Dec 09, 2013 at 12:43:37PM +0530, Kishon Vijay Abraham I wrote: On Thursday 05 December 2013 01:28 PM, Heikki Krogerus wrote: On Thu, Dec 05, 2013 at 12:04:46PM +0530, Kishon Vijay Abraham I wrote: On Wednesday 04 December 2013 08:10 PM, Heikki Krogerus wrote: On Mon, Nov 25, 2013

Re: [PATCH RFC 3/4] xhci: Tune PHY for the DWC3-Exynos host controller

2013-12-10 Thread Heikki Krogerus
Hi, On Tue, Dec 10, 2013 at 04:25:25PM +0530, Vivek Gautam wrote: @@ -170,6 +189,15 @@ static int xhci_plat_probe(struct platform_device *pdev) } /* + * The parent of the xhci-plat device may pass in a PHY via + * platform data. If it exists, store it in our struct

Re: [PATCH RFC 1/4] phy: Add provision for tuning phy.

2013-12-10 Thread Heikki Krogerus
Hi, On Tue, Dec 10, 2013 at 04:25:23PM +0530, Vivek Gautam wrote: Some PHY controllers may need to tune PHY post-initialization, so that the PHY consumers can call phy-tuning at appropriate point of time. Signed-off-by: vivek Gautam gautam.vi...@samsung.com --- drivers/phy/phy-core.c |

Re: [PATCH RFC 1/4] phy: Add provision for tuning phy.

2013-12-11 Thread Heikki Krogerus
Hi, On Wed, Dec 11, 2013 at 12:08:04PM +0530, Vivek Gautam wrote: On Tue, Dec 10, 2013 at 7:31 PM, Heikki Krogerus I think setup instead of tune is much more clear and reusable. I think setup will look more like first time setting up the phy, which is rather served by init callback

Re: [PATCH v3 04/10] usb: dwc3: use quirks to know if a particualr platform doesn't have PHY

2013-12-11 Thread Heikki Krogerus
Hi, On Mon, Dec 09, 2013 at 11:26:04AM +0200, Heikki Krogerus wrote: Can you guys explain why is something like this needed? Like with clocks and gpios, the device drivers shouldn't need to care any more if the platform has the phys or not. -ENODEV tells you your platform Shouldn't we

[RESEND PATCH] usb: dwc3: fix the glue drivers using the nop phy

2013-12-18 Thread Heikki Krogerus
. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/dwc3-exynos.c | 1 + drivers/usb/dwc3/dwc3-pci.c| 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/usb/dwc3/dwc3-exynos.c b/drivers/usb/dwc3/dwc3-exynos.c index 8b20c70..28c8ad7 100644 --- a/drivers

Re: [PATCH 1/2] usb: dwc3: core: continue probing if usb phy library returns -ENODEV/-ENXIO

2014-01-27 Thread Heikki Krogerus
Hi Felipe, On Tue, Jan 21, 2014 at 08:47:25AM -0600, Felipe Balbi wrote: On Tue, Jan 21, 2014 at 03:41:38PM +0530, Kishon Vijay Abraham I wrote: Since PHYs for dwc3 is optional (not all SoCs that have DWC3 use PHYs), do not return from probe if the USB PHY library returns -ENODEV as that

Re: [PATCHv2 5/6] base: platform: name the device already during allocation

2014-08-05 Thread Heikki Krogerus
On Mon, Jul 14, 2014 at 07:55:43AM -0700, Greg Kroah-Hartman wrote: diff --git a/drivers/base/platform.c b/drivers/base/platform.c index 9e9227e..e856bc4 100644 --- a/drivers/base/platform.c +++ b/drivers/base/platform.c @@ -177,11 +177,45 @@ struct platform_object { */

[PATCH 4/6] phy: remove the old lookup method

2014-08-21 Thread Heikki Krogerus
The users of the old method are now converted to the new one. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com --- drivers/phy/phy-bcm-kona-usb2.c | 2 +- drivers/phy/phy-core.c | 45

[PATCH 5/6] base: platform: name the device already during allocation

2014-08-21 Thread Heikki Krogerus
requested them. By naming the device already in platform_device_alloc, the resources can be assigned before platform_device_add is called. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org --- drivers/base/platform.c | 69

[PATCH 2/6] phy: improved lookup method

2014-08-21 Thread Heikki Krogerus
Removes the need for the phys to be aware of their users even when not using DT. The method is copied from clkdev.c. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com --- Documentation/phy.txt | 66 --- drivers

[PATCH 6/6] usb: dwc3: host: convey the PHYs to xhci

2014-08-21 Thread Heikki Krogerus
On some platforms a PHY may need to be handled also in the host controller driver. Exynos5420 SoC requires some PHY tuning based on the USB speed. This patch delivers dwc3's PHYs to the xhci platform device when it's created. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested

[PATCH 3/6] arm: omap3: twl: use the new lookup method with usb phy

2014-08-21 Thread Heikki Krogerus
Provide complete association for the phy and it's user (musb) with the new phy lookup method. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- arch/arm/mach-omap2/twl-common.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/arch/arm/mach

[PATCH 1/6] phy: safer to_phy() macro

2014-08-21 Thread Heikki Krogerus
This makes to_phy() macro work with other variable names besides dev. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com --- include/linux/phy/phy.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/phy

Re: [PATCH 4/6] phy: remove the old lookup method

2014-08-25 Thread Heikki Krogerus
On Mon, Aug 25, 2014 at 01:11:34PM +0530, Vivek Gautam wrote: Please squash the attached diff which removes the 'init_data' field from some of the other instances of devm_phy_create() in few other drivers. This should prevent any build errors that i could see with multi_v7_defconfig. OK, I'll

[PATCHv4 4/6] phy: remove the old lookup method

2014-08-26 Thread Heikki Krogerus
The users of the old method are now converted to the new one. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com --- drivers/phy/phy-bcm-kona-usb2.c | 2 +- drivers/phy/phy-berlin-sata.c| 2 +- drivers/phy/phy-core.c

Re: [PATCH 3/6] arm: omap3: twl: use the new lookup method with usb phy

2014-09-12 Thread Heikki Krogerus
On Thu, Sep 11, 2014 at 08:56:15PM +0530, Kishon Vijay Abraham I wrote: +static struct phy_lookup twl4030_usb_lookup = { + .phy_name = phy-twl4030_usb.0, + .dev_id = musb-hdrc.0, + .con_id = usb, }; Can use PHY_LOOKUP no? I'll fix this. Thanks, --

Re: [PATCH 2/6] phy: improved lookup method

2014-09-12 Thread Heikki Krogerus
On Thu, Sep 11, 2014 at 09:03:06PM +0530, Kishon Vijay Abraham I wrote: +static struct phy *phy_find(struct device *dev, const char *con_id) +{ + const char *dev_id = dev ? dev_name(dev) : NULL; + int match, best_found = 0, best_possible = 0; + struct phy *phy = ERR_PTR(-ENODEV);

Re: [PATCH 6/6] usb: dwc3: host: convey the PHYs to xhci

2014-09-15 Thread Heikki Krogerus
On Fri, Sep 12, 2014 at 07:41:56PM +0530, Kishon Vijay Abraham I wrote: I don't think create lookup should be in host init. If it's dt boot, the binding should be in dt data or for other boot modes the bindig should be done in the board file. This just seems hacky to me. So are you

Re: [PATCH 2/6] phy: improved lookup method

2014-09-15 Thread Heikki Krogerus
On Fri, Sep 12, 2014 at 08:16:01PM +0530, Kishon Vijay Abraham I wrote: Assume you have 2 phys in your system.. static struct phy_lookup usb_lookup = { .phy_name = phy-usb.0, .dev_id = usb.0, .con_id = usb, }; static struct phy_lookup sata_lookup =

Re: [PATCH 6/6] usb: dwc3: host: convey the PHYs to xhci

2014-09-16 Thread Heikki Krogerus
On Tue, Sep 16, 2014 at 12:07:00PM +0530, Kishon Vijay Abraham I wrote: Hi, On Monday 15 September 2014 05:36 PM, Heikki Krogerus wrote: On Fri, Sep 12, 2014 at 07:41:56PM +0530, Kishon Vijay Abraham I wrote: I don't think create lookup should be in host init. If it's dt boot

Re: [PATCH 2/6] phy: improved lookup method

2014-09-18 Thread Heikki Krogerus
On Mon, Sep 15, 2014 at 03:35:08PM +0300, Heikki Krogerus wrote: On Fri, Sep 12, 2014 at 08:16:01PM +0530, Kishon Vijay Abraham I wrote: Assume you have 2 phys in your system.. static struct phy_lookup usb_lookup = { .phy_name = phy-usb.0, .dev_id = usb.0

Re: [PATCH 2/6] phy: improved lookup method

2014-09-23 Thread Heikki Krogerus
On Mon, Sep 22, 2014 at 05:07:55PM +0530, Kishon Vijay Abraham I wrote: On Thursday 18 September 2014 03:55 PM, Heikki Krogerus wrote: On Mon, Sep 15, 2014 at 03:35:08PM +0300, Heikki Krogerus wrote: On Fri, Sep 12, 2014 at 08:16:01PM +0530, Kishon Vijay Abraham I wrote: Assume you have 2

Re: [PATCH 2/6] phy: improved lookup method

2014-09-23 Thread Heikki Krogerus
On Tue, Sep 23, 2014 at 04:33:09PM +0530, Kishon Vijay Abraham I wrote: Hi, On Tuesday 23 September 2014 04:23 PM, Heikki Krogerus wrote: On Mon, Sep 22, 2014 at 05:07:55PM +0530, Kishon Vijay Abraham I wrote: On Thursday 18 September 2014 03:55 PM, Heikki Krogerus wrote: On Mon, Sep 15

[resend PATCH] usb: dwc3: pci: Add PCI ID for Intel Braswell

2014-09-24 Thread Heikki Krogerus
From: Alan Cox a...@linux.intel.com The device controller is the same but it has different PCI ID. Add this new ID to the driver's list of supported IDs. Signed-off-by: Alan Cox a...@linux.intel.com Signed-off-by: Mika Westerberg mika.westerb...@linux.intel.com Signed-off-by: Heikki Krogerus

[resend PATCH 2/3] usb: dwc3: core: only setting the dma_mask when needed

2014-09-24 Thread Heikki Krogerus
If the probe drivers have already set the dma_mask, not replacing the value. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/core.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c

[resend PATCH 1/3] ACPI / platform: provide default DMA mask

2014-09-24 Thread Heikki Krogerus
Most devices are configured for 32-bit DMA addresses. Setting the mask to 32-bit here removes the need for the drivers to do it separately. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Rafael J. Wysocki r...@rjwysocki.net --- drivers/acpi/acpi_platform.c | 2 ++ 1 file

[resend PATCH 0/3] usb: dwc3: ACPI support

2014-09-24 Thread Heikki Krogerus
DMA mask for all the ACPI platform devices. Heikki Krogerus (3): ACPI / platform: provide default DMA mask usb: dwc3: core: only setting the dma_mask when needed usb: dwc3: add ACPI support drivers/acpi/acpi_platform.c | 2 ++ drivers/usb/dwc3/core.c | 18 +++--- 2 files

[resend PATCH 3/3] usb: dwc3: add ACPI support

2014-09-24 Thread Heikki Krogerus
Adds ACPI ID used on newer Intel SoCs. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/core.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index d08cac5..c2cf2d8 100644 --- a/drivers/usb/dwc3

Re: [PATCH 2/6] phy: improved lookup method

2014-09-25 Thread Heikki Krogerus
Assume you have 2 phys in your system.. static struct phy_lookup usb_lookup = { .phy_name = phy-usb.0, .dev_id = usb.0, .con_id = usb, }; static struct phy_lookup sata_lookup = { .phy_name = sata-usb.1, .dev_id =

[PATCHv2 3/3] usb: dwc3: add ACPI support

2014-09-25 Thread Heikki Krogerus
Adding ACPI ID used on newer Intel SoCs. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/core.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index d08cac5..88e29f5 100644 --- a/drivers/usb

Re: [RFC PATCH 3/4] usb: dwc3: add quirk to be compatible for AMD NL

2014-09-29 Thread Heikki Krogerus
A question, the dwc3 controller is the PCI-E device in my platform, but the class code of PCI header is 0x0c0330, the same with xHC. That's because it need to meet the windows enviroment. The dwc3 controller acted as only host mode to bind with windows xhci driver. But on linux,

[PATCHv4 4/6] phy: remove the old lookup method

2014-10-17 Thread Heikki Krogerus
The users of the old method are now converted to the new one. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com --- drivers/phy/phy-bcm-kona-usb2.c | 2 +- drivers/phy/phy-berlin-sata.c| 2 +- drivers/phy/phy-core.c

[PATCHv4 6/6] usb: dwc3: host: convey the PHYs to xhci

2014-10-17 Thread Heikki Krogerus
On some platforms a PHY may need to be handled also in the host controller driver. Exynos5420 SoC requires some PHY tuning based on the USB speed. This patch delivers dwc3's PHYs to the xhci platform device when it's created. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested

[PATCHv4 0/6] phy: simplified phy lookup

2014-10-17 Thread Heikki Krogerus
device is always the actual physical device. - Using PHY_LOOKUP macro in twl-common.c as suggested by Kishon. Changes since v2: - Calling ida_simple_remove in release function as pointed out by Greg Heikki Krogerus (6): phy: safer to_phy() macro phy: improved lookup method arm: omap3: twl

[PATCHv4 5/6] base: platform: name the device already during allocation

2014-10-17 Thread Heikki Krogerus
requested them. By naming the device already in platform_device_alloc, the resources can be assigned before platform_device_add is called. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org --- drivers/base/platform.c | 69

[PATCHv4 1/6] phy: safer to_phy() macro

2014-10-17 Thread Heikki Krogerus
This makes to_phy() macro work with other variable names besides dev. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com --- include/linux/phy/phy.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/phy

[PATCHv4 2/6] phy: improved lookup method

2014-10-17 Thread Heikki Krogerus
Removes the need for the phys to be aware of their users even when not using DT. The method is copied from clkdev.c. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com --- Documentation/phy.txt | 66 --- drivers

[PATCHv4 3/6] arm: omap3: twl: use the new lookup method with usb phy

2014-10-17 Thread Heikki Krogerus
Provide complete association for the phy and it's user (musb) with the new phy lookup method. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- arch/arm/mach-omap2/twl-common.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/arch/arm/mach

Re: [PATCHv4 2/6] phy: improved lookup method

2014-11-14 Thread Heikki Krogerus
Hi Vivek, On Thu, Nov 13, 2014 at 07:03:01PM +0530, Vivek Gautam wrote: Hi Heikki, Kishon, How about adding the change in attached patch [1] on top of this patch. Just introduced the phy pointer in phy_lookup structure, and modified phy_find() accordingly. [1] Attachment:

Re: [PATCHv4 2/6] phy: improved lookup method

2014-11-17 Thread Heikki Krogerus
On Thu, Nov 13, 2014 at 07:03:01PM +0530, Vivek Gautam wrote: How about adding the change in attached patch [1] on top of this patch. Just introduced the phy pointer in phy_lookup structure, and modified phy_find() accordingly. I would be fine if we used the phy pointer to match, but if we

Re: [PATCHv4 2/6] phy: improved lookup method

2014-11-18 Thread Heikki Krogerus
On Tue, Nov 18, 2014 at 10:49:18AM +0530, Kishon Vijay Abraham I wrote: On Monday 17 November 2014 09:10 PM, Heikki Krogerus wrote: On Thu, Nov 13, 2014 at 07:03:01PM +0530, Vivek Gautam wrote: How about adding the change in attached patch [1] on top of this patch. Just introduced the phy

[PATCHv5 5/7] phy: remove the old lookup method

2014-11-19 Thread Heikki Krogerus
The users of the old method are now converted to the new one. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/phy/phy-bcm-kona-usb2.c | 2 +- drivers/phy/phy-berlin-sata.c| 2 +- drivers/phy/phy-core.c | 49

[PATCHv5 6/7] base: platform: name the device already during allocation

2014-11-19 Thread Heikki Krogerus
/dwc3/host.c) to pass the phy's to xhci. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Greg Kroah-Hartman gre...@linuxfoundation.org --- drivers/base/platform.c | 69 + 1 file changed, 41 insertions(+), 28 deletions(-) diff --git

[PATCHv5 7/7] usb: dwc3: host: convey the PHYs to xhci

2014-11-19 Thread Heikki Krogerus
On some platforms a PHY may need to be handled also in the host controller driver. Exynos5420 SoC requires some PHY tuning based on the USB speed. This patch delivers dwc3's PHYs to the xhci platform device when it's created. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested

[PATCHv5 2/7] phy: improved lookup method

2014-11-19 Thread Heikki Krogerus
Separates registration of the phy and the lookup. The method is copied from clkdev.c, Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- Documentation/phy.txt | 60 ++- drivers/phy/phy-core.c | 84

[PATCHv5 3/7] phy: twl4030: use the new lookup method

2014-11-19 Thread Heikki Krogerus
Creates the lookup separately. Hard coding the consumer as it can't be anything else except musb. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/phy/phy-twl4030-usb.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/phy/phy-twl4030

[PATCHv5 4/7] arm: omap3: twl: remove usb phy init data

2014-11-19 Thread Heikki Krogerus
The driver does no use it any more. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- arch/arm/mach-omap2/twl-common.c | 12 +--- include/linux/i2c/twl.h | 2 -- 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/arch/arm/mach-omap2/twl-common.c b

[PATCHv5 0/7] phy: simplified phy lookup

2014-11-19 Thread Heikki Krogerus
of the phy itself. The parent device is always the actual physical device. - Using PHY_LOOKUP macro in twl-common.c as suggested by Kishon. Changes since v2: - Calling ida_simple_remove in release function as pointed out by Greg Heikki Krogerus (7): phy: safer to_phy() macro phy: improved lookup

[PATCHv5 1/7] phy: safer to_phy() macro

2014-11-19 Thread Heikki Krogerus
This makes to_phy() macro work with other variable names besides dev. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Tested-by: Vivek Gautam gautam.vi...@samsung.com Acked-by: Felipe Balbi ba...@ti.com --- include/linux/phy/phy.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[PATCH] usb: dwc3: pci: add support for Intel Sunrise Point PCH

2014-12-18 Thread Heikki Krogerus
Add PCI IDs for Intel Sunrise Point PCH. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/dwc3/dwc3-pci.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/usb/dwc3/dwc3-pci.c b/drivers/usb/dwc3/dwc3-pci.c index 7c4faf7..b642a2f 100644 --- a/drivers

Re: [PATCH 8/8] phy: add driver for TI TUSB1210 ULPI PHY

2015-01-26 Thread Heikki Krogerus
Hi David, On Sat, Jan 24, 2015 at 03:58:11PM -0800, David Cohen wrote: +static int tusb1210_power_on(struct phy *phy) +{ + struct tusb1210 *tusb = phy_get_drvdata(phy); + + gpiod_set_value_cansleep(tusb-gpio_reset, 1); + gpiod_set_value_cansleep(tusb-gpio_cs, 1); + + /*

Re: [PATCH 8/8] phy: add driver for TI TUSB1210 ULPI PHY

2015-02-02 Thread Heikki Krogerus
Hi David, What exactly are we breaking here? The USB on BYT-CR does not work yet with the mainline kernel, or does it? To enable it, I already suggested the BYT quirk (attached again). It used to work with mainline with minor restrictions. It stopped working (and I failed

Re: [PATCH 8/8] phy: add driver for TI TUSB1210 ULPI PHY

2015-02-02 Thread Heikki Krogerus
You can't really compare a bus like i2c, which can't enumerate devices natively, to ULPI which can. why not ? The BIOS might not need to use the PHY (or USB) at all, it can very well decide to never turn it on, right ? If ULPI was seen as a bus, then no. BIOS would have

Re: [PATCH 8/8] phy: add driver for TI TUSB1210 ULPI PHY

2015-02-03 Thread Heikki Krogerus
Hi David, Felipe, why would you have dwc3 mess around with the PHY's gpios ? Doesn't look very good. ..but unfortunately we can't use the bus without it :(. We depend on being able to read the vendor and product id's in the bus driver. Doesn't the ugly platform device case

[PATCH 2/3] usb: dwc3: add ULPI interface support

2015-01-20 Thread Heikki Krogerus
Registers ULPI interface with the ULPI bus if HSPHY type is ULPI. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Felipe Balbi ba...@ti.com --- drivers/usb/dwc3/Kconfig | 7 drivers/usb/dwc3/Makefile | 4 ++ drivers/usb/dwc3/core.c | 9 +++- drivers/usb/dwc3

[PATCH] usb: phy: never defer probe in non-OF case

2015-01-16 Thread Heikki Krogerus
into deferred probe when there are no phys. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com --- drivers/usb/phy/phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c index ccfdfb2..2f9735b 100644 --- a/drivers/usb/phy

Re: [PATCH 3/3] phy: ulpi: add driver for TI TUSB1210

2015-01-21 Thread Heikki Krogerus
Hi, On Wed, Jan 21, 2015 at 11:17:49AM +0200, Heikki Krogerus wrote: On Tue, Jan 20, 2015 at 09:45:39AM -0600, Felipe Balbi wrote: diff --git a/drivers/phy/ulpi/tusb1210.c b/drivers/phy/ulpi/tusb1210.c new file mode 100644 index 000..ac77f98 --- /dev/null +++ b/drivers/phy

Re: [PATCH 2/3] usb: dwc3: add ULPI interface support

2015-01-21 Thread Heikki Krogerus
On Tue, Jan 20, 2015 at 09:23:37AM -0600, Felipe Balbi wrote: Hi, On Tue, Jan 20, 2015 at 11:18:21AM +0200, Heikki Krogerus wrote: Registers ULPI interface with the ULPI bus if HSPHY type is ULPI. Signed-off-by: Heikki Krogerus heikki.kroge...@linux.intel.com Cc: Felipe Balbi ba

Re: [PATCH 3/3] phy: ulpi: add driver for TI TUSB1210

2015-01-21 Thread Heikki Krogerus
On Tue, Jan 20, 2015 at 09:45:39AM -0600, Felipe Balbi wrote: Hi, On Tue, Jan 20, 2015 at 11:18:22AM +0200, Heikki Krogerus wrote: TUSB1210 ULPI PHY has vendor specific register for eye diagram tuning. On some platforms the system firmware has set optimized value to it. In order

Re: [PATCH 6/8] usb: dwc3: add ULPI interface support

2015-02-12 Thread Heikki Krogerus
diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index a8c9062..66cbf38 100644 --- a/drivers/usb/dwc3/core.c +++ b/drivers/usb/dwc3/core.c @@ -879,6 +879,10 @@ static int dwc3_probe(struct platform_device *pdev) platform_set_drvdata(pdev, dwc);

Re: [PATCH 1/8] usb: add bus type for USB ULPI

2015-02-13 Thread Heikki Krogerus
On Thu, Feb 12, 2015 at 05:44:20PM -0800, Stephen Boyd wrote: On 01/23/15 07:12, Heikki Krogerus wrote: diff --git a/scripts/mod/file2alias.c b/scripts/mod/file2alias.c index e614ef6..753cb08 100644 --- a/scripts/mod/file2alias.c +++ b/scripts/mod/file2alias.c @@ -1176,6 +1176,19

Re: [PATCH 8/8] phy: add driver for TI TUSB1210 ULPI PHY

2015-01-27 Thread Heikki Krogerus
On Mon, Jan 26, 2015 at 11:23:37AM -0800, David Cohen wrote: On Mon, Jan 26, 2015 at 02:55:03PM +0200, Heikki Krogerus wrote: On Sat, Jan 24, 2015 at 03:58:11PM -0800, David Cohen wrote: +static int tusb1210_power_on(struct phy *phy) +{ + struct tusb1210 *tusb

Re: [PATCH 6/8] usb: dwc3: add ULPI interface support

2015-01-27 Thread Heikki Krogerus
look at your patch again. In case DWC3_ULPI isn't enabled, this file won't be linked at all. You're using stubs, so taht's fine. In case it _is_ enabled, you're breaking out early if you can't register ulpi interface, meaning you're exitting probe() (which, in fact, seems wrong as I want to

  1   2   3   4   5   6   7   8   9   10   >