kernel/linux.git/drivers/crypto/caam/error.c, branch v4.11.5

crypto: caam - don't include unneeded headers

2017-02-15T05:23:36+00:00

intern.h, jr.h are not needed in error.c error.h is not needed in ctrl.c Signed-off-by: Horia Geantă Signed-off-by: Herbert Xu

crypto: caam - remove unreachable code in report_ccb_status()

2016-11-13T09:45:12+00:00

ERRID is a 4-bit field. Since err_id values are in [0..15] and err_id_list array size is 16, the condition "err_id < ARRAY_SIZE(err_id_list)" is always true. Signed-off-by: Horia Geantă Signed-off-by: Herbert Xu

crypto: caam - don't emit ICV check failures to dmesg

2015-01-26T00:34:23+00:00

ICV check failures are part of normal operation; leave user notification up to the higher levels, as is done in s/w algorithm implementations. Signed-off-by: Kim Phillips Tested-by: Cristian Stoica Signed-off-by: Herbert Xu

crypto: caam - fix error reporting

2014-11-06T15:15:05+00:00

The error code returned by hardware is four bits wide with an expected zero MSB. A hardware error condition where the error code can get between 0x8 and 0xf will trigger an out of bound array access on the error message table. This patch fixes the invalid array access following such an error and reports the condition. Signed-off-by: Cristian Stoica Signed-off-by: Herbert Xu

crypto: caam - Fix the 'quoted string split across lines'

2014-05-08T13:58:11+00:00

Fix the checkpatch warnings that the strings were split across multiple lines. Checkpatch now complains about lines over 80, but this is better, since we can actually grep the source code for these strings now. Signed-off-by: Marek Vasut Cc: Herbert Xu Cc: Horia Geanta Signed-off-by: Herbert Xu

crypto: caam - Sweep the remnants

2014-05-08T13:58:11+00:00

Clean up the remnants from the rework. Constify function arguments. Note that checkpatch again complains about this space before newline, but this is the original code behavior, so I'm keeping it. Signed-off-by: Marek Vasut Cc: Herbert Xu Cc: Horia Geanta Signed-off-by: Herbert Xu

crypto: caam - Kill SPRINTFCAT() with fire

2014-05-08T13:58:10+00:00

This macro is just like an encyclopedia of string handling done wrong. This must die. This is so wrong on so many levels. Signed-off-by: Marek Vasut Cc: Herbert Xu Cc: Horia Geanta Signed-off-by: Herbert Xu

crypto: caam - Clean up report_deco_status()

2014-05-08T13:58:10+00:00

Clean this function up and rework it into sensible shape. This function now contains one single dev_err() instead of the previous insanity full of memory allocation, chaotic string handling and use of SPRINTFCAT(). Signed-off-by: Marek Vasut Cc: Herbert Xu Cc: Horia Geanta Signed-off-by: Herbert Xu

crypto: caam - Clean up report_ccb_status()

2014-05-08T13:58:09+00:00

Clean this function up and rework it into sensible shape. This function now contains one single dev_err() instead of the previous insanity full of memory allocation, possible stack overwriting, chaotic string handling and use of SPRINTFCAT(). Signed-off-by: Marek Vasut Cc: Herbert Xu Cc: Horia Geanta Signed-off-by: Herbert Xu

crypto: caam - Dissolve report_jump_idx()

2014-05-08T13:58:09+00:00

Just dissolve this function so it's not in the way of applying further white magic cleanup down the line. Signed-off-by: Marek Vasut Cc: Herbert Xu Cc: Horia Geanta Signed-off-by: Herbert Xu