Received: by 2002:a25:d7c1:0:0:0:0:0 with SMTP id o184csp367406ybg; Tue, 22 Oct 2019 22:34:19 -0700 (PDT) X-Google-Smtp-Source: APXvYqyxxuv/BArzcNegAg9ls0hYjD+xElyZQnAH5phc33oo/NlsuIj1aN+PwgagOybMGQMmRptU X-Received: by 2002:a05:6402:1b91:: with SMTP id cc17mr34877495edb.180.1571808859746; Tue, 22 Oct 2019 22:34:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1571808859; cv=none; d=google.com; s=arc-20160816; b=X1Czp7DPTr7DCkO5LbupOge9t7tOZpEg9fsgTMdW8cb1UCV8zY+X9wIF0NDaSF/oky gw2fEQ1z/eggOkXb8LKMTWFeaiqaCyj+Kh1w18AdcLhh7Cqw90Sj4Y2yLpGqu02jj6k9 U6I0pibDN0Qd43CIQbcFRNd1tEHNZeXif5lwA8Yfkno/Y9HBbNLZ0A/gPgRsDLJ2SClT EBtDRJhPvhIzlxcj6RCJdTCHKEPyJGD6GI+FzWSAdEO7kd3vl5KUolcvwItyjL5E6+sS 3LB0kqp/sFN2TCjD1U0cqlU35HDNkq1dNoButvdgNLUVH2PWYnYXxCZTU0lEr8ick8CX OiaQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=RV4x8pAxIfyyBEBnwd2Po3FGzDi6rTXFawjDYqEnXK0=; b=aBokFsWwbTE4RcLxW3yEhWmCXrjobhnzC5t8ZC8KSWcJKBynWtg9uogaFX3T+hCDwY VeoqOYJjraUcci5czaa5mApRmBe2HVcgbER3WNeDSxut37Xu6LqGPvgMfEzBD9zRVAbn 7YEKeBCXEg+gJUeDle1k6tkkr1L6Ei/7dRsO03Q8CLmXnWF1JxnaIf/YuiCtln3yLV/3 ksL/Yu6PJ4XQZuLAGiRjhB2+gbYr3ySVhblr8CZlhANbbVgzdgyG/ule4B5N7MZUWDuM XZnujOUHefytm8HrYBaUhlDkLfjVmp12OArj2ldMBBKBgnFumID862g8c1HV+v2NRgCI 4oCg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=iqwY2860; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id m33si13923706edc.94.2019.10.22.22.33.55; Tue, 22 Oct 2019 22:34:19 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=iqwY2860; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727582AbfJWE21 (ORCPT + 99 others); Wed, 23 Oct 2019 00:28:27 -0400 Received: from mail.kernel.org ([198.145.29.99]:56286 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725773AbfJWE21 (ORCPT ); Wed, 23 Oct 2019 00:28:27 -0400 Received: from localhost (unknown [122.181.210.10]) (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 9B2BA214B2; Wed, 23 Oct 2019 04:28:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1571804906; bh=pud1AqfrL2+yGH3/Mj/96RE5LhtzfD6/WYjBiXtPT38=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=iqwY2860h37CrasP50CZJnw67mckyQb0+sRAUhAsM0LM9RQyIAMU0IbWjZJbzb3D0 9yHNOVAqpoNp60Vc8vVAxZnBGrn6Q0IZ7+b9Ht2e5+W2/iHgQ+Ocso03WHoEwj/Zaq LG2USbNiXQ0tVBlfewWmUCCYm0ZJWeSr6aHHlUBs= Date: Wed, 23 Oct 2019 09:58:18 +0530 From: Vinod Koul To: Peng Ma Cc: dan.j.williams@intel.com, leoyang.li@nxp.com, anders.roxell@linaro.org, linux-kernel@vger.kernel.org, dmaengine@vger.kernel.org Subject: Re: [PATCH] dmaengine: fsl-dpaa2-qdma: Fixed build error when enable dpaa2 qdma module driver Message-ID: <20191023042818.GN2654@vkoul-mobl> References: <20191023021959.35596-1-peng.ma@nxp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191023021959.35596-1-peng.ma@nxp.com> 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 Hi Peng, On 23-10-19, 10:19, Peng Ma wrote: A patch title should detail the change it is doing so a better patch title would be: "dmaengine: fsl-dpaa2-qdma: export the symbols" > Fixed the following error: > WARNING: modpost: missing MODULE_LICENSE() in drivers/dma/fsl-dpaa2-qdma/dpdmai.o > see include/linux/module.h for more information > GZIP arch/arm64/boot/Image.gz > ERROR: "dpdmai_enable" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_set_rx_queue" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_get_tx_queue" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_get_rx_queue" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_get_attributes" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_open" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_close" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_disable" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > ERROR: "dpdmai_reset" [drivers/dma/fsl-dpaa2-qdma/dpaa2-qdma.ko] undefined! > WARNING: "HYPERVISOR_platform_op" [vmlinux] is a static EXPORT_SYMBOL_GPL > make[2]: *** [__modpost] Error 1 > make[1]: *** [modules] Error 2 > make[1]: *** Waiting for unfinished jobs.... > make: *** [sub-make] Error 2 And here in the log, you should say the symbols were not exported leading to error So export it > > Signed-off-by: Peng Ma > Reported-by: Anders Roxell > --- > drivers/dma/fsl-dpaa2-qdma/dpdmai.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/dma/fsl-dpaa2-qdma/dpdmai.c b/drivers/dma/fsl-dpaa2-qdma/dpdmai.c > index fbc2b2f..f8a1f66 100644 > --- a/drivers/dma/fsl-dpaa2-qdma/dpdmai.c > +++ b/drivers/dma/fsl-dpaa2-qdma/dpdmai.c > @@ -1,6 +1,7 @@ > // SPDX-License-Identifier: GPL-2.0 > // Copyright 2019 NXP > > +#include > #include > #include > #include > @@ -90,6 +91,7 @@ int dpdmai_open(struct fsl_mc_io *mc_io, u32 cmd_flags, > > return 0; > } > +EXPORT_SYMBOL_GPL(dpdmai_open); > > /** > * dpdmai_close() - Close the control session of the object > @@ -113,6 +115,7 @@ int dpdmai_close(struct fsl_mc_io *mc_io, u32 cmd_flags, u16 token) > /* send command to mc*/ > return mc_send_command(mc_io, &cmd); > } > +EXPORT_SYMBOL_GPL(dpdmai_close); > > /** > * dpdmai_create() - Create the DPDMAI object > @@ -177,6 +180,7 @@ int dpdmai_enable(struct fsl_mc_io *mc_io, u32 cmd_flags, u16 token) > /* send command to mc*/ > return mc_send_command(mc_io, &cmd); > } > +EXPORT_SYMBOL_GPL(dpdmai_enable); > > /** > * dpdmai_disable() - Disable the DPDMAI, stop sending and receiving frames. > @@ -197,6 +201,7 @@ int dpdmai_disable(struct fsl_mc_io *mc_io, u32 cmd_flags, u16 token) > /* send command to mc*/ > return mc_send_command(mc_io, &cmd); > } > +EXPORT_SYMBOL_GPL(dpdmai_disable); > > /** > * dpdmai_reset() - Reset the DPDMAI, returns the object to initial state. > @@ -217,6 +222,7 @@ int dpdmai_reset(struct fsl_mc_io *mc_io, u32 cmd_flags, u16 token) > /* send command to mc*/ > return mc_send_command(mc_io, &cmd); > } > +EXPORT_SYMBOL_GPL(dpdmai_reset); > > /** > * dpdmai_get_attributes() - Retrieve DPDMAI attributes. > @@ -252,6 +258,7 @@ int dpdmai_get_attributes(struct fsl_mc_io *mc_io, u32 cmd_flags, > > return 0; > } > +EXPORT_SYMBOL_GPL(dpdmai_get_attributes); > > /** > * dpdmai_set_rx_queue() - Set Rx queue configuration > @@ -285,6 +292,7 @@ int dpdmai_set_rx_queue(struct fsl_mc_io *mc_io, u32 cmd_flags, u16 token, > /* send command to mc*/ > return mc_send_command(mc_io, &cmd); > } > +EXPORT_SYMBOL_GPL(dpdmai_set_rx_queue); > > /** > * dpdmai_get_rx_queue() - Retrieve Rx queue attributes. > @@ -325,6 +333,7 @@ int dpdmai_get_rx_queue(struct fsl_mc_io *mc_io, u32 cmd_flags, u16 token, > > return 0; > } > +EXPORT_SYMBOL_GPL(dpdmai_get_rx_queue); > > /** > * dpdmai_get_tx_queue() - Retrieve Tx queue attributes. > @@ -364,3 +373,6 @@ int dpdmai_get_tx_queue(struct fsl_mc_io *mc_io, u32 cmd_flags, > > return 0; > } > +EXPORT_SYMBOL_GPL(dpdmai_get_tx_queue); > + > +MODULE_LICENSE("GPL v2"); > -- > 2.9.5 -- ~Vinod