All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Liam Girdwood <lrg@slimlogic.co.uk>
Cc: Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
	alsa-devel@alsa-project.org, linux-sh@vger.kernel.org
Subject: Re: [alsa-devel] [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only user and other codecs
Date: Mon, 20 Sep 2010 10:29:08 +0100	[thread overview]
Message-ID: <20100920092908.GA3414@rakim.wolfsonmicro.main> (raw)
In-Reply-To: <1284970372.3403.15.camel@odin>

On Mon, Sep 20, 2010 at 09:12:52AM +0100, Liam Girdwood wrote:

> Fwiw, the reason for the "-codec" suffix was to distinguish codec
> devices from their other multi-function IC siblings. It also makes sense

Yes, MFDs do need something to distinguish them.

> here to use convention with the device naming too (even if the device is
> single purpose).

I don't find it adds much more than making the name longer to type - we
always use the name in a context where it's fairly obvious that the
device is a CODEC.

WARNING: multiple messages have this Message-ID (diff)
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Liam Girdwood <lrg@slimlogic.co.uk>
Cc: Guennadi Liakhovetski <g.liakhovetski@gmx.de>,
	alsa-devel@alsa-project.org, linux-sh@vger.kernel.org
Subject: Re: [alsa-devel] [PATCH 3/5] ASoC: lowercase wm8978 codec name to
Date: Mon, 20 Sep 2010 09:29:08 +0000	[thread overview]
Message-ID: <20100920092908.GA3414@rakim.wolfsonmicro.main> (raw)
In-Reply-To: <1284970372.3403.15.camel@odin>

On Mon, Sep 20, 2010 at 09:12:52AM +0100, Liam Girdwood wrote:

> Fwiw, the reason for the "-codec" suffix was to distinguish codec
> devices from their other multi-function IC siblings. It also makes sense

Yes, MFDs do need something to distinguish them.

> here to use convention with the device naming too (even if the device is
> single purpose).

I don't find it adds much more than making the name longer to type - we
always use the name in a context where it's fairly obvious that the
device is a CODEC.

  reply	other threads:[~2010-09-20  9:29 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-17 10:29 [PATCH 0/5] fix SIU and Migo-R sound for 2.6.37 Guennadi Liakhovetski
2010-09-17 10:29 ` Guennadi Liakhovetski
2010-09-17 10:29 ` [PATCH 1/5] ASoC: fix compile breakage of the sh/siu driver Guennadi Liakhovetski
2010-09-17 10:29   ` Guennadi Liakhovetski
2010-09-19 15:04   ` Mark Brown
2010-09-19 15:04     ` Mark Brown
2010-09-17 10:30 ` [PATCH 2/5] sh: fix an SIU device name mismatch Guennadi Liakhovetski
2010-09-17 10:30   ` Guennadi Liakhovetski
2010-09-19 15:04   ` Mark Brown
2010-09-19 15:04     ` Mark Brown
2010-09-17 10:30 ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only user and other codecs Guennadi Liakhovetski
2010-09-17 10:30   ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only user Guennadi Liakhovetski
2010-09-17 12:53   ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only user and other codecs Mark Brown
2010-09-17 12:53     ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its Mark Brown
2010-09-19 15:59     ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only user and other codecs Guennadi Liakhovetski
2010-09-19 15:59       ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only Guennadi Liakhovetski
2010-09-19 19:45       ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only user and other codecs Mark Brown
2010-09-19 19:45         ` [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its Mark Brown
2010-09-20  8:12     ` [alsa-devel] [PATCH 3/5] ASoC: lowercase wm8978 codec name to match its only user and other codecs Liam Girdwood
2010-09-20  8:12       ` [alsa-devel] [PATCH 3/5] ASoC: lowercase wm8978 codec name to Liam Girdwood
2010-09-20  9:29       ` Mark Brown [this message]
2010-09-20  9:29         ` Mark Brown
2010-09-17 10:30 ` [PATCH 4/5] ASoC: fix clkdev API usage in sh/migor.c Guennadi Liakhovetski
2010-09-17 10:30   ` Guennadi Liakhovetski
2010-09-19 15:05   ` Mark Brown
2010-09-19 15:05     ` Mark Brown
2010-09-17 10:30 ` [PATCH 5/5] ASoC: fix SIU driver breakage, occurred during the multi-component transition Guennadi Liakhovetski
2010-09-17 10:30   ` [PATCH 5/5] ASoC: fix SIU driver breakage, occurred during the Guennadi Liakhovetski
2010-09-19 15:05   ` [PATCH 5/5] ASoC: fix SIU driver breakage, occurred during the multi-component transition Mark Brown
2010-09-19 15:05     ` [PATCH 5/5] ASoC: fix SIU driver breakage, occurred during the Mark Brown
2010-09-17 12:19 ` [PATCH 0/5] fix SIU and Migo-R sound for 2.6.37 Mark Brown
2010-09-17 12:19   ` Mark Brown
2010-09-17 12:35   ` Guennadi Liakhovetski
2010-09-17 12:35     ` Guennadi Liakhovetski
2010-09-17 12:59     ` Mark Brown
2010-09-17 12:59       ` Mark Brown
2010-09-17 13:13       ` Guennadi Liakhovetski
2010-09-17 13:13         ` Guennadi Liakhovetski

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=20100920092908.GA3414@rakim.wolfsonmicro.main \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=g.liakhovetski@gmx.de \
    --cc=linux-sh@vger.kernel.org \
    --cc=lrg@slimlogic.co.uk \
    /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.