linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RESEND PATCH] davinci: CQ93VC: remove machine specific header file inclusion from codec driver
@ 2012-01-10 11:27 Manjunath Hadli
  2012-01-10 18:47 ` Mark Brown
  0 siblings, 1 reply; 4+ messages in thread
From: Manjunath Hadli @ 2012-01-10 11:27 UTC (permalink / raw)
  To: dlos, LAK
  Cc: Manjunath Hadli, Liam Girdwood, Mark Brown, Jaroslav Kysela,
	Takashi Iwai, alsa-devel, linux-kernel

remove unnecessary inclusion of machine specific header
file mach/dm365.h from cq93vc.c voice codec driver
which comes in the way of platform code consolidation.

Signed-off-by: Manjunath Hadli <manjunath.hadli@ti.com>
Cc: Liam Girdwood <lrg@ti.com>
Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Jaroslav Kysela <perex@perex.cz>
CC: Takashi Iwai <tiwai@suse.de>
Cc: <alsa-devel@alsa-project.org>
Cc: <linux-kernel@vger.kernel.org>
---
 sound/soc/codecs/cq93vc.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/sound/soc/codecs/cq93vc.c b/sound/soc/codecs/cq93vc.c
index 46dbfd0..ff4eb7a 100644
--- a/sound/soc/codecs/cq93vc.c
+++ b/sound/soc/codecs/cq93vc.c
@@ -38,8 +38,6 @@
 #include <sound/soc.h>
 #include <sound/initval.h>
 
-#include <mach/dm365.h>
-
 static inline unsigned int cq93vc_read(struct snd_soc_codec *codec,
 						unsigned int reg)
 {
-- 
1.6.2.4


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [RESEND PATCH] davinci: CQ93VC: remove machine specific header file inclusion from codec driver
  2012-01-10 11:27 [RESEND PATCH] davinci: CQ93VC: remove machine specific header file inclusion from codec driver Manjunath Hadli
@ 2012-01-10 18:47 ` Mark Brown
  2012-02-08 12:41   ` Hadli, Manjunath
  0 siblings, 1 reply; 4+ messages in thread
From: Mark Brown @ 2012-01-10 18:47 UTC (permalink / raw)
  To: Manjunath Hadli
  Cc: dlos, LAK, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
	alsa-devel, linux-kernel

On Tue, Jan 10, 2012 at 04:57:43PM +0530, Manjunath Hadli wrote:
> remove unnecessary inclusion of machine specific header
> file mach/dm365.h from cq93vc.c voice codec driver
> which comes in the way of platform code consolidation.

Applied but please do try to use a subject line appropriate for the
subsystem you're contributing to.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* RE: [RESEND PATCH] davinci: CQ93VC: remove machine specific header file inclusion from codec driver
  2012-01-10 18:47 ` Mark Brown
@ 2012-02-08 12:41   ` Hadli, Manjunath
  2012-02-08 12:52     ` Mark Brown
  0 siblings, 1 reply; 4+ messages in thread
From: Hadli, Manjunath @ 2012-02-08 12:41 UTC (permalink / raw)
  To: Mark Brown
  Cc: dlos, LAK, Girdwood, Liam, Jaroslav Kysela, Takashi Iwai,
	alsa-devel, linux-kernel

Mark,

On Wed, Jan 11, 2012 at 00:17:41, Mark Brown wrote:
> On Tue, Jan 10, 2012 at 04:57:43PM +0530, Manjunath Hadli wrote:
> > remove unnecessary inclusion of machine specific header file 
> > mach/dm365.h from cq93vc.c voice codec driver which comes in the way 
> > of platform code consolidation.
> 
> Applied but please do try to use a subject line appropriate for the subsystem you're contributing to.
I still can't find this patch in the mainline, any issues?

Regards,
--Manju
> 


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RESEND PATCH] davinci: CQ93VC: remove machine specific header file inclusion from codec driver
  2012-02-08 12:41   ` Hadli, Manjunath
@ 2012-02-08 12:52     ` Mark Brown
  0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2012-02-08 12:52 UTC (permalink / raw)
  To: Hadli, Manjunath
  Cc: dlos, LAK, Girdwood, Liam, Jaroslav Kysela, Takashi Iwai,
	alsa-devel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 452 bytes --]

On Wed, Feb 08, 2012 at 12:41:14PM +0000, Hadli, Manjunath wrote:
> On Wed, Jan 11, 2012 at 00:17:41, Mark Brown wrote:

Fix your mailer, not only is it not word wrapping within paragraphs it's
adding such problems to quoted text.

> > Applied but please do try to use a subject line appropriate for the subsystem you're contributing to.

> I still can't find this patch in the mainline, any issues?

It will go into mainline in the next merge window.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-02-08 12:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-10 11:27 [RESEND PATCH] davinci: CQ93VC: remove machine specific header file inclusion from codec driver Manjunath Hadli
2012-01-10 18:47 ` Mark Brown
2012-02-08 12:41   ` Hadli, Manjunath
2012-02-08 12:52     ` Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).