Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756320Ab2HFWAf (ORCPT ); Mon, 6 Aug 2012 18:00:35 -0400 Received: from opensource.wolfsonmicro.com ([80.75.67.52]:48680 "EHLO opensource.wolfsonmicro.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756180Ab2HFWAe (ORCPT ); Mon, 6 Aug 2012 18:00:34 -0400 Date: Mon, 6 Aug 2012 23:00:32 +0100 From: Mark Brown To: Russell King Cc: Haojian Zhuang , sameo@linux.intel.com, rpurdie@rpsys.net, bryan.wu@canonical.com, linux-kernel@vger.kernel.org, Bergmann Arnd Subject: Re: [PATCH 0/5] mfd: replace IORESOURCE_IO by IORESOURCE_MEM Message-ID: <20120806220032.GD26698@opensource.wolfsonmicro.com> References: <1344184373-9670-1-git-send-email-haojian.zhuang@gmail.com> <20120806143016.GK16861@opensource.wolfsonmicro.com> <20120806154619.GO16861@opensource.wolfsonmicro.com> <20120806155805.GR16861@opensource.wolfsonmicro.com> <20120806192209.GA14594@flint.arm.linux.org.uk> <20120806195352.GC16199@opensource.wolfsonmicro.com> <20120806213124.GB14594@flint.arm.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120806213124.GB14594@flint.arm.linux.org.uk> X-Cookie: You will pass away very quickly. User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1640 Lines: 36 On Mon, Aug 06, 2012 at 10:31:24PM +0100, Russell King wrote: > So, the fact that platform devices will get any resource marked > IORESOURCE_IO registered against ioport_resource isn't a problem > then... This is what providing a separate parent to ensure they're in a different tree is there to fix. > Anyway, given that this thread is broken, there's no way for me to find > out what the _original_ issue is that you're talking about. So I'm going > to guess that it's come up because we're out of IORESOURCE bits. No, that's not it. What's happened is that Haojian has posted some patching changing all the _IO resources to _MEM in the Marvell PMIC drivers, I think because you yelled at him for using _IO when he reported that the changes in ioport_resource broke things a few releases ago. Obviously this doesn't achieve a huge amount, it's a misplaced cleanup. > So, if we made this a numeric index, then we have 32 resource types > to deal with, and no need to bugger around with re-using an existing > type for something else. This seems sensible, and I'm sure if that change were made people would be delighed to use new resource types, but like I say nobody who's motivated to do anything here seems to have the time to do anything about it. Whoever looks at this would need to do some detective work, it does seem like there must have been a reason to use a bitmask here... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/