From: Moritz Fischer <mdf@kernel.org> To: Wu Hao <hao.wu@intel.com> Cc: Alan Tull <atull@kernel.org>, Moritz Fischer <mdf@kernel.org>, linux-fpga@vger.kernel.org, linux-kernel <linux-kernel@vger.kernel.org>, linux-api@vger.kernel.org, "Kang, Luwei" <luwei.kang@intel.com>, "Zhang, Yi Z" <yi.z.zhang@intel.com>, Tim Whisonant <tim.whisonant@intel.com>, Enno Luebbers <enno.luebbers@intel.com>, Shiva Rao <shiva.rao@intel.com>, Christopher Rauer <christopher.rauer@intel.com> Subject: Re: [PATCH v3 15/21] fpga: dfl: add fpga bridge platform driver for FME Date: Thu, 1 Feb 2018 16:11:08 +0100 [thread overview] Message-ID: <20180201151108.dyxzcpjagfmxgjgb@insfl13.etec.uni-karlsruhe.de> (raw) In-Reply-To: <20180201051543.GC10273@hao-dev> [-- Attachment #1: Type: text/plain, Size: 2246 bytes --] Hi Hao, On Thu, Feb 01, 2018 at 01:15:43PM +0800, Wu Hao wrote: > On Wed, Jan 31, 2018 at 09:16:58AM -0600, Alan Tull wrote: > > On Mon, Nov 27, 2017 at 12:42 AM, Wu Hao <hao.wu@intel.com> wrote: > > > > Hi Hao, > > > > One fix below. Besides that, please add my ack. > > > > > This patch adds fpga bridge platform driver for FPGA Management Engine. > > > It implements the enable_set call back for fpga bridge. > > > > > > Signed-off-by: Tim Whisonant <tim.whisonant@intel.com> > > > Signed-off-by: Enno Luebbers <enno.luebbers@intel.com> > > > Signed-off-by: Shiva Rao <shiva.rao@intel.com> > > > Signed-off-by: Christopher Rauer <christopher.rauer@intel.com> > > > Signed-off-by: Wu Hao <hao.wu@intel.com> > > > > Acked-by: Alan Tull <atull@kernel.org> Acked-by: Moritz Fischer <mdf@kernel.org> > > > > > diff --git a/drivers/fpga/fpga-dfl-fme-br.c b/drivers/fpga/fpga-dfl-fme-br.c > > > new file mode 100644 > > > index 0000000..db2603b > > > --- /dev/null > > > +++ b/drivers/fpga/fpga-dfl-fme-br.c > > > @@ -0,0 +1,87 @@ > > > +/* > > > + * FPGA Bridge Driver for FPGA Management Engine (FME) > > > + * > > > + * Copyright (C) 2017 Intel Corporation, Inc. > > > + * > > > + * Authors: > > > + * Wu Hao <hao.wu@intel.com> > > > + * Joseph Grecco <joe.grecco@intel.com> > > > + * Enno Luebbers <enno.luebbers@intel.com> > > > + * Tim Whisonant <tim.whisonant@intel.com> > > > + * Ananda Ravuri <ananda.ravuri@intel.com> > > > + * Henry Mitchel <henry.mitchel@intel.com> > > > + * > > > + * This work is licensed under the terms of the GNU GPL version 2. > > > > If we're using the SPDX, then please get rid of the above line. > > > > > + * SPDX-License-Identifier: GPL-2.0 > > > + */ > > > > The SPDX line should be the first line of the file with a // (for .c > > files), please see: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/process/license-rules.rst > > > > Same thing for other files. > > Thanks for the review. > > Sure, will fix this for all driver files in the next version. > > Thanks > Hao > > > > > Alan Looks good, feel free to add my Acked-by: with fixes Alan requested, Moritz [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Moritz Fischer <mdf-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> To: Wu Hao <hao.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> Cc: Alan Tull <atull-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Moritz Fischer <mdf-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, linux-fpga-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, "Kang, Luwei" <luwei.kang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, "Zhang, Yi Z" <yi.z.zhang-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, Tim Whisonant <tim.whisonant-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, Enno Luebbers <enno.luebbers-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, Shiva Rao <shiva.rao-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>, Christopher Rauer <christopher.rauer-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> Subject: Re: [PATCH v3 15/21] fpga: dfl: add fpga bridge platform driver for FME Date: Thu, 1 Feb 2018 16:11:08 +0100 [thread overview] Message-ID: <20180201151108.dyxzcpjagfmxgjgb@insfl13.etec.uni-karlsruhe.de> (raw) In-Reply-To: <20180201051543.GC10273@hao-dev> [-- Attachment #1: Type: text/plain, Size: 2664 bytes --] Hi Hao, On Thu, Feb 01, 2018 at 01:15:43PM +0800, Wu Hao wrote: > On Wed, Jan 31, 2018 at 09:16:58AM -0600, Alan Tull wrote: > > On Mon, Nov 27, 2017 at 12:42 AM, Wu Hao <hao.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> wrote: > > > > Hi Hao, > > > > One fix below. Besides that, please add my ack. > > > > > This patch adds fpga bridge platform driver for FPGA Management Engine. > > > It implements the enable_set call back for fpga bridge. > > > > > > Signed-off-by: Tim Whisonant <tim.whisonant-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > Signed-off-by: Enno Luebbers <enno.luebbers-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > Signed-off-by: Shiva Rao <shiva.rao-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > Signed-off-by: Christopher Rauer <christopher.rauer-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > Signed-off-by: Wu Hao <hao.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > > Acked-by: Alan Tull <atull-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Acked-by: Moritz Fischer <mdf-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > > > > > diff --git a/drivers/fpga/fpga-dfl-fme-br.c b/drivers/fpga/fpga-dfl-fme-br.c > > > new file mode 100644 > > > index 0000000..db2603b > > > --- /dev/null > > > +++ b/drivers/fpga/fpga-dfl-fme-br.c > > > @@ -0,0 +1,87 @@ > > > +/* > > > + * FPGA Bridge Driver for FPGA Management Engine (FME) > > > + * > > > + * Copyright (C) 2017 Intel Corporation, Inc. > > > + * > > > + * Authors: > > > + * Wu Hao <hao.wu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > + * Joseph Grecco <joe.grecco-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > + * Enno Luebbers <enno.luebbers-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > + * Tim Whisonant <tim.whisonant-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > + * Ananda Ravuri <ananda.ravuri-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > + * Henry Mitchel <henry.mitchel-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> > > > + * > > > + * This work is licensed under the terms of the GNU GPL version 2. > > > > If we're using the SPDX, then please get rid of the above line. > > > > > + * SPDX-License-Identifier: GPL-2.0 > > > + */ > > > > The SPDX line should be the first line of the file with a // (for .c > > files), please see: > > > > https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/Documentation/process/license-rules.rst > > > > Same thing for other files. > > Thanks for the review. > > Sure, will fix this for all driver files in the next version. > > Thanks > Hao > > > > > Alan Looks good, feel free to add my Acked-by: with fixes Alan requested, Moritz [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --]
next prev parent reply other threads:[~2018-02-01 15:11 UTC|newest] Thread overview: 151+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-11-27 6:42 [PATCH v3 00/21] Intel FPGA Device Drivers Wu Hao 2017-11-27 6:42 ` [PATCH v3 01/21] docs: fpga: add a document for Intel FPGA driver overview Wu Hao 2017-11-27 6:42 ` Wu Hao 2017-12-04 19:55 ` Alan Tull 2017-12-05 3:57 ` Wu Hao 2017-12-05 3:57 ` Wu Hao 2017-12-06 10:04 ` David Laight 2017-12-20 22:31 ` Alan Tull 2017-12-20 22:31 ` Alan Tull 2017-12-21 6:02 ` Wu Hao 2017-12-21 6:02 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 02/21] fpga: mgr: add region_id to fpga_image_info Wu Hao 2017-11-29 6:11 ` Moritz Fischer 2017-11-29 6:11 ` Moritz Fischer 2017-12-04 20:26 ` Alan Tull 2017-12-05 3:36 ` Wu Hao 2017-12-05 3:36 ` Wu Hao 2018-01-31 15:35 ` Alan Tull 2018-01-31 15:35 ` Alan Tull 2018-02-01 5:05 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 03/21] fpga: mgr: add status for fpga-manager Wu Hao 2017-12-04 20:55 ` Alan Tull 2017-12-04 20:55 ` Alan Tull 2017-12-05 4:08 ` Wu Hao 2017-12-05 4:08 ` Wu Hao 2017-12-12 18:18 ` Alan Tull 2017-12-13 4:48 ` Wu Hao 2017-12-13 4:48 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 04/21] fpga: add device feature list support Wu Hao 2017-11-27 6:42 ` Wu Hao 2017-11-29 6:07 ` Moritz Fischer 2017-11-29 6:07 ` Moritz Fischer 2017-11-30 5:59 ` Wu Hao 2017-12-20 22:29 ` Alan Tull 2017-12-21 0:58 ` Alan Tull 2017-12-21 7:22 ` Wu Hao 2017-12-21 7:22 ` Wu Hao 2017-12-22 8:45 ` Wu Hao 2018-01-31 23:22 ` Alan Tull 2018-01-31 23:22 ` Alan Tull 2017-11-27 6:42 ` [PATCH v3 05/21] fpga: dfl: add chardev support for feature devices Wu Hao 2017-11-27 6:42 ` [PATCH v3 06/21] fpga: dfl: adds fpga_cdev_find_port Wu Hao 2018-02-05 22:08 ` Alan Tull 2018-02-06 2:37 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 07/21] fpga: dfl: add feature device infrastructure Wu Hao 2017-11-27 6:42 ` [PATCH v3 08/21] fpga: add Intel FPGA DFL PCIe device Wu Hao 2017-11-27 10:28 ` David Laight 2017-11-27 10:28 ` David Laight 2017-11-28 3:15 ` Wu Hao 2017-11-28 3:15 ` Wu Hao 2017-12-04 19:46 ` Alan Tull 2017-12-04 19:46 ` Alan Tull 2017-12-05 3:33 ` Wu Hao 2017-12-05 3:33 ` Wu Hao 2017-12-05 17:00 ` Alan Tull 2017-12-06 5:30 ` Wu Hao 2017-12-06 9:44 ` David Laight 2017-12-06 9:44 ` David Laight 2017-12-06 15:29 ` Alan Tull 2017-12-06 15:29 ` Alan Tull 2017-12-06 16:28 ` David Laight 2017-12-06 16:28 ` David Laight 2017-12-06 16:28 ` David Laight 2017-12-06 22:39 ` Alan Tull 2018-02-01 21:59 ` Alan Tull 2018-02-01 21:59 ` Alan Tull 2018-02-13 9:36 ` Wu Hao 2017-12-06 9:34 ` David Laight 2017-12-06 9:34 ` David Laight 2017-12-07 3:47 ` Wu Hao 2017-12-07 3:47 ` Wu Hao 2017-12-06 9:31 ` David Laight 2017-12-06 9:31 ` David Laight 2017-12-06 9:31 ` David Laight 2017-11-27 6:42 ` [PATCH v3 09/21] fpga: intel-dfl-pci: add enumeration for feature devices Wu Hao 2017-12-07 21:41 ` Alan Tull 2017-12-07 21:41 ` Alan Tull 2017-12-08 9:25 ` Wu Hao 2017-12-08 9:25 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 10/21] fpga: dfl: add FPGA Management Engine driver basic framework Wu Hao 2017-11-27 6:42 ` [PATCH v3 11/21] fpga: dfl: fme: add header sub feature support Wu Hao 2018-02-12 16:51 ` Alan Tull 2018-02-12 16:51 ` Alan Tull 2018-02-13 3:44 ` Wu Hao 2018-02-13 3:44 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 12/21] fpga: dfl: fme: add FPGA_GET_API_VERSION/CHECK_EXTENSION ioctls support Wu Hao 2018-01-31 15:31 ` Alan Tull 2018-01-31 15:31 ` Alan Tull 2018-02-01 5:11 ` Wu Hao 2018-02-01 15:11 ` Moritz Fischer 2017-11-27 6:42 ` [PATCH v3 13/21] fpga: dfl: fme: add partial reconfiguration sub feature support Wu Hao 2017-11-27 6:42 ` [PATCH v3 14/21] fpga: dfl: add fpga manager platform driver for FME Wu Hao 2018-02-01 22:00 ` Alan Tull 2018-02-01 22:00 ` Alan Tull 2018-02-02 9:42 ` Wu Hao 2018-02-03 0:26 ` Luebbers, Enno 2018-02-03 0:26 ` Luebbers, Enno 2018-02-03 10:41 ` Moritz Fischer 2018-02-04 10:05 ` Wu Hao 2018-02-04 10:05 ` Wu Hao 2018-02-05 17:21 ` Alan Tull 2018-02-05 17:21 ` Alan Tull 2018-02-06 2:17 ` Wu Hao 2018-02-06 2:17 ` Wu Hao 2018-02-06 4:25 ` Alan Tull 2018-02-06 5:23 ` Wu Hao 2018-02-06 5:23 ` Wu Hao 2018-02-06 6:44 ` Moritz Fischer 2018-02-06 6:44 ` Moritz Fischer 2018-02-04 9:37 ` Wu Hao 2018-02-04 9:37 ` Wu Hao 2018-02-05 18:36 ` Luebbers, Enno 2018-02-05 18:36 ` Luebbers, Enno 2018-02-06 1:47 ` Wu Hao 2018-02-06 1:47 ` Wu Hao 2018-02-06 4:25 ` Alan Tull 2018-02-06 4:25 ` Alan Tull 2018-02-06 6:47 ` Wu Hao 2018-02-06 6:47 ` Wu Hao 2018-02-06 18:53 ` Alan Tull 2018-02-06 18:53 ` Alan Tull 2018-02-07 4:52 ` Wu Hao 2018-02-07 22:37 ` Alan Tull 2018-02-07 22:37 ` Alan Tull 2017-11-27 6:42 ` [PATCH v3 15/21] fpga: dfl: add fpga bridge " Wu Hao 2018-01-31 15:16 ` Alan Tull 2018-01-31 15:16 ` Alan Tull 2018-02-01 5:15 ` Wu Hao 2018-02-01 15:11 ` Moritz Fischer [this message] 2018-02-01 15:11 ` Moritz Fischer 2017-11-27 6:42 ` [PATCH v3 16/21] fpga: dfl: add fpga region " Wu Hao 2018-01-31 20:46 ` Alan Tull 2018-02-01 5:23 ` Wu Hao 2018-02-01 15:13 ` Moritz Fischer 2017-11-27 6:42 ` [PATCH v3 17/21] fpga: dfl: add FPGA Accelerated Function Unit driver basic framework Wu Hao 2017-11-27 6:42 ` [PATCH v3 18/21] fpga: dfl: afu: add header sub feature support Wu Hao 2018-02-12 17:43 ` Alan Tull 2018-02-12 17:43 ` Alan Tull 2018-02-13 3:33 ` Wu Hao 2018-02-13 3:33 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 19/21] fpga: dfl: afu: add FPGA_GET_API_VERSION/CHECK_EXTENSION ioctls support Wu Hao 2018-01-31 14:52 ` Alan Tull 2018-01-31 14:52 ` Alan Tull 2018-02-01 5:16 ` Wu Hao 2018-02-01 15:13 ` Moritz Fischer 2018-02-02 9:08 ` Wu Hao 2018-02-02 9:08 ` Wu Hao 2017-11-27 6:42 ` [PATCH v3 20/21] fpga: dfl: afu: add user afu sub feature support Wu Hao 2017-11-27 6:42 ` [PATCH v3 21/21] fpga: dfl: afu: add FPGA_PORT_DMA_MAP/UNMAP ioctls support Wu Hao 2017-11-27 21:26 ` [PATCH v3 00/21] Intel FPGA Device Drivers Alan Tull 2017-11-27 21:26 ` Alan Tull
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20180201151108.dyxzcpjagfmxgjgb@insfl13.etec.uni-karlsruhe.de \ --to=mdf@kernel.org \ --cc=atull@kernel.org \ --cc=christopher.rauer@intel.com \ --cc=enno.luebbers@intel.com \ --cc=hao.wu@intel.com \ --cc=linux-api@vger.kernel.org \ --cc=linux-fpga@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=luwei.kang@intel.com \ --cc=shiva.rao@intel.com \ --cc=tim.whisonant@intel.com \ --cc=yi.z.zhang@intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.