On Tue, Dec 5, 2017 at 1:38 PM, Dhaval Rajeshbhai Shah <[email protected]> wrote: > From: [email protected] [mailto:[email protected]] On Behalf Of > Arnd Bergmann > Sent: Tuesday, December 05, 2017 4:19 AM > To: Dhaval Rajeshbhai Shah <[email protected]> > Cc: gregkh <[email protected]>; Linux Kernel Mailing List > <[email protected]>; Michal Simek <[email protected]>; Hyun > Kwon <[email protected]>; Dhaval Rajeshbhai Shah <[email protected]> > Subject: Re: [PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU > logicoreIP init driver > > On Tue, Dec 5, 2017 at 12:43 PM, Dhaval Shah <[email protected]> wrote: >> Xilinx ZYNQMP VCU Init driver is based on the new LogiCoreIP design >> created. This driver will provide the api which can be used by the >> encoder and decoder driver to get the configured value. >> >> Signed-off-by: Dhaval Shah <[email protected]> > > Can you explain what a "VCU" is and why there is no existing subsystem for it? > > [Dhaval] : This VCU means Video codec unit. Here, this driver is for the > logicoreIP and not the VCU which is created to support the PS and PL > isolation and to provide the clock related information. So this is not a VCU > driver and but just a intermediate driver which supports logicoreIP. That's > why no subsystem for this.
What are PS and PL then? > If this is the "video codec unit", I'd suggest moving it to > drivers/media/platform and sending the patch to the > [email protected] list instead. > [Dhaval] : This is not the Video codec unit but the driver to provide the > support the IP created for the PS and PL isolation and clock related > information. That why I have placed this under the "drivers/misc/". Is the driver useful without a codec driver then? (I assume this will be answered when you explain PS and PL). If not, just put it in the same directory as the codec. Arnd
