[PATCH] staging: rtl8188eu: include: rtl8188e_xmit.h: fixed multiple blank space coding style issues

2020-07-02 Thread B K Karthik
added blank spaces to improve code readability. (coding style issue) Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtl8188e_xmit.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtl8188e_xmit.h

[PATCH] staging: rtl8188eu: include: rtw_recv.h: fixed a blank space coding style issue.

2020-07-02 Thread B K Karthik
added blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_recv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_recv.h b/drivers/staging/rtl8188eu/include/rtw_recv.h index

[PATCH 1/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed multiple parentheses coding style issues.

2020-07-02 Thread B K Karthik
add parentheses since complex valued macros must be enclosed within parentheses. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h

[PATCH 2/2] staging: rtl8188eu: include: rtw_pwrctrl.h: fixed a blank space coding style issue.

2020-07-02 Thread B K Karthik
add blank spaces to improve code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_pwrctrl.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h b/drivers/staging/rtl8188eu/include/rtw_pwrctrl.h

Re: [PATCH] wilc1000: move wilc driver out of staging

2020-07-02 Thread Ajay.Kathat
On 02/07/20 12:45 pm, Greg KH wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > On Thu, Jul 02, 2020 at 10:00:40AM +0300, Kalle Valo wrote: >> writes: >> >>> From: Ajay Singh >>> >>> WILC1000 is an IEEE 802.11 b/g/n IoT link controller

Re: [PATCH] wilc1000: move wilc driver out of staging

2020-07-02 Thread Ajay.Kathat
On 02/07/20 12:30 pm, Kalle Valo wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > writes: > >> From: Ajay Singh >> >> WILC1000 is an IEEE 802.11 b/g/n IoT link controller module. The >> WILC1000 connects to Microchip AVR/SMART MCUs,

[PATCH] staging: rtl8188eu: include: rtw_cmd.h: fixed a blank space coding style issue.

2020-07-02 Thread B K Karthik
add blank spaces for improved code readability. Signed-off-by: B K Karthik --- drivers/staging/rtl8188eu/include/rtw_cmd.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/include/rtw_cmd.h b/drivers/staging/rtl8188eu/include/rtw_cmd.h index

Re: [PATCH v3 5/9] media: staging: rkisp1: remove unecessary clocks

2020-07-02 Thread Helen Koike
On 7/2/20 1:54 PM, Helen Koike wrote: > aclk_isp_wrap is a child of aclk_isp, and hclk_isp_wrap is a child of > hclk_isp, thus we can remove parents from the list, with the goal to > cleanup the dt-bindings and remove it from staging. > > For reference, this is the isp clock topology: > >

[PATCH 2/2] staging: rtl8188eu: Fix WARNINGs of Block comments

2020-07-02 Thread Simon Fong
Fixed 5 WARNINGs of Block comments use * on subsequent lines. Signed-off-by: Simon Fong --- drivers/staging/rtl8188eu/core/rtw_xmit.c | 62 +++ 1 file changed, 31 insertions(+), 31 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_xmit.c

[PATCH 1/2] staging: rtl8188eu: Fix CHECK of coding style

2020-07-02 Thread Simon Fong
Fixed a CHECK of Lines should not end with a '('. Signed-off-by: Simon Fong --- drivers/staging/rtl8188eu/core/rtw_recv.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_recv.c b/drivers/staging/rtl8188eu/core/rtw_recv.c index

[PATCH v3 9/9] arm64: dts: rockchip: add isp and sensors for Scarlet

2020-07-02 Thread Helen Koike
From: Eddie Cai Enable ISP and camera sensor ov2685 and ov5695 for Scarlet Chromebook Verified with: make ARCH=arm64 dtbs_check Signed-off-by: Shunqian Zheng Signed-off-by: Eddie Cai Signed-off-by: Tomasz Figa Signed-off-by: Helen Koike --- This patch is based on:

[PATCH v3 2/9] media: staging: dt-bindings: rkisp1: add required items in i2c example

2020-07-02 Thread Helen Koike
Add missing required items in Rockchip ISP1 dt-bindings example for a complete i2c node. Signed-off-by: Helen Koike --- Changes in v3: None Changes in v2: - new patch in the series --- .../devicetree/bindings/media/rockchip-isp1.yaml | 5 + 1 file changed, 5 insertions(+)

[PATCH v3 4/9] media: staging: dt-bindings: rkisp1: fix "no reg" error in parent node

2020-07-02 Thread Helen Koike
Fix the following error found with make ARCH=arm64 dt_binding_check: Documentation/devicetree/bindings/media/rockchip-isp1.example.dts:24.27-101.11: Warning (unit_address_vs_reg): /example-0/parent@0: node has a unit name, but no reg or ranges property Reported-by: Johan Jonker Signed-off-by:

[PATCH v3 1/9] media: staging: dt-bindings: rkisp1: add missing required nodes

2020-07-02 Thread Helen Koike
Add missing required nodes in json-schema yaml file for Rockchip ISP1 dt-bindings. Signed-off-by: Helen Koike --- Changes in v3: None Changes in v2: - New patch in the series --- .../devicetree/bindings/media/rockchip-isp1.yaml | 8 1 file changed, 8 insertions(+) diff --git

[PATCH v3 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging

2020-07-02 Thread Helen Koike
Move rkisp1 bindings to Documentation/devicetree/bindings/media Verified with: make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml Signed-off-by: Helen Koike Acked-by: Rob Herring --- .../devicetree/bindings/media/rockchip-isp1.yaml

[PATCH v3 3/9] media: staging: dt-bindings: rkisp1: re-order properties

2020-07-02 Thread Helen Koike
Organize properties order in dt-binbings to move it out of staging. On top: compatible, reg and interrupts. Then alphabetical order, then properties starting with '#'. Signed-off-by: Helen Koike --- V3: none V2: - this is a new patch in the series --- .../bindings/media/rockchip-isp1.yaml

[PATCH v3 8/9] arm64: dts: rockchip: add isp0 node for rk3399

2020-07-02 Thread Helen Koike
From: Shunqian Zheng RK3399 has two ISPs, but only isp0 was tested. Add isp0 node in rk3399 dtsi Verified with: make ARCH=arm64 dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml Signed-off-by: Shunqian Zheng Signed-off-by: Jacob Chen Signed-off-by: Helen

[PATCH v3 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings

2020-07-02 Thread Helen Koike
The Rockchip ISP bindings was moved out of staging. Update MAINTAINERS file with the new path. Fields sorted according to output of ./scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS --order Signed-off-by: Helen Koike --- V3: - Add line: L:

Re: [RFC PATCH 4/7] crypto: remove ARC4 support from the skcipher API

2020-07-02 Thread Eric Biggers
[+linux-wireless, Marcel Holtmann, and Denis Kenzior] On Thu, Jul 02, 2020 at 12:19:44PM +0200, Ard Biesheuvel wrote: > Remove the generic ecb(arc4) skcipher, which is slightly cumbersome from > a maintenance perspective, since it does not quite behave like other > skciphers do in terms of key vs

Re: [RFC PATCH 4/7] crypto: remove ARC4 support from the skcipher API

2020-07-02 Thread Ard Biesheuvel
On Thu, 2 Jul 2020 at 19:50, Eric Biggers wrote: > > [+linux-wireless, Marcel Holtmann, and Denis Kenzior] > > On Thu, Jul 02, 2020 at 12:19:44PM +0200, Ard Biesheuvel wrote: > > Remove the generic ecb(arc4) skcipher, which is slightly cumbersome from > > a maintenance perspective, since it does

[PATCH v4 9/9] arm64: dts: rockchip: add isp and sensors for Scarlet

2020-07-02 Thread Helen Koike
From: Eddie Cai Enable ISP and camera sensor ov2685 and ov5695 for Scarlet Chromebook Verified with: make ARCH=arm64 dtbs_check Signed-off-by: Shunqian Zheng Signed-off-by: Eddie Cai Signed-off-by: Tomasz Figa Signed-off-by: Helen Koike --- This patch is based on:

[PATCH v4 8/9] arm64: dts: rockchip: add isp0 node for rk3399

2020-07-02 Thread Helen Koike
From: Shunqian Zheng RK3399 has two ISPs, but only isp0 was tested. Add isp0 node in rk3399 dtsi Verified with: make ARCH=arm64 dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml Signed-off-by: Shunqian Zheng Signed-off-by: Jacob Chen Signed-off-by: Helen

[PATCH v3 5/9] media: staging: rkisp1: remove unecessary clocks

2020-07-02 Thread Helen Koike
aclk_isp_wrap is a child of aclk_isp, and hclk_isp_wrap is a child of hclk_isp, thus we can remove parents from the list, with the goal to cleanup the dt-bindings and remove it from staging. For reference, this is the isp clock topology: xin24m pll_npll npll clk_isp1

[PATCH v3 0/9] move Rockchip ISP bindings out of staging / add

2020-07-02 Thread Helen Koike
Move the bindings out of drivers/staging and place them in Documentation/devicetree/bindings instead. Also, add DT nodes for RK3399 and verify with make ARCH=arm64 dtbs_check and make ARCH=arm64 dt_binding_check. Tested by verifying images streamed from Scarlet Chromebook Changes in v3: -

[PATCH v4 7/9] media: MAINTAINERS: rkisp1: add path to dt-bindings

2020-07-02 Thread Helen Koike
The Rockchip ISP bindings was moved out of staging. Update MAINTAINERS file with the new path. Fields sorted according to output of ./scripts/parse-maintainers.pl --input=MAINTAINERS --output=MAINTAINERS --order Signed-off-by: Helen Koike --- V3: - Add line: L:

[PATCH v4 2/9] media: staging: dt-bindings: rkisp1: add required items in i2c example

2020-07-02 Thread Helen Koike
Add missing required items in Rockchip ISP1 dt-bindings example for a complete i2c node. Signed-off-by: Helen Koike --- Changes in v2: - new patch in the series --- .../devicetree/bindings/media/rockchip-isp1.yaml | 5 + 1 file changed, 5 insertions(+) diff --git

[PATCH v4 0/9] move Rockchip ISP bindings out of staging / add ISP DT nodes for RK3399

2020-07-02 Thread Helen Koike
Move the bindings out of drivers/staging and place them in Documentation/devicetree/bindings instead. Also, add DT nodes for RK3399 and verify with make ARCH=arm64 dtbs_check and make ARCH=arm64 dt_binding_check. Tested by verifying images streamed from Scarlet Chromebook Changes in v4: -

[PATCH v4 1/9] media: staging: dt-bindings: rkisp1: add missing required nodes

2020-07-02 Thread Helen Koike
Add missing required nodes in json-schema yaml file for Rockchip ISP1 dt-bindings. Signed-off-by: Helen Koike --- Changes in v2: - New patch in the series --- .../devicetree/bindings/media/rockchip-isp1.yaml | 8 1 file changed, 8 insertions(+) diff --git

[PATCH v4 3/9] media: staging: dt-bindings: rkisp1: re-order properties

2020-07-02 Thread Helen Koike
Organize properties order in dt-binbings to move it out of staging. On top: compatible, reg and interrupts. Then alphabetical order, then properties starting with '#'. Signed-off-by: Helen Koike --- V2: - this is a new patch in the series --- .../bindings/media/rockchip-isp1.yaml |

[PATCH v4 4/9] media: staging: dt-bindings: rkisp1: fix "no reg" error in parent node

2020-07-02 Thread Helen Koike
Fix the following error found with make ARCH=arm64 dt_binding_check: Documentation/devicetree/bindings/media/rockchip-isp1.example.dts:24.27-101.11: Warning (unit_address_vs_reg): /example-0/parent@0: node has a unit name, but no reg or ranges property Reported-by: Johan Jonker Signed-off-by:

[PATCH v4 6/9] dt-bindings: media: rkisp1: move rockchip-isp1 bindings out of staging

2020-07-02 Thread Helen Koike
Move rkisp1 bindings to Documentation/devicetree/bindings/media Verified with: make ARCH=arm64 dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/media/rockchip-isp1.yaml Signed-off-by: Helen Koike Acked-by: Rob Herring --- .../devicetree/bindings/media/rockchip-isp1.yaml

[PATCH v4 5/9] media: staging: rkisp1: remove unecessary clocks

2020-07-02 Thread Helen Koike
aclk_isp_wrap is a child of aclk_isp, and hclk_isp_wrap is a child of hclk_isp, thus we can remove parents from the list. Also, for the isp0, we only need the ISP clock, ACLK and HCLK. In the future we'll need a pixel clock for RK3288 and RK3399, and a JPEG clock for RK3288. So with the goal to

Re: [RFC PATCH 3/7] SUNRPC: remove RC4-HMAC-MD5 support from KerberosV

2020-07-02 Thread J. Bruce Fields
Acked-by: J. Bruce Fields On Thu, Jul 02, 2020 at 12:19:43PM +0200, Ard Biesheuvel wrote: > The RC4-HMAC-MD5 KerberosV algorithm is based on RFC 4757 [0], which > was specifically issued for interoperability with Windows 2000, but was > never intended to receive the same level of support. The

Re: [RFC PATCH 4/7] crypto: remove ARC4 support from the skcipher API

2020-07-02 Thread Ard Biesheuvel
On Thu, 2 Jul 2020 at 20:21, Ard Biesheuvel wrote: > > On Thu, 2 Jul 2020 at 19:50, Eric Biggers wrote: > > > > [+linux-wireless, Marcel Holtmann, and Denis Kenzior] > > > > On Thu, Jul 02, 2020 at 12:19:44PM +0200, Ard Biesheuvel wrote: > > > Remove the generic ecb(arc4) skcipher, which is

[PATCH 0/2] Patchset of coding style fixes

2020-07-02 Thread Simon Fong
This patchset fixed CHECK and WARNINGs of coding style. Simon Fong (2): staging: rtl8188eu: Fix CHECK of coding style staging: rtl8188eu: Fix WARNINGs of Block comments drivers/staging/rtl8188eu/core/rtw_recv.c | 7 ++- drivers/staging/rtl8188eu/core/rtw_xmit.c | 62 +++

Re: [RFC PATCH 2/7] staging/rtl8192u: switch to RC4 library interface

2020-07-02 Thread Greg Kroah-Hartman
On Thu, Jul 02, 2020 at 12:19:42PM +0200, Ard Biesheuvel wrote: > Switch to the ARC4 library interface, to remove the pointless > dependency on the skcipher API, from which we will hopefully be > able to drop ecb(arc4) skcipher support. > > Signed-off-by: Ard Biesheuvel Acked-by: Greg

Re: [RFC PATCH 1/7] staging/rtl8192e: switch to RC4 library interface

2020-07-02 Thread Greg Kroah-Hartman
On Thu, Jul 02, 2020 at 12:19:41PM +0200, Ard Biesheuvel wrote: > Switch to the ARC4 library interface, to remove the pointless > dependency on the skcipher API, from which we will hopefully be > able to drop ecb(arc4) skcipher support. > > Signed-off-by: Ard Biesheuvel Acked-by: Greg

Re: [PATCH] wilc1000: move wilc driver out of staging

2020-07-02 Thread Greg KH
On Thu, Jul 02, 2020 at 10:00:40AM +0300, Kalle Valo wrote: > writes: > > > From: Ajay Singh > > > > WILC1000 is an IEEE 802.11 b/g/n IoT link controller module. The > > WILC1000 connects to Microchip AVR/SMART MCUs, SMART MPUs, and other > > processors with minimal resource requirements with a

Re: [PATCH] wilc1000: move wilc driver out of staging

2020-07-02 Thread Greg KH
On Thu, Jul 02, 2020 at 07:24:34AM +, ajay.kat...@microchip.com wrote: > > > On 02/07/20 12:45 pm, Greg KH wrote: > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > > content is safe > > > > On Thu, Jul 02, 2020 at 10:00:40AM +0300, Kalle Valo wrote: > >>

Re: [PATCH v7 00/17] wilc1000: move out of staging

2020-07-02 Thread Kalle Valo
Greg KH writes: > On Fri, Jun 26, 2020 at 08:34:48AM +0300, Kalle Valo wrote: > >> And Ajay already submitted that the simple rename patch proposed, thanks >> Ajay! >> >> https://patchwork.kernel.org/patch/11625025/ >> >> And indeed the patch is simple as it can get. So Greg, if it's ok for >>

Re: [PATCH] staging/wilc1000: let wilc_mac_xmit() to NETDEV_TX_OK

2020-07-02 Thread Kalle Valo
writes: > Luc, > > Thanks for your patch... > > On 28/06/2020 at 20:32, Luc Van Oostenryck wrote: >> The method ndo_start_xmit() is defined as returning an 'netdev_tx_t', >> which is a typedef for an enum type defining 'NETDEV_TX_OK' but this >> driver returns '0' instead of 'NETDEV_TX_OK'. >>

Re: [PATCH] staging: rtl8188eu: include: rtw_cmd.h: fixed a blank space coding style issue.

2020-07-02 Thread Greg Kroah-Hartman
On Thu, Jul 02, 2020 at 12:50:04AM -0400, B K Karthik wrote: > add blank spaces for improved code readability. > > Signed-off-by: B K Karthik > --- > drivers/staging/rtl8188eu/include/rtw_cmd.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH] wilc1000: move wilc driver out of staging

2020-07-02 Thread Kalle Valo
writes: > From: Ajay Singh > > WILC1000 is an IEEE 802.11 b/g/n IoT link controller module. The > WILC1000 connects to Microchip AVR/SMART MCUs, SMART MPUs, and other > processors with minimal resource requirements with a simple > SPI/SDIO-to-Wi-Fi interface. > > WILC1000 driver has been part

[RFC PATCH 0/7] crypto: get rid of ecb(arc4)

2020-07-02 Thread Ard Biesheuvel
The RC4 algorithm does not fit the sckipher model very well: it is a stream cipher that combines the key and IV into a single vector, which implies that using the same key more than once amounts to stream cipher IV reuse, and therefore catastrophic failure. So let's replace the remaining legacy

[RFC PATCH 2/7] staging/rtl8192u: switch to RC4 library interface

2020-07-02 Thread Ard Biesheuvel
Switch to the ARC4 library interface, to remove the pointless dependency on the skcipher API, from which we will hopefully be able to drop ecb(arc4) skcipher support. Signed-off-by: Ard Biesheuvel --- drivers/staging/rtl8192u/Kconfig | 1 +

[RFC PATCH 1/7] staging/rtl8192e: switch to RC4 library interface

2020-07-02 Thread Ard Biesheuvel
Switch to the ARC4 library interface, to remove the pointless dependency on the skcipher API, from which we will hopefully be able to drop ecb(arc4) skcipher support. Signed-off-by: Ard Biesheuvel --- drivers/staging/rtl8192e/Kconfig | 4 +-

[RFC PATCH 6/7] crypto: bcm-iproc - remove ecb(arc4) support

2020-07-02 Thread Ard Biesheuvel
Signed-off-by: Ard Biesheuvel --- drivers/crypto/bcm/cipher.c | 96 +--- drivers/crypto/bcm/cipher.h | 1 - drivers/crypto/bcm/spu.c| 23 + drivers/crypto/bcm/spu.h| 1 - drivers/crypto/bcm/spu2.c | 12 +-- drivers/crypto/bcm/spu2.h | 1 - 6 files changed, 6

[RFC PATCH 7/7] crypto: tcrypt - remove ecb(arc4) testing/benchmarking support

2020-07-02 Thread Ard Biesheuvel
Signed-off-by: Ard Biesheuvel --- crypto/tcrypt.c | 21 +-- crypto/testmgr.c | 7 --- crypto/testmgr.h | 62 3 files changed, 1 insertion(+), 89 deletions(-) diff --git a/crypto/tcrypt.c b/crypto/tcrypt.c index ba0b7702f2e9..72828c4acd3a 100644 --- a/crypto/tcrypt.c

[RFC PATCH 3/7] SUNRPC: remove RC4-HMAC-MD5 support from KerberosV

2020-07-02 Thread Ard Biesheuvel
The RC4-HMAC-MD5 KerberosV algorithm is based on RFC 4757 [0], which was specifically issued for interoperability with Windows 2000, but was never intended to receive the same level of support. The RFC says The IETF Kerberos community supports publishing this specification as an informational

[RFC PATCH 4/7] crypto: remove ARC4 support from the skcipher API

2020-07-02 Thread Ard Biesheuvel
Remove the generic ecb(arc4) skcipher, which is slightly cumbersome from a maintenance perspective, since it does not quite behave like other skciphers do in terms of key vs IV lifetime. Since we are leaving the library interface in place, which is used by the various WEP and TKIP implementations

[RFC PATCH 5/7] crypto: n2 - remove ecb(arc4) support

2020-07-02 Thread Ard Biesheuvel
Signed-off-by: Ard Biesheuvel --- drivers/crypto/n2_core.c | 46 1 file changed, 46 deletions(-) diff --git a/drivers/crypto/n2_core.c b/drivers/crypto/n2_core.c index 6a828bbecea4..c347e58cd9a1 100644 --- a/drivers/crypto/n2_core.c +++ b/drivers/crypto/n2_core.c @@ -662,7

Re: [PATCH] wilc1000: move wilc driver out of staging

2020-07-02 Thread Kalle Valo
writes: > On 02/07/20 12:30 pm, Kalle Valo wrote: >> EXTERNAL EMAIL: Do not click links or open attachments unless you >> know the content is safe >> >> writes: >> >>> From: Ajay Singh >>> >>> WILC1000 is an IEEE 802.11 b/g/n IoT link controller module. The >>> WILC1000 connects to Microchip

Re: [PATCH v6 08/12] device core: Introduce DMA range map, supplanting dma_pfn_offset

2020-07-02 Thread Andy Shevchenko
On Wed, Jul 01, 2020 at 05:21:38PM -0400, Jim Quinlan wrote: > The new field 'dma_range_map' in struct device is used to facilitate the > use of single or multiple offsets between mapping regions of cpu addrs and > dma addrs. It subsumes the role of "dev->dma_pfn_offset" which was only > capable

[PATCH] staging: qlge: qlge_ethtool.c: Proper indentation.

2020-07-02 Thread Suraj Upadhyay
Remove extra indentations from if-statement. Signed-off-by: Suraj Upadhyay --- drivers/staging/qlge/qlge_ethtool.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/qlge/qlge_ethtool.c b/drivers/staging/qlge/qlge_ethtool.c index 949abd53a7a9..16fcdefa9687