From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755107AbcFQGDg (ORCPT ); Fri, 17 Jun 2016 02:03:36 -0400 Received: from mailout1.w1.samsung.com ([210.118.77.11]:31321 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752176AbcFQGDe (ORCPT ); Fri, 17 Jun 2016 02:03:34 -0400 X-AuditID: cbfec7f4-f796c6d000001486-14-576392b372c0 Subject: Re: [PATCH V2 4/4] clk: max77686: Add support for MAX77620 clocks To: Laxman Dewangan , javier@osg.samsung.com, mturquette@baylibre.com, robh+dt@kernel.org, linux@armlinux.org.uk References: <1466083407-28793-1-git-send-email-ldewangan@nvidia.com> <1466083407-28793-5-git-send-email-ldewangan@nvidia.com> Cc: linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, Javier Martinez Canillas From: Krzysztof Kozlowski Message-id: <576392B1.9040801@samsung.com> Date: Fri, 17 Jun 2016 08:03:29 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.8.0 MIME-version: 1.0 In-reply-to: <1466083407-28793-5-git-send-email-ldewangan@nvidia.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmphkeLIzCtJLcpLzFFi42I5/e/4Vd3Nk5LDDXa9ZrOYf+Qcq8W13zPY LN68XcNk8fqFocXSfatZLDY9vsZq8bHnHqvF5V1z2CxmnN/HZHFo6l5Gi4unXC1a9x5hd+Dx uHztIrPH+xut7B5/Z7cye2xa1cnmsXlJvUdv8zs2jy39d9k9+rasYvT4vEkugDOKyyYlNSez LLVI3y6BK6P93H22gl08FR9/TGBuYJzN1cXIySEhYCJx9NhMdghbTOLCvfVsILaQwFJGiU/n 8roYuYDsZ4wSNx/9ZAFJCAt4Scx9Ph2sSESgg1HizGFDiIZ6iZc/prKCNDALXGOUaNy/khkk wSZgLLF5+RKwBl4BLYnLC5+ygtgsAqoS3Q+OM4HYogIRErO2/2CCqBGU+DH5HtgyTgFXidO9 94HqOYCG6kncv6gFEmYWkJfYvOYt8wRGgVlIOmYhVM1CUrWAkXkVo2hqaXJBcVJ6rqFecWJu cWleul5yfu4mRkjcfNnBuPiY1SFGAQ5GJR7eFaLJ4UKsiWXFlbmHGCU4mJVEePf0AoV4UxIr q1KL8uOLSnNSiw8xSnOwKInzzt31PkRIID2xJDU7NbUgtQgmy8TBKdXAyJSSK/5o1Zwi7g2f QtfEbL9TMEFf97nQ60qdGVfLn9y/9HbpId1VsaGrCuTNXxn9vBO2PUTcWWz7iqcy580Fkqpn PpheuiN67+n9fUu2dEVrhNleWPTxnvpUufcq74zc75XZLa/8aFZ+yvsw93M3CePciL8Tv85N 5rVPNjt+YdfRNY0XPsvwr1RiKc5INNRiLipOBADBl7QAlwIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/16/2016 03:23 PM, Laxman Dewangan wrote: > Maxim Max77620 has one 32KHz clock output and the clock HW > IP used on this PMIC is same as what it is there in the MAX77686. > > Add clock driver support for MAX77620 on the MAX77686 driver. > > Signed-off-by: Laxman Dewangan > CC: Krzysztof Kozlowski > CC: Javier Martinez Canillas > Signed-off-by: Laxman Dewangan > Tested-by: Krzysztof Kozlowski > > --- > Changes from V1: > - Updated based on changes done in first patch. > --- > drivers/clk/Kconfig | 7 ++++--- > drivers/clk/clk-max77686.c | 18 ++++++++++++++++++ > 2 files changed, 22 insertions(+), 3 deletions(-) > > diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig > index 6afad74..d75f4c5 100644 > --- a/drivers/clk/Kconfig > +++ b/drivers/clk/Kconfig > @@ -32,10 +32,11 @@ config COMMON_CLK_WM831X > source "drivers/clk/versatile/Kconfig" > > config COMMON_CLK_MAX77686 > - tristate "Clock driver for Maxim 77686/77802 MFD" > - depends on MFD_MAX77686 > + tristate "Clock driver for Maxim 77686/77802/MAX77620 MFD" You missed Javier's comment here. Just "Maxim 77686/77802/77620" > + depends on MFD_MAX77686 || MFD_MAX77620 > ---help--- > - This driver supports Maxim 77686/77802 crystal oscillator clock. > + This driver supports Maxim 77686/77802/MAX77620 crystal oscillator Ditto Beside that: Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof