[PATCH] soc: xilinx: xlnx_vcu: Depends on HAS_IOMEM for xlnx_vcu

2018-01-15 Thread Dhaval Shah
ference to `devm_ioremap_nocache' Signed-off-by: Dhaval Shah --- drivers/soc/xilinx/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/soc/xilinx/Kconfig b/drivers/soc/xilinx/Kconfig index 266b50f..bf657ab 100644 --- a/drivers/soc/xilinx/Kconfig +++ b/drivers/soc/xilinx/Kconfig @@ -3,6 +3,7 @@ m

[PATCH v6 0/2] Documentation and driver of logicoreIP

2017-12-21 Thread Dhaval Shah
1st patch provide Device Tree binding document for logicoreIP 2nd patch provide the xlnx_vcu logicoreIP driver, Kconfig changes and Makefile changes for the driver. Dhaval Shah (2): dt-bindings: misc: Add DT bindings to xlnx_vcu driver misc: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP init

[PATCH v6 1/2] dt-bindings: soc: xilinx: Add DT bindings to xlnx_vcu driver

2017-12-21 Thread Dhaval Shah
Add Device Tree binding document for logicoreIP. This logicoreIP provides the isolation between the processing system and programmable logic. Also provides the clock related information. Signed-off-by: Dhaval Shah Reviewed-by: Rob Herring --- Changes since v6: * Updated path of the dt-bindings

[PATCH v6 2/2] soc: xilinx: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-21 Thread Dhaval Shah
Xilinx ZYNQMP logicoreIP Init driver is based on the new LogiCoreIP design created. This driver provides the processing system and programmable logic isolation. Set the frequency based on the clock information get from the logicoreIP register set. Signed-off-by: Dhaval Shah --- Changes since v6

Re: [PATCH 3/3] misc: ad525x_dpot: macros should not use a trailing semicolon

2017-12-18 Thread Dhaval Shah
Hi Greg k-h, > From: Greg KH > Sent: Monday, December 18, 2017 8:32 PM > To: Dhaval Shah > Cc: a...@arndb.de; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 3/3] misc: ad525x_dpot: macros should not use a trailing > semicolon &

[PATCH v5 2/2] misc: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-17 Thread Dhaval Shah
subsystem for this logicoreIP. Signed-off-by: Dhaval Shah --- Changes since v5: * Changes made to include index 0 of the array. Changes since v4: * Indent the help text (below) by 2 additional spaces, as documented in coding-style.rst * Spell check are resolved as per the review comment. * inline the

[PATCH v5 1/2] dt-bindings: misc: Add DT bindings to xlnx_vcu driver

2017-12-17 Thread Dhaval Shah
Add Device Tree binding document for logicoreIP. This logicoreIP provides the isolation between the processing system and programmable logic. Also provides the clock related information. Signed-off-by: Dhaval Shah --- Chnages since v5: No Changes. Chnages since v4: No Changes. Chnages since v3

[PATCH v5 0/2] Documentation and driver of logicoreIP

2017-12-17 Thread Dhaval Shah
1st patch provide Device Tree binding document for logicoreIP 2nd patch provide the xlnx_vcu logicoreIP driver, Kconfig changes and Makefile changes for the driver. Dhaval Shah (2): Documentation: devicetree: Add DT bindings to xlnx_vcu driver misc: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP

[PATCH v4 2/2] misc: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-14 Thread Dhaval Shah
subsystem for this logicoreIP. Signed-off-by: Dhaval Shah --- Changes since v4: * Indent the help text (below) by 2 additional spaces, as documented in coding-style.rst * Spell check are resolved as per the review comment. * inline the read() and write() functions.. * multi-line comment style

[PATCH v4 1/2] dt-bindings: misc: Add DT bindings to xlnx_vcu driver

2017-12-14 Thread Dhaval Shah
From: Dhaval Shah Add Device Tree binding document for logicoreIP. This logicoreIP provides the isolation between the processing system and programmable logic. Also provides the clock related information. Signed-off-by: Dhaval Shah --- Chnages since v3: No Changes. Chnages since v3

[PATCH v4 0/2] Documentation and driver of logicoreIP

2017-12-14 Thread Dhaval Shah
1st patch provide Device Tree binding document for logicoreIP 2nd patch provide the xlnx_vcu logicoreIP driver, Kconfig changes and Makefile changes for the driver. Dhaval Shah (2): Documentation: devicetree: Add DT bindings to xlnx_vcu driver misc: xlnx_vcu: Add Xilinx ZYNQMP VCU logicoreIP

Re: [PATCH] media: v4l: xilinx: Use SPDX-License-Identifier

2017-12-14 Thread Dhaval Shah
;> On Thu, 2017-12-14 at 20:37 +0200, Laurent Pinchart wrote: >> >>>>>> On Thursday, 14 December 2017 20:32:20 EET Joe Perches wrote: >> >>>>>>> On Thu, 2017-12-14 at 20:28 +0200, Laurent Pinchart wrote: >> >>>>>>>> O

Re: [PATCH] platform/x86: dell-smbios: Block comments use * on subsequent lines

2017-12-14 Thread Dhaval Shah
Hi Andy Shevchenko, > From: Andy Shevchenko > Sent: Thursday, December 14, 2017 11:29 PM > To: Dhaval Shah > Cc: Pali Rohár; Mario Limonciello; dvh...@infradead.org; Andy Shevchenko; > Platform Driver; linux-kernel@vger.kernel.org > S

Re: [PATCH 0/3] platform/x86: dell-laptop: All the warnings are resolved.

2017-12-14 Thread Dhaval Shah
Hi Andy Shevchenko, Thanks for the review. > From: Andy Shevchenko > Sent: Thursday, December 14, 2017 11:29 PM > To: Dhaval Shah > Cc: Matthew Garrett; Pali Rohár; Andy Shevchenko; Platform Driver; > linux-kernel@vger.kernel.org > Sub

Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-14 Thread Dhaval Shah
> From: Joe Perches > Sent: Wednesday, December 13, 2017 9:09 PM > To: Dhaval Shah; Philippe Ombredanne > Cc: pali.ro...@gmail.com; Darren Hart; Andy Shevchenko; > platform-driver-...@vger.kernel.org; LKML > Subject: Re: [PATCH] platf

[PATCH v3 2/2] misc: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-13 Thread Dhaval Shah
subsystem for this logicoreIP. Signed-off-by: Dhaval Shah --- Changes since v3: No Changes. Changes since v2: * Removed the "default n" from the Kconfig * More help text added to explain more about the logicoreIP driver * SPDX id is relocated at top of the file with // style comment * R

[PATCH v3 1/2] dt-bindings: misc: Add DT bindings to xlnx_vcu driver

2017-12-13 Thread Dhaval Shah
Add Device Tree binding document for logicoreIP. This logicoreIP provides the isolation between the processing system and programmable logic. Also provides the clock related information. Signed-off-by: Dhaval Shah --- Chnages since v3: * Use "dt-bindings: misc: ..." for the subjec

[PATCH v3 0/2] Documentation and driver of logicoreIP

2017-12-13 Thread Dhaval Shah
1st patch provide Device Tree binding document for logicoreIP 2nd patch provide the xlnx_vcu logicoreIP driver, Kconfig changes and Makefile changes for the driver. Dhaval Shah (2): Documentation: devicetree: Add DT bindings to xlnx_vcu driver misc: Add Xilinx ZYNQMP VCU logicoreIP init

Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Hi Philippe, > From: Philippe Ombredanne > Sent: Wednesday, December 13, 2017 6:27 PM > To: Dhaval Shah > Cc: pali.ro...@gmail.com; Darren Hart; Andy Shevchenko; > platform-driver-...@vger.kernel.org; LKML > Subject: Re: [PATCH]

Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Hi Joe, > From: Joe Perches > Sent: Wednesday, December 13, 2017 5:55 PM > To: Dhaval Shah; Philippe Ombredanne > Cc: pali.ro...@gmail.com; Darren Hart; Andy Shevchenko; > platform-driver-...@vger.kernel.org; LKML > Subject: Re: [P

Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Hi Philippe, > From: Philippe Ombredanne > Sent: Wednesday, December 13, 2017 3:31 PM > To: Dhaval Shah > Cc: pali.ro...@gmail.com; Darren Hart; Andy Shevchenko; > platform-driver-...@vger.kernel.org; LKML > Subject: Re: [PATCH] platform/x86: dell-rbtn: Block comments use

Re: [PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
HI Thanks a lot for the review. > From: Philippe Ombredanne > Sent: Wednesday, December 13, 2017 2:54 PM > To: Dhaval Shah > Cc: pali.ro...@gmail.com; Darren Hart; Andy Shevchenko; > platform-driver-...@vger.kernel.org; LKML > S

Re: [PATCH] platform/x86: dell-smo8800: Possible unnecessary 'out of memory' message

2017-12-13 Thread Dhaval Shah
Hi Pali Rohar, Thanks for the review. > From: Pali Rohár > Sent: Wednesday, December 13, 2017 2:00 PM > To: Dhaval Shah > Cc: dvh...@infradead.org; a...@infradead.org; > platform-driver-...@vger.kernel.org; linux-kernel@vger.kernel.or

[PATCH] platform/x86: dell-smo8800: Possible unnecessary 'out of memory' message

2017-12-13 Thread Dhaval Shah
Removed Possible unnecessary 'out of memory' message checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-smo8800.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/platform/x86/dell-smo8800.c b/driver

[PATCH] platform/x86: dell-rbtn: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-rbtn.c | 26 +- drivers/platform/x86/dell-rbtn.h | 26 +- 2 files changed, 26 insertions

[PATCH] platform/x86: dell-smbios: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-smbios.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/dell-smbios.c b/drivers/platform/x86

[PATCH] platform/x86: dell-wmi-aio: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-wmi-aio.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/platform/x86/dell-wmi-aio.c b/drivers/platform

[PATCH 1/3] platform/x86: dell-laptop: Block comments use * on subsequent lines

2017-12-13 Thread Dhaval Shah
Resolved Block comments use * on subsequent lines checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-laptop.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86

[PATCH 2/3] platform/x86: dell-laptop: Missing a blank line after declarations

2017-12-13 Thread Dhaval Shah
Resolved Missing a blank line after declarations checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-laptop.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/platform/x86/dell-laptop.c b/drivers/platform/x86/dell-laptop.c

[PATCH 3/3] platform/x86: dell-laptop: braces {} are not necessary for single statement blocks

2017-12-13 Thread Dhaval Shah
Resolved braces {} are not necessary for single statement blocks checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/platform/x86/dell-laptop.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/platform/x86/dell-laptop.c b/drivers

[PATCH 0/3] platform/x86: dell-laptop: All the warnings are resolved.

2017-12-13 Thread Dhaval Shah
There are total three types of warnings are resolved. First patch : Block comments use * on subsequent lines Second patch : Missing a blank line after declarations third patch : braces {} are not necessary for single statement blocks. Dhaval Shah (3): platform/x86: dell-laptop: Block comments

Re: [PATCH] media: v4l: xilinx: Use SPDX-License-Identifier

2017-12-11 Thread Dhaval Shah
Hi Laurent Pinchart, Thanks a lot for the review. On Mon, Dec 11, 2017 at 7:17 PM, Laurent Pinchart wrote: > > Hi Dhaval, > > Thank you for the patch. > > On Friday, 8 December 2017 14:35:37 EET Dhaval Shah wrote: > > SPDX-License-Identifier is used for the Xilinx

[PATCH] media: v4l: xilinx: Use SPDX-License-Identifier

2017-12-08 Thread Dhaval Shah
SPDX-License-Identifier is used for the Xilinx Video IP and related drivers. Signed-off-by: Dhaval Shah --- drivers/media/platform/xilinx/xilinx-dma.c | 5 + drivers/media/platform/xilinx/xilinx-dma.h | 5 + drivers/media/platform/xilinx/xilinx-tpg.c | 5 + drivers/media/platform

[PATCH] clkc: zynq: Missing a blank line after declarations.

2017-12-08 Thread Dhaval Shah
Resolved missing a blank line after declarations checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/clk/zynq/clkc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/clk/zynq/clkc.c b/drivers/clk/zynq/clkc.c index 88a2cab37f62..1cd4705e405e 100644

[PATCH 2/3] misc: ad525x_dpot: please, no space before tabs

2017-12-08 Thread Dhaval Shah
Resolved the please, no space beofore tabs checkpatch warning. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/ad525x_dpot.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/ad525x_dpot.c b/drivers/misc/ad525x_dpot.c index 1c6b55655f52

[PATCH 0/3] misc: ad525x_dpot: Different type of warnings are resolved.

2017-12-08 Thread Dhaval Shah
-strict drivers/misc/ad525x_dpot.c Dhaval Shah (3): misc: ad525x_dpot: Prefer 'unsigned int' to bare use of 'unsigned' misc: ad525x_dpot: please, no space before tabs misc: ad525x_dpot: macros should not use a trailing semicolon drivers/misc/ad525x_dpot.c | 30 ++

[PATCH 1/3] misc: ad525x_dpot: Prefer 'unsigned int' to bare use of 'unsigned'

2017-12-08 Thread Dhaval Shah
Resolved all the Prefer 'unsigned int' to bare use of 'unsigned' checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/ad525x_dpot.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/

[PATCH 3/3] misc: ad525x_dpot: macros should not use a trailing semicolon

2017-12-08 Thread Dhaval Shah
Resolved all the macros should not use a trailing semicolon checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/ad525x_dpot.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/misc/ad525x_dpot.c b/drivers/misc/ad525x_dpot.c

[PATCH] misc: ad525x_dpot: Unnecessary space before function pointer arguments

2017-12-08 Thread Dhaval Shah
Resolved all the Unnecessary space before function pointer arguments checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/ad525x_dpot.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/misc/ad525x_dpot.h b/drivers

[PATCH 2/3] misc: ics932s401: please, no space before tabs

2017-12-07 Thread Dhaval Shah
Resolved all the please, no space beofore tabs checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/ics932s401.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/misc/ics932s401.c b/drivers/misc/ics932s401.c index

[PATCH 3/3] misc: ics932s401: open brace should be on the previous line

2017-12-07 Thread Dhaval Shah
Resolved open brace { should be on the previous line checkpatch error. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/ics932s401.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/misc/ics932s401.c b/drivers/misc/ics932s401.c index

[PATCH 0/3] misc: ics932s401: warnings and errors are resolved.

2017-12-07 Thread Dhaval Shah
--strict drivers/misc/hpilo.c Dhaval Shah (3): misc: ics932s401: Missing a blank line after declarations misc: ics932s401: please, no space before tabs misc: ics932s401: open brace should be on the previous line drivers/misc/ics932s401.c | 8 1 file changed, 4 insertions(+), 4 deletions

[PATCH 1/3] misc: ics932s401: Missing a blank line after declarations

2017-12-07 Thread Dhaval Shah
Resolved all the missing a blank line after declarations checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/ics932s401.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/misc/ics932s401.c b/drivers/misc/ics932s401.c index 28f51e01fd2b

[PATCH] misc: apds990x: Missing a blank line after declarations.

2017-12-07 Thread Dhaval Shah
Resolved all the missing a blank line after declarations checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/apds990x.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/misc/apds990x.c b/drivers/misc/apds990x.c index c9f07032c2fc

[PATCH v2 2/2] misc: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-07 Thread Dhaval Shah
subsystem for this logicoreIP. Signed-off-by: Dhaval Shah --- Changes since v2: * Removed the "default n" from the Kconfig * More help text added to explain more about the logicoreIP driver * SPDX id is relocated at top of the file with // style comment * Removed the export API and heade

[PATCH v2 1/2] Documentation: devicetree: Add DT bindings to xlnx_vcu driver

2017-12-07 Thread Dhaval Shah
Add Device Tree binding document for logicoreIP. This logicoreIP provides the isolation between the processing system and programmable logic. Also provides the clock related information. Signed-off-by: Dhaval Shah --- Changes since v2: * Describe the h/w * compatible string is updated to make

[PATCH v2 0/2] Documentation and driver of logicoreIP

2017-12-07 Thread Dhaval Shah
1st patch provide Device Tree binding document for logicoreIP 2nd patch provide the xlnx_vcu logicoreIP driver, Kconfig changes and Makefile changes for the driver. Dhaval Shah (2): Documentation: devicetree: Add DT bindings to xlnx_vcu driver misc: Add Xilinx ZYNQMP VCU logicoreIP init

[PATCH] misc: isl29003: Missing a blank line after declarations

2017-12-07 Thread Dhaval Shah
Resolved all the missing a blank line after declarations checkpatch warnings. Issue found by checkpatch. Signed-off-by: Dhaval Shah --- drivers/misc/isl29003.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/misc/isl29003.c b/drivers/misc/isl29003.c index 976df0013633

[PATCH] [linux][master][v1] misc: Add Xilinx ZYNQMP VCU logicoreIP init driver

2017-12-05 Thread Dhaval Shah
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 --- drivers/misc/Kconfig| 6 + drivers/misc/Makefile | 1 + drivers

[PATCH] [linux][master][v1] devicetree: misc: Add binding for logicoreIP xlnx,vcu

2017-12-05 Thread Dhaval Shah
From: Dhaval Shah Added the txt file which contain the xlnx,vcu DT node properties information. This also provides the information of it's child node as well. Signed-off-by: Dhaval Shah --- .../devicetree/bindings/misc/xlnx,vcu.txt | 59 ++ 1 file change