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

["Jon Smirl" <jonsmirl@gmail.com>]

On 7/23/08, Mark Brown <broonie@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@gmail.com