Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753527AbaKDKy6 (ORCPT ); Tue, 4 Nov 2014 05:54:58 -0500 Received: from mail-ig0-f171.google.com ([209.85.213.171]:54286 "EHLO mail-ig0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751989AbaKDKyw (ORCPT ); Tue, 4 Nov 2014 05:54:52 -0500 Date: Tue, 4 Nov 2014 10:54:21 +0000 From: Lee Jones To: gyungoh@gmail.com Cc: sameo@linux.intel.com, jg1.han@samsung.com, cooloney@gmail.com, broonie@kernel.org, lgirdwood@gmail.com, jack.yoo@skyworksinc.com, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, grant.likely@linaro.org, robh+dt@kernel.org, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, pawel.moll@arm.com, heiko@sntech.de, jason@lakedaemon.net, shawn.guo@freescale.com, treding@nvidia.com, florian.vaussard@epfl.ch, trivial@kernel.org, linux@roeck-us.net, andrew@lunn.ch, jic23@kernel.org Subject: Re: [RESUBMIT PATCH v6 1/8] mfd: Add support for Skyworks SKY81452 driver Message-ID: <20141104105421.GO17577@x1> References: <1414393369-31583-1-git-send-email-jack.yoo@skyworksinc.com> <1414739322-12487-1-git-send-email-jack.yoo@skyworksinc.com> <1414739322-12487-2-git-send-email-jack.yoo@skyworksinc.com> <20141103171451.GG12011@x1> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20141103171451.GG12011@x1> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 03 Nov 2014, Lee Jones wrote: > On Fri, 31 Oct 2014, gyungoh@gmail.com wrote: > > > From: Gyungoh Yoo > > > > Signed-off-by: Gyungoh Yoo > > --- > > Changes v6: > > Added new line character at the end of line of dev_err() > > > > Changes v5: > > Move sky81452-backlight.h to include/linux/platform_data > > > > Changes v4: > > Removed MODULE_VERSION() > > Modified license to GPLv2 > > > > Changes v3: > > Fixed the backlight name from 'sky81452-bl' to 'sky81452-backlight' > > Assigned mfd_cell.of_compatible for binding device node > > Modified error messages > > > > Changes v2: > > Renamed CONFIG_SKY81452 to CONFIG_MFD_SKY81452 > > Changed the dependency from I2C=y to I2C, for CONFIG_MFD_SKY81452 > > Added message for exception or errors > > > > drivers/mfd/Kconfig | 12 +++++ > > drivers/mfd/Makefile | 1 + > > drivers/mfd/sky81452.c | 109 +++++++++++++++++++++++++++++++++++++++++++ > > include/linux/mfd/sky81452.h | 32 +++++++++++++ > > 4 files changed, 154 insertions(+) > > create mode 100644 drivers/mfd/sky81452.c > > create mode 100644 include/linux/mfd/sky81452.h > > Code looks fine, but what are its dependencies. I'm assuming nothing; > however, did Mark say that something depends on it? If so, what and > what are you going to do about it? Okay, so this driver does in fact have a dependancy on the Backlight driver. Until I get a Backlight Ack from either Brian and/or Jingoo, there is nothing I can do. Patch unapplied. > > diff --git a/drivers/mfd/Kconfig b/drivers/mfd/Kconfig > > index 1456ea7..a54244c 100644 > > --- a/drivers/mfd/Kconfig > > +++ b/drivers/mfd/Kconfig > > @@ -693,6 +693,18 @@ config MFD_SM501_GPIO > > lines on the SM501. The platform data is used to supply the > > base number for the first GPIO line to register. > > > > +config MFD_SKY81452 > > + tristate "Skyworks Solutions SKY81452" > > + select MFD_CORE > > + select REGMAP_I2C > > + depends on I2C > > + help > > + This is the core driver for the Skyworks SKY81452 backlight and > > + voltage regulator device. > > + > > + This driver can also be built as a module. If so, the module > > + will be called sky81452. > > + > > config MFD_SMSC > > bool "SMSC ECE1099 series chips" > > depends on I2C=y > > diff --git a/drivers/mfd/Makefile b/drivers/mfd/Makefile > > index 8bd54b1..d9683ea 100644 > > --- a/drivers/mfd/Makefile > > +++ b/drivers/mfd/Makefile > > @@ -174,6 +174,7 @@ obj-$(CONFIG_MFD_STW481X) += stw481x.o > > obj-$(CONFIG_MFD_IPAQ_MICRO) += ipaq-micro.o > > obj-$(CONFIG_MFD_MENF21BMC) += menf21bmc.o > > obj-$(CONFIG_MFD_HI6421_PMIC) += hi6421-pmic-core.o > > +obj-$(CONFIG_MFD_SKY81452) += sky81452.o > > > > intel-soc-pmic-objs := intel_soc_pmic_core.o intel_soc_pmic_crc.o > > obj-$(CONFIG_INTEL_SOC_PMIC) += intel-soc-pmic.o > > diff --git a/drivers/mfd/sky81452.c b/drivers/mfd/sky81452.c > > new file mode 100644 > > index 0000000..ae1b584 > > --- /dev/null > > +++ b/drivers/mfd/sky81452.c > > @@ -0,0 +1,109 @@ > > +/* > > + * sky81452.c SKY81452 MFD driver > > + * > > + * Copyright 2014 Skyworks Solutions Inc. > > + * Author : Gyungoh Yoo > > + * > > + * This program is free software; you can redistribute it and/or modify it > > + * under the terms of the GNU General Public License as published by the > > + * Free Software Foundation; either version 2, or (at your option) any > > + * later version. > > + * > > + * This program is distributed in the hope that it will be useful, but > > + * WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > + * General Public License for more details. > > + * > > + * You should have received a copy of the GNU General Public License along > > + * with this program; if not, see . > > + */ > > + > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > +#include > > + > > +static const struct regmap_config sky81452_config = { > > + .reg_bits = 8, > > + .val_bits = 8, > > +}; > > + > > +static int sky81452_probe(struct i2c_client *client, > > + const struct i2c_device_id *id) > > +{ > > + struct device *dev = &client->dev; > > + const struct sky81452_platform_data *pdata = dev_get_platdata(dev); > > + struct mfd_cell cells[2]; > > + struct regmap *regmap; > > + int ret; > > + > > + if (!pdata) { > > + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); > > + if (!pdata) > > + return -ENOMEM; > > + } > > + > > + regmap = devm_regmap_init_i2c(client, &sky81452_config); > > + if (IS_ERR(regmap)) { > > + dev_err(dev, "failed to initialize.err=%ld\n", PTR_ERR(regmap)); > > + return PTR_ERR(regmap); > > + } > > + > > + i2c_set_clientdata(client, regmap); > > + > > + memset(cells, 0, sizeof(cells)); > > + cells[0].name = "sky81452-backlight"; > > + cells[0].of_compatible = "skyworks,sky81452-backlight"; > > + cells[0].platform_data = pdata->bl_pdata; > > + cells[0].pdata_size = sizeof(*pdata->bl_pdata); > > + cells[1].name = "sky81452-regulator"; > > + cells[1].platform_data = pdata->regulator_init_data; > > + cells[1].pdata_size = sizeof(*pdata->regulator_init_data); > > + > > + ret = mfd_add_devices(dev, -1, cells, ARRAY_SIZE(cells), NULL, 0, NULL); > > + if (ret) > > + dev_err(dev, "failed to add child devices. err=%d\n", ret); > > + > > + return ret; > > +} > > + > > +static int sky81452_remove(struct i2c_client *client) > > +{ > > + mfd_remove_devices(&client->dev); > > + return 0; > > +} > > + > > +static const struct i2c_device_id sky81452_ids[] = { > > + { "sky81452" }, > > + { } > > +}; > > +MODULE_DEVICE_TABLE(i2c, sky81452_ids); > > + > > +#ifdef CONFIG_OF > > +static const struct of_device_id sky81452_of_match[] = { > > + { .compatible = "skyworks,sky81452", }, > > + { } > > +}; > > +MODULE_DEVICE_TABLE(of, sky81452_of_match); > > +#endif > > + > > +static struct i2c_driver sky81452_driver = { > > + .driver = { > > + .name = "sky81452", > > + .of_match_table = of_match_ptr(sky81452_of_match), > > + }, > > + .probe = sky81452_probe, > > + .remove = sky81452_remove, > > + .id_table = sky81452_ids, > > +}; > > + > > +module_i2c_driver(sky81452_driver); > > + > > +MODULE_DESCRIPTION("Skyworks SKY81452 MFD driver"); > > +MODULE_AUTHOR("Gyungoh Yoo "); > > +MODULE_LICENSE("GPL v2"); > > diff --git a/include/linux/mfd/sky81452.h b/include/linux/mfd/sky81452.h > > new file mode 100644 > > index 0000000..9d84872 > > --- /dev/null > > +++ b/include/linux/mfd/sky81452.h > > @@ -0,0 +1,32 @@ > > +/* > > + * sky81452.h SKY81452 MFD driver > > + * > > + * Copyright 2014 Skyworks Solutions Inc. > > + * Author : Gyungoh Yoo > > + * > > + * This program is free software; you can redistribute it and/or modify it > > + * under the terms of the GNU General Public License as published by the > > + * Free Software Foundation; either version 2, or (at your option) any > > + * later version. > > + * > > + * This program is distributed in the hope that it will be useful, but > > + * WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > + * General Public License for more details. > > + * > > + * You should have received a copy of the GNU General Public License along > > + * with this program; if not, see . > > + */ > > + > > +#ifndef _SKY81452_H > > +#define _SKY81452_H > > + > > +#include > > +#include > > + > > +struct sky81452_platform_data { > > + struct sky81452_bl_platform_data *bl_pdata; > > + struct regulator_init_data *regulator_init_data; > > +}; > > + > > +#endif > -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/