From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lubomir Rintel Subject: Re: [PATCH] ASoC: dwc: disallow building designware_pcm as a module Date: Wed, 19 Apr 2017 18:14:21 +0200 Message-ID: <1492618461.28092.18.camel@v3.sk> References: <20170418105954.23368-1-lkundrak@v3.sk> <20170418151834.c7e5fivmgn73sbwd@sirena.org.uk> <1492532010.28092.12.camel@v3.sk> <20170418171519.csiftftd434psyn5@sirena.org.uk> <2e39b000-bf25-6447-b53b-047401041f4e@synopsys.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from shell.v3.sk (shell.v3.sk [92.60.52.57]) by alsa0.perex.cz (Postfix) with ESMTP id B73E9266ADE for ; Wed, 19 Apr 2017 18:14:26 +0200 (CEST) In-Reply-To: <2e39b000-bf25-6447-b53b-047401041f4e@synopsys.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Jose Abreu , Mark Brown Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Arnd Bergmann List-Id: alsa-devel@alsa-project.org T24gV2VkLCAyMDE3LTA0LTE5IGF0IDE3OjEyICswMTAwLCBKb3NlIEFicmV1IHdyb3RlOgo+IEhp IEx1Ym9taXIsCj4gCj4gCj4gT24gMTgtMDQtMjAxNyAxODoxNSwgTWFyayBCcm93biB3cm90ZToK PiA+IE9uIFR1ZSwgQXByIDE4LCAyMDE3IGF0IDA2OjEzOjMwUE0gKzAyMDAsIEx1Ym9taXIgUmlu dGVsIHdyb3RlOgo+ID4gCj4gPiA+IEkgZG9uJ3QgdGhpbmsgZGVzaWdud2FyZV9wY20gaXMgYSBz ZXBhcmF0ZSBkcml2ZXIuIEl0IGxvb2tzCj4gPiA+IHRpZ2h0bHkKPiA+ID4gY291cGxlZCB3aXRo IGRlc2lnbndhcmVfaTJzOiB5b3UgY2FuIGVpdGhlciBkaXNhYmxlCj4gPiA+IGRlc2lnbndhcmVf cGNtCj4gPiA+IGFsdG9nZXRoZXIgYXQgYnVpbGQgdGltZSBvciBhbHdheXMgbG9hZCBpdCB0b2dl dGhlciB3aXRoCj4gPiA+IGRlc2lnbndhcmVfaTJzLgo+ID4gCj4gPiBZZXMsIHRoZXkncmUgY2xv c2VseSBjb3VwbGVkIGJ1dCB3ZSBtaWdodCBzdGlsbCB3YW50IHRoZW0gYm90aCBhcyBhCj4gPiBt b2R1bGUuCj4gCj4gVGhhbmtzIGZvciB0aGUgcGF0Y2ggYnV0IEkgYWdyZWUgd2l0aCBNYXJrLgo+ IAo+IEZvciB0aGUgcmVjb3JkIHlvdSBjYW4gYWRkICJNT0RVTEVfTElDRU5TRSjigJxEdWFsIE1J VC9HUEzigJ0pIi4KClNvcnJ5IGlmIEknbSBtaXNzaW5nIHNvbWV0aGluZzsgYnV0IEkgc3RpbGwg ZmFpbCB0byB1bmRlcnN0YW5kIHdoeSBpcwp0aGF0IGEgZ29vZCBpZGVhLgoKV2hhdCBpcyB0aGUg cG9pbnQgb2YgaGF2aW5nIGEgcGFpciBvZiBtb2R1bGVzIHRoYXQgZGVwZW5kIG9uIGVhY2ggb3Ro ZXIKYW5kIGNhbiBvbmx5IGJlIGxvYWRlZCB0b2dldGhlcj8KCj4gQmVzdCByZWdhcmRzLAo+IEpv c2UgTWlndWVsIEFicmV1CgpMdWJvCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCkFsc2EtZGV2ZWwgbWFpbGluZyBsaXN0CkFsc2EtZGV2ZWxAYWxzYS1wcm9q ZWN0Lm9yZwpodHRwOi8vbWFpbG1hbi5hbHNhLXByb2plY3Qub3JnL21haWxtYW4vbGlzdGluZm8v YWxzYS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965753AbdDSQOb convert rfc822-to-8bit (ORCPT ); Wed, 19 Apr 2017 12:14:31 -0400 Received: from shell.v3.sk ([92.60.52.57]:53519 "EHLO shell.v3.sk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965665AbdDSQO3 (ORCPT ); Wed, 19 Apr 2017 12:14:29 -0400 Message-ID: <1492618461.28092.18.camel@v3.sk> Subject: Re: [PATCH] ASoC: dwc: disallow building designware_pcm as a module From: Lubomir Rintel To: Jose Abreu , Mark Brown Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, Arnd Bergmann Date: Wed, 19 Apr 2017 18:14:21 +0200 In-Reply-To: <2e39b000-bf25-6447-b53b-047401041f4e@synopsys.com> References: <20170418105954.23368-1-lkundrak@v3.sk> <20170418151834.c7e5fivmgn73sbwd@sirena.org.uk> <1492532010.28092.12.camel@v3.sk> <20170418171519.csiftftd434psyn5@sirena.org.uk> <2e39b000-bf25-6447-b53b-047401041f4e@synopsys.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.6 (3.22.6-2.fc25) Mime-Version: 1.0 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2017-04-19 at 17:12 +0100, Jose Abreu wrote: > Hi Lubomir, > > > On 18-04-2017 18:15, Mark Brown wrote: > > On Tue, Apr 18, 2017 at 06:13:30PM +0200, Lubomir Rintel wrote: > > > > > I don't think designware_pcm is a separate driver. It looks > > > tightly > > > coupled with designware_i2s: you can either disable > > > designware_pcm > > > altogether at build time or always load it together with > > > designware_i2s. > > > > Yes, they're closely coupled but we might still want them both as a > > module. > > Thanks for the patch but I agree with Mark. > > For the record you can add "MODULE_LICENSE(“Dual MIT/GPL”)". Sorry if I'm missing something; but I still fail to understand why is that a good idea. What is the point of having a pair of modules that depend on each other and can only be loaded together? > Best regards, > Jose Miguel Abreu Lubo