From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tero Kristo Subject: Re: [PATCHv4 4/8] ARM: OMAP4: hwmod: flag hwmods/modules supporting module level context status Date: Tue, 24 Apr 2012 17:46:55 +0300 Message-ID: <1335278815.2149.93.camel@sokoban> References: <1334913591-26312-1-git-send-email-t-kristo@ti.com> <1334913591-26312-5-git-send-email-t-kristo@ti.com> <4F957ACD.2040103@ti.com> Reply-To: Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: Received: from comal.ext.ti.com ([198.47.26.152]:59556 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754710Ab2DXOrB (ORCPT ); Tue, 24 Apr 2012 10:47:01 -0400 In-Reply-To: <4F957ACD.2040103@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Jon Hunter Cc: linux-omap@vger.kernel.org, khilman@ti.com, paul@pwsan.com, Rajendra Nayak , linux-arm-kernel@lists.infradead.org On Mon, 2012-04-23 at 10:52 -0500, Jon Hunter wrote: > Hi Tero, > > On 04/20/2012 04:19 AM, Tero Kristo wrote: > > From: Rajendra Nayak > > > > On OMAP4 most modules/hwmods support module level context status. On > > OMAP3 and earlier, we relyed on the power domain level context status. > > Identify all such modules using a 'HWMOD_CONTEXT_REG' flag, all such > > hwmods already have a valid 'context_offs' populated in .prcm structure. > > Is it necessary to add another flag? Can't we just check if context_offs > is non-zero? Would save adding a lot more lines to an already large file > :-) Actually one of the older versions of this patch was just checking against a non-zero value, but it was decided to be changed as potentially the context_offs can be zero even if it is a valid offset. -Tero