[PATCH 10/10] crypto: cavium: nitrox_isr: Demote non-compliant kernel-doc headers

2021-03-18 Thread Lee Jones
'irq' not described in 'nps_core_int_isr' drivers/crypto/cavium/nitrox/nitrox_isr.c:224: warning: Function parameter or member 'data' not described in 'nps_core_int_isr' Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-crypto@vger.kernel.org Si

[PATCH 07/10] crypto: caam: caampkc: Provide the name of the function and provide missing descriptions

2021-03-18 Thread Lee Jones
'xts_key_fallback' not described in 'caam_ctx' drivers/crypto/caam/caamalg_qi2.c:87: warning: Function parameter or member 'fallback' not described in 'caam_ctx' Cc: "Horia Geantă" Cc: Aymen Sghaier Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-c

[PATCH 08/10] crypto: vmx: Source headers are not good kernel-doc candidates

2021-03-18 Thread Lee Jones
lerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Henrique Cerri Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@lists.ozlabs.org Signed-off-by: Lee Jones --- drivers/crypto/vmx/vmx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 06/10] crypto: atmel-ecc: Struct headers need to start with keyword 'struct'

2021-03-18 Thread Lee Jones
es Cc: linux-crypto@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Signed-off-by: Lee Jones --- drivers/crypto/atmel-ecc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/atmel-ecc.c b/drivers/crypto/atmel-ecc.c index 515946c99394e..333fbefbbccbd 100644

[PATCH 09/10] crypto: nx: nx-aes-cbc: Repair some kernel-doc problems

2021-03-18 Thread Lee Jones
in Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Kent Yoder Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@lists.ozlabs.org Signed-off-by: Lee Jones --- drivers/crypto/nx/nx-aes-cbc.c | 2 +- drivers/crypto/nx/nx.c | 5 +++-- drive

[PATCH 05/10] crypto: keembay: ocs-hcu: Fix incorrectly named functions/structs

2021-03-18 Thread Lee Jones
ocs_hcu_hash_finup() instead Cc: Daniele Alessandrelli Cc: Declan Murphy Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/keembay/ocs-hcu.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/driv

[PATCH 03/10] crypto: chelsio: chcr_core: Fix some kernel-doc issues

2021-03-18 Thread Lee Jones
c: "David S. Miller" Cc: Manoj Malviya Cc: Atul Gupta Cc: Jitendra Lulla Cc: M R Gowda Cc: Harsh Jain Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/chelsio/chcr_algo.c | 8 +++- drivers/crypto/chelsio/chcr_core.c | 2 +- 2 files changed, 4 i

[PATCH 04/10] crypto: ux500: hash: hash_core: Fix worthy kernel-doc headers and remove others

2021-03-18 Thread Lee Jones
re.c:91: warning: Function parameter or member 'session_id' not described in 'cryp_ctx' Cc: Herbert Xu Cc: "David S. Miller" Cc: Shujuan Chen Cc: Joakim Bech Cc: Berne Hebark Cc: Niklas Hernaeus Cc: Jonas Linde Cc: Andreas Westin Cc: linux-crypto@v

[PATCH 02/10] crypto: bcm: Fix a whole host of kernel-doc misdemeanours

2021-03-18 Thread Lee Jones
ers/crypto/bcm/cipher.c:2966: warning: Function parameter or member 'cipher' not described in 'rfc4543_gcm_esp_setkey' drivers/crypto/bcm/cipher.c:2966: warning: Function parameter or member 'key' not described in 'rfc4543_gcm_esp_setkey' drivers/crypto/

[PATCH 01/10] crypto: hisilicon: sec_drv: Supply missing description for 'sec_queue_empty()'s 'queue' param

2021-03-18 Thread Lee Jones
pto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/hisilicon/sec/sec_drv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/crypto/hisilicon/sec/sec_drv.c b/drivers/crypto/hisilicon/sec/sec_drv.c index 91ee2bb575df2..3c26871db12f3 100644 --- a/drivers/crypto/hisilicon/sec/

[PATCH v3 00/10] Rid W=1 warnings in Crypto

2021-03-18 Thread Lee Jones
This is set 1 of 2 sets required to fully clean Crypto. v2: No functional changes since v1. v3: Description change and additional struct header fix Lee Jones (10): crypto: hisilicon: sec_drv: Supply missing description for 'sec_queue_empty()'s 'queue' param crypto: b

[PATCH 03/10] crypto: chelsio: chcr_core: Fix some kernel-doc issues

2021-03-03 Thread Lee Jones
c: "David S. Miller" Cc: Manoj Malviya Cc: Atul Gupta Cc: Jitendra Lulla Cc: M R Gowda Cc: Harsh Jain Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/chelsio/chcr_algo.c | 8 +++- drivers/crypto/chelsio/chcr_core.c | 2 +- 2 files changed, 4 i

[PATCH 02/10] crypto: bcm: Fix a whole host of kernel-doc misdemeanours

2021-03-03 Thread Lee Jones
ers/crypto/bcm/cipher.c:2966: warning: Function parameter or member 'cipher' not described in 'rfc4543_gcm_esp_setkey' drivers/crypto/bcm/cipher.c:2966: warning: Function parameter or member 'key' not described in 'rfc4543_gcm_esp_setkey' drivers/crypto/

[PATCH 01/10] crypto: hisilicon: sec_drv: Supply missing description for 'sec_queue_empty()'s 'queue' param

2021-03-03 Thread Lee Jones
pto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/hisilicon/sec/sec_drv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/crypto/hisilicon/sec/sec_drv.c b/drivers/crypto/hisilicon/sec/sec_drv.c index 91ee2bb575df2..3c26871db12f3 100644 --- a/drivers/crypto/hisilicon/sec/

[PATCH 04/10] crypto: ux500: hash: hash_core: Fix worthy kernel-doc headers and remove others

2021-03-03 Thread Lee Jones
re.c:91: warning: Function parameter or member 'session_id' not described in 'cryp_ctx' Cc: Herbert Xu Cc: "David S. Miller" Cc: Shujuan Chen Cc: Joakim Bech Cc: Berne Hebark Cc: Niklas Hernaeus Cc: Jonas Linde Cc: Andreas Westin Cc: linux-crypto@v

[PATCH v2 00/10] Rid W=1 warnings in Crypto

2021-03-03 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. This is set 1 of 2 sets required to fully clean Crypto. No functional changes since v1. Lee Jones (10): crypto: hisilicon: sec_drv: Supply

[PATCH 07/10] crypto: caam: caampkc: Provide the name of the function and provide missing descriptions

2021-03-03 Thread Lee Jones
'xts_key_fallback' not described in 'caam_ctx' drivers/crypto/caam/caamalg_qi2.c:87: warning: Function parameter or member 'fallback' not described in 'caam_ctx' Cc: "Horia Geantă" Cc: Aymen Sghaier Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-c

[PATCH 10/10] crypto: cavium: nitrox_isr: Demote non-compliant kernel-doc headers

2021-03-03 Thread Lee Jones
'irq' not described in 'nps_core_int_isr' drivers/crypto/cavium/nitrox/nitrox_isr.c:224: warning: Function parameter or member 'data' not described in 'nps_core_int_isr' Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-crypto@vger.kernel.org Si

[PATCH 05/10] crypto: keembay: ocs-hcu: Fix incorrectly named functions/structs

2021-03-03 Thread Lee Jones
ocs_hcu_hash_finup() instead Cc: Daniele Alessandrelli Cc: Declan Murphy Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/keembay/ocs-hcu.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/driv

[PATCH 09/10] crypto: nx: nx-aes-cbc: Repair some kernel-doc problems

2021-03-03 Thread Lee Jones
in Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Kent Yoder Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@lists.ozlabs.org Signed-off-by: Lee Jones --- drivers/crypto/nx/nx-aes-cbc.c | 2 +- drivers/crypto/nx/nx.c | 5 +++-- drive

[PATCH 06/10] crypto: atmel-ecc: Struct headers need to start with keyword 'struct'

2021-03-03 Thread Lee Jones
es Cc: linux-crypto@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Signed-off-by: Lee Jones --- drivers/crypto/atmel-ecc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/atmel-ecc.c b/drivers/crypto/atmel-ecc.c index 9bd8e5167be34..66a31e5739f4c 100644

[PATCH 08/10] crypto: vmx: Source headers are not good kernel-doc candidates

2021-03-03 Thread Lee Jones
lerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Henrique Cerri Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@lists.ozlabs.org Signed-off-by: Lee Jones --- drivers/crypto/vmx/vmx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [PATCH 06/20] crypto: keembay: ocs-hcu: Fix incorrectly named functions/structs

2021-02-04 Thread Lee Jones
On Thu, 04 Feb 2021, Alessandrelli, Daniele wrote: > On Thu, 2021-02-04 at 11:09 +0000, Lee Jones wrote: > > Fixes the following W=1 kernel build warning(s): > > > > drivers/crypto/keembay/ocs-hcu.c:107: warning: expecting prototype for > > struct ocs_hcu_dma_lis

[PATCH 11/20] crypto: ux500: cryp_core: Fix formatting issue and add description for 'session_id'

2021-02-04 Thread Lee Jones
Jonas Linde Cc: Andreas Westin Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/ux500/cryp/cryp_core.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/ux500/cryp/cryp_core.c b/drivers/crypto/ux500/cryp/cryp_core.c index c3adeb2e58232..25

[PATCH 14/20] crypto: caam: caampkc: Provide the name of the function

2021-02-04 Thread Lee Jones
ler" Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/caam/caampkc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/crypto/caam/caampkc.c b/drivers/crypto/caam/caampkc.c index dd5f101e43f83..e313233ec6de7 100644 --- a/drivers/crypto/

[PATCH 16/20] crypto: vmx: Source headers are not good kernel-doc candidates

2021-02-04 Thread Lee Jones
lerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Henrique Cerri Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@lists.ozlabs.org Signed-off-by: Lee Jones --- drivers/crypto/vmx/vmx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 13/20] crypto: bcm: cipher: Provide description for 'req' and fix formatting issues

2021-02-04 Thread Lee Jones
fc4543_gcm_esp_setkey' Cc: Herbert Xu Cc: "David S. Miller" Cc: Rob Rice Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/bcm/cipher.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/crypto/bcm/cipher.c b/drivers/crypt

[PATCH 12/20] crypto: atmel-ecc: Struct headers need to start with keyword 'struct'

2021-02-04 Thread Lee Jones
es Cc: linux-crypto@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Signed-off-by: Lee Jones --- drivers/crypto/atmel-ecc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/atmel-ecc.c b/drivers/crypto/atmel-ecc.c index 9bd8e5167be34..66a31e5739f4c 100644

[PATCH 15/20] crypto: caam: caamalg_qi2: Supply a couple of 'fallback' related descriptions

2021-02-04 Thread Lee Jones
ot described in 'caam_ctx' Cc: "Horia Geantă" Cc: Aymen Sghaier Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/caam/caamalg_qi2.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/cr

[PATCH 20/20] crypto: cavium: nitrox_isr: Demote non-compliant kernel-doc headers

2021-02-04 Thread Lee Jones
'irq' not described in 'nps_core_int_isr' drivers/crypto/cavium/nitrox/nitrox_isr.c:224: warning: Function parameter or member 'data' not described in 'nps_core_int_isr' Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-crypto@vger.kernel.org Si

[PATCH 17/20] crypto: nx: nx-aes-cbc: Headers comments should not be kernel-doc

2021-02-04 Thread Lee Jones
nx_set_key() instead Cc: "Breno Leitão" Cc: Nayna Jain Cc: Paulo Flabiano Smorigo Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Kent Yoder Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@lists.ozlabs.

[PATCH 18/20] crypto: nx: nx_debugfs: Header comments should not be kernel-doc

2021-02-04 Thread Lee Jones
for nx_debugfs_init() instead Cc: "Breno Leitão" Cc: Nayna Jain Cc: Paulo Flabiano Smorigo Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Kent Yoder Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@l

[PATCH 19/20] crypto: nx: Demote header comment and add description for 'nbytes'

2021-02-04 Thread Lee Jones
ulo Flabiano Smorigo Cc: Michael Ellerman Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Herbert Xu Cc: "David S. Miller" Cc: Kent Yoder Cc: linux-crypto@vger.kernel.org Cc: linuxppc-...@lists.ozlabs.org Signed-off-by: Lee Jones --- drivers/crypto/nx/nx.c | 5 +++-- 1 file ch

[PATCH 09/20] crypto: ux500: cryp_irq: File headers are not good kernel-doc candidates

2021-02-04 Thread Lee Jones
erne Hebark Cc: Niklas Hernaeus Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/ux500/cryp/cryp_irq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/ux500/cryp/cryp_irq.c b/drivers/crypto/ux500/cryp/cryp_irq.c index 7ebde69e8c76b..6d

[PATCH 08/20] crypto: ux500: cryp: Demote some conformant non-kernel headers fix another

2021-02-04 Thread Lee Jones
'cryp_save_device_context' Cc: Herbert Xu Cc: "David S. Miller" Cc: "Gustavo A. R. Silva" Cc: Shujuan Chen Cc: Jonas Linde Cc: Niklas Hernaeus Cc: Joakim Bech Cc: Berne Hebark Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/ux5

[PATCH 07/20] crypto: bcm: spu2: Fix a whole host of kernel-doc misdemeanours

2021-02-04 Thread Lee Jones
1243: warning: expecting prototype for spu_request_pad(). Prototype was for spu2_request_pad() instead drivers/crypto/bcm/spu2.c:1321: warning: expecting prototype for spu_status_process(). Prototype was for spu2_status_process() instead Cc: Herbert Xu Cc: "David S. Miller" Cc:

[PATCH 10/20] crypto: chelsio: chcr_algo: Fix a couple of kernel-doc issues caused by doc-rot

2021-02-04 Thread Lee Jones
rs/crypto/chelsio/chcr_algo.c:1566: warning: Function parameter or member 'param' not described in 'create_hash_wr' Cc: Ayush Sawal Cc: Vinay Kumar Yadav Cc: Rohit Maheshwari Cc: Herbert Xu Cc: "David S. Miller" Cc: Manoj Malviya Cc: Atul Gupta Cc: Jitendra Lu

[PATCH 05/20] crypto: bcm: spu: Fix formatting and misspelling issues

2021-02-04 Thread Lee Jones
ng: bad line: drivers/crypto/bcm/spu.c:927: warning: Function parameter or member 'is_inbound' not described in 'spum_cipher_req_finish' drivers/crypto/bcm/spu.c:927: warning: Excess function parameter 'isInbound' description in 'spum_cipher_req_finish

[PATCH 06/20] crypto: keembay: ocs-hcu: Fix incorrectly named functions/structs

2021-02-04 Thread Lee Jones
ocs_hcu_hash_finup() instead Cc: Daniele Alessandrelli Cc: Declan Murphy Cc: Herbert Xu Cc: "David S. Miller" Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/keembay/ocs-hcu.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/driv

[PATCH 04/20] crypto: ux500: hash: hash_core: Fix worthy kernel-doc headers and remove others

2021-02-04 Thread Lee Jones
h_algs_unregister_all' drivers/crypto/ux500/hash/hash_core.c:1647: warning: expecting prototype for hash_algs_unregister_all(). Prototype was for ahash_algs_unregister_all() instead Cc: Herbert Xu Cc: "David S. Miller" Cc: Shujuan Chen Cc: Joakim Bech Cc: Berne Hebark Cc: Niklas

[PATCH 03/20] crypto: chelsio: chcr_core: File headers are not good candidates for kernel-doc

2021-02-04 Thread Lee Jones
lla Cc: M R Gowda Cc: Harsh Jain Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/chelsio/chcr_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/chelsio/chcr_core.c b/drivers/crypto/chelsio/chcr_core.c index f91f9d762a45e..f03

[PATCH 02/20] crypto: bcm: util: Repair a couple of documentation formatting issues

2021-02-04 Thread Lee Jones
x27;spu_msg_sg_add' Cc: Herbert Xu Cc: "David S. Miller" Cc: Takashi Iwai Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/bcm/util.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/bcm/util.c b/drivers/crypto/bc

[PATCH 01/20] crypto: hisilicon: sec_drv: Supply missing description for 'sec_queue_empty()'s 'queue' param

2021-02-04 Thread Lee Jones
pto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/hisilicon/sec/sec_drv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/crypto/hisilicon/sec/sec_drv.c b/drivers/crypto/hisilicon/sec/sec_drv.c index 91ee2bb575df2..3c26871db12f3 100644 --- a/drivers/crypto/hisilicon/sec/

[PATCH 00/20] Rid W=1 warnings in Crypto

2021-02-04 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. This is set 1 of 2 sets required to fully clean Crypto. Lee Jones (20): crypto: hisilicon: sec_drv: Supply missing description for

Re: [PATCH 1/1] crypto: ux500: hash: Add namespacing to hash_init()

2020-07-07 Thread Lee Jones
On Tue, 30 Jun 2020, Lee Jones wrote: > On Tue, 30 Jun 2020, Herbert Xu wrote: > > On Mon, Jun 29, 2020 at 01:30:03PM +0100, Lee Jones wrote: > > > A recent change to the Regulator consumer API (which this driver > > > utilises) add prototypes for the some suspend functi

Re: [PATCH 1/1] crypto: ux500: hash: Add namespacing to hash_init()

2020-06-30 Thread Lee Jones
On Tue, 30 Jun 2020, Herbert Xu wrote: > On Mon, Jun 29, 2020 at 01:30:03PM +0100, Lee Jones wrote: > > A recent change to the Regulator consumer API (which this driver > > utilises) add prototypes for the some suspend functions. These > > functions require including head

[PATCH 1/1] crypto: ux500: hash: Add namespacing to hash_init()

2020-06-29 Thread Lee Jones
the local hash_init() such that the source of confusion is removed. Cc: Linus Walleij Cc: Herbert Xu Cc: David S. Miller Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- Ideally this should go into v5.8's -rcs else it runs the risk of breaking when Linus pulls everything

Re: [PATCH v2 0/3] hwrng: st: Couple of improvements

2015-10-09 Thread Lee Jones
On Thu, 08 Oct 2015, Herbert Xu wrote: > On Wed, Oct 07, 2015 at 01:23:26PM +0100, Lee Jones wrote: > > This set contains a minor documentation fix, greater clarification > > with how the FIFO depth/size is represented and finally a fix to > > prevent early timeout du

[PATCH v2 1/3] hwrng: st: dt: Fix trivial typo in node address

2015-10-07 Thread Lee Jones
DT nodes should not append their addresses with '0x'. Suggested-by: Stephen Boyd Signed-off-by: Lee Jones --- Documentation/devicetree/bindings/rng/st,rng.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/rng/st,rng.txt b/Doc

[PATCH v2 0/3] hwrng: st: Couple of improvements

2015-10-07 Thread Lee Jones
ent call to udelay() - Suggested-by: Russell King Lee Jones (3): hwrng: st: dt: Fix trivial typo in node address hwrng: st: Use real-world device timings for timeout hwrng: st: Improve FIFO size/depth description Documentation/devicetree/bindings/rng/st,rng.txt | 2 +- drivers/c

[PATCH v2 3/3] hwrng: st: Improve FIFO size/depth description

2015-10-07 Thread Lee Jones
falling into the same trap. Signed-off-by: Lee Jones --- drivers/char/hw_random/st-rng.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/char/hw_random/st-rng.c b/drivers/char/hw_random/st-rng.c index 17f0a09..1d35363 100644 --- a/drivers/char/hw_random/st-rng.c

[PATCH v2 2/3] hwrng: st: Use real-world device timings for timeout

2015-10-07 Thread Lee Jones
delay for as long as requested. Suggested-by: Russell King : "IIRC, Linus recommends a x2 factor on delays, especially timeouts generated by these functions. Signed-off-by: Lee Jones --- drivers/char/hw_random/st-rng.c | 10 -- 1 file changed, 8 insertions(+), 2 dele

[RESEND 3/3] hwrng: st: Report correct FIFO size

2015-10-06 Thread Lee Jones
available values are being read during each invocation of .read(). This change increased the device bandwidth by a factor of 2. Reported-by: Daniel Thompson Reviewed-by: Daniel Thompson Signed-off-by: Lee Jones --- drivers/char/hw_random/st-rng.c | 3 ++- 1 file changed, 2 insertions(+),

[RESEND 2/3] hwrng: st: Use real-world device timings for timeout

2015-10-06 Thread Lee Jones
Samples are documented to be available every 0.667us, so in theory the 8 sample deep FIFO should take 5.336us to fill. However, during thorough testing, it became apparent that filling the FIFO actually takes closer to 12us. Signed-off-by: Lee Jones --- drivers/char/hw_random/st-rng.c | 9

[RESEND 1/3] hwrng: st: dt: Fix trivial typo in node address

2015-10-06 Thread Lee Jones
DT nodes should not append their addresses with '0x'. Suggested-by: Stephen Boyd Signed-off-by: Lee Jones --- Documentation/devicetree/bindings/rng/st,rng.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/rng/st,rng.txt b/Doc

[PATCH] hwrng: st: dt: Fix trivial typo in node address

2015-10-05 Thread Lee Jones
DT nodes should not append their addresses with '0x'. Suggested-by: Stephen Boyd Signed-off-by: Lee Jones --- Documentation/devicetree/bindings/rng/st,rng.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/rng/st,rng.txt b/Doc

Re: [PATCH v2 5/7] hwrng: st: Add support for ST's HW Random Number Generator

2015-10-05 Thread Lee Jones
On Mon, 05 Oct 2015, Daniel Thompson wrote: > On 05/10/15 13:11, Lee Jones wrote: > >On Mon, 05 Oct 2015, Daniel Thompson wrote: > >>Late but... > > > >That's okay. Fixup patches can always be submitted. > > > >We have forever. :) > > >

Re: [PATCH v2 5/7] hwrng: st: Add support for ST's HW Random Number Generator

2015-10-05 Thread Lee Jones
On Mon, 05 Oct 2015, Daniel Thompson wrote: > Late but... That's okay. Fixup patches can always be submitted. We have forever. :) > On 17/09/15 14:45, Lee Jones wrote: > >diff --git a/drivers/char/hw_random/Makefile > >b/drivers/char/hw_random/Makefile > >

Re: [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-30 Thread Lee Jones
On Wed, 30 Sep 2015, Herbert Xu wrote: > On Wed, Sep 30, 2015 at 03:15:39PM +0100, Lee Jones wrote: > > > > > I prefer not to merge patches that cannot be tested. Without > > > the DT bits in patch 6 the other five patches are useless. So > > > I think patch 6

Re: [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-30 Thread Lee Jones
On Wed, 30 Sep 2015, Herbert Xu wrote: > On Tue, Sep 29, 2015 at 03:29:32PM +0100, Lee Jones wrote: > > > > I see that your tree is 8 days old, so this may have been resolved > > already, but would you be kind enough to ensure you remove the 6th > > (ARM) patch from yo

Re: [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-29 Thread Lee Jones
On Fri, 18 Sep 2015, Lee Jones wrote: > On Fri, 18 Sep 2015, Herbert Xu wrote: > > On Fri, Sep 18, 2015 at 03:53:50PM +0100, Lee Jones wrote: > > > On 18 September 2015 at 15:07, Herbert Xu > > > wrote: > > > > On Thu, Sep 17, 2015 at 02:45:50PM +0

Re: [STLinux Kernel] [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-29 Thread Lee Jones
On Tue, 29 Sep 2015, Peter Griffin wrote: > On Thu, 17 Sep 2015, Lee Jones wrote: > > > v1 => v2: > > - New patch: Also fixing /dev/hw_random => /dev/hwrng in Kconfig > > - Fix 2099 => 2009 typo in commit log > > - Fix 'number of random numbers sou

Re: [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-19 Thread Lee Jones
On Sun, 20 Sep 2015, Herbert Xu wrote: > On Sat, Sep 19, 2015 at 10:21:45AM +0100, Lee Jones wrote: > > > > That's not how it works. It's helpful, more often than not, to submit > > the entire set to each maintainer concerned so they can keep up with > &g

Re: [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-19 Thread Lee Jones
On Sat, 19 Sep 2015, Herbert Xu wrote: > On Fri, Sep 18, 2015 at 04:51:12PM +0100, Lee Jones wrote: > > > > I think it's okay for you to take all but patch 6. > > > > Patch 6 is an ARM patch and needs to go into ARM SoC via > > STMicroelectronics STi tre

Re: [PATCH v2 4/7] hwrng: st: Provide DT bindings for ST's Random Number Generator

2015-09-18 Thread Lee Jones
On 18 September 2015 at 17:21, Stephen Boyd wrote: > On 09/17/2015 06:45 AM, Lee Jones wrote: >> Signed-off-by: Lee Jones >> --- >> Documentation/devicetree/bindings/rng/st,rng.txt | 15 +++ >> 1 file changed, 15 insertions(+) >> create mode 100644

Re: [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-18 Thread Lee Jones
On Fri, 18 Sep 2015, Herbert Xu wrote: > On Fri, Sep 18, 2015 at 03:53:50PM +0100, Lee Jones wrote: > > On 18 September 2015 at 15:07, Herbert Xu > > wrote: > > > On Thu, Sep 17, 2015 at 02:45:50PM +0100, Lee Jones wrote: > > >> v1 => v2: > > >&g

Re: [PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-18 Thread Lee Jones
On 18 September 2015 at 15:07, Herbert Xu wrote: > On Thu, Sep 17, 2015 at 02:45:50PM +0100, Lee Jones wrote: >> v1 => v2: >> - New patch: Also fixing /dev/hw_random => /dev/hwrng in Kconfig >> - Fix 2099 => 2009 typo in commit log >> - Fix 'numbe

[PATCH v2 5/7] hwrng: st: Add support for ST's HW Random Number Generator

2015-09-17 Thread Lee Jones
Signed-off-by: Pankaj Dev Signed-off-by: Lee Jones --- drivers/char/hw_random/Kconfig | 10 +++ drivers/char/hw_random/Makefile | 1 + drivers/char/hw_random/st-rng.c | 144 3 files changed, 155 insertions(+) create mode 100644 drivers/char

[PATCH v2 1/7] Documentation: hw_random: Fix device node name reference /dev/hw_random => /dev/hwrng

2015-09-17 Thread Lee Jones
idered ABI by now. So instead, we'll just change the documentation to match the current situation. NB: It looks like rng-tools have already been updated. Signed-off-by: Lee Jones --- Documentation/hw_random.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH v2 2/7] hwrng: Kconfig: Fix device node name reference /dev/hw_random => /dev/hwrng

2015-09-17 Thread Lee Jones
idered ABI by now. So instead, we'll just change the Kconfig help to match the current situation. NB: It looks like rng-tools have already been updated. Signed-off-by: Lee Jones --- drivers/char/hw_random/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH v2 7/7] MAINTAINERS: Add ST's Random Number Generator to the ST entry

2015-09-17 Thread Lee Jones
Signed-off-by: Lee Jones --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index fd60784..b084d69 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1502,6 +1502,7 @@ W:http://www.stlinux.com S: Maintained F: arch/arm/mach-sti/ F

[PATCH v2 4/7] hwrng: st: Provide DT bindings for ST's Random Number Generator

2015-09-17 Thread Lee Jones
Signed-off-by: Lee Jones --- Documentation/devicetree/bindings/rng/st,rng.txt | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/rng/st,rng.txt diff --git a/Documentation/devicetree/bindings/rng/st,rng.txt b/Documentation/devicetree

[PATCH v2 3/7] hwrng: core: Simplify RNG switching from sysfs

2015-09-17 Thread Lee Jones
If we attempt to use sysfs to change the current RNG in the usual way i.e. issuing something like: `echo 8a8a000.rng > /sys/devices/virtual/misc/hw_random/rng_current` ... it will fail because the code doesn't currently take the '\n' into consideration. Well, now it does.

[PATCH v2 6/7] ARM: STi: STiH407: Enable the 2 HW Random Number Generators for STiH4{07,10}

2015-09-17 Thread Lee Jones
Signed-off-by: Lee Jones --- arch/arm/boot/dts/stih407-family.dtsi | 14 ++ 1 file changed, 14 insertions(+) diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi index 838b812..9452b42 100644 --- a/arch/arm/boot/dts/stih407-family.dtsi +++ b

[PATCH v2 0/7] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-17 Thread Lee Jones
NG documentation and making the sysfs interface a little easier to use by ignoring any '\n' which may have been inadvertently passed. Lee Jones (7): Documentation: hw_random: Fix device node name reference /dev/hw_random => /dev/hwrng hwrng: Kconfig: Fix device node name referenc

Re: [PATCH 4/6] hwrng: st: Add support for ST's HW Random Number Generator

2015-09-14 Thread Lee Jones
On Sat, 12 Sep 2015, Kieran Bingham wrote: > On 11 September 2015 at 21:08, Lee Jones wrote: > > Signed-off-by: Pankaj Dev > > Signed-off-by: Lee Jones > > --- > > drivers/char/hw_random/Kconfig | 10 +++ > > drivers/char/hw_random/Makefile | 1 + > >

Re: [PATCH 4/6] hwrng: st: Add support for ST's HW Random Number Generator

2015-09-14 Thread Lee Jones
On Sat, 12 Sep 2015, Fabio Estevam wrote: > On Fri, Sep 11, 2015 at 5:08 PM, Lee Jones wrote: > > > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0); > > + base = devm_ioremap_resource(&pdev->dev, res); > > + if (IS_ERR(base)) > >

Re: [PATCH 2/6] hwrng: core: Simplify RNG switching from sysfs

2015-09-14 Thread Lee Jones
On Sun, 13 Sep 2015, Peter Korsgaard wrote: > >>>>> "Lee" == Lee Jones writes: > > > If we attempt to use sysfs to change the current RNG in the usual > > way i.e. issuing something like: > > > `echo 8a8a000.rng > /sys/devices/virtu

[PATCH 0/6] hwrng: Add support for STMicroelectronics' RNG IP

2015-09-11 Thread Lee Jones
use by ignoring any '\n' which may have been inadvertently passed. Lee Jones (6): Documentation: hw_random: Fix device node name /dev/hw_random => /dev/hwrng hwrng: core: Simplify RNG switching from sysfs hwrng: st: Provide DT bindings for ST's Random Number Generator h

[PATCH 2/6] hwrng: core: Simplify RNG switching from sysfs

2015-09-11 Thread Lee Jones
If we attempt to use sysfs to change the current RNG in the usual way i.e. issuing something like: `echo 8a8a000.rng > /sys/devices/virtual/misc/hw_random/rng_current` ... it will fail because the code doesn't currently take the '\n' into consideration. Well, now it does.

[PATCH 1/6] Documentation: hw_random: Fix device node name /dev/hw_random => /dev/hwrng

2015-09-11 Thread Lee Jones
considered ABI by now. So instead, we'll just change the documentation to match the current situation. NB: It looks like rng-tools have already been updated. Signed-off-by: Lee Jones --- Documentation/hw_random.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 3/6] hwrng: st: Provide DT bindings for ST's Random Number Generator

2015-09-11 Thread Lee Jones
Signed-off-by: Lee Jones --- Documentation/devicetree/bindings/rng/st,rng.txt | 15 +++ 1 file changed, 15 insertions(+) create mode 100644 Documentation/devicetree/bindings/rng/st,rng.txt diff --git a/Documentation/devicetree/bindings/rng/st,rng.txt b/Documentation/devicetree

[PATCH 6/6] MAINTAINERS: Add ST's Random Number Generator to the ST entry

2015-09-11 Thread Lee Jones
Signed-off-by: Lee Jones --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index fd60784..b084d69 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1502,6 +1502,7 @@ W:http://www.stlinux.com S: Maintained F: arch/arm/mach-sti/ F

[PATCH 4/6] hwrng: st: Add support for ST's HW Random Number Generator

2015-09-11 Thread Lee Jones
Signed-off-by: Pankaj Dev Signed-off-by: Lee Jones --- drivers/char/hw_random/Kconfig | 10 +++ drivers/char/hw_random/Makefile | 1 + drivers/char/hw_random/st-rng.c | 142 3 files changed, 153 insertions(+) create mode 100644 drivers/char

[PATCH 5/6] ARM: STi: STiH407: Enable the 2 HW Random Number Generators for STiH4{07,10}

2015-09-11 Thread Lee Jones
Signed-off-by: Lee Jones --- arch/arm/boot/dts/stih407-family.dtsi | 14 ++ 1 file changed, 14 insertions(+) diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi index 838b812..9452b42 100644 --- a/arch/arm/boot/dts/stih407-family.dtsi +++ b

Re: [PATCH 06/39] ARM: ux500: Move SDI (MMC) and UART devices under more descriptive heading

2013-06-10 Thread Lee Jones
On Wed, 15 May 2013, Linus Walleij wrote: > On Wed, May 15, 2013 at 11:51 AM, Lee Jones wrote: > > > Now DMA DT bindings exist and are in use by he MMC and UART drivers, it > > should be possible to remove them from the auxdata structure. However, > > after doing so the

Re: [PATCH 05/39] ARM: ux500: Stop passing MMC's platform data for Device Tree boots

2013-06-10 Thread Lee Jones
On Wed, 15 May 2013, Linus Walleij wrote: > On Wed, May 15, 2013 at 11:51 AM, Lee Jones wrote: > > > It was required to pass DMA channel configuration information to the > > MMC driver before the new DMA API was in place. Now that it is, and > > is fully compatible with

Re: [PATCH 20/39] usb: musb: ux500: move channel number knowledge into the driver

2013-05-30 Thread Lee Jones
On Thu, 30 May 2013, Linus Walleij wrote: > On Wed, May 29, 2013 at 7:57 PM, Felipe Balbi wrote: > > On Wed, May 15, 2013 at 10:51:43AM +0100, Lee Jones wrote: > >> For all ux500 based platforms the maximum number of end-points are used. > >> Move this knowledg

Re: [PATCH 20/39] usb: musb: ux500: move channel number knowledge into the driver

2013-05-28 Thread Lee Jones
ll aid us when we come to enable the driver for Device Tree. > > > > > > Cc: Felipe Balbi > > > Cc: linux-...@vger.kernel.org > > > Acked-by: Linus Walleij > > > Acked-by: Fabio Baltieri > > > Signed-off-by: Lee Jones > > > > I g

Re: [PATCH 5/6] crypto: ux500/cryp - Enable DT probing of the driver

2013-05-24 Thread Lee Jones
On Fri, 24 May 2013, Linus Walleij wrote: > On Thu, May 16, 2013 at 1:27 PM, Lee Jones wrote: > > > By providing an OF match table with a suitable compatible string, we > > can ensure the ux500-crypt driver is probed by supplying an associated > > DT node in a give

[PATCH 5/6] crypto: ux500/cryp - Enable DT probing of the driver

2013-05-16 Thread Lee Jones
By providing an OF match table with a suitable compatible string, we can ensure the ux500-crypt driver is probed by supplying an associated DT node in a given platform's Device Tree. Cc: Herbert Xu Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/ux500

[PATCH 6/6] crypto: ux500/hash - Enable DT probing of the driver

2013-05-16 Thread Lee Jones
By providing an OF match table with a suitable compatible string, we can ensure the ux500-hasht driver is probed by supplying an associated DT node in a given platform's Device Tree. Cc: Herbert Xu Cc: linux-crypto@vger.kernel.org Signed-off-by: Lee Jones --- drivers/crypto/ux500

Re: [PATCH 02/39] dmaengine: ste_dma40: Remove unnecessary call to d40_phy_cfg()

2013-05-16 Thread Lee Jones
On Thu, 16 May 2013, Vinod Koul wrote: > On Thu, May 16, 2013 at 08:25:57AM +0100, Lee Jones wrote: > > On Thu, 16 May 2013, Vinod Koul wrote: > > > > > On Wed, May 15, 2013 at 10:51:25AM +0100, Lee Jones wrote: > > > > All configuration left in d40_p

Re: [PATCH 34/39] dmaengine: ste_dma40: Convert data_width from register bit format to value

2013-05-16 Thread Lee Jones
On Thu, 16 May 2013, Vinod Koul wrote: > On Wed, May 15, 2013 at 10:51:57AM +0100, Lee Jones wrote: > > When a DMA client requests and configures a DMA channel, it requests > > data_width in Bytes. The DMA40 driver then swiftly converts it over to > > the necessa

Re: [PATCH 01/39] dmaengine: ste_dma40: Separate Logical Global Interrupt Mask (GIM) unmasking

2013-05-16 Thread Lee Jones
On Thu, 16 May 2013, Vinod Koul wrote: > On Wed, May 15, 2013 at 06:29:51PM +0200, Linus Walleij wrote: > > On Wed, May 15, 2013 at 11:51 AM, Lee Jones wrote: > > > > > During the initial setup of a logical channel, it is necessary to unmask > > > the GIM in or

Re: [PATCH 02/39] dmaengine: ste_dma40: Remove unnecessary call to d40_phy_cfg()

2013-05-16 Thread Lee Jones
On Thu, 16 May 2013, Vinod Koul wrote: > On Wed, May 15, 2013 at 10:51:25AM +0100, Lee Jones wrote: > > All configuration left in d40_phy_cfg() is runtime configurable and > > there is already a call into it from d40_runtime_config(), so let's > > rely on that. >

Re: [PATCH 31/39] dmaengine: ste_dma40: Replace ST-E's home-brew DMA direction defs with generic ones

2013-05-16 Thread Lee Jones
On Thu, 16 May 2013, Vinod Koul wrote: > On Wed, May 15, 2013 at 10:51:54AM +0100, Lee Jones wrote: > > STEDMA40_*_TO_* direction definitions are identical in all but name to > > the pre-defined generic DMA_*_TO_* ones. Let's make things easy by not > > duplicating such

Re: [PATCH 35/39] dmaengine: ste_dma40_ll: Replace meaningless register set with comment

2013-05-15 Thread Lee Jones
ready. It was his idea to place the comment in. The original author will on parental leave for the foreseeable future. -- Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: s

Re: [PATCH 12/39] crypto: ux500/hash - Prepare clock before enabling it

2013-05-15 Thread Lee Jones
On Wed, 15 May 2013, Linus Walleij wrote: > On Wed, May 15, 2013 at 11:51 AM, Lee Jones wrote: > > > If we fail to prepare the ux500-hash clock before enabling it the > > platform will fail to boot. Here we insure this happens. > > > > Cc: Herbert Xu > >

[PATCH 03/39] dmaengine: ste_dma40: Don't configure runtime configurable setup during allocate

2013-05-15 Thread Lee Jones
Using the dmaengine API, allocating and configuring a channel are two separate actions. Here we're removing logical channel configuration from the channel allocating routines. Cc: Vinod Koul Cc: Dan Williams Cc: Per Forlin Cc: Rabin Vincent Signed-off-by: Lee Jones --- drivers/dma/ste_

  1   2   >