[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