Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751971AbbHEJIB (ORCPT ); Wed, 5 Aug 2015 05:08:01 -0400 Received: from mail-pa0-f44.google.com ([209.85.220.44]:35891 "EHLO mail-pa0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750738AbbHEJH6 (ORCPT ); Wed, 5 Aug 2015 05:07:58 -0400 Message-ID: <55C1D266.5090804@linaro.org> Date: Wed, 05 Aug 2015 14:37:50 +0530 From: Vaibhav Hiremath User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 MIME-Version: 1.0 To: Lee Jones CC: linux-arm-kernel@lists.infradead.org, robh+dt@kernel.org, yhanin@marvell.com, haojian.zhuang@marvell.com, zhouqiao@marvell.com, mturquette@baylibre.com, k.kozlowski@samsung.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org Subject: Re: [PATCH 4/4] mfd: 88pm800: Add support for clk subdevice References: <1437476823-3358-1-git-send-email-vaibhav.hiremath@linaro.org> <1437476823-3358-5-git-send-email-vaibhav.hiremath@linaro.org> <20150723155008.GS3436@x1> In-Reply-To: <20150723155008.GS3436@x1> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1853 Lines: 66 On Thursday 23 July 2015 09:20 PM, Lee Jones wrote: > On Tue, 21 Jul 2015, Vaibhav Hiremath wrote: > >> This patch adds mfd_cell/clk-subdevice for 88PM800 MFD >> (and family of devices). >> >> Signed-off-by: Vaibhav Hiremath >> --- >> drivers/mfd/88pm800.c | 25 +++++++++++++++++++++++++ >> 1 file changed, 25 insertions(+) >> >> diff --git a/drivers/mfd/88pm800.c b/drivers/mfd/88pm800.c >> index f104a32..9723eac 100644 >> --- a/drivers/mfd/88pm800.c >> +++ b/drivers/mfd/88pm800.c >> @@ -173,6 +173,14 @@ static const struct mfd_cell regulator_devs[] = { >> }, >> }; >> >> +static struct mfd_cell clk_devs[] = { >> + { >> + .name = "88pm80x-clk", >> + .of_compatible = "marvell,88pm800-clk", >> + .id = -1, >> + }, >> +}; > > Why does each device in 88pm800.c have it's own mfd_cell? > Probably, the author of the driver can comment on this. I have cc'ed all of them. But looking at the driver it looks like, except onkey and regulator all other devices need per-initialization. For example, in case of clock, static int device_clk_init(struct pm80x_chip *chip) { if (chip->type == CHIP_PM800) clk_devs[0].name = "88pm800-clk"; else if (chip->type == CHIP_PM860) clk_devs[0].name = "88pm860-clk"; return mfd_add_devices(chip->dev, 0, &clk_devs[0], ARRAY_SIZE(clk_devs), NULL, 0, NULL); } > Take the opportunity to correct the tabbing in the remainder of the > file. Make that fix-up patch 1 of this set. Then fixup this patch. > Ok, I will. Thanks, Vaibhav -- 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/