Re: [PATCH 1/3] ASoC: mediatek: Add AFE platform driver

2015-06-14 Thread Koro Chen
On Fri, 2015-06-12 at 14:06 +0100, Mark Brown wrote: > On Wed, Jun 10, 2015 at 10:24:34PM +0800, Koro Chen wrote: > > This is the DPCM based platform driver of AFE (Audio Front End) unit. > > > > Signed-off-by: Koro Chen > > Signed-off-by: Sascha Hauer > > Your signoff should be last if you're

Re: [PATCH 1/3] ASoC: mediatek: Add AFE platform driver

2015-06-12 Thread Mark Brown
On Wed, Jun 10, 2015 at 10:24:34PM +0800, Koro Chen wrote: > This is the DPCM based platform driver of AFE (Audio Front End) unit. > > Signed-off-by: Koro Chen > Signed-off-by: Sascha Hauer Your signoff should be last if you're the one sending the patch. Otherwise this seems basically fine, but

Re: [PATCH 1/3] ASoC: mediatek: Add AFE platform driver

2015-06-11 Thread Paul Bolle
On Thu, 2015-06-11 at 09:03 +0200, Paul Bolle wrote: > Is SND_SOC_MEDIATEK perhaps meant to be bool? s/bool/tristate/, of course. Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at h

Re: [PATCH 1/3] ASoC: mediatek: Add AFE platform driver

2015-06-11 Thread Paul Bolle
On Wed, 2015-06-10 at 22:24 +0800, Koro Chen wrote: > --- /dev/null > +++ b/sound/soc/mediatek/Kconfig > +config SND_SOC_MEDIATEK > + bool "ASoC support for Mediatek chip" > + depends on ARCH_MEDIATEK > + help > + This adds ASoC platform driver support for Mediatek chip > +