Hi Olof,

> -----Original Message-----
> From: Olof Johansson [mailto:[email protected]]
> Sent: Sunday, September 23, 2018 6:13 AM
> To: Jolly Shah <[email protected]>
> Cc: Michal Simek <[email protected]>; [email protected]; Rajan Vaja
> <[email protected]>; [email protected]; linux-
> [email protected]
> Subject: Re: [PATCH v12 0/6] drivers: Introduce firmware driver for ZynqMP 
> core
> 
> On Thu, Sep 20, 2018 at 5:46 PM, Jolly Shah <[email protected]> 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

Reply via email to