From: Nicolin Chen <nicoleotsuka@gmail.com>
To: Zidan Wang <zidan.wang@freescale.com>
Cc: broonie@kernel.org, perex@perex.cz, tiwai@suse.de,
timur@tabi.org, Xiubo.Lee@gmail.com, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org,
linuxppc-dev@lists.ozlabs.org
Subject: Re: [alsa-devel][PATCH] ASoC: fsl: add imx-wm8960 machine driver
Date: Thu, 11 Jun 2015 08:51:07 -0700 [thread overview]
Message-ID: <20150611155106.GA3036@Asurada> (raw)
In-Reply-To: <20150611110958.GA23742@shlinux2>
On Thu, Jun 11, 2015 at 07:10:00PM +0800, Zidan Wang wrote:
> > > This imx-wm8960 device-tree-only machine driver works with sai driver
> > > and have below feature.
> > > * support codec master and slave mode
> > > * support headphone jack detect
> > > * support headphone and micphone jack event
> > > * support asrc<->sai<->wm8960 mode
> > >
> > > Signed-off-by: Zidan Wang <zidan.wang@freescale.com>
> >
> > Above all, could you merge this into fsl-asoc-card?
> >
> > It'd be nicer to have jack detection over there. And we can
> > put PLL settings to the codec driver as Mark suggested.
> I want to modify imx-wm8960 and upstream it because our release is using it now. After
> upstreaming imx-wm8960, i will try to merge it to fsl-asoc-card.
I knew your intention. That's why I suggested you to merge it directly
to fsl-asoc-card instead of making a duplicated copy of that. Remember
that getting code upstream should be a chance for us to improve thing.
If you find painstaking to handle it in parallel, you can focus on the
improvements of the WM8960 driver while I would help you on the dai-
link part. How about this?
Nicolin
next prev parent reply other threads:[~2015-06-11 15:51 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-10 8:26 [PATCH] ASoC: fsl: add imx-wm8960 machine driver Zidan Wang
2015-06-10 8:26 ` [alsa-devel][PATCH] " Zidan Wang
[not found] ` <b2a3c323a0507f3706041e1da98da8375aa61576.1433923593.git.zidan.wang-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2015-06-10 18:09 ` Mark Brown
2015-06-10 18:09 ` Mark Brown
2015-06-10 18:36 ` Nicolin Chen
2015-06-11 11:10 ` Zidan Wang
2015-06-11 11:10 ` Zidan Wang
2015-06-11 15:51 ` Nicolin Chen [this message]
2015-06-12 8:09 ` [PATCH] " Zidan Wang
2015-06-12 8:09 ` [alsa-devel][PATCH] " Zidan Wang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150611155106.GA3036@Asurada \
--to=nicoleotsuka@gmail.com \
--cc=Xiubo.Lee@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=perex@perex.cz \
--cc=timur@tabi.org \
--cc=tiwai@suse.de \
--cc=zidan.wang@freescale.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.