From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 87732C4CEC9 for ; Wed, 18 Sep 2019 19:50:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4CA4221BE5 for ; Wed, 18 Sep 2019 19:50:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1568836253; bh=L3nHUsrfa6UXv3Wh18UBzqltAVEsOn65UXCWC291020=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=g5HcW1YDdYr0crf4TH4ec+IKVM8i6f9mxUcuJfyJrTfSO2pKlswfsMJVN9HOKdZjh ubWV3DJRFljKYl4mrYBg6C00OvHd9owWTj8AbY0f49pfijG9twnatft43vRzkrhk4b iVA5zMb8c06j6Z7hrPjWqPLLA2jXNjthMRWOO/6E= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1731830AbfIRTuw (ORCPT ); Wed, 18 Sep 2019 15:50:52 -0400 Received: from mail.kernel.org ([198.145.29.99]:53950 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726268AbfIRTuw (ORCPT ); Wed, 18 Sep 2019 15:50:52 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id B342D207FC; Wed, 18 Sep 2019 19:50:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1568836249; bh=L3nHUsrfa6UXv3Wh18UBzqltAVEsOn65UXCWC291020=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=j+fH8zRN+Dq/kuopHk/1kdq2G/lrFdnvWpchuRslixgNUL4q7xwFAWvSpxsfuPLcQ 1rJpK/aiII9pzo/P9xbk95fkcgyvFn1ppHpOG+wkGW5B1z9zUaEqa2a+yNOGFHwdCV LqPV8O8vCdPfe2gIfSnRmmJOHRZuIgqFiK+9yZ+Q= Date: Wed, 18 Sep 2019 21:50:45 +0200 From: Greg KH To: Pierre-Louis Bossart Cc: alsa-devel@alsa-project.org, tiwai@suse.de, linux-kernel@vger.kernel.org, Ranjani Sridharan , vkoul@kernel.org, broonie@kernel.org, srinivas.kandagatla@linaro.org, jank@cadence.com, slawomir.blauciak@intel.com, Sanyog Kale , Bard liao , Rander Wang Subject: Re: [alsa-devel] [RFC PATCH 8/9] soundwire: intel: remove platform devices and provide new interface Message-ID: <20190918195045.GB2020317@kroah.com> References: <20190916212342.12578-1-pierre-louis.bossart@linux.intel.com> <20190916212342.12578-9-pierre-louis.bossart@linux.intel.com> <20190917055512.GE2058532@kroah.com> <20190918120629.GD1901208@kroah.com> <20190918135302.GA1919118@kroah.com> <20190918135431.GA1919350@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.12.1 (2019-06-15) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 18, 2019 at 10:14:51AM -0500, Pierre-Louis Bossart wrote: > On 9/18/19 8:54 AM, Greg KH wrote: > > On Wed, Sep 18, 2019 at 03:53:02PM +0200, Greg KH wrote: > > > On Wed, Sep 18, 2019 at 08:48:33AM -0500, Pierre-Louis Bossart wrote: > > > > On 9/18/19 7:06 AM, Greg KH wrote: > > > > > On Tue, Sep 17, 2019 at 09:29:52AM -0500, Pierre-Louis Bossart wrote: > > > > > > On 9/17/19 12:55 AM, Greg KH wrote: > > > > > > > On Mon, Sep 16, 2019 at 04:23:41PM -0500, Pierre-Louis Bossart wrote: > > > > > > > > +/** > > > > > > > > + * sdw_intel_probe() - SoundWire Intel probe routine > > > > > > > > + * @parent_handle: ACPI parent handle > > > > > > > > + * @res: resource data > > > > > > > > + * > > > > > > > > + * This creates SoundWire Master and Slave devices below the controller. > > > > > > > > + * All the information necessary is stored in the context, and the res > > > > > > > > + * argument pointer can be freed after this step. > > > > > > > > + */ > > > > > > > > +struct sdw_intel_ctx > > > > > > > > +*sdw_intel_probe(struct sdw_intel_res *res) > > > > > > > > +{ > > > > > > > > + return sdw_intel_probe_controller(res); > > > > > > > > +} > > > > > > > > +EXPORT_SYMBOL(sdw_intel_probe); > > > > > > > > + > > > > > > > > +/** > > > > > > > > + * sdw_intel_startup() - SoundWire Intel startup > > > > > > > > + * @ctx: SoundWire context allocated in the probe > > > > > > > > + * > > > > > > > > + */ > > > > > > > > +int sdw_intel_startup(struct sdw_intel_ctx *ctx) > > > > > > > > +{ > > > > > > > > + return sdw_intel_startup_controller(ctx); > > > > > > > > +} > > > > > > > > +EXPORT_SYMBOL(sdw_intel_startup); > > > > > > > > > > > > > > Why are you exporting these functions if no one calls them? > > > > > > > > > > > > They are used in the next series, see '[RFC PATCH 04/12] ASoC: SOF: Intel: > > > > > > add SoundWire configuration interface' > > > > > > > > > > That wasn't obvious :) > > > > > > > > > > Also, why not EXPORT_SYMBOL_GPL()? :) > > > > > > > > Since the beginning of this SoundWire work, the intent what that the code > > > > could be reused in non-GPL open-source circles, hence the dual license and > > > > EXPORT_SYMBOL. > > > > > > Hah, you _have_ talked to your lawyers about this, right? > > > > > > You have a chance to do something like this for header files, for .c > > > files, good luck. That's going to be a hard road to go down. Many have > > > tried in the past, all but 1 have failed. > > > > Also note, the last I checked, the _default_ license for Linux kernel > > code from Intel was GPLv2. If you got an exception for this, please > > work with your legal council on how to do this "properly" as that was > > part of getting that exception, right? > > > > If you didn't get the exception, um, you have some people to go talk to, > > and how come I am the one asking you about this? :( > > All the legal due-diligence was done when SoundWire was initially > contributed in 2018. You asked that question at the time and I will point > you to the email exchange Alan Cox and you had on this topic [1]. > > [1] https://patchwork.kernel.org/patch/10015813/ Yes, that is fine, what I am saying here is that you are now asking the community to do this for you. You said in this thread: > For this series I added a disclaimer in the cover letter that those > parts need to be reviewed further to make sure there are no conflicts > with GPL. Why are you sending code out that you think might have conflicts before your lawyers have reviewed it? That's just screaming for problems in the future (hint, you distributed something in the previous emails...) Again, go and get this sorted out before dumping that kind of work on the community as this is something that you are having to deal with (i.e. it is self-inflicted). Don't make others do this for you here in public. Otherwise I will probably just purposefully tell you the wrong thing, and then watch what kind of fun your lawyers will have :) thanks, greg k-h