Messages in this thread | | | Date | Thu, 27 Jul 2017 13:14:02 +0800 | From | Wu Hao <> | Subject | Re: [PATCH v2 04/22] fpga: mgr: add region_id to fpga_image_info |
| |
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 > >
| |