Hi, On Fri, Aug 3, 2018 at 10:53 AM, Jolly Shah <jolly.shah@xxxxxxxxxx> wrote: > From: Rajan Vaja <rajanv@xxxxxxxxxx> > > Add ZynqMP firmware IOCTL API to control and configure > devices like PLLs, SD, Gem, etc. > > Signed-off-by: Rajan Vaja <rajanv@xxxxxxxxxx> > Signed-off-by: Jolly Shah <jollys@xxxxxxxxxx> This patch worries me somewhat. It's a transparent pass-through ioctl driver. Is there a spec available for what the implemented IOCTLs are? Should some of them be proper drivers instead of an opaque pass-through like this? Could some of them have stability impact on the platform such that there are security concerns and the list of arguments should somehow be sanitized? What's the intended usecase anyway? Just a debug tool during development, or something that you expect heavy use of by some userspace middleware? -Olof
