Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1169337AbdDXLu5 (ORCPT ); Mon, 24 Apr 2017 07:50:57 -0400 Received: from mail-wr0-f175.google.com ([209.85.128.175]:34878 "EHLO mail-wr0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1168707AbdDXLus (ORCPT ); Mon, 24 Apr 2017 07:50:48 -0400 Date: Mon, 24 Apr 2017 12:50:44 +0100 From: Lee Jones To: Richard Fitzgerald Cc: broonie@kernel.org, kgene@kernel.org, krzk@kernel.org, linux-kernel@vger.kernel.org, patches@opensource.wolfsonmicro.com, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org Subject: Re: [PATCH 5/7] regulator: arizona-ldo1: Move pdata into a separate structure Message-ID: <20170424115044.rejrwmi57uifxo7j@dell> References: <1492512234-19210-1-git-send-email-rf@opensource.wolfsonmicro.com> <1492512234-19210-7-git-send-email-rf@opensource.wolfsonmicro.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1492512234-19210-7-git-send-email-rf@opensource.wolfsonmicro.com> User-Agent: Mutt/1.6.2-neo (2016-08-21) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7397 Lines: 224 On Tue, 18 Apr 2017, Richard Fitzgerald wrote: > In preparation for sharing this driver with Madera, move the pdata > for the LDO1 regulator out of struct arizona_pdata into a dedicated > pdata struct for this driver. As a result the code in > arizona_ldo1_of_get_pdata() can be made independent of struct arizona. > > This patch also updates the definition of struct arizona_pdata and > the use of this pdata in mach-crag6410-module.c > > Signed-off-by: Richard Fitzgerald > --- > arch/arm/mach-s3c64xx/mach-crag6410-module.c | 8 ++++-- > drivers/regulator/arizona-ldo1.c | 39 +++++++++++++++------------- > include/linux/mfd/arizona/pdata.h | 4 +-- Acked-by: Lee Jones > include/linux/regulator/arizona-ldo1.h | 24 +++++++++++++++++ > 4 files changed, 53 insertions(+), 22 deletions(-) > create mode 100644 include/linux/regulator/arizona-ldo1.h > > diff --git a/arch/arm/mach-s3c64xx/mach-crag6410-module.c b/arch/arm/mach-s3c64xx/mach-crag6410-module.c > index ccc3ab8..ea5f216 100644 > --- a/arch/arm/mach-s3c64xx/mach-crag6410-module.c > +++ b/arch/arm/mach-s3c64xx/mach-crag6410-module.c > @@ -209,7 +209,9 @@ > }; > > static struct arizona_pdata wm5102_reva_pdata = { > - .ldoena = S3C64XX_GPN(7), > + .ldo1 = { > + .ldoena = S3C64XX_GPN(7), > + }, > .gpio_base = CODEC_GPIO_BASE, > .irq_flags = IRQF_TRIGGER_HIGH, > .micd_pol_gpio = CODEC_GPIO_BASE + 4, > @@ -239,7 +241,9 @@ > }; > > static struct arizona_pdata wm5102_pdata = { > - .ldoena = S3C64XX_GPN(7), > + .ldo1 = { > + .ldoena = S3C64XX_GPN(7), > + }, > .gpio_base = CODEC_GPIO_BASE, > .irq_flags = IRQF_TRIGGER_HIGH, > .micd_pol_gpio = CODEC_GPIO_BASE + 2, > diff --git a/drivers/regulator/arizona-ldo1.c b/drivers/regulator/arizona-ldo1.c > index b726fa1..f5bc75a 100644 > --- a/drivers/regulator/arizona-ldo1.c > +++ b/drivers/regulator/arizona-ldo1.c > @@ -25,6 +25,8 @@ > #include > #include > > +#include > + > #include > #include > #include > @@ -186,20 +188,19 @@ static int arizona_ldo1_hc_get_voltage_sel(struct regulator_dev *rdev) > .num_consumer_supplies = 1, > }; > > -static int arizona_ldo1_of_get_pdata(struct device *dev, > - struct arizona *arizona, > +static int arizona_ldo1_of_get_pdata(struct arizona_ldo1_pdata *pdata, > struct regulator_config *config, > - const struct regulator_desc *desc) > + const struct regulator_desc *desc, > + bool *external_dcvdd) > { > - struct arizona_pdata *pdata = &arizona->pdata; > struct arizona_ldo1 *ldo1 = config->driver_data; > - struct device_node *np = arizona->dev->of_node; > + struct device_node *np = config->dev->of_node; > struct device_node *init_node, *dcvdd_node; > struct regulator_init_data *init_data; > > pdata->ldoena = of_get_named_gpio(np, "wlf,ldoena", 0); > if (pdata->ldoena < 0) { > - dev_warn(arizona->dev, > + dev_warn(config->dev, > "LDOENA GPIO property missing/malformed: %d\n", > pdata->ldoena); > pdata->ldoena = 0; > @@ -213,19 +214,19 @@ static int arizona_ldo1_of_get_pdata(struct device *dev, > if (init_node) { > config->of_node = init_node; > > - init_data = of_get_regulator_init_data(dev, init_node, desc); > - > + init_data = of_get_regulator_init_data(config->dev, init_node, > + desc); > if (init_data) { > init_data->consumer_supplies = &ldo1->supply; > init_data->num_consumer_supplies = 1; > > if (dcvdd_node && dcvdd_node != init_node) > - arizona->external_dcvdd = true; > + *external_dcvdd = true; > > - pdata->ldo1 = init_data; > + pdata->init_data = init_data; > } > } else if (dcvdd_node) { > - arizona->external_dcvdd = true; > + *external_dcvdd = true; > } > > of_node_put(dcvdd_node); > @@ -239,10 +240,9 @@ static int arizona_ldo1_probe(struct platform_device *pdev) > const struct regulator_desc *desc; > struct regulator_config config = { }; > struct arizona_ldo1 *ldo1; > + bool external_dcvdd = false; > int ret; > > - arizona->external_dcvdd = false; > - > ldo1 = devm_kzalloc(&pdev->dev, sizeof(*ldo1), GFP_KERNEL); > if (!ldo1) > return -ENOMEM; > @@ -283,17 +283,18 @@ static int arizona_ldo1_probe(struct platform_device *pdev) > > if (IS_ENABLED(CONFIG_OF)) { > if (!dev_get_platdata(arizona->dev)) { > - ret = arizona_ldo1_of_get_pdata(&pdev->dev, arizona, > - &config, desc); > + ret = arizona_ldo1_of_get_pdata(&arizona->pdata.ldo1, > + &config, desc, > + &external_dcvdd); > if (ret < 0) > return ret; > } > } > > - config.ena_gpio = arizona->pdata.ldoena; > + config.ena_gpio = arizona->pdata.ldo1.ldoena; > > - if (arizona->pdata.ldo1) > - config.init_data = arizona->pdata.ldo1; > + if (arizona->pdata.ldo1.init_data) > + config.init_data = arizona->pdata.ldo1.init_data; > else > config.init_data = &ldo1->init_data; > > @@ -303,6 +304,8 @@ static int arizona_ldo1_probe(struct platform_device *pdev) > */ > if (config.init_data->num_consumer_supplies == 0) > arizona->external_dcvdd = true; > + else > + arizona->external_dcvdd = external_dcvdd; > > ldo1->regulator = devm_regulator_register(&pdev->dev, desc, &config); > > diff --git a/include/linux/mfd/arizona/pdata.h b/include/linux/mfd/arizona/pdata.h > index 43e875f..bfeecf1 100644 > --- a/include/linux/mfd/arizona/pdata.h > +++ b/include/linux/mfd/arizona/pdata.h > @@ -12,6 +12,7 @@ > #define _ARIZONA_PDATA_H > > #include > +#include > #include > > #define ARIZONA_GPN_DIR_MASK 0x8000 /* GPN_DIR */ > @@ -77,13 +78,12 @@ struct arizona_micd_range { > > struct arizona_pdata { > int reset; /** GPIO controlling /RESET, if any */ > - int ldoena; /** GPIO controlling LODENA, if any */ > > /** Regulator configuration for MICVDD */ > struct arizona_micsupp_pdata micvdd; > > /** Regulator configuration for LDO1 */ > - struct regulator_init_data *ldo1; > + struct arizona_ldo1_pdata ldo1; > > /** If a direct 32kHz clock is provided on an MCLK specify it here */ > int clk32k_src; > diff --git a/include/linux/regulator/arizona-ldo1.h b/include/linux/regulator/arizona-ldo1.h > new file mode 100644 > index 0000000..c685f12 > --- /dev/null > +++ b/include/linux/regulator/arizona-ldo1.h > @@ -0,0 +1,24 @@ > +/* > + * Platform data for Arizona LDO1 regulator > + * > + * Copyright 2017 Cirrus Logic > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +#ifndef ARIZONA_LDO1_H > +#define ARIZONA_LDO1_H > + > +struct regulator_init_data; > + > +struct arizona_ldo1_pdata { > + /** GPIO controlling LDOENA, if any */ > + int ldoena; > + > + /** Regulator configuration for LDO1 */ > + const struct regulator_init_data *init_data; > +}; > + > +#endif > -- > 1.9.1 > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog