From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH] OMAP3: MMC: Add mux for pins Date: Tue, 16 Jun 2009 08:35:51 -0700 Message-ID: <87y6rs2nu0.fsf@deeprootsystems.com> References: <1244846488-28313-1-git-send-email-vikram.pandita@ti.com> <20090615081242.GE4474@atomide.com> <5C07676A-0A93-48DE-B4C0-5A6709BE34A8@gmail.com> <20090615110431.GO4474@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-qy0-f201.google.com ([209.85.221.201]:64231 "EHLO mail-qy0-f201.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754463AbZFPPfv (ORCPT ); Tue, 16 Jun 2009 11:35:51 -0400 Received: by qyk39 with SMTP id 39so270769qyk.33 for ; Tue, 16 Jun 2009 08:35:53 -0700 (PDT) In-Reply-To: (Vikram Pandita's message of "Tue\, 16 Jun 2009 20\:26\:24 +0530") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Pandita, Vikram" Cc: Tony Lindgren , Hugo Vincent , "linux-omap@vger.kernel.org" , "Chikkature Rajashekar, Madhusudhan" "Pandita, Vikram" writes: >>-----Original Message----- >>From: Tony Lindgren [mailto:tony@atomide.com] >>Sent: Monday, June 15, 2009 6:05 AM >>To: Hugo Vincent >>Cc: Pandita, Vikram; linux-omap@vger.kernel.org; Chikkature Rajashekar, Madhusudhan >>Subject: Re: [PATCH] OMAP3: MMC: Add mux for pins >> >>* Hugo Vincent [090615 03:44]: >>> >>> On 15/06/2009, at 8:12 PM, Tony Lindgren wrote: >>> >>>> * Vikram Pandita [090612 15:43]: >>>>> For OMAP3 add MMC1 MMC2 and MMC3 pin mux >>>>> >>>>> Signed-off-by: Chikkature Rajashekar >>>>> Signed-off-by: Vikram Pandita >>>>> --- >>>>> arch/arm/mach-omap2/devices.c | 33 ++++++++++++++++++++++ >>>>> arch/arm/mach-omap2/mux.c | 49 +++++++++++++++++++++++ >>>>> ++++++++++ >>>>> arch/arm/plat-omap/include/mach/mux.h | 28 +++++++++++++++++++ >>>> >>>> Great, just one issue: All data pins may not be connected, so you >>>> need to look at wires in struct omap_mmc_slot_data to see how many >>>> data pins to mux. >>> >>> There is another issue: different mux-outs are possible for different >>> board layouts; for example, I'm using AE10_3430_MMC3_CMD instead of >>> AC3_3430_MMC3_CMD. I'm not sure what the best way of handling this is, >>> but at a minimum, perhaps make mux setting optional, e.g. add no_mux to >>> struct omap_mmc_slot_data. >> >>Hmm, yeah that's right. I guess only the common pins should be muxed >>in devices.c, and any optional pins should be muxed in the board-*.c >>files. > > Please check this patch set: > [PATCH 1/2] OMAP3: MMC: Pass pin muxing control flag > > I used the nomux flag to do this distinction. > This still doesn't address the problem that when you do mux, you mux all OMAP3 platforms the same way, and that is not correct. Only some of it is common, the others are platform specific. Kevin