From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752666AbcEJJ4q (ORCPT ); Tue, 10 May 2016 05:56:46 -0400 Received: from mga04.intel.com ([192.55.52.120]:28880 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751041AbcEJJ4n (ORCPT ); Tue, 10 May 2016 05:56:43 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,604,1455004800"; d="asc'?scan'208";a="100408481" From: Felipe Balbi To: Roger Quadros Cc: tony@atomide.com, Joao.Pinto@synopsys.com, sergei.shtylyov@cogentembedded.com, peter.chen@freescale.com, jun.li@freescale.com, grygorii.strashko@ti.com, yoshihiro.shimoda.uh@renesas.com, nsekhar@ti.com, b-liu@ti.com, linux-usb@vger.kernel.org, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org, Roger Quadros Subject: Re: [PATCH v7 3/5] usb: dwc3: omap: Don't set POWERPRESENT In-Reply-To: <1462873919-20532-4-git-send-email-rogerq@ti.com> References: <1462873919-20532-1-git-send-email-rogerq@ti.com> <1462873919-20532-4-git-send-email-rogerq@ti.com> User-Agent: Notmuch/0.22+11~g124a67e (http://notmuchmail.org) Emacs/25.0.93.2 (x86_64-pc-linux-gnu) Date: Tue, 10 May 2016 12:54:11 +0300 Message-ID: <87futqi6gs.fsf@linux.intel.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Hi, Roger Quadros writes: > TRM [1] recommends that POWERPRESENT bit must not be > set and left at it's default value of 0. > > [1] OMAP542x TRM - http://www.ti.com/lit/pdf/swpu249 > Section 23.11.4.5.1 Mailbox VBUS/ID Management > > "Because PIPE powerpresent has a different meaning in host and in device = mode, > and because of the redundancy with the UTMI signals, the controller ORes > together the appropriate PIPE and UTMI inputs to create its internal > VBUS status. For that reason, it is recommended to leave field > USBOTGSS_UTMI_OTG_STATUS[9] POWERPRESENT at its default value (=3D0), and= only to > fill in the USB2 VBUS status fields in the same register." > > Signed-off-by: Roger Quadros to make sure we avoid regressions, do you mind sharing on which platforms you tested this patch ? =2D-=20 balbi --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJXMa/DAAoJEIaOsuA1yqREk+kP/ReHHndcqN7JTK34VQsUMQmM 4ODyqFoGNz/PQ21qXybmWI4tH11Z5gR0ma1sKpITIWCK+dqMHXrw+wL188RYYn3F 0KE9puwf7eW63Flz8w6Q4R1MDe6gDRU2jCiJfPyKPrrhYpD9qPT0QlCPNGG0Y34M 0M7/dCVOPkZ0oVoIuY1I3K30ebIDFINDmBUT4/cToD4UzMM2JQyCLlWNKbsmYc3z S5Bu3ila5BpOe8oViHCgCzfoK+66bDkWFMCX5+l+TmsCmEPcx69badZvOcUSoDw8 2MyTE6xMgxfjyCBsuENZg7P6f0iXk8ddAh/KCycl59ckGAJO0xVxXhjBD74y3KTE zbh7bANYiagXqxAUi2Jw7+q0kuoY16KFBU9JqDjnRZIXhZO/ONd5udAxCeFNH5t/ q4kWvINji2PJMjuE/tT80QPRqxr+Oyyn5Etlbc+XTvMUi8wE+VaVPtIsSqYZRnrH dz6C1bgXduols6Jay7Zg+zDRP39vk1N5AvRKlDIu9bqui42R1z295YWJ64vgxfI1 6S7OKArunEjExwpFV9/TXJ3fNUjRgewLkhFwA2S6z8rebuy5UeT0f0xE7/iKRTv1 BxBZ8hUJ3hprYxMA/GFhm1Iluvf22/Ttxu4vB9aOT6bo9mHiBlIHN7aUapoT/Evy yswP64hdK18PXPZxoKRF =39Jm -----END PGP SIGNATURE----- --=-=-=--