From: Dhaval Rajeshbhai Shah <DSHAH@xilinx.com>
To: "'Arnd Bergmann'" <arnd@arndb.de>
CC: gregkh <gregkh@linuxfoundation.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Michal Simek <michal.simek@xilinx.com>, "Hyun Kwon" <hyunk@xilinx.com>
Subject: RE: [PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU
 logicoreIP init driver
Thread-Topic: [PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU
 logicoreIP init driver
Thread-Index: AQHTbb/vvsNWt82zs0iPUUhWqjiWdqM0qvKAgAAClLCAABHvAIABC67w
Date: Wed, 6 Dec 2017 05:44:44 +0000
Message-ID: <BY2PR0201MB1879BA2E8563A89953F6A9FFC1320@BY2PR0201MB1879.namprd02.prod.outlook.com>
References: <1512474212-12803-1-git-send-email-dshah@xilinx.com>
 <CAK8P3a3AbzBkxfhffnS3K-ehd1kQC1okyh5NF4OMBYPWwZ+KpQ@mail.gmail.com>
 <BY2PR0201MB1879415B035843E955D98393C13D0@BY2PR0201MB1879.namprd02.prod.outlook.com>
 <CAK8P3a2LAHJO9GibnsLmFyTV+dzB-nsggdDu23TqnkZmMYRvwg@mail.gmail.com>
In-Reply-To: <CAK8P3a2LAHJO9GibnsLmFyTV+dzB-nsggdDu23TqnkZmMYRvwg@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: a0884c88-f8b1-475b-d691-08d53c6c742d
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Dec 2017 05:44:44.3432
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 657af505-d5df-48d0-8300-c31994686c5c
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0201MB1008
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by nfs id vB65iuQD014360
Content-Length: 2669
Lines: 44

Hi Arnd,

Replies inline.

-----Original Message-----
From: arndbergmann@gmail.com [mailto:arndbergmann@gmail.com] On Behalf Of Arnd Bergmann
Sent: Tuesday, December 05, 2017 5:32 AM
To: Dhaval Rajeshbhai Shah <DSHAH@xilinx.com>
Cc: gregkh <gregkh@linuxfoundation.org>; Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Michal Simek <michal.simek@xilinx.com>; Hyun Kwon <hyunk@xilinx.com>
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 <DSHAH@xilinx.com> wrote:
> From: arndbergmann@gmail.com [mailto:arndbergmann@gmail.com] On Behalf 
> Of Arnd Bergmann
> Sent: Tuesday, December 05, 2017 4:19 AM
> To: Dhaval Rajeshbhai Shah <DSHAH@xilinx.com>
> Cc: gregkh <gregkh@linuxfoundation.org>; Linux Kernel Mailing List 
> <linux-kernel@vger.kernel.org>; Michal Simek 
> <michal.simek@xilinx.com>; Hyun Kwon <hyunk@xilinx.com>; Dhaval 
> Rajeshbhai Shah <DSHAH@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.shah@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 <dshah@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?
[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 linux-media@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.
[Dhaval ] : Yes. Driver is useful without a codec driver as well. No dependency on the Codec driver.

       Arnd