linux-kernel - Re: [PATCH] mfd: omap: Restore TLL initialization

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]

Message-ID: <BANLkTikFFUREhKyPaZrhHZje0KeaA3ef8Q@mail.gmail.com>
Date:	Fri, 24 Jun 2011 11:52:25 +0530
From:	"Munegowda, Keshava" <keshava_mgowda@...com>
To:	Jassi Brar <jassisinghbrar@...il.com>
Cc:	linux-kernel@...r.kernel.org, ti@...ts.linaro.org,
	parthab@...ia.ti.com, sameo@...ux.intel.com, gadiyar@...com,
	balbi@...com, linux-omap@...r.kernel.org,
	linux-usb@...r.kernel.org, Jassi Brar <jaswinder.singh@...aro.org>
Subject: Re: [PATCH] mfd: omap: Restore TLL initialization

On Fri, Jun 24, 2011 at 2:31 AM, Jassi Brar <jassisinghbrar@...il.com> wrote:
> The commit
>  7e6502d577106fb5b202bbaac64c5f1b065 'mfd: Add omap-usbhs runtime PM support'
> besides moving to RPM, removes necessary TLL initialization as well.
>
> Restore the TLL initialization, without which device detection fails.
>
> Signed-off-by: Jassi Brar <jaswinder.singh@...aro.org>
> ---
>  drivers/mfd/omap-usb-host.c |   42 ++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 42 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c
> index 8552195..12ca77b 100644
> --- a/drivers/mfd/omap-usb-host.c
> +++ b/drivers/mfd/omap-usb-host.c
> @@ -640,6 +640,7 @@ static int usbhs_enable(struct device *dev)
>        struct usbhs_omap_platform_data *pdata = &omap->platdata;
>        unsigned long                   flags = 0;
>        int                             ret = 0;
> +       unsigned long                   timeout;
>        unsigned                        reg;
>
>        dev_dbg(dev, "starting TI HSUSB Controller\n");
> @@ -676,6 +677,47 @@ static int usbhs_enable(struct device *dev)
>        omap->usbhs_rev = usbhs_read(omap->uhh_base, OMAP_UHH_REVISION);
>        dev_dbg(dev, "OMAP UHH_REVISION 0x%x\n", omap->usbhs_rev);
>
> +       /* perform TLL soft reset, and wait until reset is complete */
> +       usbhs_write(omap->tll_base, OMAP_USBTLL_SYSCONFIG,
> +                       OMAP_USBTLL_SYSCONFIG_SOFTRESET);
> +
> +       /* Wait for TLL reset to complete */
> +       timeout = jiffies + msecs_to_jiffies(1000);
> +       while (!(usbhs_read(omap->tll_base, OMAP_USBTLL_SYSSTATUS)
> +                       & OMAP_USBTLL_SYSSTATUS_RESETDONE)) {
> +               cpu_relax();
> +
> +               if (time_after(jiffies, timeout)) {
> +                       dev_dbg(dev, "operation timed out\n");
> +                       ret = -EINVAL;
> +                       goto err_tll;
> +               }
> +       }
> +       dev_dbg(dev, "TLL RESET DONE\n");
> +
> +       /* (1<<3) = no idle mode only for initial debugging */
> +       usbhs_write(omap->tll_base, OMAP_USBTLL_SYSCONFIG,
> +                       OMAP_USBTLL_SYSCONFIG_ENAWAKEUP |
> +                       OMAP_USBTLL_SYSCONFIG_SIDLEMODE |
> +                       OMAP_USBTLL_SYSCONFIG_AUTOIDLE);
> +
> +       /* Put UHH in NoIdle/NoStandby mode */
> +       reg = usbhs_read(omap->uhh_base, OMAP_UHH_SYSCONFIG);
> +       if (is_omap_usbhs_rev1(omap)) {
> +               reg |= (OMAP_UHH_SYSCONFIG_ENAWAKEUP
> +                               | OMAP_UHH_SYSCONFIG_SIDLEMODE
> +                               | OMAP_UHH_SYSCONFIG_CACTIVITY
> +                               | OMAP_UHH_SYSCONFIG_MIDLEMODE);
> +               reg &= ~OMAP_UHH_SYSCONFIG_AUTOIDLE;
> +       } else if (is_omap_usbhs_rev2(omap)) {
> +               reg &= ~OMAP4_UHH_SYSCONFIG_IDLEMODE_CLEAR;
> +               reg |= OMAP4_UHH_SYSCONFIG_NOIDLE;
> +               reg &= ~OMAP4_UHH_SYSCONFIG_STDBYMODE_CLEAR;
> +               reg |= OMAP4_UHH_SYSCONFIG_NOSTDBY;
> +       }
> +
> +       usbhs_write(omap->uhh_base, OMAP_UHH_SYSCONFIG, reg);
> +
>        reg = usbhs_read(omap->uhh_base, OMAP_UHH_HOSTCONFIG);
>        /* setup ULPI bypass and burst configurations */
>        reg |= (OMAP_UHH_HOSTCONFIG_INCR4_BURST_EN
> --
> 1.7.4.1

if you move to RPM, you should'tn access these configuration/reset
registers in your driver.
moreover, the patch 'mfd: Add omap-usbhs runtime PM support' is
already reverted.

keshava
--
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/