lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 3 Mar 2014 19:57:33 +0530
From:	Kishon Vijay Abraham I <kishon@...com>
To:	Kamil Debski <k.debski@...sung.com>,
	<linux-kernel@...r.kernel.org>,
	<linux-samsung-soc@...r.kernel.org>, <linux-usb@...r.kernel.org>,
	<devicetree@...r.kernel.org>
CC:	<kyungmin.park@...sung.com>, <t.figa@...sung.com>,
	<s.nawrocki@...sung.com>, <m.szyprowski@...sung.com>,
	<gautam.vivek@...sung.com>, <mat.krawczuk@...il.com>,
	<yulgon.kim@...sung.com>, <p.paneri@...sung.com>,
	<av.tikhomirov@...sung.com>, <jg1.han@...sung.com>,
	<galak@...eaurora.org>, <matt.porter@...aro.org>,
	<tjakobi@...h.uni-bielefeld.de>, <stern@...land.harvard.edu>
Subject: Re: [PATCH v6 5/8] phy: Add new Exynos USB PHY driver

Hi,

On Wednesday 29 January 2014 10:59 PM, Kamil Debski wrote:
> Add a new driver for the Exynos USB PHY. The new driver uses the generic
> PHY framework. The driver includes support for the Exynos 4x10 and 4x12
> SoC families.

Can the PHY part of this series be merged independently of the 
controller part?

Thanks
Kishon
>
> Signed-off-by: Kamil Debski <k.debski@...sung.com>
> ---
>   .../devicetree/bindings/phy/samsung-phy.txt        |   55 ++++
>   Documentation/phy/samsung-usb2.txt                 |  135 ++++++++
>   drivers/phy/Kconfig                                |   29 ++
>   drivers/phy/Makefile                               |    3 +
>   drivers/phy/phy-exynos4210-usb2.c                  |  257 ++++++++++++++++
>   drivers/phy/phy-exynos4x12-usb2.c                  |  323 ++++++++++++++++++++
>   drivers/phy/phy-samsung-usb2.c                     |  227 ++++++++++++++
>   drivers/phy/phy-samsung-usb2.h                     |   67 ++++
>   8 files changed, 1096 insertions(+)
>   create mode 100644 Documentation/phy/samsung-usb2.txt
>   create mode 100644 drivers/phy/phy-exynos4210-usb2.c
>   create mode 100644 drivers/phy/phy-exynos4x12-usb2.c
>   create mode 100644 drivers/phy/phy-samsung-usb2.c
>   create mode 100644 drivers/phy/phy-samsung-usb2.h
>
> diff --git a/Documentation/devicetree/bindings/phy/samsung-phy.txt b/Documentation/devicetree/bindings/phy/samsung-phy.txt
> index c0fccaa..6668c41 100644
> --- a/Documentation/devicetree/bindings/phy/samsung-phy.txt
> +++ b/Documentation/devicetree/bindings/phy/samsung-phy.txt
> @@ -20,3 +20,58 @@ Required properties:
>   - compatible : should be "samsung,exynos5250-dp-video-phy";
>   - reg : offset and length of the Display Port PHY register set;
>   - #phy-cells : from the generic PHY bindings, must be 0;
> +
> +Samsung S5P/EXYNOS SoC series USB PHY
> +-------------------------------------------------
> +
> +Required properties:
> +- compatible : should be one of the listed compatibles:
> +	- "samsung,exynos4210-usb2-phy"
> +	- "samsung,exynos4x12-usb2-phy"
> +- reg : a list of registers used by phy driver
> +	- first and obligatory is the location of phy modules registers
> +- samsung,sysreg-phandle - handle to syscon used to control the system registers
> +- samsung,pmureg-phandle - handle to syscon used to control PMU registers
> +- #phy-cells : from the generic phy bindings, must be 1;
> +- clocks and clock-names:
> +	- the "phy" clocks is required by the phy module
> +	- next for each of the phys a clock has to be assigned, this clock
> +	  will be used to determine clocking frequency for the phys
> +	  (the labels are specified in the paragraph below)
> +
> +The first phandle argument in the PHY specifier identifies the PHY, its
> +meaning is compatible dependent. For the currently supported SoCs (Exynos 4210
> +and Exynos 4212) it is as follows:
> +  0 - USB device ("device"),
> +  1 - USB host ("host"),
> +  2 - HSIC0 ("hsic0"),
> +  3 - HSIC1 ("hsic1"),
> +
> +Exynos 4210 and Exynos 4212 use mode switching and require that mode switch
> +register is supplied.
> +
> +Example:
> +
> +For Exynos 4412 (compatible with Exynos 4212):
> +
> +usbphy: phy@...b0000 {
> +	compatible = "samsung,exynos4212-usb2-phy";
> +	reg = <0x125b0000 0x100 0x10020704 0x0c 0x1001021c 0x4>;
> +	clocks = <&clock 305>, <&clock 2>, <&clock 2>, <&clock 2>,
> +							<&clock 2>;
> +	clock-names = "phy", "device", "host", "hsic0", "hsic1";
> +	status = "okay";
> +	#phy-cells = <1>;
> +	samsung,sysreg-phandle = <&sys_reg>;
> +	samsung,pmureg-phandle = <&pmu_reg>;
> +};
> +
> +Then the PHY can be used in other nodes such as:
> +
> +phy-consumer@...40000 {
> +	phys = <&usbphy 2>;
> +	phy-names = "phy";
> +};
> +
> +Refer to DT bindings documentation of particular PHY consumer devices for more
> +information about required PHYs and the way of specification.
> diff --git a/Documentation/phy/samsung-usb2.txt b/Documentation/phy/samsung-usb2.txt
> new file mode 100644
> index 0000000..9f5826e
> --- /dev/null
> +++ b/Documentation/phy/samsung-usb2.txt
> @@ -0,0 +1,135 @@
> +.------------------------------------------------------------------------------+
> +|			Samsung USB 2.0 PHY adaptation layer		       |
> ++-----------------------------------------------------------------------------+'
> +
> +| 1. Description
> ++----------------
> +
> +The architecture of the USB 2.0 PHY module in Samsung SoCs is similar
> +among many SoCs. In spite of the similarities it proved difficult to
> +create a one driver that would fit all these PHY controllers. Often
> +the differences were minor and were found in particular bits of the
> +registers of the PHY. In some rare cases the order of register writes or
> +the PHY powering up process had to be altered. This adaptation layer is
> +a compromise between having separate drivers and having a single driver
> +with added support for many special cases.
> +
> +| 2. Files description
> ++----------------------
> +
> +- phy-samsung-usb2.c
> +   This is the main file of the adaptation layer. This file contains
> +   the probe function and provides two callbacks to the Generic PHY
> +   Framework. This two callbacks are used to power on and power off the
> +   phy. They carry out the common work that has to be done on all version
> +   of the PHY module. Depending on which SoC was chosen they execute SoC
> +   specific callbacks. The specific SoC version is selected by choosing
> +   the appropriate compatible string. In addition, this file contains
> +   struct of_device_id definitions for particular SoCs.
> +
> +- phy-samsung-usb2.h
> +   This is the include file. It declares the structures used by this
> +   driver. In addition it should contain extern declarations for
> +   structures that describe particular SoCs.
> +
> +| 3. Supporting SoCs
> ++--------------------
> +
> +To support a new SoC a new file should be added to the drivers/phy
> +directory. Each SoC's configuration is stored in an instance of the
> +struct samsung_usb2_phy_config.
> +
> +struct samsung_usb2_phy_config {
> +	const struct samsung_usb2_common_phy *phys;
> +	unsigned int num_phys;
> +	bool has_mode_switch;
> +};
> +
> +The num_phys is the number of phys handled by the driver. *phys is an
> +array that contains the configuration for each phy. The has_mode_switch
> +property is a boolean flag that determines whether the SoC has USB host
> +and device on a single pair of pins. If so, a special register has to
> +be modified to change the internal routing of these pins between a USB
> +device or host module.
> +
> +For example the configuration for Exynos 4210 is following:
> +
> +const struct samsung_usb2_phy_config exynos4210_usb2_phy_config = {
> +	.num_phys		= EXYNOS4210_NUM_PHYS,
> +	.phys			= exynos4210_phys,
> +}
> +
> +The exynos4210_phys configuration array is as follows:
> +
> +static const struct samsung_usb2_common_phy exynos4210_phys[] = {
> +	{
> +		.label		= "device",
> +		.id		= EXYNOS4210_DEVICE,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{
> +		.label		= "host",
> +		.id		= EXYNOS4210_HOST,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{
> +		.label		= "hsic0",
> +		.id		= EXYNOS4210_HSIC0,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{
> +		.label		= "hsic1",
> +		.id		= EXYNOS4210_HSIC1,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{},
> +};
> +
> +- int (*rate_to_clk)(unsigned long, u32 *)
> +	The rate_to_clk callback is to convert the rate of the clock
> +	used to clock the module to the value that should be written to
> +	the registers of the PHY module.
> +- int (*power_on)(struct samsung_usb2_phy_instance *);
> +- int (*power_off)(struct samsung_usb2_phy_instance *);
> +	These two callbacks are used to power on and power off the phy
> +	by modifying appropriate registers.
> +
> +Final change to the driver is adding appropriate compatible value to the
> +phy-samsung-usb2.c file. In case of Exynos 4210 the following lines were
> +added to the struct of_device_id samsung_usb2_phy_of_match[] array:
> +
> +#ifdef CONFIG_PHY_EXYNOS4210_USB2
> +	{
> +		.compatible = "samsung,exynos4210-usb2-phy",
> +		.data = &exynos4210_usb2_phy_config,
> +	},
> +#endif
> +
> +To add further flexibility to the driver the Kconfig file enables to
> +include support for selected SoCs in the compiled driver. The Kconfig
> +entry for Exynos 4210 is following:
> +
> +config PHY_EXYNOS4210_USB2
> +	bool "Support for Exynos 4210"
> +	depends on PHY_SAMSUNG_USB2
> +	depends on CPU_EXYNOS4210
> +	help
> +	  Enable USB PHY support for Exynos 4210. This option requires that
> +	  Samsung USB 2.0 PHY driver is enabled and means that support for this
> +	  particular SoC is compiled in the driver. In case of Exynos 4210 four
> +	  phys are available - device, host, HSCI0 and HSCI1.
> +
> +The newly created file that supports the new SoC has to be also added to the
> +Makefile. In case of Exynos 4210 the added line is following:
> +
> +obj-$(CONFIG_PHY_EXYNOS4210_USB2)       += phy-exynos4210-usb2.o
> +
> +After completing these steps the support for the new SoC should be ready.
> diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
> index b901c47..3691d24 100644
> --- a/drivers/phy/Kconfig
> +++ b/drivers/phy/Kconfig
> @@ -63,4 +63,33 @@ config BCM_KONA_USB2_PHY
>   	help
>   	  Enable this to support the Broadcom Kona USB 2.0 PHY.
>
> +config PHY_SAMSUNG_USB2
> +	tristate "Samsung USB 2.0 PHY driver"
> +	select GENERIC_PHY
> +	select MFD_SYSCON
> +	help
> +	  Enable this to support the Samsung USB 2.0 PHY driver for Samsung
> +	  SoCs. This driver provides the interface for USB 2.0 PHY. Support for
> +	  particular SoCs has to be enabled in addition to this driver. Number
> +	  and type of supported phys depends on the SoC.
> +
> +config PHY_EXYNOS4210_USB2
> +	bool "Support for Exynos 4210"
> +	depends on PHY_SAMSUNG_USB2
> +	depends on CPU_EXYNOS4210
> +	help
> +	  Enable USB PHY support for Exynos 4210. This option requires that
> +	  Samsung USB 2.0 PHY driver is enabled and means that support for this
> +	  particular SoC is compiled in the driver. In case of Exynos 4210 four
> +	  phys are available - device, host, HSIC0 and HSIC1.
> +
> +config PHY_EXYNOS4X12_USB2
> +	bool "Support for Exynos 4x12"
> +	depends on PHY_SAMSUNG_USB2
> +	depends on (SOC_EXYNOS4212 || SOC_EXYNOS4412)
> +	help
> +	  Enable USB PHY support for Exynos 4x12. This option requires that
> +	  Samsung USB 2.0 PHY driver is enabled and means that support for this
> +	  particular SoC is compiled in the driver. In case of Exynos 4x12 four
> +	  phys are available - device, host, HSIC0 and HSIC1.
>   endmenu
> diff --git a/drivers/phy/Makefile b/drivers/phy/Makefile
> index b57c253..69d0b3f2 100644
> --- a/drivers/phy/Makefile
> +++ b/drivers/phy/Makefile
> @@ -9,3 +9,6 @@ obj-$(CONFIG_PHY_EXYNOS_MIPI_VIDEO)	+= phy-exynos-mipi-video.o
>   obj-$(CONFIG_PHY_MVEBU_SATA)		+= phy-mvebu-sata.o
>   obj-$(CONFIG_OMAP_USB2)			+= phy-omap-usb2.o
>   obj-$(CONFIG_TWL4030_USB)		+= phy-twl4030-usb.o
> +obj-$(CONFIG_PHY_SAMSUNG_USB2)		+= phy-samsung-usb2.o
> +obj-$(CONFIG_PHY_EXYNOS4210_USB2)	+= phy-exynos4210-usb2.o
> +obj-$(CONFIG_PHY_EXYNOS4X12_USB2)	+= phy-exynos4x12-usb2.o
> diff --git a/drivers/phy/phy-exynos4210-usb2.c b/drivers/phy/phy-exynos4210-usb2.c
> new file mode 100644
> index 0000000..8f86abf
> --- /dev/null
> +++ b/drivers/phy/phy-exynos4210-usb2.c
> @@ -0,0 +1,257 @@
> +/*
> + * Samsung SoC USB 1.1/2.0 PHY driver - Exynos 4210 support
> + *
> + * Copyright (C) 2013 Samsung Electronics Co., Ltd.
> + * Author: Kamil Debski <k.debski@...sung.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/delay.h>
> +#include <linux/io.h>
> +#include <linux/phy/phy.h>
> +#include <linux/regmap.h>
> +#include "phy-samsung-usb2.h"
> +
> +/* Exynos USB PHY registers */
> +
> +/* PHY power control */
> +#define EXYNOS_4210_UPHYPWR			0x0
> +
> +#define EXYNOS_4210_UPHYPWR_PHY0_SUSPEND	BIT(0)
> +#define EXYNOS_4210_UPHYPWR_PHY0_PWR		BIT(3)
> +#define EXYNOS_4210_UPHYPWR_PHY0_OTG_PWR	BIT(4)
> +#define EXYNOS_4210_UPHYPWR_PHY0_SLEEP		BIT(5)
> +#define EXYNOS_4210_UPHYPWR_PHY0	( \
> +	EXYNOS_4210_UPHYPWR_PHY0_SUSPEND | \
> +	EXYNOS_4210_UPHYPWR_PHY0_PWR | \
> +	EXYNOS_4210_UPHYPWR_PHY0_OTG_PWR | \
> +	EXYNOS_4210_UPHYPWR_PHY0_SLEEP)
> +
> +#define EXYNOS_4210_UPHYPWR_PHY1_SUSPEND	BIT(6)
> +#define EXYNOS_4210_UPHYPWR_PHY1_PWR		BIT(7)
> +#define EXYNOS_4210_UPHYPWR_PHY1_SLEEP		BIT(8)
> +#define EXYNOS_4210_UPHYPWR_PHY1 ( \
> +	EXYNOS_4210_UPHYPWR_PHY1_SUSPEND | \
> +	EXYNOS_4210_UPHYPWR_PHY1_PWR | \
> +	EXYNOS_4210_UPHYPWR_PHY1_SLEEP)
> +
> +#define EXYNOS_4210_UPHYPWR_HSIC0_SUSPEND	BIT(9)
> +#define EXYNOS_4210_UPHYPWR_HSIC0_SLEEP		BIT(10)
> +#define EXYNOS_4210_UPHYPWR_HSIC0 ( \
> +	EXYNOS_4210_UPHYPWR_HSIC0_SUSPEND | \
> +	EXYNOS_4210_UPHYPWR_HSIC0_SLEEP)
> +
> +#define EXYNOS_4210_UPHYPWR_HSIC1_SUSPEND	BIT(11)
> +#define EXYNOS_4210_UPHYPWR_HSIC1_SLEEP		BIT(12)
> +#define EXYNOS_4210_UPHYPWR_HSIC1 ( \
> +	EXYNOS_4210_UPHYPWR_HSIC1_SUSPEND | \
> +	EXYNOS_4210_UPHYPWR_HSIC1_SLEEP)
> +
> +/* PHY clock control */
> +#define EXYNOS_4210_UPHYCLK			0x4
> +
> +#define EXYNOS_4210_UPHYCLK_PHYFSEL_MASK	(0x3 << 0)
> +#define EXYNOS_4210_UPHYCLK_PHYFSEL_48MHZ	(0x0 << 0)
> +#define EXYNOS_4210_UPHYCLK_PHYFSEL_24MHZ	(0x3 << 0)
> +#define EXYNOS_4210_UPHYCLK_PHYFSEL_12MHZ	(0x2 << 0)
> +
> +#define EXYNOS_4210_UPHYCLK_PHY0_ID_PULLUP	BIT(2)
> +#define EXYNOS_4210_UPHYCLK_PHY0_COMMON_ON	BIT(4)
> +#define EXYNOS_4210_UPHYCLK_PHY1_COMMON_ON	BIT(7)
> +
> +/* PHY reset control */
> +#define EXYNOS_4210_UPHYRST			0x8
> +
> +#define EXYNOS_4210_URSTCON_PHY0		BIT(0)
> +#define EXYNOS_4210_URSTCON_OTG_HLINK		BIT(1)
> +#define EXYNOS_4210_URSTCON_OTG_PHYLINK		BIT(2)
> +#define EXYNOS_4210_URSTCON_PHY1_ALL		BIT(3)
> +#define EXYNOS_4210_URSTCON_PHY1_P0		BIT(4)
> +#define EXYNOS_4210_URSTCON_PHY1_P1P2		BIT(5)
> +#define EXYNOS_4210_URSTCON_HOST_LINK_ALL	BIT(6)
> +#define EXYNOS_4210_URSTCON_HOST_LINK_P0	BIT(7)
> +#define EXYNOS_4210_URSTCON_HOST_LINK_P1	BIT(8)
> +#define EXYNOS_4210_URSTCON_HOST_LINK_P2	BIT(9)
> +
> +/* Isolation, configured in the power management unit */
> +#define EXYNOS_4210_USB_ISOL_DEVICE_OFFSET	0x704
> +#define EXYNOS_4210_USB_ISOL_DEVICE		BIT(0)
> +#define EXYNOS_4210_USB_ISOL_HOST_OFFSET	0x708
> +#define EXYNOS_4210_USB_ISOL_HOST		BIT(0)
> +
> +/* USBYPHY1 Floating prevention */
> +#define EXYNOS_4210_UPHY1CON			0x34
> +#define EXYNOS_4210_UPHY1CON_FLOAT_PREVENTION	0x1
> +
> +/* Mode switching SUB Device <-> Host */
> +#define EXYNOS_4210_MODE_SWITCH_OFFSET		0x21c
> +#define EXYNOS_4210_MODE_SWITCH_MASK		1
> +#define EXYNOS_4210_MODE_SWITCH_DEVICE		0
> +#define EXYNOS_4210_MODE_SWITCH_HOST		1
> +
> +enum exynos4210_phy_id {
> +	EXYNOS4210_DEVICE,
> +	EXYNOS4210_HOST,
> +	EXYNOS4210_HSIC0,
> +	EXYNOS4210_HSIC1,
> +	EXYNOS4210_NUM_PHYS,
> +};
> +
> +/*
> + * exynos4210_rate_to_clk() converts the supplied clock rate to the value that
> + * can be written to the phy register.
> + */
> +static int exynos4210_rate_to_clk(unsigned long rate, u32 *reg)
> +{
> +	switch (rate) {
> +	case 12 * MHZ:
> +		*reg = EXYNOS_4210_UPHYCLK_PHYFSEL_12MHZ;
> +		break;
> +	case 24 * MHZ:
> +		*reg = EXYNOS_4210_UPHYCLK_PHYFSEL_24MHZ;
> +		break;
> +	case 48 * MHZ:
> +		*reg = EXYNOS_4210_UPHYCLK_PHYFSEL_48MHZ;
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return 0;
> +}
> +
> +static void exynos4210_isol(struct samsung_usb2_phy_instance *inst, bool on)
> +{
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	u32 offset;
> +	u32 mask;
> +
> +	switch (inst->cfg->id) {
> +	case EXYNOS4210_DEVICE:
> +		offset = EXYNOS_4210_USB_ISOL_DEVICE_OFFSET;
> +		mask = EXYNOS_4210_USB_ISOL_DEVICE;
> +		break;
> +	case EXYNOS4210_HOST:
> +		offset = EXYNOS_4210_USB_ISOL_HOST_OFFSET;
> +		mask = EXYNOS_4210_USB_ISOL_HOST;
> +		break;
> +	default:
> +		return;
> +	};
> +
> +	regmap_update_bits(drv->reg_pmu, offset, mask, on ? 0 : mask);
> +}
> +
> +static void exynos4210_phy_pwr(struct samsung_usb2_phy_instance *inst, bool on)
> +{
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	u32 rstbits = 0;
> +	u32 phypwr = 0;
> +	u32 rst;
> +	u32 pwr;
> +
> +	switch (inst->cfg->id) {
> +	case EXYNOS4210_DEVICE:
> +		phypwr =	EXYNOS_4210_UPHYPWR_PHY0;
> +		rstbits =	EXYNOS_4210_URSTCON_PHY0;
> +		break;
> +	case EXYNOS4210_HOST:
> +		phypwr =	EXYNOS_4210_UPHYPWR_PHY1;
> +		rstbits =	EXYNOS_4210_URSTCON_PHY1_ALL |
> +				EXYNOS_4210_URSTCON_PHY1_P0 |
> +				EXYNOS_4210_URSTCON_PHY1_P1P2 |
> +				EXYNOS_4210_URSTCON_HOST_LINK_ALL |
> +				EXYNOS_4210_URSTCON_HOST_LINK_P0;
> +		writel(on, drv->reg_phy + EXYNOS_4210_UPHY1CON);
> +		break;
> +	case EXYNOS4210_HSIC0:
> +		phypwr =	EXYNOS_4210_UPHYPWR_HSIC0;
> +		rstbits =	EXYNOS_4210_URSTCON_PHY1_P1P2 |
> +				EXYNOS_4210_URSTCON_HOST_LINK_P1;
> +		break;
> +	case EXYNOS4210_HSIC1:
> +		phypwr =	EXYNOS_4210_UPHYPWR_HSIC1;
> +		rstbits =	EXYNOS_4210_URSTCON_PHY1_P1P2 |
> +				EXYNOS_4210_URSTCON_HOST_LINK_P2;
> +		break;
> +	};
> +
> +	if (on) {
> +		writel(inst->clk_reg_val, drv->reg_phy + EXYNOS_4210_UPHYCLK);
> +
> +		pwr = readl(drv->reg_phy + EXYNOS_4210_UPHYPWR);
> +		pwr &= ~phypwr;
> +		writel(pwr, drv->reg_phy + EXYNOS_4210_UPHYPWR);
> +
> +		rst = readl(drv->reg_phy + EXYNOS_4210_UPHYRST);
> +		rst |= rstbits;
> +		writel(rst, drv->reg_phy + EXYNOS_4210_UPHYRST);
> +		udelay(10);
> +		rst &= ~rstbits;
> +		writel(rst, drv->reg_phy + EXYNOS_4210_UPHYRST);
> +	} else {
> +		pwr = readl(drv->reg_phy + EXYNOS_4210_UPHYPWR);
> +		pwr |= phypwr;
> +		writel(pwr, drv->reg_phy + EXYNOS_4210_UPHYPWR);
> +	}
> +}
> +
> +static int exynos4210_power_on(struct samsung_usb2_phy_instance *inst)
> +{
> +	/* Order of initialisation is important - first power then isolation */
> +	exynos4210_phy_pwr(inst, 1);
> +	exynos4210_isol(inst, 0);
> +
> +	return 0;
> +}
> +
> +static int exynos4210_power_off(struct samsung_usb2_phy_instance *inst)
> +{
> +	exynos4210_isol(inst, 1);
> +	exynos4210_phy_pwr(inst, 0);
> +
> +	return 0;
> +}
> +
> +
> +static const struct samsung_usb2_common_phy exynos4210_phys[] = {
> +	{
> +		.label		= "device",
> +		.id		= EXYNOS4210_DEVICE,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{
> +		.label		= "host",
> +		.id		= EXYNOS4210_HOST,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{
> +		.label		= "hsic0",
> +		.id		= EXYNOS4210_HSIC0,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{
> +		.label		= "hsic1",
> +		.id		= EXYNOS4210_HSIC1,
> +		.rate_to_clk	= exynos4210_rate_to_clk,
> +		.power_on	= exynos4210_power_on,
> +		.power_off	= exynos4210_power_off,
> +	},
> +	{},
> +};
> +
> +const struct samsung_usb2_phy_config exynos4210_usb2_phy_config = {
> +	.num_phys		= EXYNOS4210_NUM_PHYS,
> +	.phys			= exynos4210_phys,
> +	.has_mode_switch	= 0,
> +};
> +
> diff --git a/drivers/phy/phy-exynos4x12-usb2.c b/drivers/phy/phy-exynos4x12-usb2.c
> new file mode 100644
> index 0000000..ce39502
> --- /dev/null
> +++ b/drivers/phy/phy-exynos4x12-usb2.c
> @@ -0,0 +1,323 @@
> +/*
> + * Samsung SoC USB 1.1/2.0 PHY driver - Exynos 4x12 support
> + *
> + * Copyright (C) 2013 Samsung Electronics Co., Ltd.
> + * Author: Kamil Debski <k.debski@...sung.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/delay.h>
> +#include <linux/io.h>
> +#include <linux/phy/phy.h>
> +#include <linux/regmap.h>
> +#include "phy-samsung-usb2.h"
> +
> +/* Exynos USB PHY registers */
> +
> +/* PHY power control */
> +#define EXYNOS_4x12_UPHYPWR			0x0
> +
> +#define EXYNOS_4x12_UPHYPWR_PHY0_SUSPEND	BIT(0)
> +#define EXYNOS_4x12_UPHYPWR_PHY0_PWR		BIT(3)
> +#define EXYNOS_4x12_UPHYPWR_PHY0_OTG_PWR	BIT(4)
> +#define EXYNOS_4x12_UPHYPWR_PHY0_SLEEP		BIT(5)
> +#define EXYNOS_4x12_UPHYPWR_PHY0 ( \
> +	EXYNOS_4x12_UPHYPWR_PHY0_SUSPEND | \
> +	EXYNOS_4x12_UPHYPWR_PHY0_PWR | \
> +	EXYNOS_4x12_UPHYPWR_PHY0_OTG_PWR | \
> +	EXYNOS_4x12_UPHYPWR_PHY0_SLEEP)
> +
> +#define EXYNOS_4x12_UPHYPWR_PHY1_SUSPEND	BIT(6)
> +#define EXYNOS_4x12_UPHYPWR_PHY1_PWR		BIT(7)
> +#define EXYNOS_4x12_UPHYPWR_PHY1_SLEEP		BIT(8)
> +#define EXYNOS_4x12_UPHYPWR_PHY1 ( \
> +	EXYNOS_4x12_UPHYPWR_PHY1_SUSPEND | \
> +	EXYNOS_4x12_UPHYPWR_PHY1_PWR | \
> +	EXYNOS_4x12_UPHYPWR_PHY1_SLEEP)
> +
> +#define EXYNOS_4x12_UPHYPWR_HSIC0_SUSPEND	BIT(9)
> +#define EXYNOS_4x12_UPHYPWR_HSIC0_PWR		BIT(10)
> +#define EXYNOS_4x12_UPHYPWR_HSIC0_SLEEP		BIT(11)
> +#define EXYNOS_4x12_UPHYPWR_HSIC0 ( \
> +	EXYNOS_4x12_UPHYPWR_HSIC0_SUSPEND | \
> +	EXYNOS_4x12_UPHYPWR_HSIC0_PWR | \
> +	EXYNOS_4x12_UPHYPWR_HSIC0_SLEEP)
> +
> +#define EXYNOS_4x12_UPHYPWR_HSIC1_SUSPEND	BIT(12)
> +#define EXYNOS_4x12_UPHYPWR_HSIC1_PWR		BIT(13)
> +#define EXYNOS_4x12_UPHYPWR_HSIC1_SLEEP		BIT(14)
> +#define EXYNOS_4x12_UPHYPWR_HSIC1 ( \
> +	EXYNOS_4x12_UPHYPWR_HSIC1_SUSPEND | \
> +	EXYNOS_4x12_UPHYPWR_HSIC1_PWR | \
> +	EXYNOS_4x12_UPHYPWR_HSIC1_SLEEP)
> +
> +/* PHY clock control */
> +#define EXYNOS_4x12_UPHYCLK			0x4
> +
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_MASK	(0x7 << 0)
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_9MHZ6	(0x0 << 0)
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_10MHZ	(0x1 << 0)
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_12MHZ	(0x2 << 0)
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_19MHZ2	(0x3 << 0)
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_20MHZ	(0x4 << 0)
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_24MHZ	(0x5 << 0)
> +#define EXYNOS_4x12_UPHYCLK_PHYFSEL_50MHZ	(0x7 << 0)
> +
> +#define EXYNOS_4x12_UPHYCLK_PHY0_ID_PULLUP	BIT(3)
> +#define EXYNOS_4x12_UPHYCLK_PHY0_COMMON_ON	BIT(4)
> +#define EXYNOS_4x12_UPHYCLK_PHY1_COMMON_ON	BIT(7)
> +
> +#define EXYNOS_4x12_UPHYCLK_HSIC_REFCLK_MASK	(0x7f << 10)
> +#define EXYNOS_4x12_UPHYCLK_HSIC_REFCLK_12MHZ	(0x24 << 10)
> +#define EXYNOS_4x12_UPHYCLK_HSIC_REFCLK_15MHZ	(0x1c << 10)
> +#define EXYNOS_4x12_UPHYCLK_HSIC_REFCLK_16MHZ	(0x1a << 10)
> +#define EXYNOS_4x12_UPHYCLK_HSIC_REFCLK_19MHZ2	(0x15 << 10)
> +#define EXYNOS_4x12_UPHYCLK_HSIC_REFCLK_20MHZ	(0x14 << 10)
> +
> +/* PHY reset control */
> +#define EXYNOS_4x12_UPHYRST			0x8
> +
> +#define EXYNOS_4x12_URSTCON_PHY0		BIT(0)
> +#define EXYNOS_4x12_URSTCON_OTG_HLINK		BIT(1)
> +#define EXYNOS_4x12_URSTCON_OTG_PHYLINK		BIT(2)
> +#define EXYNOS_4x12_URSTCON_HOST_PHY		BIT(3)
> +#define EXYNOS_4x12_URSTCON_PHY1		BIT(4)
> +#define EXYNOS_4x12_URSTCON_HSIC0		BIT(5)
> +#define EXYNOS_4x12_URSTCON_HSIC1		BIT(6)
> +#define EXYNOS_4x12_URSTCON_HOST_LINK_ALL	BIT(7)
> +#define EXYNOS_4x12_URSTCON_HOST_LINK_P0	BIT(8)
> +#define EXYNOS_4x12_URSTCON_HOST_LINK_P1	BIT(9)
> +#define EXYNOS_4x12_URSTCON_HOST_LINK_P2	BIT(10)
> +
> +/* Isolation, configured in the power management unit */
> +#define EXYNOS_4x12_USB_ISOL_OFFSET		0x704
> +#define EXYNOS_4x12_USB_ISOL_OTG		BIT(0)
> +#define EXYNOS_4x12_USB_ISOL_HSIC0_OFFSET	0x708
> +#define EXYNOS_4x12_USB_ISOL_HSIC0		BIT(0)
> +#define EXYNOS_4x12_USB_ISOL_HSIC1_OFFSET	0x70c
> +#define EXYNOS_4x12_USB_ISOL_HSIC1		BIT(0)
> +
> +/* Mode switching SUB Device <-> Host */
> +#define EXYNOS_4x12_MODE_SWITCH_OFFSET		0x21c
> +#define EXYNOS_4x12_MODE_SWITCH_MASK		1
> +#define EXYNOS_4x12_MODE_SWITCH_DEVICE		0
> +#define EXYNOS_4x12_MODE_SWITCH_HOST		1
> +
> +enum exynos4x12_phy_id {
> +	EXYNOS4x12_DEVICE,
> +	EXYNOS4x12_HOST,
> +	EXYNOS4x12_HSIC0,
> +	EXYNOS4x12_HSIC1,
> +	EXYNOS4x12_NUM_PHYS,
> +};
> +
> +/*
> + * exynos4x12_rate_to_clk() converts the supplied clock rate to the value that
> + * can be written to the phy register.
> + */
> +static int exynos4x12_rate_to_clk(unsigned long rate, u32 *reg)
> +{
> +	/* EXYNOS_4x12_UPHYCLK_PHYFSEL_MASK */
> +
> +	switch (rate) {
> +	case 9600 * KHZ:
> +		*reg = EXYNOS_4x12_UPHYCLK_PHYFSEL_9MHZ6;
> +		break;
> +	case 10 * MHZ:
> +		*reg = EXYNOS_4x12_UPHYCLK_PHYFSEL_10MHZ;
> +		break;
> +	case 12 * MHZ:
> +		*reg = EXYNOS_4x12_UPHYCLK_PHYFSEL_12MHZ;
> +		break;
> +	case 19200 * KHZ:
> +		*reg = EXYNOS_4x12_UPHYCLK_PHYFSEL_19MHZ2;
> +		break;
> +	case 20 * MHZ:
> +		*reg = EXYNOS_4x12_UPHYCLK_PHYFSEL_20MHZ;
> +		break;
> +	case 24 * MHZ:
> +		*reg = EXYNOS_4x12_UPHYCLK_PHYFSEL_24MHZ;
> +		break;
> +	case 50 * MHZ:
> +		*reg = EXYNOS_4x12_UPHYCLK_PHYFSEL_50MHZ;
> +		break;
> +	default:
> +		return -EINVAL;
> +	}
> +
> +	return 0;
> +}
> +
> +static void exynos4x12_isol(struct samsung_usb2_phy_instance *inst, bool on)
> +{
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	u32 offset;
> +	u32 mask;
> +
> +	switch (inst->cfg->id) {
> +	case EXYNOS4x12_DEVICE:
> +	case EXYNOS4x12_HOST:
> +		offset = EXYNOS_4x12_USB_ISOL_OFFSET;
> +		mask = EXYNOS_4x12_USB_ISOL_OTG;
> +		break;
> +	case EXYNOS4x12_HSIC0:
> +		offset = EXYNOS_4x12_USB_ISOL_HSIC0_OFFSET;
> +		mask = EXYNOS_4x12_USB_ISOL_HSIC0;
> +		break;
> +	case EXYNOS4x12_HSIC1:
> +		offset = EXYNOS_4x12_USB_ISOL_HSIC1_OFFSET;
> +		mask = EXYNOS_4x12_USB_ISOL_HSIC1;
> +		break;
> +	default:
> +		return;
> +	};
> +
> +	regmap_update_bits(drv->reg_pmu, offset, mask, on ? 0 : mask);
> +}
> +
> +static void exynos4x12_setup_clk(struct samsung_usb2_phy_instance *inst)
> +{
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	writel(inst->clk_reg_val, drv->reg_phy + EXYNOS_4x12_UPHYCLK);
> +}
> +
> +static void exynos4x12_phy_pwr(struct samsung_usb2_phy_instance *inst, bool on)
> +{
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	u32 rstbits = 0;
> +	u32 phypwr = 0;
> +	u32 rst;
> +	u32 pwr;
> +	u32 mode = 0;
> +	u32 switch_mode = 0;
> +
> +	switch (inst->cfg->id) {
> +	case EXYNOS4x12_DEVICE:
> +		phypwr =	EXYNOS_4x12_UPHYPWR_PHY0;
> +		rstbits =	EXYNOS_4x12_URSTCON_PHY0;
> +		mode =		EXYNOS_4x12_MODE_SWITCH_DEVICE;
> +		switch_mode =	1;
> +		break;
> +	case EXYNOS4x12_HOST:
> +		phypwr =	EXYNOS_4x12_UPHYPWR_PHY1;
> +		rstbits =	EXYNOS_4x12_URSTCON_HOST_PHY;
> +		mode =		EXYNOS_4x12_MODE_SWITCH_HOST;
> +		switch_mode =	1;
> +		break;
> +	case EXYNOS4x12_HSIC0:
> +		phypwr =	EXYNOS_4x12_UPHYPWR_HSIC0;
> +		rstbits =	EXYNOS_4x12_URSTCON_HSIC1 |
> +				EXYNOS_4x12_URSTCON_HOST_LINK_P0 |
> +				EXYNOS_4x12_URSTCON_HOST_PHY;
> +		break;
> +	case EXYNOS4x12_HSIC1:
> +		phypwr =	EXYNOS_4x12_UPHYPWR_HSIC1;
> +		rstbits =	EXYNOS_4x12_URSTCON_HSIC1 |
> +				EXYNOS_4x12_URSTCON_HOST_LINK_P1;
> +		break;
> +	};
> +
> +	if (on) {
> +		if (switch_mode)
> +			regmap_update_bits(drv->reg_sys,
> +			        EXYNOS_4x12_MODE_SWITCH_OFFSET,
> +			        EXYNOS_4x12_MODE_SWITCH_MASK,
> +				mode);
> +
> +		pwr = readl(drv->reg_phy + EXYNOS_4x12_UPHYPWR);
> +		pwr &= ~phypwr;
> +		writel(pwr, drv->reg_phy + EXYNOS_4x12_UPHYPWR);
> +
> +		rst = readl(drv->reg_phy + EXYNOS_4x12_UPHYRST);
> +		rst |= rstbits;
> +		writel(rst, drv->reg_phy + EXYNOS_4x12_UPHYRST);
> +		udelay(10);
> +		rst &= ~rstbits;
> +		writel(rst, drv->reg_phy + EXYNOS_4x12_UPHYRST);
> +	} else {
> +		pwr = readl(drv->reg_phy + EXYNOS_4x12_UPHYPWR);
> +		pwr |= phypwr;
> +		writel(pwr, drv->reg_phy + EXYNOS_4x12_UPHYPWR);
> +	}
> +}
> +
> +static int exynos4x12_power_on(struct samsung_usb2_phy_instance *inst)
> +{
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +
> +	inst->enabled = 1;
> +	exynos4x12_setup_clk(inst);
> +	exynos4x12_phy_pwr(inst, 1);
> +	exynos4x12_isol(inst, 0);
> +
> +	/* Power on the device, as it is necessary for HSIC to work */
> +	if (inst->cfg->id == EXYNOS4x12_HSIC0) {
> +		struct samsung_usb2_phy_instance *device =
> +					&drv->instances[EXYNOS4x12_DEVICE];
> +		exynos4x12_phy_pwr(device, 1);
> +		exynos4x12_isol(device, 0);
> +	}
> +
> +	return 0;
> +}
> +
> +static int exynos4x12_power_off(struct samsung_usb2_phy_instance *inst)
> +{
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	struct samsung_usb2_phy_instance *device =
> +					&drv->instances[EXYNOS4x12_DEVICE];
> +
> +	inst->enabled = 0;
> +	exynos4x12_isol(inst, 1);
> +	exynos4x12_phy_pwr(inst, 0);
> +
> +	if (inst->cfg->id == EXYNOS4x12_HSIC0 && !device->enabled) {
> +		exynos4x12_isol(device, 1);
> +		exynos4x12_phy_pwr(device, 0);
> +	}
> +
> +	return 0;
> +}
> +
> +
> +static const struct samsung_usb2_common_phy exynos4x12_phys[] = {
> +	{
> +		.label		= "device",
> +		.id		= EXYNOS4x12_DEVICE,
> +		.rate_to_clk	= exynos4x12_rate_to_clk,
> +		.power_on	= exynos4x12_power_on,
> +		.power_off	= exynos4x12_power_off,
> +	},
> +	{
> +		.label		= "host",
> +		.id		= EXYNOS4x12_HOST,
> +		.rate_to_clk	= exynos4x12_rate_to_clk,
> +		.power_on	= exynos4x12_power_on,
> +		.power_off	= exynos4x12_power_off,
> +	},
> +	{
> +		.label		= "hsic0",
> +		.id		= EXYNOS4x12_HSIC0,
> +		.rate_to_clk	= exynos4x12_rate_to_clk,
> +		.power_on	= exynos4x12_power_on,
> +		.power_off	= exynos4x12_power_off,
> +	},
> +	{
> +		.label		= "hsic1",
> +		.id		= EXYNOS4x12_HSIC1,
> +		.rate_to_clk	= exynos4x12_rate_to_clk,
> +		.power_on	= exynos4x12_power_on,
> +		.power_off	= exynos4x12_power_off,
> +	},
> +	{},
> +};
> +
> +const struct samsung_usb2_phy_config exynos4x12_usb2_phy_config = {
> +	.num_phys		= EXYNOS4x12_NUM_PHYS,
> +	.phys			= exynos4x12_phys,
> +	.has_mode_switch	= 1,
> +};
> +
> diff --git a/drivers/phy/phy-samsung-usb2.c b/drivers/phy/phy-samsung-usb2.c
> new file mode 100644
> index 0000000..c1b0fd5
> --- /dev/null
> +++ b/drivers/phy/phy-samsung-usb2.c
> @@ -0,0 +1,227 @@
> +/*
> + * Samsung SoC USB 1.1/2.0 PHY driver
> + *
> + * Copyright (C) 2013 Samsung Electronics Co., Ltd.
> + * Author: Kamil Debski <k.debski@...sung.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/mfd/syscon.h>
> +#include <linux/module.h>
> +#include <linux/of.h>
> +#include <linux/of_address.h>
> +#include <linux/phy/phy.h>
> +#include <linux/platform_device.h>
> +#include <linux/spinlock.h>
> +#include "phy-samsung-usb2.h"
> +
> +static int samsung_usb2_phy_power_on(struct phy *phy)
> +{
> +	struct samsung_usb2_phy_instance *inst = phy_get_drvdata(phy);
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	int ret;
> +
> +	dev_dbg(drv->dev, "Request to power_on \"%s\" usb phy\n",
> +							inst->cfg->label);
> +	ret = clk_prepare_enable(drv->clk);
> +	if (ret)
> +		goto err_main_clk;
> +	ret = clk_prepare_enable(inst->clk);
> +	if (ret)
> +		goto err_instance_clk;
> +	inst->rate = clk_get_rate(inst->clk);
> +	if (inst->cfg->rate_to_clk) {
> +		ret = inst->cfg->rate_to_clk(inst->rate, &inst->clk_reg_val);
> +		if (ret)
> +			goto err_get_rate;
> +	}
> +	if (inst->cfg->power_on) {
> +		spin_lock(&drv->lock);
> +		ret = inst->cfg->power_on(inst);
> +		spin_unlock(&drv->lock);
> +	}
> +
> +	return 0;
> +
> +err_get_rate:
> +	clk_disable_unprepare(inst->clk);
> +err_instance_clk:
> +	clk_disable_unprepare(drv->clk);
> +err_main_clk:
> +	return ret;
> +}
> +
> +static int samsung_usb2_phy_power_off(struct phy *phy)
> +{
> +	struct samsung_usb2_phy_instance *inst = phy_get_drvdata(phy);
> +	struct samsung_usb2_phy_driver *drv = inst->drv;
> +	int ret = 0;
> +
> +	dev_dbg(drv->dev, "Request to power_off \"%s\" usb phy\n",
> +							inst->cfg->label);
> +	if (inst->cfg->power_off) {
> +		spin_lock(&drv->lock);
> +		ret = inst->cfg->power_off(inst);
> +		spin_unlock(&drv->lock);
> +	}
> +	clk_disable_unprepare(inst->clk);
> +	clk_disable_unprepare(drv->clk);
> +	return ret;
> +}
> +
> +static struct phy_ops samsung_usb2_phy_ops = {
> +	.power_on	= samsung_usb2_phy_power_on,
> +	.power_off	= samsung_usb2_phy_power_off,
> +	.owner		= THIS_MODULE,
> +};
> +
> +static struct phy *samsung_usb2_phy_xlate(struct device *dev,
> +					struct of_phandle_args *args)
> +{
> +	struct samsung_usb2_phy_driver *drv;
> +
> +	drv = dev_get_drvdata(dev);
> +	if (!drv)
> +		return ERR_PTR(-EINVAL);
> +
> +	if (WARN_ON(args->args[0] >= drv->cfg->num_phys))
> +		return ERR_PTR(-ENODEV);
> +
> +	return drv->instances[args->args[0]].phy;
> +}
> +
> +static const struct of_device_id samsung_usb2_phy_of_match[] = {
> +#ifdef CONFIG_PHY_EXYNOS4210_USB2
> +	{
> +		.compatible = "samsung,exynos4210-usb2-phy",
> +		.data = &exynos4210_usb2_phy_config,
> +	},
> +#endif
> +#ifdef CONFIG_PHY_EXYNOS4X12_USB2
> +	{
> +		.compatible = "samsung,exynos4x12-usb2-phy",
> +		.data = &exynos4x12_usb2_phy_config,
> +	},
> +#endif
> +	{ },
> +};
> +
> +static int samsung_usb2_phy_probe(struct platform_device *pdev)
> +{
> +	const struct of_device_id *match;
> +	const struct samsung_usb2_phy_config *cfg;
> +	struct clk *clk;
> +	struct device *dev = &pdev->dev;
> +	struct phy_provider *phy_provider;
> +	struct resource *mem;
> +	struct samsung_usb2_phy_driver *drv;
> +	int i;
> +
> +	if (!pdev->dev.of_node) {
> +		dev_err(dev, "This driver is required to be instantiated from device tree\n");
> +		return -EINVAL;
> +	}
> +
> +	match = of_match_node(samsung_usb2_phy_of_match, pdev->dev.of_node);
> +	if (!match) {
> +		dev_err(dev, "of_match_node() failed\n");
> +		return -EINVAL;
> +	}
> +	cfg = match->data;
> +
> +	drv = devm_kzalloc(dev, sizeof(struct samsung_usb2_phy_driver) +
> +		cfg->num_phys * sizeof(struct samsung_usb2_phy_instance),
> +								GFP_KERNEL);
> +	if (!drv)
> +		return -ENOMEM;
> +
> +	dev_set_drvdata(dev, drv);
> +	spin_lock_init(&drv->lock);
> +
> +	drv->cfg = cfg;
> +	drv->dev = dev;
> +
> +	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> +	drv->reg_phy = devm_ioremap_resource(dev, mem);
> +	if (IS_ERR(drv->reg_phy)) {
> +		dev_err(dev, "Failed to map register memory (phy)\n");
> +		return PTR_ERR(drv->reg_phy);
> +	}
> +
> +	drv->reg_pmu = syscon_regmap_lookup_by_phandle(pdev->dev.of_node,
> +		"samsung,pmureg-phandle");
> +	if (IS_ERR(drv->reg_pmu)) {
> +		dev_err(dev, "Failed to map PMU registers (via syscon)\n");
> +		return PTR_ERR(drv->reg_pmu);
> +	}
> +
> +	if (drv->cfg->has_mode_switch) {
> +		drv->reg_sys = syscon_regmap_lookup_by_phandle(
> +				pdev->dev.of_node, "samsung,sysreg-phandle");
> +		if (IS_ERR(drv->reg_sys)) {
> +			dev_err(dev, "Failed to map system registers (via syscon)\n");
> +			return PTR_ERR(drv->reg_sys);
> +		}
> +	}
> +
> +	drv->clk = devm_clk_get(dev, "phy");
> +	if (IS_ERR(drv->clk)) {
> +		dev_err(dev, "Failed to get clock of phy controller\n");
> +		return PTR_ERR(drv->clk);
> +	}
> +
> +	for (i = 0; i < drv->cfg->num_phys; i++) {
> +		char *label = drv->cfg->phys[i].label;
> +		struct samsung_usb2_phy_instance *p = &drv->instances[i];
> +
> +		dev_dbg(dev, "Creating phy \"%s\"\n", label);
> +		p->phy = devm_phy_create(dev, &samsung_usb2_phy_ops, NULL);
> +		if (IS_ERR(p->phy)) {
> +			dev_err(drv->dev, "Failed to create usb2_phy \"%s\"\n",
> +						label);
> +			return PTR_ERR(p->phy);
> +		}
> +
> +		p->cfg = &drv->cfg->phys[i];
> +		p->drv = drv;
> +		phy_set_bus_width(p->phy, 8);
> +		phy_set_drvdata(p->phy, p);
> +
> +		clk = devm_clk_get(dev, p->cfg->label);
> +		if (IS_ERR(clk)) {
> +			dev_err(dev, "Failed to get clock of \"%s\" phy\n",
> +								p->cfg->label);
> +			return PTR_ERR(clk);
> +		}
> +		p->clk = clk;
> +	}
> +
> +	phy_provider = devm_of_phy_provider_register(dev,
> +							samsung_usb2_phy_xlate);
> +	if (IS_ERR(phy_provider)) {
> +		dev_err(drv->dev, "Failed to register phy provider\n");
> +		return PTR_ERR(phy_provider);
> +	}
> +
> +	return 0;
> +}
> +
> +static struct platform_driver samsung_usb2_phy_driver = {
> +	.probe	= samsung_usb2_phy_probe,
> +	.driver = {
> +		.of_match_table	= samsung_usb2_phy_of_match,
> +		.name		= "samsung-usb2-phy",
> +		.owner		= THIS_MODULE,
> +	}
> +};
> +
> +module_platform_driver(samsung_usb2_phy_driver);
> +MODULE_DESCRIPTION("Samsung S5P/EXYNOS SoC USB PHY driver");
> +MODULE_AUTHOR("Kamil Debski <k.debski@...sung.com>");
> +MODULE_LICENSE("GPL v2");
> +MODULE_ALIAS("platform:samsung-usb2-phy");
> +
> diff --git a/drivers/phy/phy-samsung-usb2.h b/drivers/phy/phy-samsung-usb2.h
> new file mode 100644
> index 0000000..868dd33
> --- /dev/null
> +++ b/drivers/phy/phy-samsung-usb2.h
> @@ -0,0 +1,67 @@
> +/*
> + * Samsung SoC USB 1.1/2.0 PHY driver
> + *
> + * Copyright (C) 2013 Samsung Electronics Co., Ltd.
> + * Author: Kamil Debski <k.debski@...sung.com>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#ifndef _PHY_EXYNOS_USB2_H
> +#define _PHY_EXYNOS_USB2_H
> +
> +#include <linux/clk.h>
> +#include <linux/phy/phy.h>
> +#include <linux/device.h>
> +#include <linux/regmap.h>
> +#include <linux/spinlock.h>
> +
> +#define KHZ 1000
> +#define MHZ (KHZ * KHZ)
> +
> +struct samsung_usb2_phy_driver;
> +struct samsung_usb2_phy_instance;
> +struct samsung_usb2_phy_config;
> +
> +struct samsung_usb2_phy_instance {
> +	const struct samsung_usb2_common_phy *cfg;
> +	struct clk *clk;
> +	struct phy *phy;
> +	struct samsung_usb2_phy_driver *drv;
> +	unsigned long rate;
> +	u32 clk_reg_val;
> +	bool enabled;
> +};
> +
> +struct samsung_usb2_phy_driver {
> +	const struct samsung_usb2_phy_config *cfg;
> +	struct clk *clk;
> +	struct device *dev;
> +	void __iomem *reg_phy;
> +	struct regmap *reg_pmu;
> +	struct regmap *reg_sys;
> +	spinlock_t lock;
> +	struct samsung_usb2_phy_instance instances[0];
> +};
> +
> +struct samsung_usb2_common_phy {
> +	int (*rate_to_clk)(unsigned long, u32 *);
> +	int (*power_on)(struct samsung_usb2_phy_instance *);
> +	int (*power_off)(struct samsung_usb2_phy_instance *);
> +	unsigned int id;
> +	char *label;
> +};
> +
> +
> +struct samsung_usb2_phy_config {
> +	const struct samsung_usb2_common_phy *phys;
> +	unsigned int num_phys;
> +	bool has_mode_switch;
> +};
> +
> +extern const struct samsung_usb2_phy_config exynos4210_usb2_phy_config;
> +extern const struct samsung_usb2_phy_config exynos4x12_usb2_phy_config;
> +#endif
> +
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ