[PATCH 09/16] ARM: ux500: Provide regulator support for SMSC911x via Device Tree
Linus Walleij
linus.walleij at linaro.org
Mon May 21 17:42:30 EDT 2012
On Fri, May 18, 2012 at 10:39 AM, Lee Jones <lee.jones at linaro.org> wrote:
> This patch adds a fixed regulator for use by the SMSC911x Ethernet
> chip driver into the db8500 Device Tree. It also references other
> regulators required by the same device.
>
> Signed-off-by: Lee Jones <lee.jones at linaro.org>
> arch/arm/boot/dts/snowball.dts | 13 +++++++++++++
> arch/arm/configs/u8500_defconfig | 1 +
> drivers/mfd/db8500-prcmu.c | 1 +
We usually don't mix code and defconfig changes in the same patch.
Why can't the snowball Kconfig select the fixed regulator instead,
so you get rid of the defconfig change?
> diff --git a/drivers/mfd/db8500-prcmu.c b/drivers/mfd/db8500-prcmu.c
> index d534449..40204e1 100644
> --- a/drivers/mfd/db8500-prcmu.c
> +++ b/drivers/mfd/db8500-prcmu.c
> @@ -2735,6 +2735,7 @@ static struct regulator_consumer_supply db8500_vape_consumers[] = {
> REGULATOR_SUPPLY("vcore", "uart2"),
> REGULATOR_SUPPLY("v-ape", "nmk-ske-keypad.0"),
> REGULATOR_SUPPLY("v-hsi", "ste_hsi.0"),
> + REGULATOR_SUPPLY("vddvario", "smsc911x.0"),
> };
Hm we have discussed previously that these regulator supplies should be passed
from the platform rather than open-coded in the driver.
Well, Device Tree solves that problem long term I guess.
Yours,
Lins Walleij
More information about the linux-arm-kernel
mailing list