[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170914074300.iobuwn2hrmnf6sen@dell>
Date: Thu, 14 Sep 2017 08:43:00 +0100
From: Lee Jones <lee.jones@...aro.org>
To: Robert Jarzmik <robert.jarzmik@...e.fr>
Cc: Dmitry Torokhov <dmitry.torokhov@...il.com>,
Jaroslav Kysela <perex@...ex.cz>,
Takashi Iwai <tiwai@...e.com>,
Liam Girdwood <lgirdwood@...il.com>,
Mark Brown <broonie@...nel.org>,
Daniel Mack <daniel@...que.org>,
Haojian Zhuang <haojian.zhuang@...il.com>,
Lars-Peter Clausen <lars@...afoo.de>,
Charles Keepax <ckeepax@...nsource.wolfsonmicro.com>,
linux-kernel@...r.kernel.org, linux-input@...r.kernel.org,
patches@...nsource.wolfsonmicro.com, alsa-devel@...a-project.org,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v7 2/8] mfd: wm97xx-core: core support for wm97xx Codec
On Thu, 14 Sep 2017, Lee Jones wrote:
> On Wed, 13 Sep 2017, Robert Jarzmik wrote:
>
> > The WM9705, WM9712 and WM9713 are highly integrated codecs, with an
> > audio codec, DAC and ADC, GPIO unit and a touchscreen interface.
> >
> > Historically the support was spread across drivers/input/touchscreen and
> > sound/soc/codecs. The sharing was done through ac97 bus sharing. This
> > model will not withstand the new AC97 bus model, where codecs are
> > discovered on runtime.
> >
> > Signed-off-by: Robert Jarzmik <robert.jarzmik@...e.fr>
> > Acked-by: Charles Keepax <ckeepax@...nsource.wolfsonmicro.com>
> > Acked-by: Lee Jones <lee.jones@...aro.org>
> > ---
> > Since v3:
> > - added a "depends on AC97_BUS_NEW" Kconfig statement
> > - added default values for wm9705, wm9712 per Charles's comment
> > Since v4:
> > - added Charles's ack
> > Since v5:
> > - took into account Lee's comments
> > Since v6:
> > - took into account Lee's comments
> > ---
> > drivers/mfd/Kconfig | 14 ++
> > drivers/mfd/Makefile | 1 +
> > drivers/mfd/wm97xx-core.c | 366 +++++++++++++++++++++++++++++++++++++++++++++
> > include/linux/mfd/wm97xx.h | 25 ++++
> > 4 files changed, 406 insertions(+)
> > create mode 100644 drivers/mfd/wm97xx-core.c
> > create mode 100644 include/linux/mfd/wm97xx.h
>
> Applied, thanks.
Change of plan. It looks like there are deps.
Unapplied.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Powered by blists - more mailing lists