[alsa-devel] [PATCH] ASoC: tegra: Add platform driver for rt5677 audio codec

[Mark Brown]

On Mon, Dec 29, 2014 at 05:25:59AM -0800, Anatol Pomozov wrote:
> On Fri, Dec 26, 2014 at 8:38 AM, Mark Brown <broonie at kernel.org> wrote:

> >> +#if defined(CONFIG_MODULES) && defined(MODULE)
> >> +     request_module("snd_soc_tegra30_i2s");
> >> +#endif

> > What's this all about then?

> Sorry, my question was not clear. I meant is it ok to put this
> request_module() to upstream code? If not then just remove it before
> merging my patch.

What question, I'm not seeing a question in the patch (I'd expect it to
be after the --- if one was being asked)?  I've no idea what it's there
for so it's hard to tell, my first thought would be no and you need to
fix whatever you're working around directly.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20141229/c8741afa/attachment.sig>