From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752984Ab3LBNE0 (ORCPT ); Mon, 2 Dec 2013 08:04:26 -0500 Received: from www.linutronix.de ([62.245.132.108]:44364 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751087Ab3LBNEZ (ORCPT ); Mon, 2 Dec 2013 08:04:25 -0500 Message-ID: <529C854F.3010101@linutronix.de> Date: Mon, 02 Dec 2013 14:04:15 +0100 From: Sebastian Andrzej Siewior User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20131103 Icedove/17.0.10 MIME-Version: 1.0 To: Roger Quadros CC: Michael Trimarchi , Felipe Balbi , lee.jones@linaro.org, sameo@linux.intel.com, tomi.valkeinen@ti.com, Stefan Roese , ljkenny.mailinglists@gmail.com, Linux OMAP Mailing List , USB list , linux-kernel@vger.kernel.org, Keshava Munegowda Subject: Re: [PATCH 1/1] mfd: omap-usb-host: Fix USB device detection problems on OMAP4 Panda References: <1385730118-26402-1-git-send-email-rogerq@ti.com> <1385730118-26402-2-git-send-email-rogerq@ti.com> <529C55DF.3000808@ti.com> <529C76AF.10106@linutronix.de> <529C7922.5000601@ti.com> In-Reply-To: <529C7922.5000601@ti.com> X-Enigmail-Version: 1.6 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/02/2013 01:12 PM, Roger Quadros wrote: > You are right. The musb hwmod data doesn't have the HWMOD_INIT_NO_RESET > flag, so hwmod should reset it during boot. > > Maybe I too should just remove HWMOD_INIT_NO_RESET flag from the USB Host > hwmod data, then we don't need to change this driver at all. > > Benoit, Tony any comments? Almost two years ago in commit de231388 ("ARM: OMAP: USB: EHCI and OHCI hwmod structures for OMAP3") the "don't reset flag" has been added by Keshava. It refers to "Errata Id:i660" why it is required. Once you figured what why it has been added you could have an idea if it is okay to remove it and if the reset you do here might lead to it (I dunno). > > cheers, > -roger Sebastian