Re: [PATCH v3 08/10] clk: clock-wizard: Make the output names unique

2019-11-27 Thread Dan Carpenter
On Thu, Nov 28, 2019 at 12:06:15PM +0530, shubhrajyoti.da...@gmail.com wrote: > From: Shubhrajyoti Datta > > Incase there are more than one instance of the clocking wizard. > And if the output name given is the same then the probe fails. > Fix the same by appending the device name to the output

[PATCH v3 08/10] clk: clock-wizard: Make the output names unique

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta Incase there are more than one instance of the clocking wizard. And if the output name given is the same then the probe fails. Fix the same by appending the device name to the output name to make it unique. Signed-off-by: Shubhrajyoti Datta ---

[PATCH v3 09/10] staging: clocking-wizard: Delete the driver from the staging

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta Delete the driver from the staging as it is in drivers/clk. Signed-off-by: Shubhrajyoti Datta --- drivers/staging/Kconfig| 2 - drivers/staging/Makefile | 1 - drivers/staging/clocking-wizard/Kconfig

[PATCH v3 07/10] clk: clock-wizard: Update the fixed factor divisors

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta Update the fixed factor clock registration to register the divisors. Signed-off-by: Shubhrajyoti Datta --- drivers/clk/clk-xlnx-clock-wizard.c | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/drivers/clk/clk-xlnx-clock-wizard.c

[PATCH v3 03/10] clk: clock-wizard: Fix kernel-doc warning

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta Update description for the clocking wizard structure Signed-off-by: Shubhrajyoti Datta --- drivers/clk/clk-xlnx-clock-wizard.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/clk/clk-xlnx-clock-wizard.c

[PATCH v3 00/10] clk: clk-wizard: clock-wizard: Driver updates

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta In the thread [1] Greg suggested that we move the driver to the clk from the staging. Add patches to address the concerns regarding the fractional and set rate support in the TODO. The patch set does the following - Trivial fixes for kernel doc. - Move the driver to the

[PATCH v3 06/10] clk: clock-wizard: Remove the hardcoding of the clock outputs

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta The number of output clocks are configurable in the hardware. Currently the driver registers the maximum number of outputs. Fix the same by registering only the outputs that are there. Signed-off-by: Shubhrajyoti Datta --- drivers/clk/clk-xlnx-clock-wizard.c | 5 +++--

[PATCH v3 10/10] clk: clock-wizard: Fix the compilation failure

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta After 90b6c5c73 (clk: Remove CLK_IS_BASIC clk flag) The CLK_IS_BASIC is deleted. Adapt the driver for the same. Signed-off-by: Shubhrajyoti Datta --- drivers/clk/clk-xlnx-clock-wizard.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v3 05/10] clk: clock-wizard: Add support for fractional support

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta Currently the set rate granularity is to integral divisors. Add support for the fractional divisors. Only the first output0 is fractional in the hardware. Signed-off-by: Shubhrajyoti Datta --- drivers/clk/clk-xlnx-clock-wizard.c | 192

[PATCH v3 01/10] dt-bindings: add documentation of xilinx clocking wizard

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta Add the devicetree binding for the xilinx clocking wizard. Signed-off-by: Shubhrajyoti Datta --- .../bindings/clock/xlnx,clocking-wizard.txt| 32 ++ 1 file changed, 32 insertions(+) create mode 100644

[PATCH v3 02/10] clk: clock-wizard: Move the clockwizard to clk

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta Move the clocking wizard driver from staging to clk. Signed-off-by: Shubhrajyoti Datta --- drivers/clk/Kconfig | 6 + drivers/clk/Makefile| 1 + drivers/clk/clk-xlnx-clock-wizard.c | 335 3 files

[PATCH v3 04/10] clk: clock-wizard: Add support for dynamic reconfiguration

2019-11-27 Thread shubhrajyoti . datta
From: Shubhrajyoti Datta The patch adds support for dynamic reconfiguration of clock output rate. Output clocks are registered as dividers and set rate callback function is used for dynamic reconfiguration. Based on the initial work from Chirag. Signed-off-by: Chirag Parekh Signed-off-by:

Re: [PATCH v1 1/1] staging: rtl8192e: rename camelcase variable 'ShortData'

2019-11-27 Thread Dan Carpenter
On Wed, Nov 27, 2019 at 10:13:31AM -0800, Travis Davies wrote: > This patch renames camelcase variable 'ShortData' to 'short_data' > Fixes checkpatch message 'CHECK: Avoid CamelCase: > > Signed-off-by: Travis Davies > > --- > drivers/staging/rtl8192e/rtl819x_BA.h | 2 +- >

Re: [GIT PULL] Staging / IIO driver patches for 5.5-rc1

2019-11-27 Thread pr-tracker-bot
The pull request you sent on Wed, 27 Nov 2019 17:35:15 +0100: > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git > tags/staging-5.5-rc1 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/0dd09bc02c1bad55e92306ca83b38b3cf48b9f40 Thank you! --

[PATCH v1 1/1] staging: rtl8192e: rename camelcase variable 'ShortData'

2019-11-27 Thread Travis Davies
This patch renames camelcase variable 'ShortData' to 'short_data' Fixes checkpatch message 'CHECK: Avoid CamelCase: Signed-off-by: Travis Davies --- drivers/staging/rtl8192e/rtl819x_BA.h | 2 +- drivers/staging/rtl8192e/rtl819x_BAProc.c | 2 +- 2 files changed, 2 insertions(+), 2

[GIT PULL] Staging / IIO driver patches for 5.5-rc1

2019-11-27 Thread Greg KH
The following changes since commit 31f4f5b495a62c9a8b15b1c3581acd5efeb9af8c: Linux 5.4-rc7 (2019-11-10 16:17:15 -0800) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git tags/staging-5.5-rc1 for you to fetch changes up to

[PATCH 2/5] staging/qlge: add blank lines after declarations

2019-11-27 Thread Dorothea Ehrl
This patch fixes "WARNING: Missing a blank line after declarations" by checkpatch.pl. Signed-off-by: Dorothea Ehrl Co-developed-by: Vanessa Hack Signed-off-by: Vanessa Hack --- drivers/staging/qlge/qlge_ethtool.c | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 1/5] staging/qlge: remove initialising of static local variable

2019-11-27 Thread Dorothea Ehrl
This patch fixes "ERROR: do not initialise statics to 0" by checkpatch.pl. Signed-off-by: Dorothea Ehrl Co-developed-by: Vanessa Hack Signed-off-by: Vanessa Hack --- drivers/staging/qlge/qlge_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 3/5] staging/qlge: add braces to conditional statement

2019-11-27 Thread Dorothea Ehrl
This patch fixes "CHECK: braces {} should be used on all arms of this statement" by checkpatch.pl. Signed-off-by: Dorothea Ehrl Co-developed-by: Vanessa Hack Signed-off-by: Vanessa Hack --- drivers/staging/qlge/qlge_main.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff

[PATCH 5/5] staging/qlge: fix block comment coding style

2019-11-27 Thread Dorothea Ehrl
This patch fixes: "WARNING: block comment use * on subsequent lines" "WARNING: block comments should align the * on each line" "WARNING: block comments use a trailing */ on a separate line" by checkpatch.pl. Signed-off-by: Dorothea Ehrl Co-developed-by: Vanessa Hack Signed-off-by: Vanessa Hack

[PATCH 4/5] staging/qlge: remove braces in conditional statement

2019-11-27 Thread Dorothea Ehrl
This patch fixes "WARNING: braces {} are not necessary for single statement blocks" and "WARNING: braces {} are not necessary for any arm of this statement" by checkpatch.pl. Signed-off-by: Dorothea Ehrl Co-developed-by: Vanessa Hack Signed-off-by: Vanessa Hack ---

Re: staging: most: configfs: PAGE_SIZE char arrays?

2019-11-27 Thread Christian.Gromm
On Di, 2019-11-26 at 11:30 -0800, Joe Perches wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you > know the content is safe > > drivers/staging/most/configfs.c:struct mdev_link { > drivers/staging/most/configfs.c-struct config_item item; >