From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Avinash.H.M." Subject: Re: [PATCH] omap: hwmod: add syss reset done flags to omap2, omap3 hwmods Date: Fri, 25 Mar 2011 11:56:34 +0530 Message-ID: <20110325062534.GA25480@avinash-laptop> References: <1298542123-3654-1-git-send-email-avinashhm@ti.com> <20110314161609.GA10506@avinash-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from na3sys009aog115.obsmtp.com ([74.125.149.238]:50742 "EHLO na3sys009aog115.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753307Ab1CYG0s (ORCPT ); Fri, 25 Mar 2011 02:26:48 -0400 Received: by gxk8 with SMTP id 8so322240gxk.9 for ; Thu, 24 Mar 2011 23:26:47 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Paul Walmsley Cc: linux-omap@vger.kernel.org, Rajendra Nayak , Benoit Cousson , Kevin Hilman On Thu, Mar 24, 2011 at 11:38:15PM -0600, Paul Walmsley wrote: > Hi Avinash, Hi Paul, > > > > On Thu, 24 Feb 2011, Avinash.H.M wrote: > > > > > > > Some of the omap2, omap3 peripherals support software reset. This > > > > can be done through the softreset bit in sysconfig register. > > > > The reset status can be checked through resetdone bit of > > > > sysstatus register. syss_has_reset_status is added to the hwmod > > > > database of peripherals which have resetdone bit in sysstatus register. > > > > > > > > Cc: Rajendra Nayak > > > > Cc: Paul Walmsley > > > > Cc: Benoit Cousson > > > > Cc: Kevin Hilman > > > > Reviewed-by: Govindraj.R > > > > Signed-off-by: Avinash.H.M > > This patch is causing I2C softreset timeouts in the hwmod layer on OMAP2 > and 3. Could you please take a look at this and figure out what is going > on? I ll start looking at this. I have a 3430 ES3.1 SDP, where i will check the difference in behaviour with this patch. BTW, is there a test case which i should run to produce 'I2C softreset timeouts' ? Can you give more information on how to reproduce the issue ?. br, - avinash > > thanks, > > - Paul