From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: matti.j.aaltonen@nokia.com
Cc: alsa-devel@alsa-project.org, peter.ujfalusi@nokia.com,
lrg@slimlogic.co.uk
Subject: Re: [PATCH v2 0/1] *** SUBJECT HERE ***
Date: Thu, 22 Jul 2010 14:05:49 +0100 [thread overview]
Message-ID: <20100722130548.GI4737@rakim.wolfsonmicro.main> (raw)
In-Reply-To: <A1C7E76B4973664FBB6C554A75234AC62BEEEB72A7@NOK-EUMSG-02.mgdnok.nokia.com>
On Thu, Jul 22, 2010 at 02:11:27PM +0200, matti.j.aaltonen@nokia.com wrote:
> >You've still not provided any information about what use this will be
> >put to.
> Yes, sorry I didn't get your point. I guess I tought that it's irrelevant, what
> the soc_card driver actually does. But it actually does this:
OK, so the DAI format for the CODEC changes dependant on the mode. For
this I'd suggest just changing the function to actually export the DAI
format directly rather than requring all users to replicate the lookup
you've currently got in your driver. This makes it clear what the
purpose of providing the export is and avoids external drivers needing
to know too much about the internals of this one.
In terms of what's relevant if someone's asking questions like this
they're normally trying to figure out what your high level goal is
because your code looks suspicous and are trying to understand the
purpose of the code so they can suggest a way of doing things that is
more in line with the intentions of the surrounding code.
next prev parent reply other threads:[~2010-07-22 13:05 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-22 9:38 [PATCH v2 0/1] *** SUBJECT HERE *** Matti J. Aaltonen
2010-07-22 9:38 ` [PATCH v2 1/1] ASoC: TI WL1273 FM Radio Codec Matti J. Aaltonen
2010-07-22 9:48 ` [PATCH v2 0/1] *** SUBJECT HERE *** Mark Brown
2010-07-22 10:57 ` Peter Ujfalusi
2010-07-22 11:02 ` Mark Brown
2010-07-22 11:07 ` Peter Ujfalusi
2010-07-22 11:30 ` Takashi Iwai
2010-07-22 12:04 ` Mark Brown
2010-07-22 11:18 ` matti.j.aaltonen
2010-07-22 11:25 ` Peter Ujfalusi
2010-07-22 11:37 ` matti.j.aaltonen
2010-07-22 12:02 ` Mark Brown
2010-07-22 12:11 ` matti.j.aaltonen
2010-07-22 13:05 ` Mark Brown [this message]
2010-07-22 14:13 ` Matti J. Aaltonen
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=20100722130548.GI4737@rakim.wolfsonmicro.main \
--to=broonie@opensource.wolfsonmicro.com \
--cc=alsa-devel@alsa-project.org \
--cc=lrg@slimlogic.co.uk \
--cc=matti.j.aaltonen@nokia.com \
--cc=peter.ujfalusi@nokia.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.