On Tue, Dec 5, 2017 at 1:38 PM, Dhaval Rajeshbhai Shah <ds...@xilinx.com> wrote:
> From: arndbergm...@gmail.com [mailto:arndbergm...@gmail.com] On Behalf Of
> Arnd Bergmann
> Sent: Tuesday, December 05, 2017 4:19 AM
> To: Dhaval Rajeshbhai Shah <ds...@xilinx.com>
> Cc: gregkh <gre...@linuxfoundation.org>; Linux Kernel Mailing List
> <linux-kernel@vger.kernel.org>; Michal Simek <michal.si...@xilinx.com>; Hyun
> Kwon <hy...@xilinx.com>; Dhaval Rajeshbhai Shah <ds...@xilinx.com>
> 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 <dhaval.s...@xilinx.com> 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 <ds...@xilinx.com>
>
> 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
> linux-me...@vger.kernel.org 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
