Hi Alan, Thanks for the response. Please find my response inline. > -----Original Message----- > From: Alan Tull [mailto:atull@xxxxxxxxxx] > Sent: Tuesday, April 9, 2019 1:57 AM > To: Moritz Fischer <mdf@xxxxxxxxxx> > Cc: Michal Simek <michals@xxxxxxxxxx>; Nava kishore Manne > <navam@xxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland > <mark.rutland@xxxxxxx>; Rajan Vaja <RAJANV@xxxxxxxxxx>; Jolly Shah > <JOLLYS@xxxxxxxxxx>; linux-fpga@xxxxxxxxxxxxxxx; open list:OPEN FIRMWARE > AND FLATTENED DEVICE TREE BINDINGS <devicetree@xxxxxxxxxxxxxxx>; > moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE <linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx>; linux-kernel <linux-kernel@xxxxxxxxxxxxxxx>; > kishore m <chinnikishore369@xxxxxxxxx> > Subject: Re: [PATCH v4 3/3] fpga manager: Adding FPGA Manager support for > Xilinx zynqmp > > On Mon, Apr 8, 2019 at 11:51 AM Moritz Fischer <mdf@xxxxxxxxxx> wrote: > > > > Hi Michal, > > > > On Mon, Apr 08, 2019 at 04:36:15PM +0200, Michal Simek wrote: > > > On 08. 04. 19 16:17, Alan Tull wrote: > > > > On Mon, Apr 8, 2019 at 7:39 AM Nava kishore Manne > <navam@xxxxxxxxxx> wrote: > > > >> > > > >> Hi Alan, > > > >> > > > >> Thanks for look into it and providing the ACK. > > > >> I got one minor comments from Moritz Fischer do you want me fix that > issue now or I can fix it later as it’s a minor comment? > > > > > > > > Please fix for Moritz comment. Will fix in the next version. Regards, Navakishore.