linux-kernel - Re: [PATCH 5/5] usb: musb: dsps: Enable sw babble control for newer silicon

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>] [day] [month] [year] [list]

Message-ID: <536A4138.80603@ti.com>
Date:	Wed, 7 May 2014 19:50:40 +0530
From:	George Cherian <george.cherian@...com>
To:	<linux-kernel@...r.kernel.org>, <linux-usb@...r.kernel.org>
CC:	<gregkh@...uxfoundation.org>, <balbi@...com>, <zonque@...il.com>
Subject: Re: [PATCH 5/5] usb: musb: dsps: Enable sw babble control for newer
 silicon

Hi,

Please ignore this patch.
It actually doesnot call sw_babble_ control() at all.
It should have.
Will post a fixed version.

On 5/7/2014 7:39 PM, George Cherian wrote:
> Find whether we are running on newer silicon. The babble control
> register reads 0x4 by default in newer silicon as opposed to 0
> in old versions of AM335x. Based on this enable the sw babble
> control logic.
>
> Signed-off-by: George Cherian <george.cherian@...com>
> ---
>   drivers/usb/musb/musb_dsps.c | 28 +++++++++++++++++++++++++++-
>   1 file changed, 27 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/musb/musb_dsps.c b/drivers/usb/musb/musb_dsps.c
> index 6647e7e..da719d7 100644
> --- a/drivers/usb/musb/musb_dsps.c
> +++ b/drivers/usb/musb/musb_dsps.c
> @@ -136,6 +136,7 @@ struct dsps_glue {
>   	const struct dsps_musb_wrapper *wrp; /* wrapper register offsets */
>   	struct timer_list timer;	/* otg_workaround timer */
>   	unsigned long last_timer;    /* last timer data for each instance */
> +	bool sw_babble_enabled;
>   
>   	struct dsps_context context;
>   	struct debugfs_regset32 regset;
> @@ -469,6 +470,16 @@ static int dsps_musb_init(struct musb *musb)
>   	val &= ~(1 << wrp->otg_disable);
>   	dsps_writel(musb->ctrl_base, wrp->phy_utmi, val);
>   
> +	/*
> +	 *  Check whether the dsps version has babble control enabled.
> +	 * In latest silicon revision the babble control logic is enabled.
> +	 * If MUSB_BABBLE_CTL returns 0x4 then we have the babble control
> +	 * logic enabled.
> +	 */
> +	val = dsps_readb(musb->mregs, MUSB_BABBLE_CTL);
> +	if (val == MUSB_BABBLE_RCV_DISABLE)
> +		glue->sw_babble_enabled = true;
> +
>   	ret = dsps_musb_dbg_init(musb, glue);
>   	if (ret)
>   		return ret;
> @@ -591,6 +602,7 @@ static int dsps_musb_reset(struct musb *musb)
>   	struct device *dev = musb->controller;
>   	struct dsps_glue *glue = dev_get_drvdata(dev->parent);
>   	const struct dsps_musb_wrapper *wrp = glue->wrp;
> +	int session_restart = 0;
>   
>   	dsps_writel(musb->ctrl_base, wrp->control, (1 << wrp->reset));
>   	usleep_range(100, 200);
> @@ -598,7 +610,21 @@ static int dsps_musb_reset(struct musb *musb)
>   	usleep_range(100, 200);
>   	usb_phy_init(musb->xceiv);
>   
> -	return 1;
> +	/*
> +	 * In case of new silicon version babble condition can be recovered
> +	 * without resetting the MUSB. But for older silicon versions, MUSB
> +	 * reset is needed
> +	 */
> +	if (session_restart || !glue->sw_babble_enabled) {
> +		dsps_writel(musb->ctrl_base, wrp->control, (1 << wrp->reset));
> +		usleep_range(100, 200);
> +		usb_phy_shutdown(musb->xceiv);
> +		usleep_range(100, 200);
> +		usb_phy_init(musb->xceiv);
> +		session_restart = 1;
> +	}
> +
> +	return session_restart;
>   }
>   
>   static struct musb_platform_ops dsps_ops = {


-- 
-George

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