From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964850AbcHaOgi (ORCPT ); Wed, 31 Aug 2016 10:36:38 -0400 Received: from mail-wm0-f41.google.com ([74.125.82.41]:38699 "EHLO mail-wm0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935210AbcHaOge (ORCPT ); Wed, 31 Aug 2016 10:36:34 -0400 Date: Wed, 31 Aug 2016 15:38:20 +0100 From: Lee Jones To: Peter Ujfalusi Cc: linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH RESEND 2/2] mfd: twl6040: Register child device for twl6040-pdmclk Message-ID: <20160831143820.GL9415@dell> References: <20160831114621.6301-1-peter.ujfalusi@ti.com> <20160831114621.6301-3-peter.ujfalusi@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20160831114621.6301-3-peter.ujfalusi@ti.com> User-Agent: Mutt/1.6.2 (2016-07-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 31 Aug 2016, Peter Ujfalusi wrote: > The McPDM in OMAP4/5 is using the pdmclk from twl6040 as functional clock. > The twl6040-pdmclk driver provides a clock which can be used to make sure > that the pdmclk is active when the McPDM is in use. > > Signed-off-by: Peter Ujfalusi > Acked-by: Rob Herring > --- > Documentation/devicetree/bindings/mfd/twl6040.txt | 1 + > drivers/mfd/twl6040.c | 5 +++++ > include/linux/mfd/twl6040.h | 2 +- > 3 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/mfd/twl6040.txt b/Documentation/devicetree/bindings/mfd/twl6040.txt > index e6afdfa3543d..9a98ee7c323d 100644 > --- a/Documentation/devicetree/bindings/mfd/twl6040.txt > +++ b/Documentation/devicetree/bindings/mfd/twl6040.txt > @@ -12,6 +12,7 @@ Required properties: > - interrupt-parent: The parent interrupt controller > - gpio-controller: > - #gpio-cells = <1>: twl6040 provides GPO lines. > +- #clock-cells = <0>; twl6040 is a provider of pdmclk which is used by McPDM > - twl6040,audpwron-gpio: Power on GPIO line for the twl6040 > > - vio-supply: Regulator for the twl6040 VIO supply > diff --git a/drivers/mfd/twl6040.c b/drivers/mfd/twl6040.c > index c3ad59ef43dd..d66502d36ba0 100644 > --- a/drivers/mfd/twl6040.c > +++ b/drivers/mfd/twl6040.c > @@ -783,6 +783,11 @@ static int twl6040_probe(struct i2c_client *client, > cell->name = "twl6040-gpo"; > children++; > > + /* PDM clock support */ > + cell = &twl6040->cells[children]; > + cell->name = "twl6040-pdmclk"; > + children++; What an awful way of adding cells. That wouldn't happen in my day! ;) But ... it fits in with the current way of doing things, so: For my own reference: Acked-by: Lee Jones > /* The chip is powered down so mark regmap to cache only and dirty */ > regcache_cache_only(twl6040->regmap, true); > regcache_mark_dirty(twl6040->regmap); > diff --git a/include/linux/mfd/twl6040.h b/include/linux/mfd/twl6040.h > index 36795a1be479..a2e88761c09f 100644 > --- a/include/linux/mfd/twl6040.h > +++ b/include/linux/mfd/twl6040.h > @@ -168,7 +168,7 @@ > #define TWL6040_VIBROCDET 0x20 > #define TWL6040_TSHUTDET 0x40 > > -#define TWL6040_CELLS 3 > +#define TWL6040_CELLS 4 > > #define TWL6040_REV_ES1_0 0x00 > #define TWL6040_REV_ES1_1 0x01 /* Rev ES1.1 and ES1.2 */ -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog