From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v3 08/14] mfd: max77686/802: Remove support for board files Date: Mon, 3 Nov 2014 17:01:28 +0000 Message-ID: <20141103170128.GD12011@x1> References: <1414668053-31370-1-git-send-email-k.kozlowski@samsung.com> <1414668053-31370-9-git-send-email-k.kozlowski@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <1414668053-31370-9-git-send-email-k.kozlowski@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org To: Krzysztof Kozlowski Cc: Samuel Ortiz , Liam Girdwood , Mark Brown , linux-kernel@vger.kernel.org, Ben Dooks , Kukjin Kim , Russell King , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, Kyungmin Park , Marek Szyprowski , Bartlomiej Zolnierkiewicz , Javier Martinez Canillas , Chanwoo Choi List-Id: devicetree@vger.kernel.org On Thu, 30 Oct 2014, Krzysztof Kozlowski wrote: > The driver is used only on Exynos based boards with DTS support. > Convert the driver to DTS-only version. This simplifies a little the > code: > 1. No dead (unused) entries in platform_data structure. > 2. More code removed. > 3. Regulator driver does not depend on allocated memory > from MFD driver. > 4. It makes also easier extending the regulator driver. >=20 > Add to the max77686 MFD driver dependency on CONFIG_OF because withou= t > DTS the regulator drivers won't bind. >=20 > Signed-off-by: Krzysztof Kozlowski > Reviewed-by: Javier Martinez Canillas > --- > drivers/mfd/Kconfig | 1 + > drivers/mfd/max77686.c | 23 ----------------------- > include/linux/mfd/max77686-private.h | 1 - > include/linux/mfd/max77686.h | 28 --------------------------= -- > 4 files changed, 1 insertion(+), 52 deletions(-) Applied, thanks. > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > index cbdb10918af1..fd9d19ccf8c0 100644 > --- a/drivers/mfd/Kconfig > +++ b/drivers/mfd/Kconfig > @@ -401,6 +401,7 @@ config MFD_MAX14577 > config MFD_MAX77686 > bool "Maxim Semiconductor MAX77686/802 PMIC Support" > depends on I2C=3Dy > + depends on OF > select MFD_CORE > select REGMAP_I2C > select REGMAP_IRQ > diff --git a/drivers/mfd/max77686.c b/drivers/mfd/max77686.c > index 9e1046bdef90..418b08b6c066 100644 > --- a/drivers/mfd/max77686.c > +++ b/drivers/mfd/max77686.c > @@ -211,24 +211,10 @@ static const struct of_device_id max77686_pmic_= dt_match[] =3D { > { }, > }; > =20 > -static struct max77686_platform_data *max77686_i2c_parse_dt_pdata(st= ruct device > - *dev) > -{ > - struct max77686_platform_data *pd; > - > - pd =3D devm_kzalloc(dev, sizeof(*pd), GFP_KERNEL); > - if (!pd) > - return NULL; > - > - dev->platform_data =3D pd; > - return pd; > -} > - > static int max77686_i2c_probe(struct i2c_client *i2c, > const struct i2c_device_id *id) > { > struct max77686_dev *max77686 =3D NULL; > - struct max77686_platform_data *pdata =3D dev_get_platdata(&i2c->dev= ); > const struct of_device_id *match; > unsigned int data; > int ret =3D 0; > @@ -239,14 +225,6 @@ static int max77686_i2c_probe(struct i2c_client = *i2c, > const struct mfd_cell *cells; > int n_devs; > =20 > - if (IS_ENABLED(CONFIG_OF) && i2c->dev.of_node && !pdata) > - pdata =3D max77686_i2c_parse_dt_pdata(&i2c->dev); > - > - if (!pdata) { > - dev_err(&i2c->dev, "No platform data found.\n"); > - return -EINVAL; > - } > - > max77686 =3D devm_kzalloc(&i2c->dev, > sizeof(struct max77686_dev), GFP_KERNEL); > if (!max77686) > @@ -265,7 +243,6 @@ static int max77686_i2c_probe(struct i2c_client *= i2c, > max77686->dev =3D &i2c->dev; > max77686->i2c =3D i2c; > =20 > - max77686->wakeup =3D pdata->wakeup; > max77686->irq =3D i2c->irq; > =20 > if (max77686->type =3D=3D TYPE_MAX77686) { > diff --git a/include/linux/mfd/max77686-private.h b/include/linux/mfd= /max77686-private.h > index 960b92ad450d..f5043490d67c 100644 > --- a/include/linux/mfd/max77686-private.h > +++ b/include/linux/mfd/max77686-private.h > @@ -447,7 +447,6 @@ struct max77686_dev { > struct regmap_irq_chip_data *rtc_irq_data; > =20 > int irq; > - bool wakeup; > struct mutex irqlock; > int irq_masks_cur[MAX77686_IRQ_GROUP_NR]; > int irq_masks_cache[MAX77686_IRQ_GROUP_NR]; > diff --git a/include/linux/mfd/max77686.h b/include/linux/mfd/max7768= 6.h > index 553f7d09258a..bb995ab9a575 100644 > --- a/include/linux/mfd/max77686.h > +++ b/include/linux/mfd/max77686.h > @@ -119,12 +119,6 @@ enum max77802_regulators { > MAX77802_REG_MAX, > }; > =20 > -struct max77686_regulator_data { > - int id; > - struct regulator_init_data *initdata; > - struct device_node *of_node; > -}; > - > enum max77686_opmode { > MAX77686_OPMODE_NORMAL, > MAX77686_OPMODE_LP, > @@ -136,26 +130,4 @@ struct max77686_opmode_data { > int mode; > }; > =20 > -struct max77686_platform_data { > - int ono; > - int wakeup; > - > - /* ---- PMIC ---- */ > - struct max77686_regulator_data *regulators; > - int num_regulators; > - > - struct max77686_opmode_data *opmode_data; > - > - /* > - * GPIO-DVS feature is not enabled with the current version of > - * MAX77686 driver. Buck2/3/4_voltages[0] is used as the default > - * voltage at probe. DVS/SELB gpios are set as OUTPUT-LOW. > - */ > - int buck234_gpio_dvs[3]; /* GPIO of [0]DVS1, [1]DVS2, [2]DVS3 */ > - int buck234_gpio_selb[3]; /* [0]SELB2, [1]SELB3, [2]SELB4 */ > - unsigned int buck2_voltage[8]; /* buckx_voltage in uV */ > - unsigned int buck3_voltage[8]; > - unsigned int buck4_voltage[8]; > -}; > - > #endif /* __LINUX_MFD_MAX77686_H */ --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog