Hi Arnd, Thanks a lot for the review. Replies inline.
-----Original Message----- 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. 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/". Arnd
