Hi Arnd, Replies inline.
-----Original Message----- From: [email protected] [mailto:[email protected]] On Behalf Of Arnd Bergmann Sent: Tuesday, December 05, 2017 5:32 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]> Subject: Re: [PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU logicoreIP init driver 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? [Dhaval ] : PS is the Processing system. PL is the Programmable logic. There are few register in the logicoreIP which provides the isolation between this two in the chip. > 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. [Dhaval ] : Yes. Driver is useful without a codec driver as well. No dependency on the Codec driver. Arnd
