Re: [PATCH -next] staging: ks7010: Macros with complex values

2021-02-11 Thread Fatih Yildirim
On Thu, Feb 11, 2021 at 12:10:44PM +0100, Greg KH wrote: > On Thu, Feb 11, 2021 at 01:57:04PM +0300, Fatih YILDIRIM wrote: > > On Thu, Feb 11, 2021 at 11:02:51AM +0100, Greg KH wrote: > > > On Thu, Feb 11, 2021 at 12:22:39PM +0300, Fatih Yildirim wrote: > > > > Fix for checkpatch.pl warning: > > >

Re: [PATCH] staging: wfx: avoid defining array of flexible struct

2021-02-11 Thread Jérôme Pouiller
On Thursday 11 February 2021 11:50:26 CET Muhammad Usama Anjum wrote: > > In this particular case, the struct element is already flexible struct. > Thus struct element ie[] is ambiguous inside another struct. The members > of struct element ie aren't being accessed in code anywhere. The data of >

[PATCH v4 2/2] staging: rtl8723bs: remove obsolete commented out code

2021-02-11 Thread karthik alapati
There is a bunch of messy, commented out code. Just delete it. Suggested-by: Dan Carpenter Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 40 +-- 1 file changed, 2 insertions(+), 38 deletions(-) diff --git

Re: [PATCH 1/1] staging: greybus: Added do - while in multi statement macro

2021-02-11 Thread kernel test robot
Hi Hemansh, Thank you for the patch! Yet something to improve: [auto build test ERROR on staging/staging-testing] url: https://github.com/0day-ci/linux/commits/Hemansh-Agnihotri/staging-greybus-Added-do-while-in-multi-statement-macro/20210211-175717 base: https://git.kernel.org/pub/scm

[PATCH v1 3/9] sfi: Remove framework for deprecated firmware

2021-02-11 Thread Andy Shevchenko
SFI-based platforms are gone. So does this framework. This removes mention of SFI through the drivers and other code as well. Signed-off-by: Andy Shevchenko Reviewed-by: Hans de Goede Acked-by: Linus Walleij --- Documentation/ABI/testing/sysfs-firmware-sfi | 15 -

[PATCH v1 9/9] x86/platform/intel-mid: Update Copyright year and drop file names

2021-02-11 Thread Andy Shevchenko
Update Copyright year and drop file names from files themselves. Signed-off-by: Andy Shevchenko --- arch/x86/include/asm/intel-mid.h| 4 ++-- arch/x86/platform/intel-mid/intel-mid.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/x86/include/asm/intel-mid.h

[PATCH v1 5/9] x86/PCI: Describe @reg for type1_access_ok()

2021-02-11 Thread Andy Shevchenko
Describe missed parameter in documentation of type1_access_ok(). Otherwise "make W=1 arch/x86/pci/" produces the following warning: CHECK arch/x86/pci/intel_mid_pci.c CC arch/x86/pci/intel_mid_pci.o arch/x86/pci/intel_mid_pci.c:152: warning: Function parameter or member 'reg' not

[PATCH v1 7/9] x86/platform/intel-mid: Drop unused __intel_mid_cpu_chip and Co.

2021-02-11 Thread Andy Shevchenko
Since there is no more user of this global variable and associated custom API, we may safely drop this legacy reinvented a wheel from the kernel sources. Signed-off-by: Andy Shevchenko --- arch/x86/include/asm/intel-mid.h| 23 ---

[PATCH v1 6/9] x86/platform/intel-mid: Get rid of intel_scu_ipc_legacy.h

2021-02-11 Thread Andy Shevchenko
The header is used by a single user. Move header content to that user. Signed-off-by: Andy Shevchenko Reviewed-by: Mika Westerberg --- arch/x86/include/asm/intel_scu_ipc.h| 2 -- arch/x86/include/asm/intel_scu_ipc_legacy.h | 18 --

[PATCH v1 8/9] x86/platform/intel-mid: Remove unused header inclusion in intel-mid.h

2021-02-11 Thread Andy Shevchenko
After the commit f1be6cdaf57c ("x86/platform/intel-mid: Make intel_scu_device_register() static") the platform_device.h is not being used anymore by intel-mid.h. Remove it. Signed-off-by: Andy Shevchenko --- arch/x86/include/asm/intel-mid.h | 1 - 1 file changed, 1 deletion(-) diff --git

Re: [PATCH -next] staging: ks7010: Macros with complex values

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 03:23:24PM +0300, Fatih Yildirim wrote: > On Thu, Feb 11, 2021 at 12:10:44PM +0100, Greg KH wrote: > > On Thu, Feb 11, 2021 at 01:57:04PM +0300, Fatih YILDIRIM wrote: > > > On Thu, Feb 11, 2021 at 11:02:51AM +0100, Greg KH wrote: > > > > On Thu, Feb 11, 2021 at 12:22:39PM

[PATCH v4 1/2] staging: rtl8723bs: fix function comments to follow kernel-doc

2021-02-11 Thread karthik alapati
there are some good function comments not following kernel-doc. Make them follow kernel-doc style Signed-off-by: karthik alapati --- .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 185 +++--- 1 file changed, 73 insertions(+), 112 deletions(-) diff --git

[PATCH v4 0/2] staging: rtl8723bs: driver cleanup

2021-02-11 Thread karthik alapati
karthik alapati (2): staging: rtl8723bs: fix function comments to follow kernel-doc staging: rtl8723bs: remove obsolete commented out code .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 225 ++ 1 file changed, 75 insertions(+), 150 deletions(-) -- 2.30.0

RE: [PATCH v4] staging: gdm724x: Fix DMA from stack

2021-02-11 Thread David Laight
> Stack allocated buffers cannot be used for DMA > on all architectures so allocate hci_packet buffer > using kmalloc. I wonder if the usb stack ought/could support a short bounce buffer within the memory is already has to allocate? For hci and lengths less than 8 the immediate data can be

Re: [PATCH -next] staging: ks7010: Macros with complex values

2021-02-11 Thread Fatih YILDIRIM
Ok, thanks! On Thu, Feb 11, 2021 at 3:52 PM Greg KH wrote: > > On Thu, Feb 11, 2021 at 03:23:24PM +0300, Fatih Yildirim wrote: > > On Thu, Feb 11, 2021 at 12:10:44PM +0100, Greg KH wrote: > > > On Thu, Feb 11, 2021 at 01:57:04PM +0300, Fatih YILDIRIM wrote: > > > > On Thu, Feb 11, 2021 at

[PATCH v1 0/9] x86/platform: Remove SFI framework and users

2021-02-11 Thread Andy Shevchenko
This is last part of Intel MID (SFI based) removal. We have no more users of it in the kernel and since SFI has been marked Obsolete for a few years already, Remove all the stuff altogether. Note, the more recent platforms (Intel Merrifield and Moorefield) still work as long as they provide

[PATCH v1 4/9] x86/PCI: Get rid of custom x86 model comparison

2021-02-11 Thread Andy Shevchenko
Switch the platform code to use x86_id_table and accompanying API instead of custom comparison against x86 CPU model. This is one of the last users of custom API for that and following changes will remove it for the good. Signed-off-by: Andy Shevchenko --- arch/x86/pci/intel_mid_pci.c | 17

[PATCH v1 2/9] cpufreq: sfi-cpufreq: Remove driver for deprecated firmware

2021-02-11 Thread Andy Shevchenko
SFI-based platforms are gone. So does this driver. Signed-off-by: Andy Shevchenko Acked-by: Linus Walleij --- drivers/cpufreq/Kconfig.x86 | 10 --- drivers/cpufreq/Makefile | 1 - drivers/cpufreq/sfi-cpufreq.c | 127 -- 3 files changed, 138

[PATCH v1 1/9] media: atomisp: Remove unused header

2021-02-11 Thread Andy Shevchenko
sfi.h is not anyhow used by the driver. Remove it. Signed-off-by: Andy Shevchenko Acked-by: Sakari Ailus Acked-by: Linus Walleij --- drivers/staging/media/atomisp/include/linux/atomisp_platform.h | 1 - 1 file changed, 1 deletion(-) diff --git

Re: [PATCH v4] staging: gdm724x: Fix DMA from stack

2021-02-11 Thread gre...@linuxfoundation.org
On Thu, Feb 11, 2021 at 01:28:41PM +, David Laight wrote: > > Stack allocated buffers cannot be used for DMA > > on all architectures so allocate hci_packet buffer > > using kmalloc. > > I wonder if the usb stack ought/could support a short bounce > buffer within the memory is already has to

Re: [PATCH] staging: wfx: avoid defining array of flexible struct

2021-02-11 Thread Muhammad Usama Anjum
> I think that "#include " is no more necessary. Good catch. I'll send another patch. Thanks, Usama ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: wfx: remove unused included header files

2021-02-11 Thread Muhammad Usama Anjum
Many header files have been included, but never used. Those header files have been removed. Signed-off-by: Muhammad Usama Anjum --- drivers/staging/wfx/bh.c | 1 - drivers/staging/wfx/bh.h | 4 drivers/staging/wfx/bus.h | 3 ---

[PATCH] staging: ks7010: enclosed complex macro definitions with parentheses

2021-02-11 Thread Pritthijit Nath
kshostif.h fixed ERROR: Macros with complex values should be enclosed in paranthesis Signed-off-by: Pritthijit Nath --- drivers/staging/ks7010/ks_hostif.h | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h

[PATCH -next] staging: ks7010: Macros with complex values

2021-02-11 Thread Fatih Yildirim
Fix for checkpatch.pl warning: Macros with complex values should be enclosed in parentheses. Signed-off-by: Fatih Yildirim --- drivers/staging/ks7010/ks_hostif.h | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h

[PATCH 1/1] staging: greybus: Added do - while in multi statement macro

2021-02-11 Thread Hemansh Agnihotri
This patch add fixes an checkpatch error for "Macros with multiple statements should be enclosed in a do - while loop" Signed-off-by: Hemansh Agnihotri --- drivers/staging/greybus/loopback.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

Re: [PATCH -next] staging: ks7010: Macros with complex values

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 12:22:39PM +0300, Fatih Yildirim wrote: > Fix for checkpatch.pl warning: > Macros with complex values should be enclosed in parentheses. > > Signed-off-by: Fatih Yildirim > --- > drivers/staging/ks7010/ks_hostif.h | 24 > 1 file changed, 12

Re: [PATCH 1/1] staging: greybus: Added do - while in multi statement macro

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 03:24:44PM +0530, Hemansh Agnihotri wrote: > This patch add fixes an checkpatch error for "Macros with multiple statements > should be enclosed in a do - while loop" > > Signed-off-by: Hemansh Agnihotri Any reason you didn't test-build your patch before sending it out?

Re: [greybus-dev] [PATCH 1/1] staging: greybus: Added do - while in multi statement macro

2021-02-11 Thread Viresh Kumar
On 11-02-21, 11:00, Greg KH wrote: > On Thu, Feb 11, 2021 at 03:24:44PM +0530, Hemansh Agnihotri wrote: > > This patch add fixes an checkpatch error for "Macros with multiple > > statements > > should be enclosed in a do - while loop" > > > > Signed-off-by: Hemansh Agnihotri > > Any reason you

Re: [PATCH] staging: rtl8723bs: fix function comments to follow kernel-doc

2021-02-11 Thread Greg Kroah-Hartman
On Thu, Feb 11, 2021 at 12:48:16AM +0530, karthek wrote: > check this out Why ask us again when you already sent a patch? Do you see any other developers doing that on the mailing lists? thanks, greg k-h ___ devel mailing list

Re: [PATCH] staging: rtl8723bs: remove obsolete commented out code

2021-02-11 Thread karthek
On Thu, Feb 11, 2021 at 4:16 PM Greg Kroah-Hartman wrote: > > A: http://en.wikipedia.org/wiki/Top_post > Q: Were do I find info about this thing called top-posting? > A: Because it messes up the order in which people normally read text. > Q: Why is top-posting such a bad thing? > A: Top-posting.

[PATCH] staging: wfx: avoid defining array of flexible struct

2021-02-11 Thread Muhammad Usama Anjum
In this particular case, the struct element is already flexible struct. Thus struct element ie[] is ambiguous inside another struct. The members of struct element ie aren't being accessed in code anywhere. The data of u8 type is copied in it. So it has been changed to u8 ie[] to make the sparse

Re: [PATCH -next] staging: ks7010: Macros with complex values

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 01:57:04PM +0300, Fatih YILDIRIM wrote: > On Thu, Feb 11, 2021 at 11:02:51AM +0100, Greg KH wrote: > > On Thu, Feb 11, 2021 at 12:22:39PM +0300, Fatih Yildirim wrote: > > > Fix for checkpatch.pl warning: > > > Macros with complex values should be enclosed in parentheses. >

Re: [PATCH][next] staging: rtl8723bs: Replace one-element array with flexible-array member in struct ndis_80211_var_ie

2021-02-11 Thread Gustavo A. R. Silva
On 2/11/21 05:06, Dan Carpenter wrote: > On Wed, Feb 10, 2021 at 04:49:37PM -0600, Gustavo A. R. Silva wrote: >> There is a regular need in the kernel to provide a way to declare having >> a dynamically sized set of trailing elements in a structure. Kernel code >> should always use “flexible

[PATCH 2/4] reset: Add reset driver for IMX8MQ VPU block

2021-02-11 Thread Benjamin Gaignard
IMX8MQ SoC got a dedicated hardware block to reset the video processor units (G1 and G2). Signed-off-by: Benjamin Gaignard --- drivers/reset/Kconfig| 8 ++ drivers/reset/Makefile | 1 + drivers/reset/reset-imx8mq-vpu.c | 169 +++ 3 files

[PATCH 0/4] Reset driver for IMX8MQ VPU hardware block

2021-02-11 Thread Benjamin Gaignard
The two VPUs inside IMX8MQ share the same control block which can be see as a reset hardware block. In order to be able to add the second VPU (for HECV decoding) it will be more handy if the both VPU drivers instance don't have to share the control block registers. This lead to implement it as an

[PATCH 3/4] media: hantro: Use reset driver

2021-02-11 Thread Benjamin Gaignard
Rather use a reset like feature inside the driver use the reset controller API to get the same result. Signed-off-by: Benjamin Gaignard --- drivers/staging/media/hantro/Kconfig| 1 + drivers/staging/media/hantro/imx8m_vpu_hw.c | 61 - 2 files changed, 12

[PATCH 4/4] arm64: dts: imx8mq: Use reset driver for VPU hardware block

2021-02-11 Thread Benjamin Gaignard
Add a vpu reset hardware block node. Signed-off-by: Benjamin Gaignard --- arch/arm64/boot/dts/freescale/imx8mq.dtsi | 31 ++- 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi b/arch/arm64/boot/dts/freescale/imx8mq.dtsi

Re: [PATCH] staging: rtl8723bs: remove obsolete commented out code

2021-02-11 Thread karthek
Should i send them as patch series? On Thu, Feb 11, 2021 at 1:27 PM Dan Carpenter wrote: > > On Thu, Feb 11, 2021 at 12:40:41AM +0530, karthik alapati wrote: > > @@ -867,10 +845,8 @@ static void PHY_HandleSwChnlAndSetBW8723B( > > if (bSetBandWidth) > > pHalData->bSetChnlBW =

Re: [PATCH][next] staging: rtl8723bs: Replace one-element array with flexible-array member in struct ndis_80211_var_ie

2021-02-11 Thread Dan Carpenter
On Wed, Feb 10, 2021 at 04:49:37PM -0600, Gustavo A. R. Silva wrote: > There is a regular need in the kernel to provide a way to declare having > a dynamically sized set of trailing elements in a structure. Kernel code > should always use “flexible array members”[1] for these cases. The older >

Re: [PATCH] staging: rtl8723bs: fix function comments to follow kernel-doc

2021-02-11 Thread Greg Kroah-Hartman
On Thu, Feb 11, 2021 at 12:40:15AM +0530, karthik alapati wrote: > fix checkpatch.pl warning for "block comments should align the > * on each line" and make function comments follow kernel-doc > > Signed-off-by: karthik alapati > --- > .../staging/rtl8723bs/hal/rtl8723b_phycfg.c | 185

[PATCH 1/4] dt-bindings: reset: IMX8MQ VPU reset

2021-02-11 Thread Benjamin Gaignard
Document bindings for IMX8MQ VPU reset hardware block Signed-off-by: Benjamin Gaignard --- .../bindings/reset/fsl,imx8mq-vpu-reset.yaml | 54 +++ include/dt-bindings/reset/imx8mq-vpu-reset.h | 16 ++ 2 files changed, 70 insertions(+) create mode 100644

Re: [PATCH] staging: rtl8723bs: fix function comments to follow kernel-doc

2021-02-11 Thread karthek
Sorry On Thu, Feb 11, 2021 at 3:34 PM Greg Kroah-Hartman wrote: > > On Thu, Feb 11, 2021 at 12:48:16AM +0530, karthek wrote: > > check this out > > Why ask us again when you already sent a patch? Do you see any other > developers doing that on the mailing lists? > > thanks, > > greg k-h

Re: [PATCH] staging: rtl8723bs: remove obsolete commented out code

2021-02-11 Thread Greg Kroah-Hartman
A: http://en.wikipedia.org/wiki/Top_post Q: Were do I find info about this thing called top-posting? A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I

Re: [PATCH -next] staging: ks7010: Macros with complex values

2021-02-11 Thread Fatih YILDIRIM
On Thu, Feb 11, 2021 at 11:02:51AM +0100, Greg KH wrote: > On Thu, Feb 11, 2021 at 12:22:39PM +0300, Fatih Yildirim wrote: > > Fix for checkpatch.pl warning: > > Macros with complex values should be enclosed in parentheses. > > > > Signed-off-by: Fatih Yildirim > > --- > >

Re: [PATCH] staging: ks7010: enclosed complex macro definitions with parentheses

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 04:26:36PM +0530, Pritthijit Nath wrote: > kshostif.h Why is this in the changelog text? > > fixed ERROR: Macros with complex values should be enclosed in > paranthesis What does that mean? > > Signed-off-by: Pritthijit Nath > --- >

Re: [PATCH v1 0/9] x86/platform: Remove SFI framework and users

2021-02-11 Thread Rafael J. Wysocki
On Thu, Feb 11, 2021 at 2:50 PM Andy Shevchenko wrote: > > This is last part of Intel MID (SFI based) removal. We have no more users of > it > in the kernel and since SFI has been marked Obsolete for a few years already, > Remove all the stuff altogether. > > Note, the more recent platforms

Re: [PATCH v1 0/9] x86/platform: Remove SFI framework and users

2021-02-11 Thread Hans de Goede
Hi, On 2/11/21 4:24 PM, Rafael J. Wysocki wrote: > On Thu, Feb 11, 2021 at 2:50 PM Andy Shevchenko > wrote: >> >> This is last part of Intel MID (SFI based) removal. We have no more users of >> it >> in the kernel and since SFI has been marked Obsolete for a few years already, >> Remove all the

[PATCH] staging: wimax: Fix some coding style problems

2021-02-11 Thread Hemansh Agnihotri
This fixes checkpatch errors :- "else should follow close brace '}'", "space required before the open parenthesis '('" and "spaces required around that '?' (ctx:VxW)" in drivers/staging/wimax/i2400m/rx.c file. Signed-off-by: Hemansh Agnihotri --- drivers/staging/wimax/i2400m/rx.c | 11

[PATCH] staging: wimax: Fix some coding style problems

2021-02-11 Thread Hemansh Agnihotri
This fixes following warnings and errors as reported by checkpatch.pl: 1) WARNING: Missing a blank line after declarations 2) WARNING: Block comments use a trailing */ on a separate line 3) ERROR: code indent should use tabs where possible 4) ERROR: space required

[PATCH] staging: vt6656: Fixed alignment with issue in rf.c

2021-02-11 Thread Pritthijit Nath
This change fixes a checkpatch CHECK style issue for "Alignment should match open parenthesis". Signed-off-by: Pritthijit Nath --- drivers/staging/vt6656/rf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c index

Re: [PATCH] staging: wimax: Fix some coding style problems

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 09:45:53PM +0530, Hemansh Agnihotri wrote: > This fixes following warnings and errors as reported by checkpatch.pl: > 1) WARNING: Missing a blank line after declarations > 2) WARNING: Block comments use a trailing */ on a separate line > 3) ERROR: code

Re: [PATCH] staging: wlan-ng: Fix comments typos

2021-02-11 Thread Mairo P. Rufus
> Hi, > > This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him > a patch that has triggered this response. He used to manually respond > to these common problems, but in order to save his sanity (he kept > writing the same thing over and over, yet to different people), I was >

Re: [PATCH] staging: wlan-ng: Fix comments typos

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 05:55:18PM +0100, Mairo P. Rufus wrote: > > Hi, > > > > This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him > > a patch that has triggered this response. He used to manually respond > > to these common problems, but in order to save his sanity (he kept >

[PATCH -next] staging: nvec: minor coding style fix

2021-02-11 Thread Fatih Yildirim
Fix for the below coding style warning. Warning: Move const after static - use 'static const int' Signed-off-by: Fatih Yildirim --- drivers/staging/nvec/nvec_power.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/nvec/nvec_power.c

[RESEND PATCH v5 4/6] staging: mt7621-dts: make use of new 'mt7621-clk'

2021-02-11 Thread Sergio Paracuellos
Clocks for SoC mt7621 have been properly integrated so there is no need to declare fixed clocks at all in the device tree. Remove all of them, add new device tree nodes for mt7621-clk and update the rest of the nodes to use them. Signed-off-by: Sergio Paracuellos ---

[RESEND PATCH v5 2/6] dt: bindings: add mt7621-clk device tree binding documentation

2021-02-11 Thread Sergio Paracuellos
Adds device tree binding documentation for clocks in the MT7621 SOC. Signed-off-by: Sergio Paracuellos --- .../bindings/clock/mediatek,mt7621-clk.yaml | 52 +++ 1 file changed, 52 insertions(+) create mode 100644

[RESEND PATCH v5 5/6] staging: mt7621-dts: use valid vendor 'mediatek' instead of invalid 'mtk'

2021-02-11 Thread Sergio Paracuellos
Vendor listed for mediatek in kernel vendor file 'vendor-prefixes.yaml' contains 'mediatek' as a valid vendor string. Some nodes in the device tree are using an invalid vendor string vfor 'mtk' instead. Fix all of them in dts file. Update also ralink mt7621 related code to properly match new

[RESEND PATCH v5 0/6] MIPS: ralink: add CPU clock detection and clock driver for MT7621

2021-02-11 Thread Sergio Paracuellos
This patchset ports CPU clock detection for MT7621 from OpenWrt and adds a complete clock plan for the mt7621 SOC. The documentation for this SOC only talks about two registers regarding to the clocks: * SYSC_REG_CPLL_CLKCFG0 - provides some information about boostrapped refclock. PLL and

[RESEND PATCH v5 3/6] clk: ralink: add clock driver for mt7621 SoC

2021-02-11 Thread Sergio Paracuellos
The documentation for this SOC only talks about two registers regarding to the clocks: * SYSC_REG_CPLL_CLKCFG0 - provides some information about boostrapped refclock. PLL and dividers used for CPU and some sort of BUS. * SYSC_REG_CPLL_CLKCFG1 - a banch of gates to enable/disable clocks for all or

[RESEND PATCH v5 6/6] MAINTAINERS: add MT7621 CLOCK maintainer

2021-02-11 Thread Sergio Paracuellos
Adding myself as maintainer for mt7621 clock driver. Signed-off-by: Sergio Paracuellos --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index f5eafee83bc6..f0c51d9760ec 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11247,6 +11247,12 @@ L:

[RESEND PATCH v5 1/6] dt-bindings: clock: add dt binding header for mt7621 clocks

2021-02-11 Thread Sergio Paracuellos
Adds dt binding header for 'mediatek,mt7621-clk' clocks. Acked-by: Rob Herring Signed-off-by: Sergio Paracuellos --- include/dt-bindings/clock/mt7621-clk.h | 41 ++ 1 file changed, 41 insertions(+) create mode 100644 include/dt-bindings/clock/mt7621-clk.h diff --git

[PATCH] ks7010: removed checkpatch.pl format errors

2021-02-11 Thread shivang upadhyay
Signed-off-by: shivang upadhyay --- drivers/staging/ks7010/ks_hostif.h | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/drivers/staging/ks7010/ks_hostif.h index 39138191a556..c62a494ed6bb 100644 ---

Re: [PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Pritthijit Nath
On 12/02/21 1:59 am, Greg KH wrote: > On Thu, Feb 11, 2021 at 08:54:26PM +0530, Pritthijit Nath wrote: >> This change fixes a checkpatch CHECK style issue for "Alignment should match >> open parenthesis". >> >> Signed-off-by: Pritthijit Nath >> --- >> drivers/staging/vt6656/rf.c | 2 +- >> 1

Re: [PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Pritthijit Nath
On 12/02/21 2:14 am, Greg KH wrote: > On Fri, Feb 12, 2021 at 02:07:50AM +0530, Pritthijit Nath wrote: >> On 12/02/21 1:59 am, Greg KH wrote: >>> On Thu, Feb 11, 2021 at 08:54:26PM +0530, Pritthijit Nath wrote: This change fixes a checkpatch CHECK style issue for "Alignment should match

11-02-2021

2021-02-11 Thread Mrs. Yara Mariam Abdul Azeez
I am Mrs. Yara Mariam Abdul-Azeez 57 years old suffering from long term cancer of the blood (leukemia) and according to the doctors i have very few months to spend in this world before I die due to the fact that the cancer is in it's third and final stage and has affected many vital organs in

[PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Pritthijit Nath
This change fixes a checkpatch CHECK style issue for "Alignment should match open parenthesis". Signed-off-by: Pritthijit Nath --- drivers/staging/vt6656/rf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c index

Re: [PATCH] staging: wimax: Fix some coding style problems

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 10:43:20PM +0530, Hemansh Agnihotri wrote: > This fixes checkpatch errors :- "else should follow close brace '}'", > "space required before the open parenthesis '('" and "spaces required > around that '?' (ctx:VxW)" in drivers/staging/wimax/i2400m/rx.c file. > >

Re: [PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Greg KH
On Fri, Feb 12, 2021 at 02:07:50AM +0530, Pritthijit Nath wrote: > On 12/02/21 1:59 am, Greg KH wrote: > > On Thu, Feb 11, 2021 at 08:54:26PM +0530, Pritthijit Nath wrote: > >> This change fixes a checkpatch CHECK style issue for "Alignment should > >> match open parenthesis". > >> > >>

Re: [PATCH] removed enclosed parenthesis errors given by checkpatch.pl

2021-02-11 Thread Greg KH
On Fri, Feb 12, 2021 at 02:27:39AM +0530, roz wrote: > Signed-off-by: roz > --- > drivers/staging/ks7010/ks_hostif.h | 24 > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/drivers/staging/ks7010/ks_hostif.h > b/drivers/staging/ks7010/ks_hostif.h >

Re: [PATCH] staging: vt6656: Fixed alignment with issue in rf.c

2021-02-11 Thread Pritthijit Nath
On 11/02/21 7:15 pm, Pritthijit Nath wrote: > This change fixes a checkpatch CHECK style issue for "Alignment should match > open parenthesis". > > Signed-off-by: Pritthijit Nath > --- > drivers/staging/vt6656/rf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 08:54:26PM +0530, Pritthijit Nath wrote: > This change fixes a checkpatch CHECK style issue for "Alignment should match > open parenthesis". > > Signed-off-by: Pritthijit Nath > --- > drivers/staging/vt6656/rf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

Re: [PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-11 Thread Greg KH
On Fri, Feb 12, 2021 at 01:59:08AM +0530, Anirudh Rayabharam wrote: > Fix sparse byte-order warnings in the i2400m_bm_cmd_prepare() > function: > > wimax/i2400m/fw.c:194:36: warning: restricted __le32 degrades to integer > wimax/i2400m/fw.c:195:34: warning: invalid assignment: += >

[PATCH] removed enclosed parenthesis errors given by checkpatch.pl

2021-02-11 Thread roz
Signed-off-by: roz --- drivers/staging/ks7010/ks_hostif.h | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/staging/ks7010/ks_hostif.h b/drivers/staging/ks7010/ks_hostif.h index 39138191a556..c62a494ed6bb 100644 ---

[PATCH] staging: greybus: Fixed misspelling and alignment issue in hid.c

2021-02-11 Thread Pritthijit Nath
This change fixes a checkpatch CHECK style issue for "Alignment should match open parenthesis". In addition the misspelling of "transferred" also has been fixed. Signed-off-by: Pritthijit Nath --- drivers/staging/greybus/hid.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff

[PATCH] staging: wimax/i2400m: fix some byte order issues found by sparse

2021-02-11 Thread Anirudh Rayabharam
Fix sparse byte-order warnings in the i2400m_bm_cmd_prepare() function: wimax/i2400m/fw.c:194:36: warning: restricted __le32 degrades to integer wimax/i2400m/fw.c:195:34: warning: invalid assignment: += wimax/i2400m/fw.c:195:34:left side has type unsigned int wimax/i2400m/fw.c:195:34:

Re: [PATCH] staging: greybus: Fixed misspelling and alignment issue in hid.c

2021-02-11 Thread Greg KH
On Thu, Feb 11, 2021 at 09:00:01PM +0530, Pritthijit Nath wrote: > This change fixes a checkpatch CHECK style issue for "Alignment should match > open parenthesis". > In addition the misspelling of "transferred" also has been fixed. When you say "also" or "in addition" in a changelog, that is a

Re: [PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Greg KH
On Fri, Feb 12, 2021 at 02:32:51AM +0530, Pritthijit Nath wrote: > On 12/02/21 2:14 am, Greg KH wrote: > > On Fri, Feb 12, 2021 at 02:07:50AM +0530, Pritthijit Nath wrote: > >> On 12/02/21 1:59 am, Greg KH wrote: > >>> On Thu, Feb 11, 2021 at 08:54:26PM +0530, Pritthijit Nath wrote: > This

Re: [PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Greg KH
On Fri, Feb 12, 2021 at 02:56:47AM +0530, Pritthijit Nath wrote: > This change fixes a checkpatch CHECK style issue for "Alignment should > match open parenthesis". > > Signed-off-by: Pritthijit Nath > --- > drivers/staging/vt6656/rf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) >

[PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Pritthijit Nath
This change fixes a checkpatch CHECK style issue for "Alignment should match open parenthesis". Signed-off-by: Pritthijit Nath --- Fixed trailing space in changelog. drivers/staging/vt6656/rf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/rf.c

Re: [PATCH v5 0/6] MIPS: ralink: add CPU clock detection and clock driver for MT7621

2021-02-11 Thread Stephen Boyd
Quoting Sergio Paracuellos (2021-01-17 06:19:36) > Hi all, > > On Sun, Dec 20, 2020 at 10:37 AM Sergio Paracuellos > wrote: > > > > - Hardcode "xtal" as parent in FIXED macro. > > - Change 'else if' clause into 'if' clause since a return statement was > >being used in

[PATCH] staging: wimax: Fix some coding style problem

2021-02-11 Thread Hemansh Agnihotri
This fixes checkpatch error "open brace '{' following struct go on the same line" in file drivers/staging/wimax/i2400m/rx.c . Signed-off-by: Hemansh Agnihotri --- drivers/staging/wimax/i2400m/rx.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

Re: [PATCH v1 2/9] cpufreq: sfi-cpufreq: Remove driver for deprecated firmware

2021-02-11 Thread Viresh Kumar
On 11-02-21, 15:40, Andy Shevchenko wrote: > SFI-based platforms are gone. So does this driver. > > Signed-off-by: Andy Shevchenko > Acked-by: Linus Walleij > --- > drivers/cpufreq/Kconfig.x86 | 10 --- > drivers/cpufreq/Makefile | 1 - > drivers/cpufreq/sfi-cpufreq.c | 127

[PATCH] staging: vt6656: Fixed issue with alignment in rf.c

2021-02-11 Thread Pritthijit Nath
This change fixes a checkpatch CHECK style issue for "Alignment should match open parenthesis". Signed-off-by: Pritthijit Nath --- drivers/staging/vt6656/rf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/rf.c b/drivers/staging/vt6656/rf.c index