[alsa-devel] [PATCH 2/2] Allow a custom ASOC machine driver with soc-of-simple

Jon Smirl jonsmirl at gmail.com
Wed Jul 23 16:09:01 CEST 2008


On 7/23/08, Mark Brown <broonie at opensource.wolfsonmicro.com> wrote:
> On Tue, Jul 22, 2008 at 07:53:51PM -0400, Jon Smirl wrote:
>
>  > Allow a custom ASOC machine driver with soc-of-simple
>
>
> As with the clocking configuration: this looks fine from an ASoC point
>  of view but please fix the checkpatch warnings.  However...
>
>
>  >       /* Only register the device if both the codec and platform have
>  >        * been registered */
>  > -     if ((!of_soc->device.codec_data) || (!of_soc->platform_node))
>  > +     if ((!of_soc->device.codec_data) || (!of_soc->platform_node) || !machine_name)
>  >               return;
>
>
> ...this doesn't just allow a custom machine driver, it requires that
>  something configures at least the machine name.  That's not a problem
>  from the ASoC point of view but possibly from the PowerPC side?

You have to configure at least the name. Otherwise if the machine
driver is the last to register, how do you know to hold off the final
registration and wait for the machine driver to appear?

Or is it ok for me to change these after the platform device has been created?
		of_soc->dai_link.ops = machine_ops;
		of_soc->machine.name = machine_name;

I have to have the machine driver in order to control my external clock chips.

-- 
Jon Smirl
jonsmirl at gmail.com


More information about the Alsa-devel mailing list