Hi,
On Tue, Jun 21, 2011 at 12:33:46PM +0530, p.paneri@xxxxxxxxxxx wrote:
> From: Praveen Paneri <p.paneri@xxxxxxxxxxx>
>
> A generic method to initialize and exit OTG PHY which can be
> used for all the samsung SoCs.
> OTG platdata structure added in platform to pass required
> platform specific functions and data to the driver.
>
> Signed-off-by: Praveen Paneri <p.paneri@xxxxxxxxxxx>
> ---
> arch/arm/mach-s5p64x0/Makefile | 1 +
> arch/arm/mach-s5p64x0/include/mach/map.h | 4 +
> arch/arm/mach-s5p64x0/setup-otg-phy.c | 89 ++++++++++++++++++++++++++++++
> arch/arm/plat-s5p/include/plat/otg.h | 29 ++++++++++
> 4 files changed, 123 insertions(+), 0 deletions(-)
> create mode 100644 arch/arm/mach-s5p64x0/setup-otg-phy.c
> create mode 100644 arch/arm/plat-s5p/include/plat/otg.h
>
> diff --git a/arch/arm/mach-s5p64x0/Makefile b/arch/arm/mach-s5p64x0/Makefile
> index ae6bf6f..611fb3a 100644
> --- a/arch/arm/mach-s5p64x0/Makefile
> +++ b/arch/arm/mach-s5p64x0/Makefile
> @@ -28,3 +28,4 @@ obj-y += dev-audio.o
> obj-$(CONFIG_S3C64XX_DEV_SPI) += dev-spi.o
>
> obj-$(CONFIG_S5P64X0_SETUP_I2C1) += setup-i2c1.o
> +obj-$(CONFIG_S3C_DEV_DWC_OTG) += setup-otg-phy.o
> diff --git a/arch/arm/mach-s5p64x0/include/mach/map.h b/arch/arm/mach-s5p64x0/include/mach/map.h
> index 95c9125..717c279 100644
> --- a/arch/arm/mach-s5p64x0/include/mach/map.h
> +++ b/arch/arm/mach-s5p64x0/include/mach/map.h
> @@ -44,6 +44,8 @@
> #define S5P64X0_PA_SPI1 0xEC500000
>
> #define S5P64X0_PA_HSOTG 0xED100000
> +#define S5P64X0_PA_USB_HSPHY 0xED200000
> +#define S5P64X0_VA_USB_HSPHY S3C_ADDR_CPU(0x00100000)
>
> #define S5P64X0_PA_HSMMC(x) (0xED800000 + ((x) * 0x100000))
>
> @@ -71,6 +73,8 @@
> #define S5P_PA_TIMER S5P64X0_PA_TIMER
>
> #define SAMSUNG_PA_ADC S5P64X0_PA_ADC
> +#define S3C_PA_USB_HSOTG S5P64X0_PA_HSOTG
> +#define S3C_VA_USB_HSPHY S5P64X0_VA_USB_HSPHY
>
> /* UART */
>
> diff --git a/arch/arm/mach-s5p64x0/setup-otg-phy.c b/arch/arm/mach-s5p64x0/setup-otg-phy.c
drivers should not live in arch/arm/*, why don't you move this to
drivers/usb/otg where the PHYs are staying right now ?
> new file mode 100644
> index 0000000..c351554
> --- /dev/null
> +++ b/arch/arm/mach-s5p64x0/setup-otg-phy.c
> @@ -0,0 +1,89 @@
> +/*
> + * Copyright (C) 2011 Samsung Electronics Co.Ltd
> + * Author: Praveen Paneri <p.paneri@xxxxxxxxxxx>
> + * based on arch/arm/mach-exynos4/setup-usb-phy.c
> + * written by Joonyoung Shim <jy0922.shim@xxxxxxxxxxx>
> + *
> + * This program is free software; you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License as published by the
> + * Free Software Foundation; either version 2 of the License, or (at your
> + * option) any later version.
> + *
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/delay.h>
> +#include <linux/err.h>
> +#include <linux/io.h>
> +#include <linux/platform_device.h>
> +#include <mach/regs-clock.h>
> +#include <mach/gpio.h>
> +#include <plat/regs-usb-hsotg-phy.h>
> +#include <plat/usb-phy.h>
> +
> +struct clk *otg_clk;
> +static int s5p64x0_otg_phy_init(struct platform_device *pdev)
> +{
> + int err;
> +
> + otg_clk = clk_get(&pdev->dev, "otg");
> + if (IS_ERR(otg_clk)) {
> + dev_err(&pdev->dev, "Failed to get otg clock\n");
> + return PTR_ERR(otg_clk);
> + }
> +
> + err = clk_enable(otg_clk);
> + if (err) {
> + clk_put(otg_clk);
> + return err;
> + }
> +
> + if (gpio_is_valid(S5P6440_GPN(1))) {
> + err = gpio_request(S5P6440_GPN(1), "GPN");
> + if (err)
> + printk(KERN_ERR "failed to request GPN1\n");
> + gpio_direction_output(S5P6440_GPN(1), 1);
> + }
> +
> + writel(readl(S5P64X0_OTHERS)&~S5P64X0_OTHERS_USB_SIG_MASK,
> + S5P64X0_OTHERS);
> + writel(0x0, S3C_PHYPWR); /* Power up */
> + writel(S3C_PHYCLK_CLKSEL_12M, S3C_PHYCLK);
> + writel(S3C_RSTCON_PHY, S3C_RSTCON);
> +
> + udelay(50);
> + writel(0x0, S3C_RSTCON);
> + udelay(50);
> +
> + return 0;
> +}
> +
> +static int s5p64x0_otg_phy_exit(struct platform_device *pdev)
> +{
> + writel(readl(S3C_PHYPWR)|(0x1F<<1), S3C_PHYPWR);
> + writel(readl(S5P64X0_OTHERS)&~S5P64X0_OTHERS_USB_SIG_MASK,
> + S5P64X0_OTHERS);
> +
> + gpio_free(S5P6440_GPN(1));
> +
> + clk_disable(otg_clk);
> + clk_put(otg_clk);
> +
> + return 0;
> +}
> +
> +int s5p_usb_phy_init(struct platform_device *pdev, int type)
> +{
> + if (type == S5P_USB_PHY_DEVICE)
> + return s5p64x0_otg_phy_init(pdev);
> +
> + return -EINVAL;
> +}
> +
> +int s5p_usb_phy_exit(struct platform_device *pdev, int type)
> +{
> + if (type == S5P_USB_PHY_DEVICE)
> + return s5p64x0_otg_phy_exit(pdev);
> +
> + return -EINVAL;
> +}
no way to make this a real driver instead of set a exported functions ??
--
balbi
Attachment:
signature.asc
Description: Digital signature
