Message ID | 1498441938-14046-5-git-send-email-hao.wu@intel.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers | show |
On Sun, Jun 25, 2017 at 8:52 PM, Wu Hao <hao.wu@intel.com> wrote: Hi Hao, > This patch adds region_id to fpga_image_info data structure, it > allows driver to pass region id information to fpga-mgr via > fpga_image_info for fpga reconfiguration function. > > Signed-off-by: Wu Hao <hao.wu@intel.com> > --- > include/linux/fpga/fpga-mgr.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/linux/fpga/fpga-mgr.h b/include/linux/fpga/fpga-mgr.h > index 66d0e32..b222a57 100644 > --- a/include/linux/fpga/fpga-mgr.h > +++ b/include/linux/fpga/fpga-mgr.h > @@ -95,6 +95,7 @@ struct fpga_image_info { > struct sg_table *sgt; > const char *buf; > size_t count; > + int region_id; Please add a line to document this region_id above where all the other struct members are documented. Alan > #ifdef CONFIG_OF > struct device_node *overlay; > #endif > -- > 1.8.3.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-fpga" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Jul 26, 2017 at 01:33:53PM -0500, Alan Tull wrote: > On Sun, Jun 25, 2017 at 8:52 PM, Wu Hao <hao.wu@intel.com> wrote: > > Hi Hao, > > > This patch adds region_id to fpga_image_info data structure, it > > allows driver to pass region id information to fpga-mgr via > > fpga_image_info for fpga reconfiguration function. > > > > Signed-off-by: Wu Hao <hao.wu@intel.com> > > --- > > include/linux/fpga/fpga-mgr.h | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/include/linux/fpga/fpga-mgr.h b/include/linux/fpga/fpga-mgr.h > > index 66d0e32..b222a57 100644 > > --- a/include/linux/fpga/fpga-mgr.h > > +++ b/include/linux/fpga/fpga-mgr.h > > @@ -95,6 +95,7 @@ struct fpga_image_info { > > struct sg_table *sgt; > > const char *buf; > > size_t count; > > + int region_id; > > Please add a line to document this region_id above where all the other > struct members are documented. Hi Alan Thanks for the comments, I will fix this in the next version. * @sgt: scatter/gather table containing FPGA image * @buf: contiguous buffer containing FPGA image * @count: size of buf + * @region_id: id of target region * @overlay: Device Tree overlay */ struct fpga_image_info { Thanks Hao > > Alan > > > #ifdef CONFIG_OF > > struct device_node *overlay; > > #endif > > -- > > 1.8.3.1 > > -- To unsubscribe from this list: send the line "unsubscribe linux-fpga" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/include/linux/fpga/fpga-mgr.h b/include/linux/fpga/fpga-mgr.h index 66d0e32..b222a57 100644 --- a/include/linux/fpga/fpga-mgr.h +++ b/include/linux/fpga/fpga-mgr.h @@ -95,6 +95,7 @@ struct fpga_image_info { struct sg_table *sgt; const char *buf; size_t count; + int region_id; #ifdef CONFIG_OF struct device_node *overlay; #endif
This patch adds region_id to fpga_image_info data structure, it allows driver to pass region id information to fpga-mgr via fpga_image_info for fpga reconfiguration function. Signed-off-by: Wu Hao <hao.wu@intel.com> --- include/linux/fpga/fpga-mgr.h | 1 + 1 file changed, 1 insertion(+)