From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ranjith Lohithakshan Subject: Re: [PATCH] omap: Remove DEBUGFS dependency for mux name checking Date: Wed, 17 Feb 2010 22:11:50 +0530 Message-ID: <4B7C1C4E.2090709@ti.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from bear.ext.ti.com ([192.94.94.41]:38823 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751630Ab0BQQmF (ORCPT ); Wed, 17 Feb 2010 11:42:05 -0500 In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Premi, Sanjeev" Cc: "linux-omap@vger.kernel.org" , "tony@atomide.com" Sanjeev, On Wed, 17-Feb-10 3:46 PM +0530, Premi, Sanjeev wrote: >> -----Original Message----- >> From: linux-omap-owner@vger.kernel.org >> [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of >> Lohithakshan, Ranjith >> Sent: Wednesday, February 17, 2010 11:48 AM >> To: linux-omap@vger.kernel.org >> Cc: tony@atomide.com; Lohithakshan, Ranjith >> Subject: [PATCH] omap: Remove DEBUGFS dependency for mux name checking >> >> The check for a valid mux name should be performed regardless >> of whether >> DEBUGFS is enabled or not. >> >> This fixes the issue currently seen with boards not booting up >> if DEBUGFS is not enabled in defconfig. >> >> Signed-off-by: Ranjith Lohithakshan >> --- >> arch/arm/mach-omap2/mux.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> > > Ranjith, > > For better readability, I suggest the following: > > diff --git a/arch/arm/mach-omap2/mux.c b/arch/arm/mach-omap2/mux.c > index 19001dd..7c0c4ad 100644 > --- a/arch/arm/mach-omap2/mux.c > +++ b/arch/arm/mach-omap2/mux.c > @@ -960,16 +960,14 @@ static void __init omap_mux_init_list(struct omap_mux *sup > while (superset->reg_offset != OMAP_MUX_TERMINATOR) { > struct omap_mux *entry; > > -#ifndef CONFIG_OMAP_MUX > - /* Skip pins that are not muxed as GPIO by bootloader */ > - if (!OMAP_MODE_GPIO(omap_mux_read(superset->reg_offset))) { > +#ifdef CONFIG_OMAP_MUX > + if (!superset->muxnames || !superset->muxnames[0]) { > superset++; > continue; > } > -#endif > - > -#if defined(CONFIG_OMAP_MUX) && defined(CONFIG_DEBUG_FS) > - if (!superset->muxnames || !superset->muxnames[0]) { > +#else > + /* Skip pins that are not muxed as GPIO by bootloader */ > + if (!OMAP_MODE_GPIO(omap_mux_read(superset->reg_offset))) { > superset++; > continue; Sounds good. I will issue an updated patch shortly. - Ranjith > Best regards, > Sanjeev > > >> diff --git a/arch/arm/mach-omap2/mux.c b/arch/arm/mach-omap2/mux.c >> index 986077f..1a6ff8f 100644 >> --- a/arch/arm/mach-omap2/mux.c >> +++ b/arch/arm/mach-omap2/mux.c >> @@ -969,7 +969,7 @@ static void __init >> omap_mux_init_list(struct omap_mux *superset) >> } >> #endif >> >> -#if defined(CONFIG_OMAP_MUX) && defined(CONFIG_DEBUG_FS) >> +#ifdef CONFIG_OMAP_MUX >> if (!superset->muxnames || !superset->muxnames[0]) { >> superset++; >> continue; >> -- >> 1.6.2.4 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe >> linux-omap" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html