From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Cousson, Benoit" Subject: Re: [PATCH 3/3] ARM: OMAP2+: hwmod code: convert missing clockdomain warnings to debug messages Date: Wed, 19 Sep 2012 17:58:40 -0500 Message-ID: <505A4E20.5040903@ti.com> References: <20120918211146.16484.85893.stgit@dusk.lan> <20120918211251.16484.18401.stgit@dusk.lan> <5058ECAF.7010609@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from arroyo.ext.ti.com ([192.94.94.40]:50576 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752436Ab2ISW6o (ORCPT ); Wed, 19 Sep 2012 18:58:44 -0400 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, linux-arm-kernel@lists.infradead.org Hi Paul, On 9/19/2012 5:48 PM, Paul Walmsley wrote: > Hi Beno=C3=AEt, > > On Tue, 18 Sep 2012, Cousson, Benoit wrote: > >> On 9/18/2012 4:12 PM, Paul Walmsley wrote: >>> The decision was made a few months ago to allow struct omap_hwmod >>> records and struct clk records to omit clockdomain information if t= he >>> clockdomain is not software-controllable. See for example commit >>> 868c157df9721675c19729eed2c96bac6c3f1d01 ("ARM: OMAP2+: hwmod: remo= ve >>> prm_clkdm, cm_clkdm; allow hwmods to have no clockdomain"). >>> >>> So convert an existing pr_warning() to a pr_debug() (regarding miss= ing >>> clockdomains in clocks), and add a pr_debug() for missing hwmod >>> clockdomains. >> >> At some point I sent you a small RFC to still keep the message if ne= ither >> oh->clkdm nor oh->_clk->clkdm was there. I've never had the time to = test it, >> but that should then cover OMAP2, 3 & 4 cases. > > OK let's plan to test and add something like that during the 3.8 time > frame... probably it should also depend on the value of clkdm_control= =2E > It's not clear to me exactly what that conditional should look like..= =2E OK, I was thinking of a more basic thing. If some more values have to b= e=20 checked it might be a little bit more tricky. Let's see that later. Thanks, Benoit -- To unsubscribe from this list: send the line "unsubscribe linux-omap" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: b-cousson@ti.com (Cousson, Benoit) Date: Wed, 19 Sep 2012 17:58:40 -0500 Subject: [PATCH 3/3] ARM: OMAP2+: hwmod code: convert missing clockdomain warnings to debug messages In-Reply-To: References: <20120918211146.16484.85893.stgit@dusk.lan> <20120918211251.16484.18401.stgit@dusk.lan> <5058ECAF.7010609@ti.com> Message-ID: <505A4E20.5040903@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Paul, On 9/19/2012 5:48 PM, Paul Walmsley wrote: > Hi Beno?t, > > On Tue, 18 Sep 2012, Cousson, Benoit wrote: > >> On 9/18/2012 4:12 PM, Paul Walmsley wrote: >>> The decision was made a few months ago to allow struct omap_hwmod >>> records and struct clk records to omit clockdomain information if the >>> clockdomain is not software-controllable. See for example commit >>> 868c157df9721675c19729eed2c96bac6c3f1d01 ("ARM: OMAP2+: hwmod: remove >>> prm_clkdm, cm_clkdm; allow hwmods to have no clockdomain"). >>> >>> So convert an existing pr_warning() to a pr_debug() (regarding missing >>> clockdomains in clocks), and add a pr_debug() for missing hwmod >>> clockdomains. >> >> At some point I sent you a small RFC to still keep the message if neither >> oh->clkdm nor oh->_clk->clkdm was there. I've never had the time to test it, >> but that should then cover OMAP2, 3 & 4 cases. > > OK let's plan to test and add something like that during the 3.8 time > frame... probably it should also depend on the value of clkdm_control. > It's not clear to me exactly what that conditional should look like... OK, I was thinking of a more basic thing. If some more values have to be checked it might be a little bit more tricky. Let's see that later. Thanks, Benoit