From mboxrd@z Thu Jan 1 00:00:00 1970 From: broonie@opensource.wolfsonmicro.com (Mark Brown) Date: Wed, 30 May 2012 11:03:03 +0100 Subject: [PATCH 08/22] regulator: Change ab8500 match names to reflect Device Tree In-Reply-To: References: <1338353260-10097-1-git-send-email-lee.jones@linaro.org> <1338353260-10097-9-git-send-email-lee.jones@linaro.org> Message-ID: <20120530100302.GD9947@opensource.wolfsonmicro.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, May 30, 2012 at 02:58:44PM +0800, Linus Walleij wrote: > On Wed, May 30, 2012 at 12:47 PM, Lee Jones wrote: > > + ? ? ? { .name = "ab8500_ldo_ana", ? ? .driver_data = (void *) AB8500_LDO_ANA, }, > So why is the device tree going to use these lowercase names etc? > I guess there is a rationale... If we are changing the names then not including the ab8500 bit would seem more idiomatic - we've already worked out which chip this is by the time we're looking here. > Anyway: the name is used in more parts of the file, so please change > it everywhere if you change it. It's only used for a few debug prints AFAICT? -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: