Hi Olof, > -----Original Message----- > From: Olof Johansson [mailto:o...@lixom.net] > Sent: Sunday, September 23, 2018 6:13 AM > To: Jolly Shah <jol...@xilinx.com> > Cc: Michal Simek <mich...@xilinx.com>; a...@kernel.org; Rajan Vaja > <raj...@xilinx.com>; linux-arm-ker...@lists.infradead.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH v12 0/6] drivers: Introduce firmware driver for ZynqMP > core > > On Thu, Sep 20, 2018 at 5:46 PM, Jolly Shah <jol...@xilinx.com> wrote: > > Hi Olof, > > > > As suggested, this patchset is generated without ioctl interface. Please > > review. > If you are ok, Michal can create a pull request for merge. > > Hi, > > I replied on the other thread as well; it sounds like you will need something > _like_ the ioctl interface for some of the dependent drivers. For those, > would it > be possible to only expose some of the ioctls through the interface, and not > just > pass through any call? I think that'd probably take care of my concern. >
I have pushed clock driver patch set including ioctl changes. Please review and let me know if any changes are required. Thanks, Jolly Shah > But yeah, go ahead and stage these in a pull request for now, that way we can > take care of these and merge them as the rest are getting ready. I don't want > to > hold you up more than needed; the patch set is already up to a very high > version > number and it's not fair to make you rework it completely yet again. > > > > -Olof
