[alsa-devel] [PATCH 4/5] ASoC: codecs: Add AB8500 codec-driver
broonie at opensource.wolfsonmicro.com
Wed May 30 17:09:38 CEST 2012
On Wed, May 30, 2012 at 03:49:51PM +0200, Ola Lilja wrote:
> On 05/30/2012 03:14 PM, Mark Brown wrote:
> > What problems are you finding when you try to use the debug
> > infrastructure in both the regulator API and DAPM to discover the state
> > of the regulators?
> My vision here is that in a simple way, in one place, activate all
> debug-information we need in our driver, prefixed with our dev_xxx. This is very
> valuable for us when debugging, especially when a customer is told to activate
> debug-information that we can use to debug.
> I removed the menuconfig flag on your request, and then we lost the information
> for regulators and clocks when I implemented the clock/regulator-widgets. I'm
> just trying to keep some aspects of what we want to have but still conforming
> what you want to see on mainline.
All the existing diagnostics use dev_name() too... just turning on the
standard logging ought to give you exactly the same stuff, indeed if
debugfs is on there should be no need to even manually enable equivalent
> >> +int ab8500_audio_init_audioblock(struct snd_soc_codec *codec)
> > static. Lots of other functions in the rest of the driver have the same
> > issue.
> This one should be static, yes. Cannot find any other non-static functions in
> the codec-driver that is missing static.
IIRC the function immediately below was one.
> >> +#define PRE_PMU_POST_PMD (SND_SOC_DAPM_PRE_PMU | \
> >> + SND_SOC_DAPM_POST_PMD)
> > You shouldn't define stuff like this in your driver!
> This was mainly to avoid impossible situations trying to comply with the
Adding things like this if they're useful are fine but they shouldn't be
driver specific things. There is a SND_SOC_DAPM_PRE_POST_PMD already
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 836 bytes
Desc: Digital signature
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20120530/1e34294f/attachment.sig
More information about the Alsa-devel