Re: [PATCH 2/4 v2] ARM: Exynos5250: Enabling samsung-usbphy driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Mon, Oct 22, 2012 at 10:33 AM, Jingoo Han <jg1.han@xxxxxxxxxxx> wrote:
> On Wednesday, October 10, 2012 7:43 PM Vivek Gautam wrote
>>
>> Adding usbphy node for Exynos5250 along with the platform data.
>>
>> Signed-off-by: Vivek Gautam <gautam.vivek@xxxxxxxxxxx>
>> ---
>>  .../devicetree/bindings/usb/samsung-usbphy.txt     |   12 +++++-
>>  arch/arm/boot/dts/exynos5250.dtsi                  |    5 ++
>>  arch/arm/mach-exynos/Kconfig                       |    1 +
>>  arch/arm/mach-exynos/include/mach/map.h            |    1 +
>>  arch/arm/mach-exynos/mach-exynos5-dt.c             |   10 +++++
>>  arch/arm/mach-exynos/setup-usb-phy.c               |   43 ++++++++++++++++---
>>  arch/arm/plat-samsung/include/plat/usb-phy.h       |    1 +
>>  include/linux/platform_data/samsung-usbphy.h       |    1 +
>>  8 files changed, 65 insertions(+), 9 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/usb/samsung-usbphy.txt
>> b/Documentation/devicetree/bindings/usb/samsung-usbphy.txt
>> index 7d54d59..c644d01 100644
>> --- a/Documentation/devicetree/bindings/usb/samsung-usbphy.txt
>> +++ b/Documentation/devicetree/bindings/usb/samsung-usbphy.txt
>> @@ -1,11 +1,19 @@
>>  * Samsung's usb phy transceiver
>>
>> -The Samsung's phy transceiver is used for controlling usb otg phy for
>> -s3c-hsotg usb device controller.
>> +The Samsung's phy transceiver is used for controlling usb phy for
>> +s3c-hsotg as well as ehci-s5p and ohci-exynos usb controllers
>> +across Samsung SOCs.
>>  TODO: Adding the PHY binding with controller(s) according to the under
>>  developement generic PHY driver.
>>
>> +Exynos4210:
>>  Required properties:
>>  - compatible : should be "samsung,exynos4210-usbphy"
>>  - reg : base physical address of the phy registers and length of memory mapped
>>       region.
>> +
>> +Exynos5250:
>> +Required properties:
>> +- compatible : should be "samsung,exynos5250-usbphy"
>> +- reg : base physical address of the phy registers and length of memory mapped
>> +     region.
>> diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
>> index dddfd6e..82bf042 100644
>> --- a/arch/arm/boot/dts/exynos5250.dtsi
>> +++ b/arch/arm/boot/dts/exynos5250.dtsi
>> @@ -218,6 +218,11 @@
>>               #size-cells = <0>;
>>       };
>>
>> +     usbphy {
>> +             compatible = "samsung,exynos5250-usbphy";
>> +             reg = <0x12130000 0x100>;
>> +     };
>> +
>>       amba {
>>               #address-cells = <1>;
>>               #size-cells = <1>;
>> diff --git a/arch/arm/mach-exynos/Kconfig b/arch/arm/mach-exynos/Kconfig
>> index 4372075..e8e3d7b 100644
>> --- a/arch/arm/mach-exynos/Kconfig
>> +++ b/arch/arm/mach-exynos/Kconfig
>> @@ -422,6 +422,7 @@ config MACH_EXYNOS5_DT
>>       select SOC_EXYNOS5250
>>       select USE_OF
>>       select ARM_AMBA
>> +     select EXYNOS4_SETUP_USB_PHY
>>       help
>>         Machine support for Samsung EXYNOS5 machine with device tree enabled.
>>         Select this if a fdt blob is available for the EXYNOS5 SoC based board.
>> diff --git a/arch/arm/mach-exynos/include/mach/map.h b/arch/arm/mach-exynos/include/mach/map.h
>> index 9694424..7f5eb03 100644
>> --- a/arch/arm/mach-exynos/include/mach/map.h
>> +++ b/arch/arm/mach-exynos/include/mach/map.h
>> @@ -195,6 +195,7 @@
>>  #define EXYNOS4_PA_EHCI                      0x12580000
>>  #define EXYNOS4_PA_OHCI                      0x12590000
>>  #define EXYNOS4_PA_HSPHY             0x125B0000
>> +#define EXYNOS5_PA_HSPHY             0x12130000
>>  #define EXYNOS4_PA_MFC                       0x13400000
>>
>>  #define EXYNOS4_PA_UART                      0x13800000
>> diff --git a/arch/arm/mach-exynos/mach-exynos5-dt.c b/arch/arm/mach-exynos/mach-exynos5-dt.c
>> index db1cd8e..bc041be 100644
>> --- a/arch/arm/mach-exynos/mach-exynos5-dt.c
>> +++ b/arch/arm/mach-exynos/mach-exynos5-dt.c
>> @@ -11,6 +11,7 @@
>>
>>  #include <linux/of_platform.h>
>>  #include <linux/serial_core.h>
>> +#include <linux/platform_data/samsung-usbphy.h>
>>
>>  #include <asm/mach/arch.h>
>>  #include <asm/hardware/gic.h>
>> @@ -18,9 +19,16 @@
>>
>>  #include <plat/cpu.h>
>>  #include <plat/regs-serial.h>
>> +#include <plat/usb-phy.h>
>>
>>  #include "common.h"
>>
>> +static struct samsung_usbphy_data exynos5_usbphy_pdata = {
>> +     .pmu_isolation = s5p_usb_phy_pmu_isolation,
>> +     .phy_cfg_sel = s5p_usb_phy_cfg_sel,
>> +     .type = TYPE_HOST,
>> +};
>> +
>>  /*
>>   * The following lookup table is used to override device names when devices
>>   * are registered from device tree. This is temporarily added to enable
>> @@ -72,6 +80,8 @@ static const struct of_dev_auxdata exynos5250_auxdata_lookup[] __initconst = {
>>                               "exynos-gsc.2", NULL),
>>       OF_DEV_AUXDATA("samsung,exynos5-gsc", EXYNOS5_PA_GSC3,
>>                               "exynos-gsc.3", NULL),
>> +     OF_DEV_AUXDATA("samsung,exynos5250-usbphy", EXYNOS5_PA_HSPHY,
>> +                             "s3c-usbphy", &exynos5_usbphy_pdata),
>>       {},
>>  };
>>
>> diff --git a/arch/arm/mach-exynos/setup-usb-phy.c b/arch/arm/mach-exynos/setup-usb-phy.c
>> index be6cd4f..bebc99b 100644
>> --- a/arch/arm/mach-exynos/setup-usb-phy.c
>> +++ b/arch/arm/mach-exynos/setup-usb-phy.c
>> @@ -14,11 +14,15 @@
>>  #include <linux/err.h>
>>  #include <linux/io.h>
>>  #include <linux/platform_device.h>
>> +#include <linux/platform_data/samsung-usbphy.h>
>>  #include <mach/regs-pmu.h>
>>  #include <mach/regs-usb-phy.h>
>>  #include <plat/cpu.h>
>> +#include <plat/map-base.h>
>>  #include <plat/usb-phy.h>
>>
>> +#define EXYNOS5_USB_CFG                      (S3C_VA_SYS + (0x230))
>
> The parentheses around 0x230 is not necessary.
>
> +#define EXYNOS5_USB_CFG                        (S3C_VA_SYS + 0x230)
>
Ok, will remove them.

>
>> +
>>  static atomic_t host_usage;
>>
>>  static int exynos4_usb_host_phy_is_on(void)
>> @@ -224,13 +228,38 @@ int s5p_usb_phy_exit(struct platform_device *pdev, int type)
>>
>>  void s5p_usb_phy_pmu_isolation(int on, int type)
>>  {
>> -     if (on) {
>> -             writel(readl(S5P_USBDEVICE_PHY_CONTROL)
>> -                     & ~S5P_USBDEVICE_PHY_ENABLE,
>> -                             S5P_USBDEVICE_PHY_CONTROL);
>> +     if (type == TYPE_HOST) {
>> +             if (on) {
>> +                     writel(readl(S5P_USBHOST_PHY_CONTROL)
>> +                             & ~S5P_USBHOST_PHY_ENABLE,
>> +                                     S5P_USBHOST_PHY_CONTROL);
>> +             } else {
>> +                     writel(readl(S5P_USBHOST_PHY_CONTROL)
>> +                             | S5P_USBHOST_PHY_ENABLE,
>> +                                     S5P_USBHOST_PHY_CONTROL);
>> +             }
>>       } else {
>> -             writel(readl(S5P_USBDEVICE_PHY_CONTROL)
>> -                     | S5P_USBDEVICE_PHY_ENABLE,
>> -                             S5P_USBDEVICE_PHY_CONTROL);
>> +             if (on) {
>> +                     writel(readl(S5P_USBDEVICE_PHY_CONTROL)
>> +                             & ~S5P_USBDEVICE_PHY_ENABLE,
>> +                                     S5P_USBDEVICE_PHY_CONTROL);
>> +             } else {
>> +                     writel(readl(S5P_USBDEVICE_PHY_CONTROL)
>> +                             | S5P_USBDEVICE_PHY_ENABLE,
>> +                                     S5P_USBDEVICE_PHY_CONTROL);
>> +             }
>>       }
>>  }
>> +
>> +/* Switch between HOST and OTG link from PHY_CFG */
>> +void s5p_usb_phy_cfg_sel(struct device *dev, int type)
>> +{
>> +     u32 is_host;
>> +
>> +     is_host = readl(EXYNOS5_USB_CFG);
>> +     writel(type, EXYNOS5_USB_CFG);
>> +
>> +     if (is_host != type)
>> +             dev_dbg(dev, "Changed USB MUX from %s to %s",
>> +                     is_host ? "Host" : "Device", type ? "Host" : "Device");
>> +}
>> diff --git a/arch/arm/plat-samsung/include/plat/usb-phy.h b/arch/arm/plat-samsung/include/plat/usb-phy.h
>> index 7a4a959..a57b8bb 100644
>> --- a/arch/arm/plat-samsung/include/plat/usb-phy.h
>> +++ b/arch/arm/plat-samsung/include/plat/usb-phy.h
>> @@ -19,5 +19,6 @@ enum s5p_usb_phy_type {
>>  extern int s5p_usb_phy_init(struct platform_device *pdev, int type);
>>  extern int s5p_usb_phy_exit(struct platform_device *pdev, int type);
>>  extern void s5p_usb_phy_pmu_isolation(int on, int type);
>> +extern void s5p_usb_phy_cfg_sel(struct device *dev, int type);
>>
>>  #endif /* __PLAT_SAMSUNG_USB_PHY_H */
>> diff --git a/include/linux/platform_data/samsung-usbphy.h b/include/linux/platform_data/samsung-usbphy.h
>> index 364a38a..2b400e0 100644
>> --- a/include/linux/platform_data/samsung-usbphy.h
>> +++ b/include/linux/platform_data/samsung-usbphy.h
>> @@ -25,6 +25,7 @@ enum phy_type {
>>
>>  struct samsung_usbphy_data {
>>       void (*pmu_isolation)(int on, int phy_type);
>> +     void (*phy_cfg_sel)(struct device *dev, int phy_type);
>>       int type;
>>  };
>>
>> --
>> 1.7.6.5
>>
>> --
>> 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
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Thanks & Regards
Vivek
--
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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux