Re: [PATCH 1/2] ASoC: Set up debugfs only once per CODEC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Aug 13, 2010 at 07:42:44AM +0300, Peter Ujfalusi wrote:
> On Thursday 12 August 2010 17:50:31 ext Mark Brown wrote:
> > Since the debugfs directory is current per CODEC we should only init
> > it when the CODEC is initialised, otherwise we end up with errors
> > being generated when an attempt is made to add duplicate debugfs
> > entries.

> But if you move the init of debugfs where you have moved it, than the machine 
> driver given DAPM widgets will have no debugfs entries, since the machine init 
> is going to be called later (dai_link->init)...

Sure, but the big problem there is that these aren't really CODEC
specific objects.

> > Since most of this stuff is actually for the card we should refactor
> > but this can come later.

> Sure, refactoring will help, but with this change we have missing debugfs files 
> for widgets, which is kind of confusing.

Meh, it's a debug facility and I hope it'll be sorted out within a week
or so anyway.  Besides, you're not really loosing much information since
there tends to be no interesting routing outside the CODEC.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux