Hi Lee, On 06/16/2014 10:03 PM, Lee Jones wrote: > On Mon, 16 Jun 2014, Chanwoo Choi wrote: > >> Add support for Samsung S2MPU02 PMIC device to the MFD sec-core driver. >> The S2MPU02 device includes PMIC/RTC/Clock devices. >> >> Signed-off-by: Chanwoo Choi <cw00.c...@samsung.com> >> Reviewed-by: Krzysztof Kozlowski <k.kozlow...@samsung.com> >> --- >> drivers/mfd/sec-core.c | 19 +++++++++ >> drivers/mfd/sec-irq.c | 88 >> ++++++++++++++++++++++++++++++++++++++++ >> include/linux/mfd/samsung/core.h | 1 + >> include/linux/mfd/samsung/irq.h | 24 +++++++++++ >> 4 files changed, 132 insertions(+) >> >> diff --git a/drivers/mfd/sec-core.c b/drivers/mfd/sec-core.c >> index be06d0a..5e8784b 100644 >> --- a/drivers/mfd/sec-core.c >> +++ b/drivers/mfd/sec-core.c >> @@ -89,6 +89,17 @@ static const struct mfd_cell s2mpa01_devs[] = { >> }, >> }; >> >> +static const struct mfd_cell s2mpu02_devs[] = { >> + { >> + .name = "s2mpu02-pmic", >> + }, { >> + .name = "s2mpu02-rtc", > > Make these two one liners, so: > > { .name = "s2mpu02-pmic" }, > { .name = "s2mpu02-rtc" ),
OK, I'll modify it. > >> + }, { >> + .name = "s2mpu02-clk", >> + .of_compatible = "samsung,s2mpu02-clk", >> + } >> +}; >> + >> #ifdef CONFIG_OF >> static const struct of_device_id sec_dt_match[] = { >> { .compatible = "samsung,s5m8767-pmic", >> @@ -103,6 +114,9 @@ static const struct of_device_id sec_dt_match[] = { >> .compatible = "samsung,s2mpa01-pmic", >> .data = (void *)S2MPA01, >> }, { >> + .compatible = "samsung,s2mpu02-pmic", >> + .data = (void *)S2MPU02, >> + }, { >> /* Sentinel */ >> }, >> }; >> @@ -342,6 +356,11 @@ static int sec_pmic_probe(struct i2c_client *i2c, >> ret = mfd_add_devices(sec_pmic->dev, -1, s2mps14_devs, >> ARRAY_SIZE(s2mps14_devs), NULL, 0, NULL); >> break; >> + case S2MPU02: >> + ret = mfd_add_devices(sec_pmic->dev, -1, s2mpu02_devs, >> + ARRAY_SIZE(s2mpu02_devs), NULL, 0, NULL); >> + break; > > This is a bit bonkers. Instead of passing S2MPU02, why don't you pass > a new container which holds s2mpu02_devs and s2mpu02_irq_chip, then > you won't need new mfd_add_devices() and regmap_add_irq_chip() calls > for every device you want to support. OK, I'll remove duplicate code of mfd_add_devices() and regmap_add_irq_chip(). Best Regards, Chanwoo Choi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/