Re: [U-Boot] [PATCH] drivers/crypto/fsl: remove redundant logical contraint

2017-04-16 Thread Simon Glass
On 15 April 2017 at 08:37, Heinrich Schuchardt wrote: > 'A || (!A && B)' is equivalent to 'A || B'. > Let's reduce the complexity of the statement in start_jr0(). > > The problem was indicated by cppcheck. > > Signed-off-by: Heinrich Schuchardt > --- > drivers/crypto/fsl/jr.c | 3 +-- > 1 file c

Re: [U-Boot] [PATCH] drivers/crypto/fsl: remove redundant logical contraint

2017-04-15 Thread York Sun
On 04/15/2017 07:38 AM, Heinrich Schuchardt wrote: 'A || (!A && B)' is equivalent to 'A || B'. Let's reduce the complexity of the statement in start_jr0(). The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- drivers/crypto/fsl/jr.c | 3 +-- 1 file changed, 1 insertio

[U-Boot] [PATCH] drivers/crypto/fsl: remove redundant logical contraint

2017-04-15 Thread Heinrich Schuchardt
'A || (!A && B)' is equivalent to 'A || B'. Let's reduce the complexity of the statement in start_jr0(). The problem was indicated by cppcheck. Signed-off-by: Heinrich Schuchardt --- drivers/crypto/fsl/jr.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/crypto/fsl