From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Date: Sun, 21 Aug 2011 00:03:11 +0000 Subject: Re: [PATCH 4/4] ARM: shmobile: ag5evm, ap4: Make use of irq index Message-Id: <20110821000308.GA27615@verge.net.au> List-Id: References: <1313716221-20136-1-git-send-email-horms@verge.net.au> <1313716221-20136-5-git-send-email-horms@verge.net.au> <20110819052004.GB17548@verge.net.au> <20110819063857.GC16419@verge.net.au> <20110819071720.GD16419@verge.net.au> In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Guennadi Liakhovetski Cc: Magnus Damm , linux-mmc@vger.kernel.org, linux-sh@vger.kernel.org, Chris Ball , Paul Mundt On Fri, Aug 19, 2011 at 09:52:17AM +0200, Guennadi Liakhovetski wrote: > On Fri, 19 Aug 2011, Simon Horman wrote: > > > On Fri, Aug 19, 2011 at 03:51:49PM +0900, Magnus Damm wrote: > > > On Fri, Aug 19, 2011 at 3:39 PM, Simon Horman wrote: > > [snip] > > > > > As we are already on the slippery slope of allowing combinations > > > > other than 1 (legacy) or 3 (specific) IRQ sources I plan to implement > > > > a variant of your flag idea. The variation being to use names instead > > > > because a) that allows the use of platform_get_irq_byname() and b) > > > > the flags bits seem to be full and not driver-specific. > > > > > > Great, platform_get_irq_byname() seems like a perfect match. > > > > > > May I suggest "hotplug", "data" and "sdio" as names? I don't care very > > > much about names except keeping them short and precise to prevent > > > errors that can only be caught during runtime. > > > > Earlier on in the life of this series Guennadi suggested > > the names "card_detect", "sdcard" and "sdio". While I am > > not particularly attached to those names the do seem > > reasonable and are already used consistently by this series. > > So I would prefer to use those names. > > Just one more thing I forgot to mention in the previous mail: using names > also makes the transition simple: first patch all platforms with names (at > least those with multiple IRQs, if your legacy fallback implementation > will accept unnamed IRQs), and then patch sh_mobile_sdhi.c Yes, I agree that is a nice feature of using names.