[PATCH] staging: emxx_udc: Remove misplaced debugging aid

2020-09-11 Thread Mikhail Gusarov
emxx_udc.h contained a #if 0 block with debugging macros. These should
be supplied via command line or Kconfig.

Fixes checkstyle's
WARNING: Consider removing the code enclosed by this #if 0 and its #endif

Signed-off-by: Mikhail Gusarov 
---
 drivers/staging/emxx_udc/emxx_udc.h | 5 -
 1 file changed, 5 deletions(-)

diff --git a/drivers/staging/emxx_udc/emxx_udc.h 
b/drivers/staging/emxx_udc/emxx_udc.h
index 45f076e5fdb0..bca614d69aca 100644
--- a/drivers/staging/emxx_udc/emxx_udc.h
+++ b/drivers/staging/emxx_udc/emxx_udc.h
@@ -9,11 +9,6 @@
 #define _LINUX_EMXX_H
 
 /*---*/
-/*- Default undef */
-#if 0
-#define DEBUG
-#define UDC_DEBUG_DUMP
-#endif
 
 /*- Default define */
 #defineUSE_DMA 1
-- 
2.28.0

___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH][next] binder: remove redundant assignment to pointer n

2020-09-11 Thread Christian Brauner
On Thu, Sep 10, 2020 at 04:12:21PM +0100, Colin King wrote:
> From: Colin Ian King 
> 
> The pointer n is being initialized with a value that is
> never read and it is being updated later with a new value. The
> initialization is redundant and can be removed.
> 
> Addresses-Coverity: ("Unused value")
> Signed-off-by: Colin Ian King 
> ---

Thanks!
Acked-by: Christian Brauner 
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel


Re: [PATCH v3 0/7] crypto: mark ecb(arc4) skcipher as obsolete

2020-09-11 Thread Herbert Xu
On Mon, Aug 31, 2020 at 06:16:42PM +0300, Ard Biesheuvel wrote:
> RC4 hasn't aged very well, and is a poor fit for the skcipher API so it
> would be good if we could get rid of the ecb(arc4) drivers in the kernel
> at some point in the future. This prevents new users from creeping in, and
> allows us to improve the skcipher API without having to care too much about
> obsolete algorithms that may be difficult to support going forward.
> 
> So let's get rid of any remaining in-kernel users, either by switching them
> to the arc4 library API (for cases which simply cannot change algorithms,
> e.g., WEP), or dropping the code entirely. Also remove the remaining h/w
> accelerated implementations, and mark the generic s/w implementation as
> obsolete in Kconfig.
> 
> Changes since v2:
> - depend on CRYPTO_USER_API not CRYPTO_USER
> - rename CRYPTO_USER_ENABLE_OBSOLETE to CRYPTO_USER_API_ENABLE_OBSOLETE for
>   clarity
> 
> Changes since RFC [0]:
> - keep ecb(arc4) generic C implementation, and the associated test vectors,
>   but print a warning about ecb(arc4) being obsolete so we can identify
>   remaining users
> - add a Kconfig option to en/disable obsolete algorithms that are only kept
>   around to prevent breaking users that rely on it via the socket interface
> - add a patch to clean up some bogus Kconfig dependencies
> - add acks to patches #1, #2 and #3
> 
> [0] 
> https://lore.kernel.org/driverdev-devel/20200702101947.682-1-a...@kernel.org/
> 
> Cc: Herbert Xu 
> Cc: "David S. Miller" 
> Cc: Greg Kroah-Hartman 
> Cc: Trond Myklebust 
> Cc: Anna Schumaker 
> Cc: "J. Bruce Fields" 
> Cc: Chuck Lever 
> Cc: Eric Biggers 
> Cc: Arnd Bergmann 
> Cc: linux-cry...@vger.kernel.org
> Cc: net...@vger.kernel.org
> Cc: de...@driverdev.osuosl.org
> Cc: linux-...@vger.kernel.org
> 
> Ard Biesheuvel (7):
>   staging/rtl8192e: switch to RC4 library interface
>   staging/rtl8192u: switch to RC4 library interface
>   SUNRPC: remove RC4-HMAC-MD5 support from KerberosV
>   crypto: n2 - remove ecb(arc4) support
>   crypto: bcm-iproc - remove ecb(arc4) support
>   net: wireless: drop bogus CRYPTO_xxx Kconfig selects
>   crypto: arc4 - mark ecb(arc4) skcipher as obsolete
> 
>  crypto/Kconfig|  10 +
>  crypto/arc4.c |  10 +
>  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 -
>  drivers/crypto/n2_core.c  |  46 ---
>  drivers/net/wireless/intel/ipw2x00/Kconfig|   4 -
>  drivers/net/wireless/intersil/hostap/Kconfig  |   4 -
>  drivers/staging/rtl8192e/Kconfig  |   4 +-
>  drivers/staging/rtl8192e/rtllib_crypt_tkip.c  |  70 +
>  drivers/staging/rtl8192e/rtllib_crypt_wep.c   |  72 +
>  drivers/staging/rtl8192u/Kconfig  |   1 +
>  .../rtl8192u/ieee80211/ieee80211_crypt_tkip.c |  81 +
>  .../rtl8192u/ieee80211/ieee80211_crypt_wep.c  |  64 +---
>  include/linux/sunrpc/gss_krb5.h   |  11 -
>  include/linux/sunrpc/gss_krb5_enctypes.h  |   9 +-
>  net/sunrpc/Kconfig|   1 -
>  net/sunrpc/auth_gss/gss_krb5_crypto.c | 276 --
>  net/sunrpc/auth_gss/gss_krb5_mech.c   |  95 --
>  net/sunrpc/auth_gss/gss_krb5_seal.c   |   1 -
>  net/sunrpc/auth_gss/gss_krb5_seqnum.c |  87 --
>  net/sunrpc/auth_gss/gss_krb5_unseal.c |   1 -
>  net/sunrpc/auth_gss/gss_krb5_wrap.c   |  65 +
>  26 files changed, 97 insertions(+), 949 deletions(-)

All applied.  Thanks.
-- 
Email: Herbert Xu 
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel