[PATCH v2] staging: ccree: remove unused function argument

From: Arnd Bergmann "gcc -Wunused" warns about one argument being assigned but not used: drivers/staging/ccree/ssi_cipher.c: In function 'ssi_blkcipher_complete': drivers/staging/ccree/ssi_cipher.c:747:41: error: parameter 'info' set but not used [-Werror=unused-but-set-parameter] We can simpl

Re: [PATCH] staging: ccree: remove unused function argument

Hi Arnd, On Thu, May 11, 2017 at 2:38 PM, Arnd Bergmann wrote: > "gcc -Wunused" warns about one argument being assigned but not used: > > drivers/staging/ccree/ssi_cipher.c: In function 'ssi_blkcipher_complete': > drivers/staging/ccree/ssi_cipher.c:747:41: error: parameter 'info' set but > not u

[PATCH 3/9] staging: sm750fb: reordering of macro definitions

This patch reorder definition of macros so all macros are in same order. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_display.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/sm750fb/ddk750_display.h b/drivers/staging/sm750fb/ddk750_display

[PATCH 1/9] staging: sm750fb: fix length of lines

This patch breaks lines that are longer than 80 characters and joins together those, that are too short and can be placed at one. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_chip.c | 7 +++-- drivers/staging/sm750fb/ddk750_dvi.c| 35 +-- drivers/stag

[PATCH 8/9] staging: sm750fb: Remove typedef from "typedef enum _DPMS_t"

This patch removes typedefs from enum and renames it from "typedef enum _DPMS_t" to "enum DPMS" as per kernel coding standards. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_power.c | 2 +- drivers/staging/sm750fb/ddk750_power.h | 7 +++ 2 files changed, 4 insertions(+), 5 d

[PATCH 6/9] staging: sm750fb: Remove typedef from "typedef enum _clock_type_t"

This patch removes typedefs from enum and renames it from "typedef enum _clock_type_t" to "enum clock_type" as per kernel coding standards. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_chip.h | 8 drivers/staging/sm750fb/ddk750_mode.c | 2 +- drivers/staging/sm750fb/dd

[PATCH 9/9] staging: sm750fb: Remove typedef from "typedef enum _sii164_hot_plug_mode_t"

This patch removes typedefs from enum and renames it from "typedef enum _sii164_hot_plug_mode_t" to "enum sii164_hot_plug_mode" as per kernel coding standards. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_sii164.c | 2 +- drivers/staging/sm750fb/ddk750_sii164.h | 4 ++-- 2 file

[PATCH 2/9] staging: sm750fb: unifying macro definitions

This patch adds tabs into macro definitions so all rhs are on same column. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_display.h | 74 drivers/staging/sm750fb/ddk750_hwi2c.c | 4 +- drivers/staging/sm750fb/sm750.h | 6 +-- 3 files

[PATCH 0/9] staging: sm750fb: cleaning code

Folowing patches are cleaning some warnings and checkups from checkpatch.pl Matej Dujava (9): staging: sm750fb: fix length of lines staging: sm750fb: unifying macro definitions staging: sm750fb: reordering of macro definitions staging: sm750fb: removing unnecessary binary operations stag

[PATCH 4/9] staging: sm750fb: removing unnecessary binary operations

This patch remove unnecessary operation (eg. ``X | (0x0 << Y)`` to ``X``). Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_display.h | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/staging/sm750fb/ddk750_display.h b/d

[PATCH 5/9] staging: sm750fb: Remove typedef from "typedef enum _logical_chip_type_t"

This patch removes typedefs from enum and renames it from "typedef enum _logical_chip_type_t" to "enum logical_chip_type" as per kernel coding standards. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_chip.c | 4 ++-- drivers/staging/sm750fb/ddk750_chip.h | 8 2 files ch

[PATCH 7/9] staging: sm750fb: Remove typedef from "typedef enum _disp_output_t"

This patch removes typedefs from enum and renames it from "typedef enum _disp_output_t" to "enum disp_output" as per kernel coding standards. Signed-off-by: Matej Dujava --- drivers/staging/sm750fb/ddk750_display.c | 2 +- drivers/staging/sm750fb/ddk750_display.h | 8 drivers/staging/sm

[PATCH] staging: rtl8188eu, rtl8723bs: fix spelling mistake "Cancle" -> "Cancel"

From: Colin Ian King Trivial fix to spelling mistakes in a comments and RT_TRACE text. Signed-off-by: Colin Ian King --- drivers/staging/rtl8188eu/core/rtw_mlme.c | 4 ++-- drivers/staging/rtl8723bs/core/rtw_mlme.c | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/driver

[PATCH] staging: atomisp: fix non static symbol warnings

Fix some unneeded exported symbols by marking them as static. This was found with the 'sparse' tool. Signed-off-by: Juan Antonio Pedreira Martos --- .../media/atomisp/platform/intel-mid/atomisp_gmin_platform.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/driv

[PATCH v2] staging: ccree: Fix blank lines codestyle issue

Checkpatch emits CHECK: Please don't use multiple blank lines. Remove multiple blank lines. Signed-off-by: Alexander Mazyrin --- drivers/staging/ccree/ssi_aead.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/ccree/ssi_aead.c b/drivers/staging/ccree/ssi_aead.c index 03829

[PATCH 1/5] staging: rtl8192e: Remove RX: IEEE802.1X EAPOL frame! warning.

RX will receive countless EAPOL frames over the life of the connection. A number of conditional calls to rtllib_is_eapol_frame are made in this function. So this call serves no purpose other than to spam logs with false warning that it is indeed a EAPOL frame, remove it. Signed-off-by: Malcolm

[PATCH 4/5] staging: rtl8192e: Let user know mac address associated with.

User cannot tell which mac address(BSIDD) associated with so add this to info message. Signed-off-by: Malcolm Priestley --- drivers/staging/rtl8192e/rtllib_softmac.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192e/rtllib_softmac.c b/drivers/staging

[PATCH 2/5] staging: rtl8192e: print alg name as debug.

alg name will be printed a number times during a connection it is only really useful as a debug message. Change to netdev_dbg. Signed-off-by: Malcolm Priestley --- drivers/staging/rtl8192e/rtllib_wx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8192e/r

[PATCH 5/5] staging: rtl8192e: _rtl92e_dm_check_edca_turbo remove peername message.

This kinda reports this as if it was an error message. Now that bssid is reported at associate remove this piece of code serves no purpose as there is no code for peers so remove it. Signed-off-by: Malcolm Priestley --- drivers/staging/rtl8192e/rtl8192e/rtl_dm.c | 16 1 file ch

[PATCH 3/5] staging: rtl8192e: HTSetConnectBwMode message replace with debug.

The flag status of bCurBW40MHz is printed as info and is only useful as debug message. Replace with netdev_dbg in line with rest of driver. Signed-off-by: Malcolm Priestley --- drivers/staging/rtl8192e/rtl819x_HTProc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/driv

[PATCH] staging: atomisp: Fix -Werror=int-in-bool-context compile errors

With gcc-7.1.1 I was getting the following compile error: error: ‘*’ in boolean context, suggest ‘&&’ instead The problem is the definition of CEIL_DIV: #define CEIL_DIV(a, b) ((b) ? ((a) + (b) - 1) / (b) : 0) Which when called as: CEIL_DIV(x, y * z) triggers this error, note we cannot do

[PATCH 12622/12622] Staging: ccree: ssi_cipher: fix brace coding style issue

Fixed some coding style issues after running scripts/checkpatch.pl Fixed spelling in ssi_blkcipher_complete(), should be counter not couter. Signed-off-by: Ammly Fredrick --- drivers/staging/ccree/ssi_cipher.c | 72 ++ 1 file changed, 34 insertions(+), 38 del

Re: [Bug] VCHIQ functional test broken

On Sat, May 13, 2017 at 11:07:28AM +0200, Stefan Wahren wrote: > In the meantime this issue has been fixed by Phil [1]. Right - definitely a driver bug. Mapping more memory for DMA than is actually going to be DMA'd to and expecting data to be preserved is really horrid. > Unfortunately i found

Re: [Bug] VCHIQ functional test broken

> Stefan Wahren hat am 24. April 2017 um 21:35 > geschrieben: > > > > > Russell King - ARM Linux hat am 24. April 2017 um > > 20:59 geschrieben: > > > > > > On Mon, Apr 24, 2017 at 07:42:27PM +0200, Stefan Wahren wrote: > > > > What I can't see is how changing flush_dcache_page() has poss