RE: [PATCH v12 0/6] drivers: Introduce firmware driver for ZynqMP core

From: Jolly Shah
Date: Mon Sep 24 2018 - 14:22:04 EST


Hi Olof,

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

Sounds good. I will make required changes and submit as incremental patches along with clock driver.

> 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.
>
>

Thanks. Michal will create pull request for same.

Thanks,
Jolly Shah

>
> -Olof