From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH] OMAP2+: OPP: allow OPP enumeration to continue if device is not present Date: Mon, 19 Mar 2012 14:48:31 -0700 Message-ID: <8739945kk0.fsf@ti.com> References: <1331914749-23171-1-git-send-email-nm@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from na3sys009aog107.obsmtp.com ([74.125.149.197]:41112 "EHLO na3sys009aog107.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753631Ab2CSVsN (ORCPT ); Mon, 19 Mar 2012 17:48:13 -0400 Received: by mail-pz0-f42.google.com with SMTP id g27so14344754dan.1 for ; Mon, 19 Mar 2012 14:48:11 -0700 (PDT) In-Reply-To: <1331914749-23171-1-git-send-email-nm@ti.com> (Nishanth Menon's message of "Fri, 16 Mar 2012 11:19:09 -0500") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Nishanth Menon Cc: l-o , l-a , Tony , Steve Sakoman , Maximilian Schwerin Nishanth Menon writes: > On platforms such as OMAP3, certain variants may not have IVA, SGX > or some specific component. We currently have a check to aid fixing > wrong population of OPP entries for issues such as typos. This however > causes a conflict with valid requirement where the SoC variant does > not actually have the module present. > > So, reduce the severity of the print to a debug statement and skip > registering that specific OPP, but continue down the list. > > Reported-by: Steve Sakoman > Reported-by: Maximilian Schwerin > Acked-by: Steve Sakoman > Tested-by: Maximilian Schwerin > Signed-off-by: Nishanth Menon Thanks, queuing as a fix for for v3.4-rc (branch: for_3.4/fixes/pm) Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@ti.com (Kevin Hilman) Date: Mon, 19 Mar 2012 14:48:31 -0700 Subject: [PATCH] OMAP2+: OPP: allow OPP enumeration to continue if device is not present In-Reply-To: <1331914749-23171-1-git-send-email-nm@ti.com> (Nishanth Menon's message of "Fri, 16 Mar 2012 11:19:09 -0500") References: <1331914749-23171-1-git-send-email-nm@ti.com> Message-ID: <8739945kk0.fsf@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Nishanth Menon writes: > On platforms such as OMAP3, certain variants may not have IVA, SGX > or some specific component. We currently have a check to aid fixing > wrong population of OPP entries for issues such as typos. This however > causes a conflict with valid requirement where the SoC variant does > not actually have the module present. > > So, reduce the severity of the print to a debug statement and skip > registering that specific OPP, but continue down the list. > > Reported-by: Steve Sakoman > Reported-by: Maximilian Schwerin > Acked-by: Steve Sakoman > Tested-by: Maximilian Schwerin > Signed-off-by: Nishanth Menon Thanks, queuing as a fix for for v3.4-rc (branch: for_3.4/fixes/pm) Kevin