Messages in this thread | | | From | "Thokala, Srikanth" <> | Subject | RE: [PATCH v2 15/34] misc: xlink-pcie: Add XLink API interface | Date | Sun, 24 Jan 2021 11:46:00 +0000 |
| |
Hi Greg,
Thank you for the review.
> -----Original Message----- > From: Greg KH <gregkh@linuxfoundation.org> > Sent: Wednesday, January 20, 2021 11:30 PM > To: mgross@linux.intel.com > Cc: markgross@kernel.org; arnd@arndb.de; bp@suse.de; > damien.lemoal@wdc.com; dragan.cvetic@xilinx.com; corbet@lwn.net; > leonard.crestez@nxp.com; palmerdabbelt@google.com; > paul.walmsley@sifive.com; peng.fan@nxp.com; robh+dt@kernel.org; > shawnguo@kernel.org; jassisinghbrar@gmail.com; linux- > kernel@vger.kernel.org; Thokala, Srikanth <srikanth.thokala@intel.com> > Subject: Re: [PATCH v2 15/34] misc: xlink-pcie: Add XLink API interface > > On Fri, Jan 08, 2021 at 01:25:41PM -0800, mgross@linux.intel.com wrote: > > From: Srikanth Thokala <srikanth.thokala@intel.com> > > > > Provide interface for XLink layer to interact with XLink PCIe transport > > layer on both local host and remote host. > > > > Cc: Arnd Bergmann <arnd@arndb.de> > > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Reviewed-by: Mark Gross <mgross@linux.intel.com> > > Signed-off-by: Srikanth Thokala <srikanth.thokala@intel.com> > > --- > > drivers/misc/xlink-pcie/common/interface.c | 109 +++++++++++++++++++ > > drivers/misc/xlink-pcie/local_host/Makefile | 1 + > > drivers/misc/xlink-pcie/remote_host/Makefile | 1 + > > 3 files changed, 111 insertions(+) > > create mode 100644 drivers/misc/xlink-pcie/common/interface.c > > > > diff --git a/drivers/misc/xlink-pcie/common/interface.c > b/drivers/misc/xlink-pcie/common/interface.c > > new file mode 100644 > > index 000000000000..56c1d9ed9d8f > > --- /dev/null > > +++ b/drivers/misc/xlink-pcie/common/interface.c > > @@ -0,0 +1,109 @@ > > +// SPDX-License-Identifier: GPL-2.0-only > > > +/************************************************************************ > ***** > > + * > > + * Intel Keem Bay XLink PCIe Driver > > + * > > + * Copyright (C) 2020 Intel Corporation > > + * > > + > ************************************************************************** > **/ > > Do you really need the ******* mess? :)
It is not required; I will clean up.
> > > + > > +#include <linux/xlink_drv_inf.h> > > + > > +#include "core.h" > > +#include "xpcie.h" > > + > > +/* Define xpcie driver interface API */ > > +int xlink_pcie_get_device_list(u32 *sw_device_id_list, u32 > *num_devices) > > +{ > > + if (!sw_device_id_list || !num_devices) > > + return -EINVAL; > > + > > + *num_devices = intel_xpcie_get_device_num(sw_device_id_list); > > + > > + return 0; > > +} > > +EXPORT_SYMBOL(xlink_pcie_get_device_list); > > EXPORT_SYMBOL_GPL() for all of these perhaps? I have to ask...
I agree with Mark, will make the change to use EXPORT_SYMBOL_GPL().
Thanks! Srikanth
> > thanks, > > greg k-h
| |